今天同事给了一段代码,说调试有问题,让给看看。功能是把int类型变量转换为16进制字符串打印输出。
问题描述:输出全为乱码。
一开始的时候,也没看出是什么问题,
在inttohex函数里面吧buffer打印出来,没有问题。
可是在main中的printf(“%s”, b);倒全是乱码了。
后来想了想,发现 buffer是个局部变量,位于堆栈,在每次跳出inttohex函数后,内存会被释放掉,导致数据不为实际内容。
只需要把buffer作为全局变量或者改为静态变量,则就不用担心内存会被释放掉。
原问题代码如下:
#include <iostream>
#include <stdlib.h>
#include <math.h>
using namespace std;
char *inttohex(int aa)
{
char buffer[3];
if (aa / 16 < 10) //计算十位,并转换成字符
buffer[0] = aa / 16 + '0';
else
buffer[0] = aa / 16 - 10 + 'A';
if (aa % 16 < 10) //计算个位,并转换成字符
buffer[1] = aa % 16 + '0';
else
buffer[1] = aa % 16 - 10 + 'A';
buffer[2] = '\0'; //字符串结束标志
return (buffer);
}
int main()
{
int a = 12;
char *b;
b = (char *)malloc(sizeof(char) * 3);
b = inttohex(a);
printf("a= %d, b= %s", a, b);
getchar();
}
下面是作为全局变量修改后的结果:
#include <stdio.h>
#include <string.h>
char buffer[3] = { 0 }; /*全局变量*/
char *inttohex(int aa)
{
if (aa / 16 < 10) //计算十位,并转换成字符
buffer[0] = aa / 16 + '0';
else
buffer[0] = aa / 16 - 10 + 'A';
if (aa % 16 < 10) //计算个位,并转换成字符
buffer[1] = aa % 16 + '0';
else
buffer[1] = aa % 16 - 10 + 'A';
return buffer;
}
int main()
{
int a = 12;
char buffer[3] = { 0 };
char *b = inttohex(a);;
printf("a= %d, b= %s", a, b);
getchar();
}