線形リストの１つのノードに文字列を格納したいです。

Question

前提・実現したいこと

複数行に渡る単語の入力において
各単語の現れる回数を知るために線形リストを用いて新しい単語の入力であればノードを追加し、リストにあればcount+1とするプログラムを作りたいのですが、コンパイルしたところ
線形リストのノード挿入時において, ある文字列をそのノードのメンバに代入したいのですがうまくいかず警告文も理解できないため。
ご教示願います。

発生している問題・エラーメッセージ

reduce.c: 関数 ‘insert’ 内:
reduce.c:14:3: 警告: 2 番目の ‘strcpy’ の引数へ渡すときに整数からキャスト無しにポインタを作成しています [デフォルトで有効]
   strcpy(n->element, e);
   ^
In file included from reduce.c:2:0:
/usr/include/string.h:125:14: 備考: expected ‘const char * __restrict__’ but argument is of type ‘elementtype’
 extern char *strcpy (char *__restrict __dest, const char *__restrict __src)
              ^
reduce.c: 関数 ‘main’ 内:
reduce.c:43:5: 警告: passing argument 2 番目の ‘insert’ の引数を渡すときにポインタからキャスト無しに整数を作成しています [デフォルトで有効]
     if(l->next == NULL) { insert(l, buf); }
     ^
reduce.c:11:6: 備考: expected ‘elementtype’ but argument is of type ‘char *’
 void insert(list l, elementtype e) {
      ^

該当のソースコード

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

char buf[128];
typedef char elementtype;

struct node { int count; elementtype element[128]; struct node *next; };
typedef struct node* list;

void insert(list l, elementtype e) {
  list n;
  n = (struct node *) malloc (sizeof (struct node));
  strcpy(n->element, e);
  n->count = 0;
  n->next = l->next;
  l->next = n;
}

void print_list(list l) {
  l = l->next;
  while(l != NULL) {
    printf("%s %d\n",l->element, l->count);
    l = l->next;
  }
  printf("\n");
}

int main(void) {
  list l, head;
  //ダミーノードを使います                                                 
  l = (struct node *)malloc(sizeof(struct node));
  l->next = NULL;
  head = l;

  while(fgets(buf,sizeof(buf),stdin) !=NULL) {
    while(l->next != NULL) {
      //同じ文字列を持つリストがある場合count+1                            
      if(strcmp(l->element, buf)==0) { l->count++; break;}
      else { l = l->next; }
    }
    //もし同じ文字列を持つリストが存在しなかった場合                       
    if(l->next == NULL) { insert(l, buf); }
    l = head;
  }
 print_list(head);
 return 0;
}

###入出力例
入力　　
ねこ　　　　
ぶた
たぬき
ねこ
いぬ
ねこ
いぬ

出力
ねこ　3
ぶた　1
たぬき　1
いぬ　2

補足情報（FW/ツールのバージョンなど）

gccコンパイラ

Answer 1

警告文も理解できない

引数で定義している型と、実際の値の型が異なるために発生している警告です。

void insert(list l, elementtype e) {

insert関数では「list型（node型のポインタ）」と「elementtype型（char型）」を
引数として受け取るように定義しています。

しかし、実際のinsert関数の呼び出しを見てみると、

if(l->next == NULL) { insert(l, buf); }

と書いてあり、それぞれの宣言部を見ると、
lは「list型」で、bufは「char*型」となっています。

※char buf[128]と宣言したときのbufは&buf[0]と同義になるので、
char型のアドレスを意味するchar*型になります。

解決策として、型を正しい方に合わせましょう。

Comments

[apeirogon0813]

ありがとうございます解決しました！