我是否错误地使用了atoi？

Question

我的解析函数遇到了一些问题，因此我放置了一些 cout 语句来告诉我运行时某些变量的值，并且我相信 atoi 错误地转换了字符。

这是我的代码的一小段，其行为很奇怪：

c = data_file.get();
if (data_index == 50)
    cout << "50 digit 0 = '" << c << "' number = " << atoi(&c) << endl;

此语句的输出是： 50 digital 0 = '5' number = 52

我在循环中调用此代码，奇怪的是它正确转换了前 47 个字符，然后在第 48 个字符后添加了 0整数，在第 49 个字符上添加 1，在第 50 个字符（见此处）添加 2，一直到第 57 个字符添加 9，然后继续正确转换一直到第239个字符。

这很奇怪还是什么？

只是为了澄清一点，我将发布整个功能。该函数传递一个指向空双精度数组 (ping_data) 的指针：

int parse_ping_data(double* ping_data)
{
    ifstream data_file(DATA_FILE);

    int pulled_digits [4];
    int add_data;
    int loop_count;
    int data_index = 0;

    for (char c = data_file.get(); !data_file.eof(); c = data_file.get())
    {
        if (c == 't' && data_file.get() == 'i' && data_file.get() == 'm' && data_file.get() == 'e' && data_file.get() == '=')
        {
            loop_count = 0;
            c = data_file.get();
            if (data_index == 50)
                    cout << "50 digit 0 = '" << c << "' number = " << atoi(&c) << endl;
            pulled_digits[loop_count] = atoi(&c);

            while ((c = data_file.get()) != 'm')
            {
                loop_count++;
                if (data_index == 50)
                    cout << "50 digit " << loop_count << " = '" << c << "' number = " << atoi(&c) << endl;
                pulled_digits[loop_count] = atoi(&c);
            }
            add_data = 0;
            for (int i = 0; i <= loop_count; i++)
                add_data += pulled_digits[loop_count - i] * (int)pow(10.0,i);

            if (data_index == 50)
                cout << "50 index = " << add_data << endl;
            ping_data[data_index] = add_data;
            data_index++;

            if (data_index >= MAX_PING_DATA)
            {
                cout << "Error parsing data. Exceeded maximum allocated memory for ping data." << endl;
                return MAX_PING_DATA;
            }   
        }
    }

    data_file.close();

    return data_index;
}

Answer 1

atoi 接受一个字符串，即一个以 null 结尾的 char 数组，而不是指向单个 char 的指针，所以这是不正确的并且会让你不可预测的结果。

char c;
//...
/* ... */ atoi(&c) /* ... */

此外，atoi 不提供任何检测错误的方法，因此更喜欢 strtol 和类似的函数。

例如

char *endptr;
char c[2] = {0}; // initalize c to all zero

c[0] = data.file.get(); // c[1] is the null terminator

long l = strtol(c, &endptr, 10);

if (endptr == c)
    // an error occured

Answer 2

atoi 需要一个以 null 结尾的字符串作为输入。您提供的不是以空结尾的字符串。

话虽如此，值得补充的是，正确使用 atoi 是非常困难的（如果可能的话）。 atoi 是一个不提供错误控制和溢出控制的函数。在 C 标准库中执行字符串表示形式到数字转换的唯一正确方法是来自 strto... 组的函数。

实际上，如果您只需要转换单个字符数字，那么使用atoi或任何其他字符串转换函数都是一种奇怪的矫枉过正。正如已经建议的那样，您只需从字符数字值中减去 0 值即可得到相应的数值。语言规范保证这是一个可移植的解决方案。

Answer 3

没关系，我只是需要将字符转换为以 \0 结尾的字符串。我将其更改为以下代码：

char buffer [2];

缓冲区[1] = '\0';

缓冲区[0] = data_file.get();

if (data_index == 50)

cout << "50 digit 0 = '" << buffer[0] << "' number = " << atoi(buffer) << endl;

并且它起作用了。