0

10個の値と探索する値を問い、その探索する値が入力した数値の中に存在するか存在しないかを出力させるプログラムを作成したいのですが、何番目のデータにそれが存在するのかを出力する方法がわかりませんでした。

私は以下のようにプログラムを作成しました。

ソースコード

#include <stdio.h>

/* size個 の入力されたデータを 配列の先頭から順番に格納する */
void readIntArray(int a[], int size)
{
    int i;
    
    for (i=0;i<size;i=i+1) {
        printf("%d 番目? ", i+1);
        scanf("%d", &a[i]);
    }
}


/* na個 のデータが入っている集合に x と等しい要素があるかどうかを判定する \ */
int memberOf(int x, int a[], int na)
{
     int i,result=0;
     for(i=0;i<na;i=i+1) {
       if (x==a[i]) {
         result = 1;
        }
     }
     return result;
}


int main(void)
{
    int data[1024];
    int i,x;
    
    readIntArray(data, 10);
    
    printf("探索する数字を入力してください: ");  scanf("%d",&x);

  
    for(i=1;i<10;i++)
    if (memberOf(x, data, 10)) {
        printf(" data[%d]=%d .\n",i ,x);
    } else {
        printf(" %d は存在しない.\n", x);     
    }   

    return 0;
}

実行結果

$ ./a.out
1 番目? 1
2 番目? 2
3 番目? 3
4 番目? 4
5 番目? 5
6 番目? 6
7 番目? 7
8 番目? 8
9 番目? 9
10 番目? 10
探索する数字を入力してください: 6
 data[1]=6 .
 data[2]=6 .
 data[3]=6 .
 data[4]=6 .
 data[5]=6.
 data[6]=6 .
 data[7]=6 .
 data[8]=6 .
 data[9]=6 .

単に十個の値と、探索する値を入力して、その入力した値が存在するのか存在しないのかを調べるためのプログラムなら実装できるのですが、何番のデータにそれが存在するのか出力すべくdata[%d]を読み込む関数の実装方法がわかりません。
for(i=1;i<=10;i++)では上記のように繰り返しとなるためすべてのデータ番号が出力され、for文を入力せずにコンパイルをするとdata[0]=6のように出力されてしまいます。

どのようにすればデータ番号がうまく出力されるのでしょうか。

期待する実行結果

$ ./a.out
1 番目? 1
2 番目? 2
3 番目? 3
4 番目? 4
5 番目? 5
6 番目? 6
7 番目? 7
8 番目? 8
9 番目? 9
10 番目? 10
探索する数字を入力してください: 6
data[6]=6
4
  • どういった環境で開発をされているか明記してみてください。デバッグができる環境であれば、ステップ実行して一行ずつ変数がどう変わっていくか追っていけば問題はすぐに解決できます。 Commented 2020年10月19日 15:05
  • プログラミングを始めたばかりなので、デバッグど言うものがどういうものかは知りませんが、調べたところ、C言語などの低級言語は気づかないうちにメモリ破壊を引き起こすこともあると書いてありました。また、例えばどういう環境だとそのデバッグというものが使えるのですか。(例えばUbuntuとか・・・)
    – user41837
    Commented 2020年10月19日 16:07
  • ご自身が今どんな環境でコードを作成してコンパイルして実行しているかをまず書いてみてください。今現在Ubuntuでコードを書かれているのであれば、そのように書いていただければよいです。デバッグとはプログラムのバグを潰すことを指しますが、今されているようにプログラムを動かして、問題があれば、ソースコードを見直して修正、というやり方は初学者にはちょっときついと思います。できれば一行ずつ結果を確認しながら意図したとおりに動いているか確認することができた方が、理解も早くなると思いますし、自力で問題解決することができるようになると思います。 Commented 2020年10月19日 16:28
  • 1
    IDEを使っていない場合は、printfを活用して、動作がおかしい所を調べて試します。
    – Light
    Commented 2020年10月19日 16:35

1 件の回答 1

0
    for(i=1;i<10;i++)
    if (memberOf(x, data, 10)) {
        printf(" data[%d]=%d .\n",i ,x);
    } else {
        printf(" %d は存在しない.\n", x);     
    }   

現在のコードは、「dataに値xが入っているかどうかを表示する、を9回繰り返す」になっています。

dataに値xが複数個入っている場合に、最初に見つかった添字だけを表示すれば良い場合は、memberOf() が見つかった添字を返すようにすれば良いでしょう。見つからなかった場合は-1や配列サイズを返します。そういった仕様にしたうえで、memberOf()indexOf() に名前を変えると、呼び出し側は以下のようになります:

    i = indexOf(x, data, 10);
    if (i <= 0 && i < 10) {
        printf(" data[%d]=%d .\n", i, x);
    } else {
        printf(" %d は存在しない.\n", x);
    }

見つかった複数の添字を表示する必要がある場合は、memberOf()は使わないほうがシンプルです。

#include <stdbool.h>
...
    bool found = false;
    for (i = 0; i < 10; ++i) {
        if (data[i] == x) {
            printf(" data[%d]=%d .\n", i ,x);
            found = true;
        }
    }
    if (!found)
        printf(" %d は存在しない.\n", x);     

または、memberOf() の中で表示してしまえば良いでしょう。

int memberOf(int x, int a[], int na)
    ...
       if (x == a[i]) {
           result = 1;
           printf(" data[%d]=%d .\n", i, x);
       }
...
  
    if (!memberOf(x, data, 10)) {
        printf(" %d は存在しない.\n", x);     
    }   

ちなみに、Ubuntu ですと、gdb や lldb という無骨なデバッガが利用可能かもしれません。

4
  • insight ってまだ active なんでしたっけ? sourceware.org/insight
    – 774RR
    Commented 2020年10月19日 23:23
  • insight は Ubuntu の標準のパッケージにはないですね。
    – int32_t
    Commented 2020年10月20日 0:03
  • じゃあ自分でインストールしてってのは無理か...
    – 774RR
    Commented 2020年10月20日 1:19
  • @int32_t 回答ありがとうございます。menber関数の中にdata[%d]文を入れたことで、きちんと処理できました。その方法は全く思いつかなかったので、勉強になりました。
    – user41837
    Commented 2020年10月20日 4:12

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。