为什么 atoi 给我一个分段错误？

发布于 2024-09-30 11:53:53 字数 496 浏览 3 评论 0原文

我有以下代码：

#include <stdio.h>

int main ( int argc, char *argv[] )
{
    int M, N;

    M = 1;
    N = 1;
    curr = 1;

    if ( argv[1][0] == '-' )
    {
        curr = 2;

        char *a = argv[1][1];
        char *b = argv[1][3];

        M = atoi(a);
        N = atoi(b);
    }

    printf("%d\n%d", M, N);
}

因此，我传递了这样的程序：

a.out -1,2

而不是获得预期的输出

1
2

我遇到分段错误。什么给？

原文

I have the following piece of code:

#include <stdio.h>

int main ( int argc, char *argv[] )
{
    int M, N;

    M = 1;
    N = 1;
    curr = 1;

    if ( argv[1][0] == '-' )
    {
        curr = 2;

        char *a = argv[1][1];
        char *b = argv[1][3];

        M = atoi(a);
        N = atoi(b);
    }

    printf("%d\n%d", M, N);
}

So, I pass this program something like this:

a.out -1,2

and instead of getting expected output

1
2

I get a segmentation fault. What gives?

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

九八野马 2024-10-07 11:53:53

这可以编译吗？！

char argv*[] 是一个 char 指针数组。

char *a = argv[1][1] 将

获取第二个 char 指针，所以现在你有了一个 char *。
获取该指针中的第二个元素，该元素将是一个字符。

所以现在您将一个 char 分配给一个 char 指针（这应该是一个编译错误）。

我只能假设您的意思是 char *a = &argv[1][1]。顺便说一句，const 正确性也很好，所以 const char *a = &argv[1][1]。

顺便说一句，你的代码仍然非常不安全 - 你甚至没有检查字符串的大小。想象一下，如果您的字符串只有两个字符，&argv[1][3] 会做什么。

回复收藏 0 原文

梅窗月明清似水 2024-10-07 11:53:53

#include 它应该变得显而易见。

详细说明：您将一个整数传递给需要指针的函数，并且编译器无法警告您，因为您忘记使用原型声明该函数。这就是事故发生的原因。

此外，您只是误用了atoi。 atoi 解析字符串，而不是单个字符。如果您希望字符的值为数字，只需减去'0'：

M = argv[1][1]-'0';
N = argv[1][3]-'0';

实际上，您还应该检查该字符实际上是否是数字。

编辑：我不记得原始帖子中存在char *a = argv[1][1];（也许早期的编辑不会显示为编辑？），但任何理智的编译器都应该在该行给出编译时错误。在 C 中，整数不会隐式转换为指针。如果编译器确实允许这种情况发生，那么包含 atoi 的原型将不再有帮助，因为之前发生了类型错误。

#include <stdlib.h> and it should become apparent.

To elaborate: you're passing an integer to a function which expects a pointer, and the compiler could not warn you because you forgot to declare the function with a prototype. This is the cause of the crash.

Moreover, you're simply misusing atoi. The atoi parses strings, not individual characters. If you want the value of a character as a digit, simply subtract '0':

M = argv[1][1]-'0';
N = argv[1][3]-'0';

In practice you should also check that the character is actually a digit.

Edit: I don't recall char *a = argv[1][1]; being in the original post (maybe early edits don't show up as edits?), but any sane compiler should give a compile-time error on that line. Integers do not implicitly convert to pointers in C. If the compiler does let this get by, then including a prototype for atoi will no longer help, since the type error occurred earlier.

回复收藏 0 原文