C言語 ポインタを返す関数について

Question

前提・実現したいこと

文字列の中に特定の文字cが含まれていれば、そのポインタを返す関数

　

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

異なるconst修飾子です

該当のソースコード

C言語

#include <stdio.h>

char *str_chr(const char *s, int c)
{
	while (*s) {
	　if (c == *s) 
	　　return s;			
	　*s++;
	}
	return NULL;
}

int main(void)
{
    char s[128];
    int c;

    printf("kye:");
    c = getchar();

    printf("文字列入力:");
    scanf_s("%s",s,128);

	printf("%s\n",str_chr(s, c));

    return 0;
}

### 試したこと

constを外せばうまくいったんですが、constありではどうすればいいのでしょう
### 補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

解決済みですが、気づいたことがあるので。

*s++; という行があります。これは間違いです。
s++; とすべきです。

なぜなら、ここで行うのはアドレスを前に進める（アドレスを＋１する）ことであって、そのアドレスのメモリを読むことでは無いから。なので、参照演算子「*」は不要です。

同じ間違いを良く見ます。エラーではないし、警告レベルを上げない限りコンパイラは警告も出さないようですが、場合によっては動作時に不具合を起こす可能性も考えられます。何よりもこの区別が曖昧だと、ポインタ変数とは何かがあやふやになってポインタが苦手になる・・・と私は見ています。

Comments

[soboro]

＊はポインタの指す中身にアクセスするから*は不要ということですね。
s++;は格納されていたアドレス値を1つ後ろに書き換えるという認識であっているでしょうか？

[rubato6809]

> s++;は格納されていたアドレス値を1つ後ろに

「後ろ」って具体的にどういうことですか？　後置インクリメントの「後」なら混乱してますよ。

s++ はポインタ変数の値をひとつ分増やします。s は char*型のポインタだから、アドレスがひとつ増えます。たとえば、１０００番地を指していたら、1001番地を指すようになる。それを私は「アドレスを前に進める」と書きました。

[rubato6809]

念の為。この場合、前置インクリメントの ++s も、後置インクリメントの s++ でも、どちらでも結果は同じ、 s の値がひとつ増えます。 s = s + 1; でも s += 1; でも、結果はみな同じ。

[soboro]

紛らわしい言い方をしてすいません。+1されるという意味でした(0からみて後ろ)。
間違いをご指摘いただきありがとうございました。

Answer 2

constを外せばうまくいったんですが、constありではどうすればいいのでしょう

返り値もconst char *にしてください。

Comments

[y_waiwai]

うむ、それが本筋ですね

[fu7mu4]

str_chr関数の返り値の型をconst char *に変えて、returnの引数の型とあわせるというとこですね。

[soboro]

問題では返値の型がcharになってたので、ミスなのでしょうか。
わかりやすい説明ありがとうございます。

[fu7mu4]

こうかも?
````c
#include <stdio.h>

char *str_chr(const char *x, int c)
{
char *s = (char *) x;
while (*s) {
if (c == *s)
return s;
*s++;
}
return NULL;
}
````

[rubato6809]

出題者の意図はわかりませんが、それもアリかな。

で、*s++; は s++; に直してください笑。

Answer 3

return (void*)s;

これではどでしょ

Comments

[soboro]

そのような方法もあるのだと勉強になりました。

[y_waiwai]

ワーニングを潰すための方法、と取っておいてください
キャストというのは強力すぎるので、むやみに多用するとデメリットが大きくなります

[soboro]

ありがとうございます。

[rubato6809]

y_waiwaiさん
そこでキャストするなら、 return (char*)s; とするのが自然でしょ。void*にしなきゃならない理由はありませんよ。