Question

我正在从一个字节的十六进制值转换为它的ascii表示，但是当我尝试打印出结果字符串时，由于某种原因，长度是错误的。

以下是我生成的代码和输出：

#include <math.h>
#include <string.h>

unsigned char numDigits(unsigned char aNum)
{
    unsigned char digits = 0x00;
    while (aNum)
    {
      //  printf("aNum: %d\n",aNum);
        aNum /= 0x0a;
        digits++;
    }
    if(digits == 0x00)
      return 1; // aNum was 0 but we still need to send the ascii representation of 0 so set lengeth to 1
    return digits;
}

string numToAscii(unsigned char aNum)
{
  int len = numDigits(aNum);
  printf("Num digits: %d\n", len);
  string asciiNum = new char[len];
  for(int i = 0; i < len; i++)
  {
    int nthPlace = (int)pow(10.0,(len-(i+1)));  
    int nthNum = aNum/nthPlace;
    asciiNum[i] = nthNum+0x30;
    printf("adding %c\n", asciiNum[i]);
    aNum-=nthNum*nthPlace;
  }
  return asciiNum;
}

int main ()
{
unsigned char one = 0xfd;
unsigned char two = 0x64;
unsigned char three = 0x40;
unsigned char four = 0x0f;
unsigned char five = 0x01;

string cone = numToAscii(one);
string ctwo= numToAscii(two );
string cthree = numToAscii(three );
string cfour = numToAscii(four );
string cfive= numToAscii(five );

printf("Len: %d One: %s\n", cone.length(), cone.c_str());
printf("Len: %d two : %s\n", ctwo.length(), ctwo.c_str());
printf("Len: %d three : %s\n", cthree.length(), cthree.c_str() );
printf("Len: %d four : %s\n", cfour.length(), cfour.c_str() );
printf("Len: %d five : %s\n",cfive.length(), cfive.c_str());

return 0;
}

输出：

Num digits: 3
adding 2
adding 5
adding 3
Num digits: 3
adding 1
adding 0
adding 0
Num digits: 2
adding 6
adding 4
Num digits: 2
adding 1
adding 5
Num digits: 1
adding 1
Len: 5 One: 253�)
Len: 6 two : 100�i
Len: 3 three : 64�
Len: 3 four : 15�
Len: 2 five : 1�

Answer 1

这是错误的：

string asciiNum = new char[len];

当您从C样式字符数组初始化C ++字符串时，它假定数组已终止 - 也就是说，有一个零值字节来标记字符串的结尾。这不是这里的情况 - 你分配一些内存（填充垃圾），尝试从它初始化字符串（具有未定义的结果，因为它可能找不到终结符），然后泄漏内存，因为你已经丢失了指向它的指针。

相反，你想要：

string asciiNum(len, '\0');

或者，如果您愿意，请将其设为空：

string asciiNum;

并使用push_back或+=向其添加字符。

（这假定string是标准库中的类，即使您没有包含<string>或使用std::对其进行限定。也许您的图书馆距离标准库已有20年了日期？）

Answer 2

这是因为您错误地初始化了字符串。当你打电话

string asciiNum = new char[len];

您正在使用未初始化的数据初始化字符串，并且还会创建内存泄漏，因为您从未在delete[]上调用new char[len]。

将其替换为

string asciiNum(len, '0');

解决问题。

Answer 3

因为没有其他人发布过它，你试过了吗？

int main ()
{
unsigned char one = 0xfd;
std::cout << static_cast<unsigned int>(one) << std::endl;
return 0;
}

字符串长度不正确

3 个答案: