int型への文字の入力について

Question

###前提・実現したいこと
「文字を入れるのではなく、文字コードを入れているだけ」と認識しています。
int型の変数にその文字コードというのを入力させて、、とやろうとしたのですが、うまくいきません。なぜダメなのでしょうか。

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

文字列に入力した文字が含まれているかどうかを調べる関数 を書いたのですが、含まれていても含まれてないとなってしまいます。

###該当のソースコード

#include <stdio.h>
/* 文字列の中の最も先頭に位置する文字ｃを探索 */
int str_char(const char s[] ,int c)
{
    int i=0;
    while(s[i]){
        if(s[i]==c){
            return i;
        }
        i++;
    }

    return -1;
}



int main()
{
    char str[128];
    puts("文字列を入力");
    scanf("%s",str);
    puts("探したい文字を入力");
    int key;
    scanf("%d",&key);
    int ans=str_char(str,key);
    if(ans!=-1){
        printf("[%d]にあります\n",ans);
    }
    else{
        puts("ありませんでした");
    }

    return 0;

}

###試したこと
key をchar型の配列として、str_char関数に渡すときにchar[0]として渡すとうまくいきます。

###補足情報(言語/FW/ツール等のバージョンなど)
C言語

Answer 1

%d は画面から数字を受け取って、それを数値として変換して key に設定する識別です。
ですので文字を画面から受け取る為には %c である必要があります。

scanf("%c", &key);

おそらくこう変更されると思いますが、標準入出力の動作でつまづくかと思いますので、先に助言を足しておきます。scanf は改行文字を抜き取りません。ですのでこれを呼び出すと前の文字列入力 %s の時の改行コードが次の %c に読み取られてしまい、結果入力がスキップされる事になります。

これを避ける為には一旦 getchar() で改行コードを抜き取っておく必要があります。

やろうとされておられる事はおそらくこうだろうなと思う物を貼っておきます。

#include <stdio.h>

/* 文字列の中の最も先頭に位置する文字ｃを探索 */
int str_char(const char s[], char c)
{
    int i=0;
    while(s[i]){
        if(s[i]==c){
            return i;
        }
        i++;
    }

    return -1;
}

int main()
{
    char str[128];
    char key[1];

    puts("文字列を入力");
    scanf("%s", str);

    getchar(); /* 改行文字を抜き取っておく */

    puts("探したい文字を入力");
    scanf("%c", &key[0]);

    int ans = str_char(str, key[0]);
    if(ans != -1){
        printf("[%d]にあります\n", ans);
    }
    else{
        puts("ありませんでした");
    }

    return 0;

}

勘違いされたのは、おそらく変数が数字だから %d を使うべきと思われた部分かと思います。scanf や sscanf、fscanf といった関数は、%d が与えられると入力から数字のみを読み取り、それ以外がくると入力を中断し、それを数値に変換して引数で与えられたアドレスにコピーします。

ですので 文字 を入力して貰うには %d でなく %c でなければなりません。

であれば getchar() でいいじゃんと思うかもしれませんが、getchar() は行入力でないため、1文字入力してバックスペースで直すといった事が出来なくなります。

Comments

[macg3n]

うまくできました！勉強になります。ありがとうございました！

ここで質問させていただきます。
char key[1] ; →　char key ;
とやっても同じように動きますが、配列として扱った方が良いのでしょうか？

[mattn]

コードを書いている時にテストで puts(key); (実際は char key[2]; key[1] = 0; と宣言して) でチェックしていたのでその残骸でした。すみません。char key と書いた方が混乱が無かったですね。

[macg3n]

わかりました！非常に助かりました。

Answer 2

puts("探したい文字を入力");
を
puts("探したい文字の文字コード値を10進数で入力");
に変えて、その通りすればいいです。

Comments

[macg3n]

ありがとうございました！

Answer 3

key をchar型の配列として、str_char関数に渡すときにchar[0]として渡すとうまくいきます。

とおっしゃっているので、たぶんkeyに入力しているのは、文字コードではなく文字を入力しているのではないでしょうか（aとかbとか）
scanfで書式%dに対して文字を入力しても、受け付けられず0が入ります。
なので現状のコードのまま成功させるには、文字を入力するのではなく、文字コードを入力する必要があります。
aであれば文字コードは97
bであれば文字コードは98
アスキーコード表

コード自体は問題ないように思えます。
文字コード（アスキーコード）に対する理解度が足りていないのだと思います。

Comments

[macg3n]

なるほどです。ありがとうございました！

Answer 4

プログラムはあなたの指示通り動きますが、どのあたりがあなたの期待と異なるのでしょうか？

https://wandbox.org/permlink/6s6aheQ6dEPDxsWT

Answer 5

どうしてもint引数でやりたい場合

#include <stdio.h>
/* 文字列の中の最も先頭に位置する文字ｃを探索 */
int str_char(const char s[] ,int c)
{
    int i=0;
	printf("search '%xh/%c' from [%s]\n", c, c, s);
    while(s[i]){
        if(s[i]==c){
            return i;
        }
        i++;
    }

    return -1;
}



int main()
{
    char str[128];
    int key = 0;  // 0入れとかないと scanf が1バイトしか書き込まない場合ゴミを渡してしまう
    int ans;
    
    puts("文字列を入力");
    scanf("%s",str);
    printf("\n[%s]\n\n", str);
    puts("探したい文字を入力");
    fflush(stdin);
//    scanf("%d",&key);
    scanf("%c",&key);

    ans=str_char(str,key);
    if(ans!=-1){
        printf("[%d]にあります\n",ans);
    }
    else{
        puts("ありませんでした");
    }

    return 0;

}

Comments

[mattn]

念のため、fflush(stdin) は windows でしか動きません。

[macg3n]

ありがとうございました！

Answer 6

scanf("%d",&key);とした場合「%dは、10進数で入力する」なので文字（数字以外）の入力では０となり検索できないのではないでしょうか？
また、数字を入れた場合、値が&keyに入りますが（1の場合に(int)1）、最初の文字列の中での数字はasciiコードなので（例として1は49）keyの値とは異なってしまいます。

無理にif((s[i]-'0')==c)とすれば数値であれば比較はできると思いますが、ご意図とされる動きでは無いと思います。

Comments

[macg3n]

知りませんでした。。ありがとうございます！！