文字列の探索について

Question

###前提
書籍で勉強している学生です。
書籍の解答がないため問題のヒントや解説をしていただけると嬉しいです。

###問題
文字列ｓの中に、文字ｃが含まれていれば、その添字（文字列中に文字ｃが複数ある場合は、最も先頭側の添字とする）を返し、含まれていなければ－１を返す関数を作成せよ。

int str_char(const char s[], int c){/* ... */}

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

#define NUMBER 100   //文字列の要素数
#define FAILED -1   //探索失敗

int str_char (const char s[], int c)
{
    int i;

    while(1){
        if(i == NUMBER){
            return FAILED;
            break;
        }
        if(s[i] == c){
            return i;
            break;
        }
        i++;
    }
}

###疑問点
この演習を解いていて不思議に思ったことは、文字ｃであるのにint型で指定されている事です。
また、このプログラムの確認として、
文字ｃをscanf関数で取得しようとしたのですが、関数にはint型で渡さないといけないのに、文字はchar型であるため、変換指定子を何にすれば良いのか迷っています。
そして、そこで悩んでいるため確認ができず、このプログラムが上手く動作しているかどうか分かりません。

###補足情報
書籍 ： 新明解c言語 入門編
演習 ９－５

Answer 1

文字ｃをscanf関数で取得しようとした

パラメータがintなのは、キー入力時に、scanf()ではなく返り値がintの getchar() (getc(),fgetc() ?)を前提に書かれているのではないでしょうか？

Comments

[kamecha]

なるほど、この問題の前の例題では、scanf関数を使っていたけれども、この問題は違いますね。

Answer 2

この方法だとセグメントエラーを起こします。
引数の文字列sの要素数が、常に100である保証がないからです。

次のようなコードにすべきです。
無限ループを採用するメリットも感じられないので、for文を用いています。

#define FAILED -1   //探索失敗

int str_char(const char s[], int c) {
    for(int i = 0; s[i] != '\0'; i++) {
        if(s[i] == c) return i;
    }
    return FAILED;
}

ついでに言うと、return文の後のbreakも不要です。
returnされると処理が元の関数に戻るので、breakは評価されません。

---

文字ｃをscanf関数で取得しようとしたのですが、関数にはint型で渡さないといけないのに、文字はchar型であるため、変換指定子を何にすれば良いのか迷っています。

%cで受け取って(int)でキャストすればよいのでは？

Comments

[kamecha]

なるほど、素早い回答有り難うございます。