C - читать строку из stdin с функцией fgets() в функции

Question

Я пытаюсь прочитать строку из stdin с помощью fgets(), я хочу использовать функцию fgets() в моей функции, которая, по моему мнению, является проблемой. Строка может быть не более 1024 символов. Когда я запускаю этот код я получаю «ошибка сегментации (ядро сбрасывали)»

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

#define MAX_SIZE 1025 

void print_fgets(); 

int main() 
{ 
    print_select(); 
    return 0; 
} 

void print_select() 
{ 
    char *str; 
    int length; 

    while (fgets(str, MAX_SIZE, stdin)!=NULL) 
    { 
     length=strlen(str); 

     if (length==MAX_SIZE-1 && str[length-1]!='\n') 
     { 
      printf("Error, line overeached buffer!\n"); 
      return 1; 
     } 

     if (str[length-1]=='\n') 
      str[length-1]='\0'; 
     printf("%s\n", str); 
    } 
} 

Answer 1

Проблема заключается в том, что вы пытаетесь записать на место, что str указатель указывает на. Первоначально он укажет на некоторый адрес мусора (из-за char *str не инициализируется).

Вы можете попробовать стек на основе решения, а не за счет изменения:

/* This is just a pointer */ 
char *str; 

к:

/* This is a character array (on the stack) of size MAX_SIZE, initialized to zero */ 
char str[MAX_SIZE] = {0}; 

Или, если вы хотите выделить память динамически для массива, сделать это вместо:

char *str; 
str = calloc(1, MAX_SIZE); 
if (str == NULL) { 
    /* Failed to allocate memory */ 
} 

... Your code 

free(str); /* You should always free dynamically allocated memory when you are done with it */ 
str = NULL; /* It's a good practice to point your pointer to NULL after it's free'd */ 

Не забывайте такие вещи, как индекс массива, начиная с 0 и переходит в MAX_SIZE - 1 (в вашем случае) и завершение NUL (строки должны быть завершены).

Answer 2

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

#define MAX_SIZE 1025 

int print_select(); /* Use correct name (instead of print_fgets()) */ 

int main() 
{ 
    print_select(); 
    return 0; 
} 

int print_select() /* Fix. Dhould return int if you have a return <int> statement. */ 
{ 
    char str[MAX_SIZE]; /* Initialize static memory. */ 
    int length; 

    while (fgets(str, MAX_SIZE, stdin)!=NULL) 
    { 
     length=strlen(str); 
     if (length==MAX_SIZE-1 && str[length-1]!='\n') 
     { 
      printf("Error, line overeached buffer!\n"); 
      return 1; 
     } 
     if (str[length-1]=='\n') 
     { 
      str[length-1]='\0'; 
     } 
     printf("%s\n", str); 
    } 
    return 0; /* Function may not be returning an int. Return it in those cases. */ 
}