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. */ 
}