2つの言葉を入力して同じかどうかを判定するプログラム

Question

前提・実現したいこと

２つの言葉を入力してその言葉が同じであればidebtical,ちがっていればdifferentと出力する
プログラムを作っています。

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

Segmantation fault

該当のソースコード

#include<studio.h>

int identical(char *word_a,char *word_b)
{
   int i;

   while(word_a[i] != '\0' && word_b[i] != '\0'){
        if(word_a[i] = word_b[i]){
             return 1;
             break;
            }
       i++
   }
  return 0;
}

int main(void)
{
    char *word_a[100],char *word_b[100];
    int han;

      printf("input a word :");
      scanf("%99s",word_a);

      printf("input a word :");
      scanf("%99s",word_b);

      han = identical(*word_a,*word_b);

     if(han == 0){
         printf("different\n");
       }else if(han ==1){
         printf("identical\n");
      }
    return 0;
}

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

identical関数がなければできるため、identical関数がある状態でどうやってやるのかが
知りたいです。何かきずいたことがあれば教えてください。

Comments

[otn]

> char *word_a[100]
は、何を宣言しているつもりですか？

そもそも、このプログラムだと、コンパイルエラーになります。
コピペするプログラムを間違えてますね。実行したプログラムに置き換えてください。

[nanami12]

2つの文字列を比較する標準関数をご存知ですか？

Answer 1

勉強がてらで書いてみました。

#include <stdio.h>

int identical(char *word_a, char *word_b)
{
   while(*word_a != '\0' || *word_b != '\0'){
      if(*word_a != *word_b)
          return 1;
      word_a++;
      word_b++;
   }
   return 0;
}

int main(void)
{
    char word_a[100];
    char word_b[100];

    printf("input a word :");
    scanf("%99s", word_a);

    printf("input a word :");
    scanf("%99s", word_b);

    if(identical(word_a, word_b)){
       printf("different\n");
    }else{
       printf("identical\n");
    }
    return 0;
}

Comments

[fana]

質問者のコードのidentical()の戻り値の利用部分から，identical()の戻り値の仕様が（一致のとき1, 不一致のとき0なのだと）推測できますが…
それに対して，あえて仕様が異なる形のコードを示すのであれば，その旨を一言添えた方が無用な混乱を抑制できるのではないかと思います．
（わざとやってるのかもしれませんが）

Answer 2

Segmentatin Fault が起きる前に、そもそもこのプログラムはコンパイルできない（コンパイルエラーが発生する）と思いますが、きちんとコンパイルは通りましたか？

#include<studio.h>

これは

#include<stdio.h>

の間違いですね（studioという間違いは意外に多いのですが）。

int identical(char *word_a,char *word_b)
{
   int i;

   while(word_a[i] != '\0' && word_b[i] != '\0'){
        if(word_a[i] = word_b[i]){
             return 1;
             break;
            }
       i++
   }
  return 0;
}

while()の条件式がおかしくありませんか？

他の方も指摘されているように、変数iが初期化されないまま使われています。

if()文の中の比較式が代入になっていますね。
おそらく、二つの変数の値が等しいならば、というつもりで書かれたと思いますが、アルゴリズムとしてはそれで大丈夫でしょうか？
このままだと、文字列の一番目の要素を比較し、それが2つの文字列で同じならば判定終了になってしまいます。

int main(void)
{
    char *word_a[100],char *word_b[100];

    int han;

      printf("input a word :");
      scanf("%99s",word_a);

      printf("input a word :");
      scanf("%99s",word_b);

      han = identical(*word_a,*word_b);

     if(han == 0){
         printf("different\n");
       }else if(han ==1){
         printf("identical\n");
      }
    return 0;
}

これも他の方が指摘されていますが、変数宣言でのword_a,word_b両変数とも、アスタリスクは不要です。

また、変数をidentical()に渡す際もアスタリスクは不要です。

Answer 3

int i;

iが初期化されていません。

    if(word_a[i] = word_b[i]){

これは間違いを含んでいるのでよく条件式について復習するとよいでしょう。

char *word_a[100],char *word_b[100];
  han = identical(*word_a,*word_b);

*が不要です。

Answer 4

identical関数がなければできるため

標準関数でstrcmpが存在します。（学習目的などでないなら）自分で作る必要はありません。