C錯誤：格式'％s'需要類型爲'char *'的參數，但參數2的類型爲'char（*）[100]'

Question

我正在做最後幾天的練習，有這個警告（如標題所示）。我嘗試了一堆東西，但我不知道如何完全解決這個問題。我不擅長編程，所以有錯誤。下面是我使用的結構（這不能被改變，因爲這是他們是如何給出）：

typedef struct bookR* book; 
struct bookR{ 
    char author[MAXSTRING]; 
    enum genres{fiction,scientific,politics}; 
    int id; 
    char review[MAXLINES][MAXSTRING]; 

}; 

typedef struct nodeR* node; 
struct nodeR{ 
    book b; 
    node next; 

}; 

typedef struct listR* list; 
struct listR{ 
    node head, tail; 
    int size; 
}; 

這裏是代碼的一部分發生問題：

void addBook(book b, list bList){ 
char author [MAXSTRING]; 
int id; 
char review [MAXSTRING][MAXLINES]; 
printf ("Give the author,`enter code here` id and review of the new book respectively"); 
scanf("%s",author); 
scanf("%d",&id); 
scanf("%s",review); 
node k=(node)malloc(sizeof(struct nodeR)); 
assert(k); 
k->next=NULL; 
strcpy(k->b->author,author); 
k->b->id=id; 
strcpy(k->b->review,review[MAXSTRING]);} 

這是我得到的警告：

warning: format '%s' expects argument of type 'char *' but argument 2 has type 'char (*)[100]' [-Wformat=] 
scanf("%s",review); 
warining:passing argument 1 of 'strcpy' from incompatible pointer tupe [-Wincompatible-pointer-types] 
strcpy(k->b->review,review[MAXSTRING]); 

任何幫助，非常感謝。感謝您的時間，併爲長篇文章感到遺憾。

Answer 1

• 警告一號

  爲了使用scanf，你需要一個指針傳遞給它。您已經聲明：

  char review [MAXSTRING][MAXLINES]; 
  

  ，而是你讀：

  scanf("%s",review); 
  

  您需要將此更改爲：

  scanf("%s", review[i]); 
  

  其中i爲0到MAXSTRING-1的索引。

• 警告NO2

  此外，聲明：

  strcpy(k->b->review,review[MAXSTRING]); 
  

  得到出界，爲您的陣列的位置到達review[MAXSTRING-1]。除此之外，您正在將一個行分配給整個數組。所以，你應該將其更改爲：

  strcpy(k->b->review[index], review[MAXSTRING-1]); 
  

---

還有兩個注意事項：

1. 看到這個link on why not to cast the result of malloc。

2. 請記住，在聲明中，例如：

   array[x][y]; 
   

   x指示線和y指明列。您以相反的方式使用它們，所以請確保您不會感到困惑，並且在需要列時需要行和列時才能訪問行。

Answer 2

第一警告

char review [MAXSTRING][MAXLINES]; 

它是一個矩陣，其可以被看作是C-串的你的情況的陣列。

每個C-string是review[index]其中index去從0到MAXSTRING-1

所以

scanf("%s",review) 

是錯誤的，因爲你必須通過一個C字符串的功能，那麼你必須寫：

scanf("%s",review[index]); 

我建議你將輸入字符串限制爲每個字符串的最大允許字符數MAXLINES-1使用，而不是scanf：

fgets(review[index], MAXLINES, stdin); 

2日警告

同樣的事情的struct bookRreview成員。 所以

strcpy(k->b->review,review[MAXSTRING]); 

必須

strcpy(k->b->review[index],review[MAXSTRING-1]); 

正如你可以看到有一個第二個問題到你的strcpy的呼叫：第二個參數滿足您的字符串數組越界，即調用Undefined Behavior。

其他警告

有一個警告的到你的代碼：

test.c:666:45: warning: declaration does not declare anything 
    enum genres{fiction,scientific,politics}; 
              ^

最後考慮

我猜你要定義切換到您的矩陣的定義，你做成struct bookR，如：

char review [MAXLINES][MAXSTRING]; 

我認爲可以通過特定的prinf和scanf/fgets來詢問每個單一數據的最佳選擇。

printf ("Give the author: "); 
fgets(author, MAXSTRING, stdin); 
printf ("Enter id: "); 
scanf("%d",&id); 
printf ("Enter review of the new book respectively: "); 
fgets(review[index], MAXSTRING, stdin);