このプログラムに問題はあるでしょうか?
2つの文字列に対して,同じ位置にある文字が一致する箇所をカウントするプログラムです.
やはりwhile分の||を&&にしなければならないのでしょうか?
#include <stdio.h>
int main(void) {
char mojiA[] = "Please Help Me!" ;
char mojiC[] = "eeeeeeeeeeeeeeeee";
char mojiB[] = "Please give me some money" ;
int count ; /* 一致した文字数 */
int i ;
count = 0 ;
i=0;
do{
if (mojiA[i] == mojiB[i]) {
count++ ;
}
i++;
} while(mojiA[i]!=0 || mojiB[i]!=0);
printf("%d\n", count) ;
return 0 ;
} ;
No.2ベストアンサー
- 回答日時:
★文字列の最後は NULL 文字です。
・このため、mojiA、mojiB のどちらか一方が NULL 文字に到達したときに while 文のループを
抜けるようにします。こうしないと NULL 文字以降の文字列ではないメモリとの比較・カウント
が行われ正しくカウントされませんよ。
・よって問題ありです。→回答者 No.1 さんのアドバイスどおりに do-while 文だと mojiA、mojiB
が最初から『空文字列』のときに1カウントしたり、do-while 文を抜け出せなくなってしまい
無限ループしてしまうかも…。怖いよ~無限ループは。
・それではどうするか?
・do-while 文から while 文または、for 文に書き直します。
・私は、for 文の方が分かりやすいと思います。→for(初期化 ; 条件式 ; 増減式 ){ … }
手直しサンプル:
int main(void)
{
char mojiA[] = "Please Help Me!";
char mojiC[] = "eeeeeeeeeeeeeeeee"; ←これ必要なの?
char mojiB[] = "Please give me some money";
int count = 0; ←宣言と同時に初期化も可能
int i;
for ( i = 0 ; (mojiA[i] != '\0') && (mojiB[i] != '\0') ; i++ ){ ←『&&』ですね
if ( mojiA[i] == mojiB[i] ){
count++;
}
}
printf( "%d\n", count );
return( 0 );
}←ここにあった『;』文字は必要ありません。
最後に:
・質問者さんも『やはり while 文の || を && にしなければ』と分かっているようですね。
・この場合は『&&』の演算子になります。
・また、文字列の最後を表す NULL 文字は数値の 0 ですが、一般的に『'\0'』という表現をします。
もちろん、数値の 0 でも動作しますが…。
あと NULL 文字の間違った表現に『NULL』という記号定数を使う人がいますが、こちらは
『NULLポインタ』という意味です。これも、数値的には 0 だったりしますが厳密にはポインタ
ですので NULL 文字を表現するのに使ってはいけません。注意。
・『NULL文字』、『NULLポインタ』、『空文字列』をしっかり区別しましょう。
・以上。おわり。
No.1
- 回答日時:
>やはりwhile分の||を&&にしなければならないのでしょうか?
うん。
これだと mojiB の方が長かった場合に(この場合まさにそう) Please Help Me! を越えてメモリを読みにいっちゃうからね。
mojiC が mojiA と mojiB の間に挟んであるけど、メモリの配置までその順になるとは保証されないと思う。
ついでに do { } while() でループをまわすと mojiA, mojiB が ""(空)の場合に mojiA[0] を終了判定せずに mojiA[1] をアクセスしてしまうので while ループにしましょう。
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
似たような質問が見つかりました
- C言語・C++・C# C言語のエラーについて 2 2022/07/11 13:56
- C言語・C++・C# c言語配列の結合についてです。 なぜうまくいかないのでしょうか。 #include <stdio.h 4 2022/05/30 22:42
- C言語・C++・C# 宣言する関数の形が決まっている状態で、 str1とstr2の文字列をこの順に引っ付けてstrに保存し 2 2022/05/30 18:21
- C言語・C++・C# プログラミングの授業の課題です 1 2023/01/17 22:15
- C言語・C++・C# c言語 プログラムのエラー 1 2023/02/11 20:31
- C言語・C++・C# c言語でユーザ関数を利用して入力された文字列を反転させるプログラムを作りたいです。 3 2023/01/29 19:47
- C言語・C++・C# プログラミング c言語 4 2023/03/07 01:05
- UNIX・Linux 次の要件を満たすにはどのように修正したらよろしいでしょうか 1 2022/11/24 20:57
- C言語・C++・C# C言語 3 2022/11/09 13:27
- C言語・C++・C# バイナリファイルをコピーするのにかかる時間を測りたいのですが実行するとFatel error:gli 2 2022/11/03 01:10
関連するカテゴリからQ&Aを探す
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
C言語 strstrの実装
-
DWORDとcharの変換
-
char型の文字列をウィンドウに...
-
C言語のintとcharの違いってな...
-
苦しんでCを読んでchar型は文字...
-
csvファイルをfscanfで読み込む...
-
fgetc( )の戻り値はなぜ整数??
-
CStringからchar*への型変換に...
-
文字列についていい方法はあり...
-
C言語にて構造体のメンバがNULL...
-
文字列のswap
-
なぜ?counterintuitive
-
C言語 配列の長さの上限
-
セグメントエラー
-
VBA基本構文の作り方 2列の...
-
c言語
-
mallocについて
-
C言語の関数と配列に関する質問
-
_tcscpy_s(wcscpy_s)の第二引数...
-
c#について、現在テトリスを作...
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
C言語のintとcharの違いってな...
-
char*を初期化したいのですが
-
C言語にて構造体のメンバがNULL...
-
CStringからchar*への型変換に...
-
strcat関数を自作したいです
-
new charとnew char[N]の違いは?
-
csvファイルをfscanfで読み込む...
-
char型にint型の数値を代入する。
-
動的メモリの初期化方法について。
-
C言語で文字列をかえす正しい書...
-
char 文字列型 の表現範囲が-12...
-
文字列str内の全ての数字を...
-
DWORDとcharの変換
-
fstream型オブジェクトを関数の...
-
C言語のプログラムについてです
-
小数点入りの文字列をfloat型に...
-
szとlpszの違い
-
const char* s1とただのchar s1...
-
文字列内の数字削除
-
c言語でポインタ変数を用いた配...
おすすめ情報