我错误地使用了atoi吗？

Question

我的解析函数遇到了一些问题,所以我在运行时把一些cout语句告诉我某些变量的值,我相信atoi错误地转换了字符.

下面是我的代码的一小部分奇怪的行为:

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

该语句的输出是: 50 digit 0 = '5' number = 52

我在一个循环中调用这个代码,有什么奇怪的是它正确地转换了前47个字符,然后在第48个字符上它在整数后面加了0,在第49个字符上它加了1,在50号(见过它增加了两个,一直到第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接受一个字符串,即一个空终止的chars 数组,而不是一个单指针,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