C言語 誰か教えて [転載禁止]©2ch.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押しても次に進まないです。
どうすればいいですか。 入門読め
基本編ではCharは配列で宣言するはずだ #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; } >>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++;
にする
これで動くはず スタック破壊とchar*にはconst付けろというのは初心者には理解できないか 関数の引数としてconst char*またはchar const*となっていると、引数を見た人がこの関数内でポインタをデリファレンスした値が変更されないと理解できる
また、コンパイラにもそれを明示的に指示できるので最適化される可能性が高くなる
ちなみにchar *constの場合は、(デリファレンス前の)変数値が変更されないと理解できる
しかしながら、ポインタ変数のコピー渡しなので意味がないし、const char*とchar *constでは互換性がないため
strchr()を実装するという観点からはchar *constという宣言は不適切である
もちろんchar const*constという宣言も可能であることを付け加えておく
上記の性質より、char const*const*というようにポインタのポインタ変数を宣言する際に変数値を変更しないし、デリファレンスした変数値も変更しないというときに使うことになる >>16
難しいですね。
ありがとうございました。 strchr(char *str){
for(n = 0; *str ; ++str) ++n;
return n;
} 誰でも簡単にパソコン1台で稼げる方法など
参考までに、
⇒ 『宮本のゴウリエセレレ』 というブログで見ることができるらしいです。
グーグル検索⇒『宮本のゴウリエセレレ』
3GUVEYI7D4 とても簡単な自宅で稼げる方法
参考までに書いておきます
グーグルで検索するといいかも『ネットで稼ぐ方法 モニアレフヌノ』
HGK39 ■ このスレッドは過去ログ倉庫に格納されています