検索させるプログラムです。

質問

文字列を入力させて、その後に検索キー入力をし、先ほど入力した文字列のなかから
一致するものを検索するプログラムです。

#include <stdio.h>

int searchidx(char a[],char key)
{
int i=0,j=0;

a[3]=key;

for(i=0;i<3;i++){
if (a[i] == key){
j++;
break;
}
}
if(j==0){
return(-1);
}else{
return(i);
}
}

int main(void)
{
int a,i;
char ky=0;
char x[3];

printf("3個の文字を入力してください。\n");
for (i = 0; i < 3; i++) {
printf("x[%d]：", i);
scanf("%s", &x[i]);
}
printf("検索キー：");
scanf("%s", &ky);

a=searchidx(x,ky);

if (a == -1){
puts("一致しません。");
}else{
printf("%d\n",a);
printf("%s\n",x[a]);
}
}

メイン関数で文字列を入力させ、その配列とキー値をサーチ関数に渡して
一致した時の配列の添え字をaに返してメイン関数で文字列の配列のその添え字x[a]で表示されるのではないか？と思ってそうしてるつもりなのですが
実行してみると最後の一致した結果を表示させるところでエラーがでます。
表示させるところの前のprintfで一致するものの要素数が帰ってくるかどうかのチェックを入れてますが
ちゃんと添え字番号は正しく返ってくるのになぜ最後の
printf("%s\n",x[a]);
ではダメなのでしょうか？

それとscanfって%sのときは＆いらなかったと思うのですが
この場合＆を入れないとエラーになってしまいます。

回答No.4

wolv · ベストアンサー

表示部分は、
printf("%c",x[a]);
にしないといけません。
xは文字列ですが、x[a]は文字です。
%sは文字列を表示する場合の書式指定です。

回答No.3

wolv

とりあえず、searchidxは、以下のような関数でいいはずです。ただし、aが３文字ではない場合に問題が起きるので、そのあたりの例外処理は将来足す必要があるかもしれません。

ちなみに、元の関数は「一致する文字があったら1を返す。なかったら－１を返す」という関数になってますよ。（j++の位置に問題がある）

int searchidx(char a[],char key) {
int i=0;

for(i=0;i<3;i++){
if (a[i] == key) return i;
}
return -1;
}

回答No.2

winterofmeei

No1です

じっくりとコードを見ていたら他にも原因かもしれない部分がありました。

#include <stdio.h>

int searchidx(char a[], char key)
{
int i = 0, j = 0;

a[3] = key; //////////////// メモリ破壊が起きる箇所（１）

for (i = 0; i < 3; i++)
{
if (a[i] == key)
{
j++;
break;
}
}

if (j == 0)
{
return(-1);
}
else
{
return(i);
}
}

int main(void)
{
int a, i;
char ky = 0;
char x[3];

printf("3個の文字を入力してください。\n");
for (i = 0; i < 3; i++)
{
printf("x[%d]：", i);
scanf("%s", &x[i]); //////////////// メモリ破壊が起きる箇所（２）
}

printf("検索キー：");
scanf("%s", &ky); //////////////// メモリ破壊が起きる箇所（３）

a = searchidx(x, ky);

if (a == -1)
{
puts("一致しません。");
}
else
{
printf("%d\n",a);
printf("%s\n",x[a]);
}
}

（１）で最初のメモリ破壊が起きます。このコードがある意図が分かりません。たぶん必要ないと思いますが……

（２）の書き方は「文字」の入力ではなく、「文字列」の入力を要求するものです。ここでもやはりメモリ破壊が起きます。私なら以下のように修正します。
scanf("%c", &x[i]);

（３）の書き方も「文字」の入力ではなく、「文字列」の入力を要求するものです。私なら以下のように修正します。
scanf("%c", &ky);

> それとscanfって%sのときは＆いらなかったと思うのです> が
> この場合＆を入れないとエラーになってしまいます。

本当はC言語は、「%sなら&はいらない」という単純なものではないです。この説明はちょっと難しく、ポインタやメモリの知識が十分にないと理解できないと思うので省略します。

回答No.1

winterofmeei

main()関数内の

scanf("%s", &x[i]);

ですが、%sは文字列を入力するときに使用します。一文字だけ入力したいときは%cを使います。

ちなみにこのプログラムは、このままでは使用メモリ以外も破壊するプログラムになってます。