C言語 誰か教えて

1 ：仕様書無しさん：2015/06/16(火) 18:14:20.10 .net

ライブラリ関数のstrchr()を自分で作成したくて書いてみたんだがうまく動かない
どこをどうすればいいか教えてください

#include <stdio.h>

main() {
char text, c, *pc;
int i;

printf("文字列を入力してください");
scanf("%s", text);

printf("検索する文字列を1文字入力してください");
scanf("%c%*[^n]%*c", &c);

pc = &text;

while(*pc != '\0') {
pc++;
i++;

if (*pc == c)
printf("%cは前から%d番目に存在します\n", c, i);

return(0);
}

printf("存在しません\n");
}

こうすると二つ目のscanfが実行された後にenter押しても次に進まないです。
どうすればいいですか。

2 ：仕様書無しさん：2015/06/16(火) 23:21:23.08 .net

まず服を脱ぎます

3 ：仕様書無しさん：2015/06/17(水) 01:09:15.97 .net

脱がずにする方が好きです

4 ：仕様書無しさん：2015/06/17(水) 02:48:09.40 .net

入門読め
基本編ではCharは配列で宣言するはずだ

5 ：仕様書無しさん：2015/06/17(水) 08:09:05.70 .net

>>1
プログラム板でやれ

6 ：仕様書無しさん：2015/06/19(金) 09:35:52.02 .net

>>1いるならマジレスしてやる

7 ：仕様書無しさん：2015/06/20(土) 02:43:50.15 .net

>>6
います

8 ：片山博文MZ ◆T6xkBnTXz7B0 ：2015/06/20(土) 03:39:48.17 .net

#include <stdio.h>
#include <string.h>
void chomp(char *s) {
size_t len = strlen(s);
if (len && s[len - 1] == '\n') s[len - 1] = 0;
}
char *my_strchr(char *s, char ch) {
while (*s) { if (*s == ch) return s;
++s;
} return NULL; }
int main(void) {
char ch, *pch, text[512]; int i;
printf("文字列を入力してください"); fgets(text, 512, stdin); chomp(text);
printf("検索する文字を入力してください"); scanf("%c", &ch);
pch = my_strchr(text, ch);
if (pch) { i = (int)(pch - text) + 1;
printf("%cは前から%d番目に存在します\n", ch, i); return 0;
}
puts("存在しません"); return 0; }

9 ：仕様書無しさん：2015/06/20(土) 10:30:17.88 .net

>>7
まず宣言のところは
char text[100] , c , *pc ;
int i = 1 ;
にする
textは文字型配列にする
ただのポインタに文字列入れても確保されてないメモリにデータ入って異常動作しかねない
iに初期値ないとi++;しても正常にカウントアップされない
ポインタの代入文はtext[100]で宣言してtextもcharのポインタになってるから
pc = text;
にする
whileループの条件文はそのまま
中は
pc++;
i++;
if(){}
にすると1文字目が飛ばされるから
if(){}
pc++;
i++;
にする
これで動くはず

10 ：仕様書無しさん：2015/06/20(土) 15:01:17.02 .net

>>8>>9
本当にありがとう
感謝します

11 ：仕様書無しさん：2015/06/20(土) 16:28:05.04 .net

厶板でやれ

12 ：仕様書無しさん：2015/06/20(土) 16:56:07.21 .net

チンチンの皮をム板

13 ：仕様書無しさん：2016/10/15(土) 04:46:14.07 .net

https://www.youtube.com/watch?v=kUwXubIo19I

14 ：仕様書無しさん：2016/10/15(土) 23:40:35.33 .net

スタック破壊とchar*にはconst付けろというのは初心者には理解できないか

15 ：仕様書無しさん：2016/10/19(水) 20:46:16.67 .net

>>14
付けるとどんなメリットがあるんですか？

16 ：仕様書無しさん：2016/10/19(水) 22:57:32.25 .net

関数の引数としてconst char*またはchar const*となっていると、引数を見た人がこの関数内でポインタをデリファレンスした値が変更されないと理解できる
また、コンパイラにもそれを明示的に指示できるので最適化される可能性が高くなる

ちなみにchar *constの場合は、(デリファレンス前の)変数値が変更されないと理解できる
しかしながら、ポインタ変数のコピー渡しなので意味がないし、const char*とchar *constでは互換性がないため
strchr()を実装するという観点からはchar *constという宣言は不適切である
もちろんchar const*constという宣言も可能であることを付け加えておく

上記の性質より、char const*const*というようにポインタのポインタ変数を宣言する際に変数値を変更しないし、デリファレンスした変数値も変更しないというときに使うことになる

17 ：15：2016/10/20(木) 02:05:47.40 .net

>>16
難しいですね。
ありがとうございました。

18 ：仕様書無しさん：2016/10/21(金) 02:15:05.88 .net

strchr(char *str){
　for(n = 0; *str ; ++str) ++n;
　return n;
}

19 ：仕様書無しさん：2017/06/11(日) 15:31:12.19 .net

嫌です

20 ：仕様書無しさん：2017/12/29(金) 21:52:50.23 .net

誰でも簡単にパソコン1台で稼げる方法など
参考までに、
⇒　『宮本のゴウリエセレレ』 というブログで見ることができるらしいです。

グーグル検索⇒『宮本のゴウリエセレレ』

3GUVEYI7D4

21 ：仕様書無しさん：2018/05/22(火) 13:22:49.63 .net

とても簡単な自宅で稼げる方法
参考までに書いておきます
グーグルで検索するといいかも『ネットで稼ぐ方法　モニアレフヌノ』

HGK39