質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

Q&A

解決済

2回答

1103閲覧

入力した文字列から特定の文字の数を数えるプログラムで数えてくれない。

MAIHIME

総合スコア7

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

0グッド

0クリップ

投稿2019/07/24 09:11

編集2019/07/24 09:52

入力した文字列から特定の文字の数を数えてプログラムで同じような質問の(https://teratail.com/questions/62066)のベストアンサーを真似て作ったのですがcount_ter1とcouunt_str3の飛んだ先のint a;やint c;が初期化されていないとエラーがでて、=0や=NULLで初期化してみたのですが、文字列中の文字は0文字です。となってしまい一向に数えてくれません。どこがおかしいのでしょうか?
ーーー追記ーーー
c言語を習ってて出てきた問題の自己応用書いたものです。
ソフトとしてはc++のソフトでcを書いています。
ーーーーーーーー
また質問形式がおかしかったらおかしいと言ってくれると嬉しいです。

該当のソースコード

#include<stdio.h> #include<string.h> int count_ter1(const char* FLname, const char* str1); int count_ter3(const char* FLname, const char* str3); int main(void) { char FLname[64]; size_t len; printf("アルファベットで64文字以内してください->"); fgets(FLname, sizeof(FLname), stdin); FLname[strlen(FLname) - 1] = '\0'; printf("\n"); char str1[] = "dr"; char str3[] = "auo"; int num_ter1,num_ter3; num_ter1 = count_ter1(FLname, str1); printf("drは%d文字ありました\n",num_ter1); printf("\n"); num_ter3 = count_ter3(FLname, str3); printf("auoは%d文字ありました\n", num_ter3); printf("\n"); return 0; } int count_ter1(const char* FLname, const char* str1) { int cut1 = 0; int a; while ((a==*FLname++)) { if (strchr(str1, a)) { cut1++; } } return cut1; } int count_ter3(const char* FLname, const char* str3) { int cut3 = 0; int c; while ((c == *FLname++)) { if (strchr(str3, c)) { cut3++; } } return cut3; }

補足情報

visual studio2019を使ってます。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

jimbe

2019/07/24 09:16

コードは 「```」の行("`"は"shift+@"で表示)で上下を囲むと、枠内に専用のフォントで表示されるようになり, "#"等でのフォーマットはされなくなります. その際, 最初の「```」を「```c」としますと, c言語のコードとして認識されるようにもなりますので, ご活用ください.
MAIHIME

2019/07/24 09:18

ご指摘ありがとうございます。 初めて使ったので使い方がわからず投稿した節があります。 もう1度投稿しなしてみようとおもいます。
gentaro

2019/07/24 09:20

同一内容での複数ポストは問題があるので、その際は「投稿し直し」ではなく「編集」してください。
guest

回答2

0

ベストアンサー

まず, count_ter1 と count_ter3 は変数名以外全く同じですので, 1つに纏められたほうが良いかと思います.

そして, a と c の「初期化されてない」と言いますのは, 例えば a ですと

while ((a==*FLname++))

では a と *FLname を比較しているだけで a に値(文字)が入りません. 次の strchr で a を使うので 値を入れる必要があります.

while ((a=*FLname++))

と "==" を "=" にする必要があります.
c も同様です.
恐らくコードの意味をご理解頂けないまま書かれたのではないでしょうか.
ポインタを使って代入と(比較と)カウントアップを一度に行うこの書き方は, 動作をご理解頂いていなければ間違え易いです.

投稿2019/07/24 09:55

jimbe

総合スコア12646

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

MAIHIME

2019/07/24 10:55

ありがとうございます。無事”==”を"="にしたら動きました。
guest

0

C言語でコードを書くなら、VisualStudioやEclipseなどのデバッグできる環境を整えましょう
任意の行で実行を止め、変数のナカミを見れるようになります。
また、1行づつ実行することもできます。

これを使えるようになれば、アテずっぽでコードを書かなくて済むようになります

投稿2019/07/24 09:20

y_waiwai

総合スコア87774

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問