将 char 数组转换为 uint16_t 数组 C/C++

Question

我编写了以下代码，将数据从名为str的字符串（字符数组）转换并存储到名为arr16bit的16位整数数组中

代码有效。但是，我想说有一种更好或更简洁的方式来实现这个逻辑，使用更少的变量等。

我不想使用索引i 来获取模数% 2，因为如果使用小端序，我有相同的算法，但i 从字符串的最后一个索引开始并倒计时而不是向上倒计时.任何建议表示赞赏。

// assuming str had already been initialised before this ..

int strLength        = CalculateStringLength(str);      // function implementation now shown 
uint16_t*  arr16bit  = new uint16_t[ (strLength /2) + 1];  // The only C++ feature used here , so I didn't want to tag it
int indexWrite       = 0;
int counter          = 0;

for(int i = 0; i < strLength; ++i)
{
    arr16bit[indexWrite] <<= 8;
    arr16bit[indexWrite] |= str[i];
    if ( (counter  % 2) != 0)
    {
        indexWrite++;
    }
    counter++;
}

Answer 1

是的，这里有一些多余的变量。

您同时拥有counter 和i，它们执行完全相同的操作并始终保持相同的值。你有indexWrite，它总是正好是它们的一半（每个整数除法）。

您也移动得太远了（16 位而不是 8 位）。

const std::size_t strLength = CalculateStringLength(str);
std::vector<uint16_t> arr16bit((strLength/2) + 1);

for (std::size_t i = 0; i < strLength; ++i)
{
    arr16bit[i/2] <<= 8;
    arr16bit[i/2] |= str[i];
}

虽然我可能会这样做，以避免 N 冗余 |= 操作：

const std::size_t strLength = CalculateStringLength(str);
std::vector<uint16_t> arr16bit((strLength/2) + 1);

for (std::size_t i = 0; i < strLength+1; i += 2)
{
    arr16bit[i/2]      = (str[i] << 8);
    arr16bit[(i/2)+1] |= str[i+1];
}

如果您的字节序适合，您可能还希望在整个 dang 缓冲区上考虑一个简单的 std::copy。