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

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*というようにポインタのポインタ変数を宣言する際に変数値を変更しないし、デリファレンスした変数値も変更しないというときに使うことになる

17152016/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;
}

新着レスの表示
レスを投稿する