C言語についてです（構造体と標準入力について）

Question

循環リストを３つ作り、与えられた標準入力から一行ずつ構造体メンバに代入、更新して回したいのですが、

warning: passing argument 1 of ‘strcpy’ from incompatible pointer type [-Wincompatible-pointer-types] strcpy(tmp->next,t);

というエラーの直し方と意味がよくわかりません。

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

typedef struct test
{
    //int count;
    char tmpbuf[100];
    struct test* next;
}tests;



int main(){
tests *tmp = NULL;
tests *head=NULL;
tests*tail=NULL;
tmp=malloc(sizeof(tests));

head=tail=tmp;
tmp->next=malloc(sizeof(tests));

tmp=tmp->next;
tmp->next=malloc(sizeof(tests));

tail=tmp->next;
tail->next=head;

head=tmp;
char ch;
int count = 0;
char t[100];
while ((ch=fgetc(stdin)) != EOF){

t[count]=ch;


    if(ch=='\n'){
    strcpy(t,tmp->next); //エラー箇所
    tmp=tmp->next; 
    count=0;
    }
count+=1;
}

for (int i=0;i<=3;i++){
printf("%s",tmp->tmpbuf);

tmp=tmp->next;

}

return 0;
}

よろしくお願いいたします。

Answer 1

warning: passing argument 1 of ‘strcpy’ from incompatible pointer type [-Wincompatible-pointer-types] strcpy(tmp->next,t);は警告であってエラーではありません。

---

strcpyに渡された第１引数がstrcpyの引数とは非互換とのことなので、
strcpyの定義を確認するとchar *strcpy(char *s1, const char *s2);です。
渡しているtmp->nextは構造体へのポインタなのでchar*ではありません。
渡すべきはtmp->next->tmpbufとなります。

---

なお、偶然でしょうけど、C言語としては間違ってはいません。

構造体へのポインタはC言語の場合私の知る限りの処理系では(N1570 §6.7.2.1p15で保証されてます)先頭要素へのポインタと同値です。
そして、C言語の場合ポインタは暗黙の変換を規格上認めています。

ただ、tmpbufが先頭要素でなくなった途端によく分からない事がおきるので、明示的にtmpbufを指定した方がよいでしょう。

ついでに
tが\0で終わる文字列であることが担保されていませんので
その状態でstrcpyを使うのは不適切です。

Comments

[Cchan]

ありがとうございます！

Answer 2

英語を訳せば、答えが書いてあります。

tmp->nextはstruct test *であって、char *ではありません。