C言語でエラーの意味が分かりません。

Question

前提

C言語で寿司打を作っています。
エラーの意味がよくわかりません。
お願いします。

実現したいこと

エラーの解消

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

natu_g21288.c:350:17: warning: passing argument 1 of ‘word’ from incompatible pointer type [-Wincompatible-pointer-types]
  350 |  word(word_kanji[100], word_romaji[100]);
      |       ~~~~~~~~~~^~~~~
      |                 |
      |                 char *
natu_g21288.c:140:18: note: expected ‘char **’ but argument is of type ‘char *’
  140 | char *word(char *word_kanji[100], char *word_romaji[100])
      |            ~~~~~~^~~~~~~~~~~~~~~
natu_g21288.c:350:35: warning: passing argument 2 of ‘word’ from incompatible pointer type [-Wincompatible-pointer-types]
  350 |  word(word_kanji[100], word_romaji[100]);
      |                        ~~~~~~~~~~~^~~~~
      |                                   |
      |                                   char *
natu_g21288.c:140:41: note: expected ‘char **’ but argument is of type ‘char *’
  140 | char *word(char *word_kanji[100], char *word_romaji[100])
      |                                   ~~~~~~^~~~~~~~~~~~~~~~
akira0809@ubuntu:~/js1$ 

該当のソースコード

int sushida()
{
        int enter, index;
        char *word_kanji[100], *word_romaji[100], *word_play[100];
        word(word_kanji[100], word_romaji[100]);
        printf("問題数を選択してください\n");
        printf("10 20 30 40 50\n");
        scanf("%d", &enter);
        printf("Enterを押すと始まります\n");
        getchar();
        getchar();
        for(int i = 3; i > 0 ; i--)
        {
                printf("%d\n", i);　//カウント
                sleep(1);
        }
        printf("START\n");
        printf("\n");
        for(int i = 0; i < enter; i++)
        {
                srand((unsigned int)time(NULL));
                index = rand() % 99 + 0;
                printf("%s\n", word_kanji[0]);
                printf("%s\n", word_romaji[0]);
                //printf("%s\n", word_kanji[index]);
                //printf("%s\n", word_romaji[index]);
                //scanf("%s", &word_play[i]);
        }
        return 0;
}

char *word(char *word_kanji[100], char *word_romaji[100])
{
        word_kanji[0] = "漢字";
        word_romaji[0] = "kannzi";
                     :
                     :
        word_kanji[99] = "こんにちは";
        word_romaji[99] = "konnnitiha";
}

Answer 1

まず、「エラー/error」と「警告/warning」の違いはしっかり認識してください。
コンパイル時のエラーは、文法的に誤りで、それ以上プログラムの動作が不可能になるものです。
一方、警告は、Cの文法上間違いではないのでコンパイルをすることができるものです。ただし、意味上間違いである可能性が高いのでそれを知らせてくれたものです。
エラーのときはプログラムを実行することはできませんが、警告の場合はとりあえず実行することができます。但し、少なくない場合で警告は実際には(意味の上で)間違いで、プログラムは期待した動作をしないことになります。原則として、警告は出ないようにしなければいけません。何故その警告が出るのか、実際にどういう動作をするのかを完全に理解して、それこそが意図したプログラムの解である、ということを言い切れる場合以外は警告が出ないように(意味上の誤りを)修正する必要があります(決して、理解していないキャストで警告をとにかく消すような対応はしてはいけません)。

で、質問の警告の言葉の意味がわからないということはないかと思いますが、何故警告になるのかわからないというなら、C言語での配列についての理解が足りないということになるかと思います。
Cでは、宣言のときにこそ要素数を用いて
T array[N];
というように記述しますが、多くのシーンで要素数の情報は欠落します。そして、配列名arrayを単独で記述したときは配列全体ではなく、配列の先頭要素へのポインタを表し、[ ]で囲んだ添字を付けた場合は添字番目の要素一つを変数と表すことになります。
その知識をもってすれば、
word(word_kanji[100], word_romaji[100]);
は、
「word関数を、引数としてword_kanjiの101番目の要素とword_romajiの101番目の要素の値をもって呼び出す」
という意味になっていることがわかります。これは、多分やりたいことに合致していないと思います。

さらに言えば、関数の引数において配列形式が記述された場合、要素数は無視され(そもそも要素数が記述されなくても文法エラーではない)、配列の要素の型へのポインタとして解釈されることになっています。
つまり、word関数のシグニチャ
char *word(char *word_kanji[100], char *word_romaji[100])
を読み解くと、charの値を返す関数wordは、charへのポインタ型(つまりchar* *型)の仮引き数word_kanjiとword_romajiを持つ、ということを宣言していることがわかります。100という数字無視され、今回は使用していませんが、sizeof(word_kanji)の値を使ってなにかしようとしたりすると意図しない結果になってしまうかも知れません。

ついでに余計なことを言えば、srand関数の使い方は間違っているだろう(警告にもならないだろうけど、きっと期待した動作ではない)とか、word関数は返り値を持たないとこれはエラーになるんじゃないかとか。

---

さて。私が自分で書くなら、多分wordのような関数は設定しないかなぁ...(関数が間違いということではないけれど)

struct Word{
  const char* kanji;
  const char* romaji;
}words[]={
  {"漢字", "kannzi"},
  // :
  {"こんにちは", "konnnitiha"}
};
const size_t size=sizeof(words)/sizeof(struct Word);

と初期化で書いてしまえば、変換テーブルの数の管理も兼ねられて追加/削除も容易になったりしそうで。
(もしかして構造体を知らなかったりするかな?)

Comments

[Akira_0809]

ありがとうございました。
構造体を知らなかったので勉強になりました。エラーとワーニングの違いや配列についての説明が分かりやすかったです。

Answer 2

それらはエラーではなくワーニングです
ポインタの型が違う、とぼやいてらっしゃいます

Comments

[Akira_0809]

解消する必要はありますか。
また、実行するとSegmentation fault (コアダンプ)が発生します。

[dodox86]

@質問者さん

適切なコードを書けば警告は自然と消えます。ポインターとポインター配列をちゃんと区別しましょう。

[y_waiwai]

> word(word_kanji[100], word_romaji[100]);
word_kanji、word_romajiの配列の100番目の要素を与えて呼び出してますが、これはあなたの意図しているコードなんでしょうか。
これはそもそもアクセス違反となります
コアダンプするってのは当たり前の話です

小手先の修正でそのワーニングを出ないようにするのは簡単ですが、そもそものコードが間違っていますよ

[Akira_0809]

ありがとうございました。
問題が無事解決しました。

Answer 3

とりあえず

char *word(char *word_kanji, char *word_romaji)
{
        strcpy(word_kanji, "漢字");