Назначение значений из массива массиву структур

Question

Я пытаюсь назначить значения из массива, содержащего все строки, в массив структур, где некоторые из членов структуры являются целыми числами. Однако, как я пытался это сделать, это вызывает некоторое неопределенное поведение. Как ниже код генерирует следующее:

0 surname ▒▒ 
1 forename & 
2 id 0 

, когда он должен сказать

0 surname Boatswain 
1 forename Michael Jr 
2 id 109993267 

Я не совсем уверен, что случилось с тем, как я присвоенным этими значениями, хотя.

Код:

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

//There are 7 values for each student 
#define VALUES 7 

struct Students{ 
    int term; 
    int id; // NEED TO GENERATE ERROR IF WRONG NUM OF CHARACTERS 
    char surname[15]; 
    char forename[15]; 
    char subject[3]; 
    int catnum; 
    char section[3]; 
}; 

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

//Get contents of input file 
    unsigned char filename; 
    printf("Enter the input file name: \n"); 
    scanf("%s", &filename); 
    FILE *file = fopen(&filename, "r"); 
    int filesize=0; 
    fseek(file, 0L, SEEK_END); 
    filesize = ftell(file); 
    fseek(file, 0L, SEEK_SET); 
    char *contents = malloc(filesize+1); 
    size_t size=fread(contents,1,filesize,file); 
    contents[size]=0; // what does all this do exactly? 

//Get number of lines in input file 
    int total_line = 0; 
    const char *str; 
    for(str = contents; *str; ++str) 
    total_line += *str == '\n'; 

//Tokenize string 
    int n=0,nn; 
    char *b[VALUES*total_line]; 
    char *ds=strdup(contents); 
    b[n]=strtok(ds, ",=\"\r\n\""); 
    while(b[n] && n<((VALUES*total_line)-1)) b[++n]=strtok(NULL, ",=\"\r\n\""); 
    //for(nn=0; nn<=n; ++nn) printf("%s %d ", b[nn], nn); 
    //putchar('\n'); 
    free(ds); 

    struct Students record[total_line]; 

    int i, j; 
    for(i=0;i++;i<total_line){ 
    for(j=0;j++;j<VALUES){ 
     int n=(7*i)+j; 
     record[i].term=atoi(b[n]); 
     record[i].id=atoi(b[n]); 
     strcpy(record[i].surname, b[n]); 
     strcpy(record[i].forename, b[n]); 
     strcpy(record[i].subject, b[n]); 
     record[i].catnum=atoi(b[n]); 
     strcpy(record[i].section, b[n]); 
    } 
    } 

// try printing some values here 
    printf("0 surname %s\n",record[0].surname); 
    printf("1 forename %s\n",record[1].forename); 
    printf("2 id %d\n", record[2].id); 

    free(contents); 
    return 0; 
} 

входного файла:

1301,107515018,"Boatswain","Michael R.",CSE, 230,="R01" 
1301,109993269,"Castille","Michael Jr",CSE, 230,="R03" 
1301,109993267,"Castille","Janice",CSE, 230,="R03" 

Заранее спасибо за помощь!

Редактировать: В чем проблема при изменении цикла for так?

for(i=0;i<total_line;i++) { 
    record[i].term=atoi(b[(7*i)]); 
    record[i].id=atoi(b[(7*i)+1]); 
    strcpy(record[i].surname, b[(7*i)+2]); 
    strcpy(record[i].forename, b[(7*i)+3]); 
    strcpy(record[i].subject, b[(7*i)+4]); 
    record[i].catnum=atoi(b[(7*i)+5]); 
    strcpy(record[i].section, b[(7*i)+6]); 
} 

Редактировать: Я получил его на работу. Просто нужно изменить цикл for-while на цикл while. Edit2: ohh У меня были i ++ и i total_line. Derp. теперь исправлено. Ну, я думаю, это решает эту тайну.

Answer 1

В вашей петле for(j=0;j++;j<VALUES) вы конвертируете то же самое b[n] несколько раз. Скорее всего, вам не нужен этот цикл, но вместо этого нужно преобразовать b[n++].

Answer 2

Предварительно, эта линия (которая, по-видимому вы уже нашли подозрительный) ...

contents[size]=0; // what does all this do exactly? 

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

@DavetheSax достоверно следит за индексами ваших жетонов. Эта проблема не вызывает ошибки , но полученное поведение, конечно же, не то, что вы хотели.

Однако у вас действительно есть неопределенное поведение в вашей программе. Вы заполняете массив b путем обмана contents на ds, а затем на токенизацию ds с strtok(). Это само по себе хорошо, но необходимо, только если вам нужно сохранить contents, поскольку он был первоначально прочитан, чего вы, вероятно, не знаете. Однако после токенизации вы освобождаете ds. Это, опять же, само по себе не проблема, но это делает недействительными все указатели маркеров, которые вы только что вычислили, потому что они указывают на ранее выделенную память для ds (из которой они были маркированы). Вам нужно сохранить это до тех пор, пока вы не проанализируете токены. Выделение этих указателей токенов после освобождения пространства, в которое они указывают, создает UB.

Как правило, нельзя прогнозировать UB (иначе это не будет UB). Тем не менее, конкретные результаты, о которых вы сообщаете, кажутся маловероятным проявлением, поэтому, возможно, здесь все больше происходит.