Ожидаемый инициализатор перед именем функции в заголовке. C

Question

Я получаю «ожидаемый инициализатор до« read_file »как ошибку. Ошибка находится в строке« код команды [] read_file (код команды []) ». Его строка В поисках веб-страницы для справки, все im нахождение - это связанный с C++ пост, поэтому, чтобы это объяснить, для C.

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

Спасибо за помощь.

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

typedef struct instruction{ 
    int op; //opcode 
    int l; // L 
    int m; // M 
} instr; 

FILE * ifp; //input file pointer 
FILE * ofp; //output file pointer 
instruction code[501]; 

instruction code[] read_file(instruction code[]); 
char* lookup_OP(int OP); 
void print_program(instruction code[]); 
void print_input_list(instruction code[]); 


int main(){ 

    code = read_file(code); 
    print_input_list(code);//used for debugging 
    print_program(code); 
} 

instruction code[] read_file(instruction code[]){ 
    int i = 0; 

    ifp = fopen("input.txt", "r"); 

    while(!feof(ifp)){ 
     fscanf(ifp,"%d%d%d",&code[i]->op, &code[i]->l, &code[i]->m); 
     i++; 
    } 
    code[i]->op = -1; //identifies the end of the code in the array 
    fclose(ifp); 
    return code; 
} 

Answer 1

instruction code[] read_file(instruction code[]) не является юридическим синтаксисом. Вы не можете вернуть массив из функции. Кроме того, глобальный code представляет собой массив. Таким образом, это назначение также является незаконным - вам придется исправить оба места.

code = read_file(code); 

Что это выглядит, как вы хотите, это просто:

void read_file(instruction code[]) 

А просто назвать это нравится:

read_file(code); 

Нет необходимости назначение.

Фактически теперь, когда я читаю еще несколько, так как code является глобальным, вам не нужны параметры вообще.

Answer 2

Попробуйте функцию, возвращающую calloc -ed (см. calloc(3) man page) указатель instr.

Так

instr* read_file(const char*filename) 
{ 
    instr* arr=NULL; 
    int len=0, size=0; 
    FILE* f= fopen(filename, "r"); 
    if (!f) { perror(filename); exit(EXIT_FAILURE); }; 
    while (!feof (f)) { 
    if (len>=size-1) { 
     int newsize = 5*len/4+50; 
     instr* newarr = calloc(newsize, sizeof(instr)); 
     if (!newarr) { perror("calloc"); exit(EXIT_FAILURE); }; 
     if (arr) memcpy (newarr, arr, sizeof(instr)*len); 
     free (arr); 
     arr = newarr; 
     size = newsize; 
    }; 
    if (fscanf(f, "%d %d %d", 
       &arr[len]->op, &arr[len]->l, &arr[len]->m)<3) 
     break; 
    len++; 
} 
arr[len]->op = -1; // end of array marker 
fclose(f); 
return arr; 
} 

^{выше функция читает кучу выделенный массив instr [в] указатель arr и перераспределить его по мере необходимости.}

Не забудьте указать free результат read_file в конце вашей программы.

С вышеуказанным кодом вы можете прочитать много instr (возможно, миллионы на среднем ПК и более 500). Затем вы укажете

int main() { 
    instr* code = read_file("input.txt"); 
    print_program(code); 
    // at the very end of main 
    free(code); 
} 

Answer 3

Это скорректированный код.

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


typedef struct instruction{ 
int op; //opcode 
int l; // L 
int m; // M 
} instruction; 

FILE * ifp; //input file pointer 
FILE * ofp; //output file pointer 
instruction code[501]; 

instruction * read_file(instruction code[]); 
char* lookup_OP(int OP); 
void print_program(instruction code[]); 
void print_input_list(instruction code[]); 


int main(){ 

instruction * code = read_file(code); 
print_input_list(code);//used for debugging 
print_program(code); 

} 

instruction * read_file(instruction code[]){ 
int i = 0; 

ifp = fopen("input.txt", "r"); 


while(!feof(ifp)){ 
    fscanf(ifp,"%d%d%d",&code[i].op, &code[i].l, &code[i].m); 
    i++; 
} 
code[i].op = -1; //identifies the end of the code in the array 
fclose(ifp); 
return code; 
}