単方向リスト

質問

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
struct Address{
char name[100];
char tel[100];
char email[100];
};
struct AddressList {
struct Address addr; //データそのもの
struct AddressList *next; //後続ノードへのポインタ
struct AddressList *prev;

};

struct AddressList *this,*last,*new,*first;
struct Address *addr;

/*--------ここからmain関数------------*/

int main(void){

int n,i; //何番目に入れるか？
char quit[100]; //繰り返しを止める時に利用。

first=last=NULL; //まず初期化
addr =(struct Address *)malloc(sizeof(struct Address));
if(addr==NULL){
fprintf(stderr,"malloc:error\n");
exit(1);
}
while(-1){
printf("整数を入力してください。\n");
scanf("%d",&n);
printf("名前を入力してください。\n");
scanf("%s",addr->name);
printf("電話番号を入力してください。\n");
scanf("%s",addr->tel);
printf("メールアドレスを入力してください。\n");
scanf("%s",addr->email);

new =(struct AddressList *)malloc(sizeof(struct AddressList)); //新たに加えるリストの動的メモリー確保
if(new==NULL){
fprintf(stderr,"malloc:error\n");
exit(1);
}
new->addr=addr; //ここでエラーが発生します。
new->next=NULL;

incompatible types in assignment
とエラーがでます。
型はあってると思うんですが、、、
よろしくおねがいします。

回答No.3

e_watt · ベストアンサー

new->addr = *addr; // 構造体のコピー
とすれば動作するはずですが、

addr を廃止して while(-1){ の次の行でいきなり new を確保し、
scanf("%s", new->addr.name);
のように直接書いちゃったほうが見通しがいいかもしれません。

# ついでに、new　という変数名は C では問題ないのですが、
# C++ では予約語なので C++ を知っている人が見ると
# ちょっと違和感があったりします。

お礼

ありがとうございます。
できました。
そういう方法もあるんですね。
参考になりました。

回答No.4

e_watt

No.3です。ちょっと補足。

No.2さんの回答のデータ構造に変更する場合は、
addr をループの中で毎回確保するように変更しないと
リストに加える全要素が同一の要素を指すことになるのでご注意を。

回答No.2

mac_res

struct AddressList {
struct Address *addr; //データそのもの
struct AddressList *next; //後続ノードへのポインタ
struct AddressList *prev;
};
のあやまりでしょうか？

お礼

型あってませんでしたね。。
ありがとうございました。

回答No.1

jacta

> 型はあってると思うんですが、、、

あっていません。

お礼

ありがとうございました。