從文件讀取到字符串數組，C

Question

我正在嘗試編寫一個 function ，它使用 fgets() 將文件中的字符串讀取到字符串數組中，但我不斷收到zsh: segmentation fault 。

這是我的代碼：

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

void read(FILE*, char[][31],int*,int);
void printAll(char[][31],int*);

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


    FILE *in = fopen("strings.dat", "r");
    char strarr[50][31];
    int items=0;

    read(in,strarr,&items,31);
    printAll(strarr,&items);

}

void read(FILE *in,char arr[][31],int *items,int len){
    int i = 0;
    char temp[31];
    while (fgets(temp, len, in) != NULL) {
        strcpy(arr[i], temp);
        i++;
        items++;
    }
}
void printAll(char arr[][31],int *items){
    for(int i = 0;i<*items;i++){
        printf("String %d: %s\n",i+1,arr[i]);
    }
}

我該如何解決？

Answer 1

您的代碼幾乎是正確的。

錯誤：

1. 在read function items++是錯誤的，這會增加指針但您想增加指針指向的變量，因此您需要(*items)++代替。

2. 根據平台read將與 posix read function 沖突。

---

該程序將在以下情況下失敗：

1. 如果文件strings.dat不存在in NULL並對 NULL FILE指針執行文件操作會導致未定義的行為，這通常表現為段錯誤。 這可能就是發生在你身上的事情。

2. 如果文件包含超過 50 行，您將溢出strarr這也會導致未定義的行為

3. 如果文件中的行超過 31 個字符，行將被截斷和/或拆分。

---

獎金：

temp 變量不是必需的，您可以直接讀入數組：

而不是這個：

 char temp[31];
 while (fgets(temp, len, in) != NULL) {
    strcpy(arr[i], temp);
    ...

你可以有這個：

 while (fgets(arr[i], len, in) != NULL) {
    ...

---

帶有更正的最終代碼：

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

void read(FILE*, char[][31], int*, int);
void printAll(char[][31], int*);

int main(int argc, char** argv) {
  FILE* in = fopen("strings.dat", "r");
  if (in == NULL)
  {
    printf("file does not exist\n");
    exit(1);
  }
  char strarr[50][31];
  int items = 0;

  read(in, strarr, &items, 31);
  printAll(strarr, &items);

}

void read(FILE* in, char arr[][31], int* items, int len) {
  int i = 0;
  while (fgets(arr[i], len, in) != NULL) {
    i++;
    (*items)++;
  }
}

void printAll(char arr[][31], int* items) {
  for (int i = 0; i < *items; i++) {
    printf("String %d: %s\n", i + 1, arr[i]);
  }
}

---

read的替代版本

void read(FILE* in, char arr[][31], int* items, int len) {
  int i = 0;
  while (fgets(arr[i++], len, in) != NULL) {
    (*items)++;
  }
}

更短：

void read(FILE* in, char arr[][31], int* items, int len) {
  for (int i = 0; fgets(arr[i], len, in) != NULL; i++)
    (*items)++;
}