将int分配给数组元素时出现分段错误[关闭]

Question

我的程序的这一部分旨在遍历字符串的chars，并选择字符串中的所有数字并将它们放入一个名为helper 的数组中。这是我正在处理的一个较大程序的一小部分，我已尽力只提供有用的代码段。我也知道没有正确使用指针会导致分段错误，但是我的问题是我找不到错误使用指针的位置。

所以，当我尝试编译以下代码时，我遇到了一个分段错误（核心转储）...

#include <stdio.h>
#include <string.h>

int main(void)
{
    char *eq = "y=344+99";
    int helper[50];
    unsigned short int helperWriter = 0, i;

    for (i = 0; i < eq[i]; i++) //
    {
        if (isdigit(eq[i]))
        {
            unsigned short int d;
            for (d = i; eq[d]; d++)
            {

                if (isdigit(eq[d]))
                {
                    int temp = atoi(eq[d]);
                    helper[helperWriter] = temp;
                    printf("%d", helperWriter);
                }
                helperWriter++; 
            }
        }
    }
}

我对 C 很陌生，反过来，我对指针也很陌生，所以我的错误可能很愚蠢。如果需要任何额外信息，请询问。

Answer 1

当您使用库函数而不包括其原型时，编译器会假设其参数和返回类型为int。你没有#include <stdlib.h>（或ctype.h）所以在这一行

int temp = atoi(eq[d]);

你正在传递一个char，编译器很乐意将它提升为int。但是atoi 需要char* 类型。所以它试图访问机器中低地址的内存，导致段错误。该行应该是

int temp = atoi(eq + d);    // or `atoi(&eq[d])`

将每个字符串字符的地址传递给atoi。

第一个循环不正确，你应该测试nul终止符。

helperWriter++; 位于错误位置时也会出错：将其移至将 temp 放入数组的位置下方。

您还忽略了我的建议，即在每条调试消息的末尾加上 newline（以确保在崩溃发生之前实际看到它）。

在您拿起号码然后移过它的方式中可能存在其他错误。您不需要两个嵌套循环，atoi 会将数字转换为下一个非数字字符。

Answer 2

由于@Weather Vane 指出了您代码中的问题，我想向您展示一个替代解决方案：

#include <stdio.h>          /* printf */
#include <string.h>         /* strcspn */
#include <stdlib.h>         /* strtol */

int main(void)
{
    char *eq = "y=344+99", *ptr = eq;
    int helper[50];
    size_t i = 0;
    while(*ptr) // Is there any more numbers in the string?
    {
        ptr += strcspn(ptr, "1234567890"); // Move ptr to the next number
        helper[i++] = strtol(ptr, &ptr, 10);
        printf("%d\n", helper[i - 1]);
    }
}

虽然这段代码看起来有点复杂，但它更紧​​凑，并且利用了库函数，效率更高。