C言語で入力した文字列と一致しているか調べる

Question

実現したいこと

あなたが食べたい料理の名前と、メニューに載っている料理名が 1 つ与えられるので、食べたい料理に合致するメニューであれば "Yes" を、そうでなければ "No" を出力するプログラムを作成してください。
食べたい料理に合致するメニューとは、メニューの単語の中に食べたい料理名が入っていることを指します。

入力例 1 では、あなたは hamburg を食べたいと思っていて、与えられた料理名は cheese hamburg です。2 つ目の単語が、あなたの食べたい hamburg と一致しているため、"Yes" と出力します。

入力例1
hamburg
2
cheese hamburg

出力例1
Yes

入力例2
apple
4
spicy pizza with pineapple

出力例2
No

発生している問題・分からないこと

以上の問題を解きたいのですが、いくら考えてもよくわかりません。
以下のようなソースコードを書いてみたのですが、結果が出ないです。

該当のソースコード

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

int main(void){
    char s[100]; 
    char t[10][100];
    int n,i;  
    
    scanf("%s",s);
    scanf("%d",&n);
    for(i=0;i<n;++i){
        scanf("%s",t[i]);
        if (strcmp(s, t[i]) == 0){
            printf("Yes");
        }
    }
    return 0;
}

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

以下のようなコードを書いてみたのですが、この場合Noの場合の処理はどうのように書けばいいのでしょうか

補足

特になし

Comments

[fana]

「"Yes" か "No" を出力するものを作れ」なる指定に対して，そのどちらの出力も存在しないコードを「書いてみた」とか，その行動原理が理解できませんが……

とりえあえず文字列の比較というのは

> t[i]==s

では　できない　ので，そこらへんから復習されるべきかと見えます．

[hiroki-o]

入力例の2とか4は何ですか？ 次に入力する単語数でしょうか？
食べたい料理も、メニューの料理も自分で入力するのですか？ 変なUIですね。

[jimbe]

入力例等を見ると、 atcoder の類じゃないでしょうか。

[fana]

Q．あなたが食べたい料理の名前は？
A. "apple"

入力例が雑すぎる

Answer 1

食べたい料理に合致するメニューであれば "Yes" を、そうでなければ "No" を出力する

よくある問題ですね。
プログラムは時に人の思考方法を知っているかどうかで難易度が変わります。
ご自身この問題のコンピュータ役になった時、どうやって Yes/No を表示するでしょうか。
Yes の場合、プログラムとして書けているように最初から順に一つ々々と比較して一致していたら…となるでしょう。
では No の場合は?
同じ流れなら、最初から一つ々々と比較して一つも一致しなかったら…となるのが自然でしょう。つまり、全てと比較→判断となるのですから、ループの外で (Yes の時の形のように) 判断して表示する形になります。
では何を判断に使えば良いのか。現状のプログラム内に一つも一致しなかったことを示す変数等はありません。
１つの重要なステップアップは、『無いものは作る』です。
たとえば int の flag という変数を用意し、 for の前で 0 に初期化します。for の中でメニューに一致して Yes を表示したら、同時に flag を 1 にするようにします。

  int flag=0;
  for(int i=0;i<n;++i){
    //…
    if (strcmp(s, t[i]) == 0){
      printf("Yes");
      flag=1;
    }
  }

そうすると for が終わった時、一致したものがあったら flag は 1 になっており、一致したものが無かったら flag は 0 のままになるはずです。
後は、この flag が 0 だったら No を表示することで期待する動作になるのではないでしょうか。

  //…
  if(flag == 0) {
    printf("No");
  }

---

上は既存のプログラムの修正を最小にするような内容ですが、もう少し形を整えることが出来ます。
新たに追加した flag 変数は No を表示する為に使いましたが、 Yes を表示することにも使えることに気が付かれるでしょうか。
つまり、for の中の if で一致した時に print するのでは無く、for の外で (No を表示する同じタイミングで) print するのです。

  //一致するものがあるか
  int flag=0;
  for(int i=0;i<n;++i){
    //…
    if(strcmp(s,t[i]) == 0){
      flag=1;
    }
  }
  //一致状態によって Yes/No を出力
  if(flag == 1) {
    printf("Yes");
  } else {
    printf("No");
  }

このようにすると一致するかを判断する部分とその結果に基づいて出力する部分が明確になり、プログラムコードとして評価が上がります。
さらに、 flag を found という名前にすることでその変数が何を表すかが明確になります。

  //一致するものがあるか
  int found=0;
  for(int i=0;i<n;++i){
    //…
    if(strcmp(s,t[i]) == 0){
      found=1;
    }
  }
  //一致状態によって Yes/No を出力
  if(found) { //if 文は () 内が 0 で無い場合を成立とみなすので、 0 か 1 にしかならない変数なら "==1" は無くても良い
    printf("Yes");
  } else {
    printf("No");
  }

if(found) print ～ が「もし見つかったら～を表示」と見えてくるでしょう。

Comments

[komori_k]

回答ありがとうございます。
試したところ問題が解決しました！
ベストアンサーに選ばせていただきました。

Answer 2

scanfでスペースを含んだ文字列を入力できると勘違いしていました。失礼いたしました。
せっかくなので一応動くコードも書いておきます

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

int main(void){
    char s[100]; 
    char t[100];
    int n,i;

    char kotae[2][5] = {"Yes","No"};
    int kotae_soeji = 1;
    
    scanf("%s",s);
    scanf("%d",&n);
    for(i=0;i<n;++i){
        scanf("%s",t);

        if(strcmp(s,t) == 0){
            kotae_soeji = 0;
        }

    }

    printf("%s", kotae[kotae_soeji]);

    return 0;
}

--↓間違い--
strcmpは文字コードの大きさを比べているのでstrstrを使ってはいかがでしょうか

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

int main(void){
    char s[100]; 
    char t[100];
    char s2[300] = " ";
    int n,i;  
    
    scanf("%s",s);
    strcat(s2, s);

    scanf("%d",&n);
    for(i=0;i<n;++i){
        scanf("%s",t);
        if (strstr(t,s2)){
            printf("Yes");
        }
    }
    printf("No");

    return 0;
}

Comments

[jimbe]

`strcmp(～)==0` は大きさの差が無い＝同じという使い方ですので大きさの比較で問題ありません。
strstr は文字列の中から一致する部分を探すものですので、部分一致(t="ABCD" で s2="BC"とか) で Yes となってしまう場合があり、完全一致が必要な本件では不適当ではないでしょうか。

[fana]

これ，"Yes" が複数回表示されたりとか，常に "No" が表示されたりとかしませんか？

[退会済みユーザー]

コメントありがとうございます。
最初に探したい食材名を入力しその後に入力される料理名の中に最初の食材名が入っていたらYesと出力し終了するというイメージで作りましたが、これだと常にNoが表示される上に料理名は単語ごとに入力されているのでstrcmpを使うべきでした。
とりあえず常にNoと表示されないように直してみます。

[jimbe]

全体の動作としては (s2 の最初をスペースにしていて t にはスペースは入らないので) Yes が表示されることは無く常に No だけになるように見えます。

[jimbe]

>Yesと出力し終了するというイメージ

現状は Yes と出力したら終了するように書かれていません。
なので for ループは最後まで回り、常に No が表示されることになります。