Чтобы прочитать содержимое файла в C

Question

У меня есть код C, чтобы прочитать текстовый файл:

#include <stdio.h> 

int main() 
{  
    FILE *pf; 
    int ii; 
    int jj; 

    char *filename; 
    printf("enter file name"); 
    scanf("%s",filename);   

    printf("%s",filename); 

    pf = fopen("filename+.txt", "r"); 
    if(pf==Null) 
    { 
     printf("cant open"); 
    } 
    else 
    { 
     fscanf(pf,"%d,%d" ,&ii,&jj); 
     printf("%d,%d\n" ,ii,jj); 
    } 
    fclose(pf);  

    return 0; 
} 

Тем не менее я получаю ошибку сегментации. Входной текстовый файл содержит

2,3

я получаю ошибки сегментации (ядро сбрасывали), когда я запустить программу как ./readfile вход.
Что здесь не так, как я могу исправить это?

Answer 1

Здесь много ошибок.

После звонка fopen(), вы должны проверить, pf is NULL, потому что fopen() может потерпеть неудачу.

Вы пытаетесь открыть файл с именем filename+.txt. Разве вы не должны открывать файл, который был указан в качестве параметра?

Кроме того, структура main() должна быть int main(int argc, char **argv), вы не можете делать все, что хотите.

Проверьте, есть ли argc > 1, и в этом случае программа была запущена с параметрами, а имя файла должно быть указано в argv[1].

Update на комментарии: Это как ваш код должен выглядеть следующим образом:

int main() 
{ 
    char filename[512]; // reserve 512 bytes to receive the file name from input 
    FILE *pf; 
    int ii; 
    int jj; 

    printf("Enter file name: "); 
    scanf("%s", filename); 

    pf = fopen(filename, "r"); 

    if (pf) 
    { 
     fscanf(pf,"%d,%d", &ii, &jj); 
     printf("%d,%d\n", ii, jj); 
     fclose(pf); 
    } 
    else 
    { 
     printf("Failed to open file name %s", filename); 
    } 

    return 0; 
} 

Вы также можете сделать это, чтобы получить имя файла из параметров:

int main(int argc, char **argv) 
{ 
    FILE *pf; 
    int ii; 
    int jj; 

    if (argc > 1) 
    { 
     pf = fopen(argv[1], "r"); 
     if (pf) 
     { 
      fscanf(pf, "%d,%d", &ii, &jj); 
      printf("%d,%d\n", ii, jj); 
      fclose(pf); 
     } 
     else 
     { 
      printf("Failed to open file name %s", argv[1]); 
     } 
    } 
    else 
    { 
     printf("Insuficient parameters"); 
    } 

    return 0; 
} 

Или даже, если вы не «т хочет передать расширение файла:

int main(int argc, char **argv) 
{ 
    char *filename; 
    FILE *pf; 
    int ii; 
    int jj; 

    if (argc > 1) 
    { 
     filename = malloc(strlen(argv[1]) + 5); // alloc necessary memory 
     strcpy(filename, argv[1]); 
     strcat(filename, ".txt"); 

     pf = fopen(filename, "r"); 
     if (pf) 
     { 
      fscanf(pf, "%d,%d", &ii, &jj); 
      printf("%d,%d\n", ii, jj); 
      fclose(pf); 
     } 
     else 
     { 
      printf("Failed to open file name %s", filename); 
     } 
    } 
    else 
    { 
     printf("Insuficient parameters"); 
    } 

    return 0; 
} 

Answer 2

int main(char *) не является юридической сигнатурой main в C. Только

int main(void) 

и

int main(int argc, char **argv) 

законны. В вашем случае вам понадобится последний.

Answer 3

Это неправильный способ указать аргументы в вашей программе. т.е. вы не можете сделать это:

int main(char *filename) 

Там должно быть ошибка компилятора при компиляции программы. Правильное определение:

int main(int argc, char **argv) 

Где argv массив строк. Попробуйте сделать этот эксперимент:

int main(int argc, char **argv) 
{ 
    int i; 
    for(i = 0; i < argc; i++) { 
     printf("arg %d is: \"%s\"\n", argv[i]); 
    } 
    return 0; 
} 

Затем напишите вашу программу, чтобы использовать список правильных аргументов, как указано выше.

Еще один момент заключается в том, что вы должны проверить возвращаемое значение fopen. Если это NULL, вам не следует пытаться получить доступ к файлу (потому что он не открывается).