C言語 誰か教えて [転載禁止]©2ch.net

[1 仕様書無しさん 2015/06/16(火) 18:14:20.10]

ライブラリ関数の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]

まず服を脱ぎます

[3 仕様書無しさん 2015/06/17(水) 01:09:15.97]

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

[4 仕様書無しさん 2015/06/17(水) 02:48:09.40]

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

[5 仕様書無しさん 2015/06/17(水) 08:09:05.70]

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

[6 仕様書無しさん 2015/06/19(金) 09:35:52.02]

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

[7 仕様書無しさん 2015/06/20(土) 02:43:50.15]

>>6
います

[8 片山博文MZ ◆T6xkBnTXz7B0 2015/06/20(土) 03:39:48.17]

#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]

>>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]

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

[11 仕様書無しさん 2015/06/20(土) 16:28:05.04]

厶板でやれ

[12 仕様書無しさん 2015/06/20(土) 16:56:07.21]

チンチンの皮をム板

[13 仕様書無しさん 2016/10/15(土) 04:46:14.07]

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

[14 仕様書無しさん 2016/10/15(土) 23:40:35.33]

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

[15 仕様書無しさん 2016/10/19(水) 20:46:16.67]

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

[16 仕様書無しさん 2016/10/19(水) 22:57:32.25]

関数の引数として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]

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

[18 仕様書無しさん 2016/10/21(金) 02:15:05.88]

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

[19 仕様書無しさん 2017/06/11(日) 15:31:12.19]

嫌です

[20 仕様書無しさん 2017/12/29(金) 21:52:50.23]

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

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

3GUVEYI7D4

[21 仕様書無しさん 2018/05/22(火) 13:22:49.63]

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

HGK39