Срочный счетчик сбрасывается неожиданно

Question

Приведенный ниже код должен взять данные из строки s и разбить их на данные двойного массива с пробелом в качестве разделителя. J счетчика неожиданно сбрасывает, когда он должен увеличить, чтобы завершить функцию

полукокса * S, передается в функции

0.0000000E00  0.0000000E00  -1.9311798E+03  8.0321814E+02  8.0294336E+02 

диагностическая функция Printf ниже отпечатков:

0.000000 | 1 
0.000000 | 2 
-1931.179800 | 3 
803.218140 | 4 
802.943360 | 1 

It приводит к сбою программы

void split_data(char *s, double *data, int fields) { 
    char buff[DATA_MAX]; 
    int j = 0, i; 

    for(; *s; *s++) { 
    while(*s == ' ' || *s == '\t') /* trim leading white space */ 
     *s++; 

    i = 0; 
    while(!(*s == ' ' || *s == '\t')) 
     buff[i++] = *s++; 
    buff[i] = 0; 

    data[j++] = atof(buff); 
    printf("%lf | %d\n", data[j-1], j); 

    if(j == fields) 
     return; 
    } 
} 

Answer 1

Ваш цикл:

while(!(*s == ' ' || *s == '\t')) 
     buff[i++] = *s++; 

должен также содержать тест на конец строки s. Например, while(!(*s == ' ' || *s == '\t') && *s) ....

В противном случае buff будет по-прежнему заполняться «шумом» до тех пор, пока он не переполнится. И тогда другие переменные в стеке начнут сбиваться, например j. Но поведение очень зависит от компилятора.

Answer 2

Возможно, вы сделали buff слишком короткий, и поэтому ваша программа перекрывает конец буфера (возможно, только одним символом). Это заставит его перезаписать следующую переменную стека, которая, вероятно, j.

Попробуйте увеличить DATA_MAX и посмотреть, не решена ли эта проблема.

Answer 3

Если fields больше, чем количество строк в вашем массиве символов, программа может потерпеть крах, так как ваше условие выхода основано только на сравнении j с fields. Можете ли вы попробовать со следующим изменением кода?

if((j == fields) || (*s == '\0')) 
    return; 

Answer 4

Проблема ваша вторая петля:

while(!(*s == ' ' || *s == '\t')) 
    buff[i++] = *s++; 
buff[i] = 0; 

Вы продолжаете заполнять буфер, если вы не достигнете «» из «\ г», к сожалению, строка окончания символа «\ 0» не является ни один из них, поэтому вы все равно продолжаете заполняться после достижения конца строки. В случае, если вам повезло, память, которую вы читаете, содержит любые «'' \ t ', и ваш цикл остановится, пока ваш буфер не будет заполнен.

Чтобы решить вашу проблему, вы также должны проверить '\ 0' в своем втором цикле - даже лучше, если вы сделаете это в первом (на случай, если для анализа не хватает значений) или добавьте дополнительный ' или '\ t' до конца строки.