[已解决]s1e29动动手0.疑问和题目没有太大关系，只是不明白哪里用了溢出

顶级太阳

s1e29动动手0，题目如下：
      你应该听说过 itoa 函数（函数文档 -> 传送门），它的作用是将一个整数转换成字符串形式存储。现在要求我们自己来实现一个类似功能的函数 myitoa(int num, char *str)，该函数的第一个参数是待转换的整型变量，第二参数传入一个字符指针，用于存放转换后的字符串。
给出主程序段：

……
int main(void)
{
        char str[10];

        printf("%s\n", myitoa(520, str));
        printf("%s\n", myitoa(-1234, str));

        return 0;
}

要求的结果：

我的代码：

#include<stdio.h>

char *myitoa(int a,char *array)
{

        int i=0,j,flag=0,m; //结果位数i计算
        char ch_array[10]; //中间数组。

        
        if(a<<0)  //判断正负值。如果是负数，将符号‘-’存入数组array第一个位置
        {
                array[0]='-';
                i++;
                a=-a;
                
        }
        m=0;
        if(a>=10)    //每次除10，提取余数存入中间数组ch_array
        {
                ch_array[m]=a%10;
                m++;        
        }

        for(j=m;j<0;j--)  //数组存储值反转存储到指定的数组里
        {
                array[i]=ch_array[j];
                i++;
        }

}

int main(void)
{
        char str[10];
        
        printf("%s\n",myitoa(520,str));
        printf("%s\n",myitoa(-1234,str));

        return 0;
}

悲惨的运行结果：


仔细分析了代码的各个部分，实在搞不明白哪里出了问题，造成这个“段错误（核心已转存）”。上网查找段错误的原因，是指发生了越界，也就是访问了不应该访问的内存，这个内存要么不存在，要么不允许使用。还有一个可能文件损坏。
那么首先，我尝试编译了好多遍了，应该排除文件损坏的原因。整个代码没有指定内存位置，应该不会被这么多次调用到不允许访问的内存，那么就是越界了。整个代码过程，根据题目要求定义了一个10元素的数组，我又定义了一个10元素组成的中间数组char ch_array[10].外加使用了4个整形变量，都是用了几个数字，根据题目，数组最大用到5个元素，那么变量值最大用到4也就结束了，不可能超出范围呀。还求哪位有耐心帮忙找找这个“段错误”错在了哪里。谢谢。

最佳答案

月排行榜 / 总排行榜

dolly_yos2

2022-11-12 08:36:22

先说编码问题，其实您的编译器本应该协助您解决这些问题的。看看我使用的编译器是怎么评价您的代码的：

test.c:10:13: warning: converting the result of '<<' to a boolean; did you mean '(a << 0) != 0'? [-Wint-in-bool-context]
        if(a<<0)  //判断正负值。如果是负数，将符号‘-’存入数组array第一个位置
            ^
test.c:6:19: warning: unused variable 'flag' [-Wunused-variable]
        int i=0,j,flag=0,m; //结果位数i计算
                  ^
test.c:30:1: warning: non-void function does not return a value [-Wreturn-type]
}
^
3 warnings generated.

理解和解决了这些问题，就理解和解决了您目前出现的段错误。
当然，您的代码除此之外还有逻辑上的问题，还需要后续的修改才能得到正确的结果，您可以先尝试，遇到无法解决的问题欢迎继续提问。

跳转到最佳答案楼层

顶级太阳

jackz007 发表于 2022-11-11 22:05
下面是我修改的版本

谢谢Jack。上次就看出来您的for循环使用的炉火纯青，在下佩服。根据你的提示，我修改了判断正负的if语句（第10行），修改了取尾数语句（第18行），在第21行插入了a=a/10;语句。虽然不明白你的第13行，计算出10进制尾数后，存入的时候要与0相加，也按照您的方式进行了修改。试运行结果还是同样的结果，编译通过，运行时出现“段错误”提示，不能出现正确结果。

我想要的不只是一个正确的答案，想要的是解决我现在遇到的问题。十分感谢。

dolly_yos2

先说编码问题，其实您的编译器本应该协助您解决这些问题的。看看我使用的编译器是怎么评价您的代码的：

test.c:10:13: warning: converting the result of '<<' to a boolean; did you mean '(a << 0) != 0'? [-Wint-in-bool-context]
        if(a<<0)  //判断正负值。如果是负数，将符号‘-’存入数组array第一个位置
            ^
test.c:6:19: warning: unused variable 'flag' [-Wunused-variable]
        int i=0,j,flag=0,m; //结果位数i计算
                  ^
test.c:30:1: warning: non-void function does not return a value [-Wreturn-type]
}
^
3 warnings generated.

理解和解决了这些问题，就理解和解决了您目前出现的段错误。
当然，您的代码除此之外还有逻辑上的问题，还需要后续的修改才能得到正确的结果，您可以先尝试，遇到无法解决的问题欢迎继续提问。

顶级太阳

dolly_yos2 发表于 2022-11-12 08:36
先说编码问题，其实您的编译器本应该协助您解决这些问题的。看看我使用的编译器是怎么评价您的代码的：
理 ...

谢谢您的答复，根据你的编译器提示，我找到了溢出的位置，也对逻辑进行了修正，已经完成了题目。好奇问一下，您用的是什么编译器，可以告诉我么？你这个编译器的提示太丰富了，我能够使用不？

dolly_yos2

顶级太阳 发表于 2022-11-13 19:28
谢谢您的答复，根据你的编译器提示，我找到了溢出的位置，也对逻辑进行了修正，已经完成了题目。好奇问一 ...

得到之前回复里的输出的编译器和版本是

clang version 14.0.6

当然， gcc 也可以得到类似的结果（这里我改了一下源代码文件名）：

test2.c: In function ‘myitoa’:
test2.c:10:13: warning: ‘<<’ in boolean context, did you mean ‘<’? [-Wint-in-bool-context]
   10 |         if(a<<0)  //判断正负值。如果是负数，将符号‘-’存入数组array第一个位置
      |            ~^~~
test2.c:6:19: warning: unused variable ‘flag’ [-Wunused-variable]
    6 |         int i=0,j,flag=0,m; //结果位数i计算
      |                   ^~~~
test2.c:30:1: warning: control reaches end of non-void function [-Wreturn-type]
   30 | }
      | ^

版本是

gcc (GCC) 12.2.0

您可以试着给您的编译命令加上额外的调试选项，我一般常用的是（在此处使用的也是） -Wall -Wextra -fsanitize=address ，即改为

gcc -Wall -Wextra -fsanitize=address test.c

相关选项的含义可以在这里查找对应您使用的版本的文档： https://gcc.gnu.org/onlinedocs/
如最新版的选项总结在这里： https://gcc.gnu.org/onlinedocs/gcc-12.2.0/gcc/Option-Summary.html

顶级太阳

dolly_yos2 发表于 2022-11-14 10:11
得到之前回复里的输出的编译器和版本是
当然， gcc 也可以得到类似的结果（这里我改了一下源代码文件名 ...

十分感谢。刚刚自己开始学习，完全靠摸索，很多东西还都不懂。