質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.49%

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

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 2,592

score 9

 前提・実現したいこと

文字列の中に特定の文字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/ツールのバージョンなど)

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

0

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/08/17 14:07

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

    キャンセル

  • 2018/08/17 15:01 編集

    こうかも?
    ````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;
    }
    ````

    キャンセル

  • 2018/08/17 16:13

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

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

    キャンセル

0

return (void*)s; 

これではどでしょ       

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/08/17 14:03

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

    キャンセル

  • 2018/08/17 14:06

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

    キャンセル

  • 2018/08/17 14:08

    ありがとうございます。

    キャンセル

  • 2018/08/17 16:15

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

    キャンセル

0

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

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/08/18 00:00

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

    キャンセル

  • 2018/08/18 00:12

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

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

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

    キャンセル

  • 2018/08/18 00:15

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

    キャンセル

  • 2018/08/19 17:38

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.49%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る