2ちゃんねる★スマホ版★■掲示板に戻る■全部1-最新50

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

新着レスの表示

★スマホ版★■掲示板に戻る■全部前100次100最新50

名前:E-mail: