配列の要素内に入力した単語を比較参照したい。

Question

下記のようなプログラムを実行したいのですが、うまくいかず悩んでいます。
要素内にうまく単語を読み込めていないようなのですが、原因がわかりません。
改善策をご教授いただけないでしょうか。

【内容】
①入力する単語数を指定
②配列を用いて各要素に単語を入力
③特定の単語（変数ｂ）があればカウントを＋１し、次の７要素は比較せず飛ばす
④集計結果を表示する

それと、要素をprintfで出力しようとするとき＆を使用しないとエラーがおきるのですがなぜなのでしょうか。

#include <stdio.h>
int main(){
    char a[2000],*b="hukuoka";
    int i=0,j,n=0;
    scanf("%d",&j);
    for(;i<j;i++)scanf("%s",&a[i]);
    for(i=0;i<j;i++){
        //printf("a[%d]=%s\n",i,&a[i]);
        if(a[i]==*b){
            i+=7;
            n++;
        }
    }
    printf("%d",n);
    return 0;
}

Answer 1

char a[2000];と定義した場合
aは2000個の文字列ではなく、2000文字分の文字です。
言い換えると最大2000文字の文字列が1つです。

それと、要素をprintfで出力しようとするとき＆を使用しないとエラーがおきるのですがなぜなのでしょうか。

a[i]とは、文字列aのi番目の1文字を指しているので
%sが文字列を表示するための書式なのでエラーになります。

---

追記

入力した単語をプログラムが覚えておく必要はあるのでしょうか？
みた感じ、カウントだけ終わったら忘れても構わないように思います。

次の７要素は比較せず飛ばす

7要素とは、単語7個ですか?
現状のプログラムでは、格納位置を7文字ずらし、末尾の7単語を捨てているだけです。

Comments

[wertyu]

回答してくださりありがとうございます。
文字を入力していくと１要素に１文字になってしまうんですね。
変数宣言のところで｛｝を使って単語を入れるやり方があったのでずっと勘違いしていました。
printfでエラーがおきたのにも納得がいきました。

>>次の７要素は比較せず飛ばす
７単語という意味で合っています。

>>入力した単語をプログラムが覚えておく必要はあるのでしょうか？
その発想が浮かばすこのような形になってしましました。

教えていただいたことを踏まえてもう一度作り直してみます。

Answer 2

こんな感じでどうでしょうか？
変数名は合わせました．

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

int NumberOfData(void) {
    char buf[256];
    fgets(buf, sizeof(buf), stdin);
    if (atoi(buf) > 0) return atoi(buf);
    exit(1);
}
int main(int argc, const char * argv[]) {

    char *a, b[]= "hukuoka";
    int i, j, n = 0;
    
    j = NumberOfData();
    a = (char *)malloc(sizeof(char) * j);
    for (i=0; i<j; i++) {
        scanf("%s", &a[i]);
        while (getchar() != '\n');
    }
    for (i=0; i<j; i++) {
        if (strcmp(&a[i], b) == 0) {
            n++;
            i += 7;
        }
    }
    printf("個数: %d\n",n);
    free(a);
    return 0;
}