質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.42%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

Q&A

解決済

2回答

19303閲覧

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

Cchan

総合スコア35

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

0グッド

0クリップ

投稿2018/10/18 08:49

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

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

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

C

1#include<stdio.h> 2#include<stdlib.h> 3#include<string.h> 4 5typedef struct test 6{ 7 //int count; 8 char tmpbuf[100]; 9 struct test* next; 10}tests; 11 12 13 14int main(){ 15tests *tmp = NULL; 16tests *head=NULL; 17tests*tail=NULL; 18tmp=malloc(sizeof(tests)); 19 20head=tail=tmp; 21tmp->next=malloc(sizeof(tests)); 22 23tmp=tmp->next; 24tmp->next=malloc(sizeof(tests)); 25 26tail=tmp->next; 27tail->next=head; 28 29head=tmp; 30char ch; 31int count = 0; 32char t[100]; 33while ((ch=fgetc(stdin)) != EOF){ 34 35t[count]=ch; 36 37 38 if(ch=='\n'){ 39 strcpy(t,tmp->next); //エラー箇所 40 tmp=tmp->next; 41 count=0; 42 } 43count+=1; 44} 45 46for (int i=0;i<=3;i++){ 47printf("%s",tmp->tmpbuf); 48 49tmp=tmp->next; 50 51} 52 53return 0; 54}

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

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

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


strcpyに渡された第1引数が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を使うのは不適切です。

投稿2018/10/18 09:12

編集2018/10/18 12:04
asm

総合スコア15149

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Cchan

2018/10/23 19:57

ありがとうございます!
guest

0

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

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

投稿2018/10/18 08:59

maisumakun

総合スコア145531

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.42%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問