作成したプログラムの失敗ケースはどんな場合か？

フォーラム(掲示板)ルール
フォーラム(掲示板)ルールはこちら　　※コードを貼り付ける場合は [code] と [/code] で囲って下さい。詳しくはこちら

トピックに返信する


答えを正確にご入力ください。答えられるかどうかでスパムボットか否かを判定します。
BBCode: ON
[img]: ON
[flash]: OFF
[url]: ON
スマイリー: OFF
トピックのレビュー
   

展開ビュー トピックのレビュー: 作成したプログラムの失敗ケースはどんな場合か？

Re: 作成したプログラムの失敗ケースはどんな場合か？

#4

by NightShift » 5年前

HelloWです。（すみません。入力した ユーザー名 は既に使用されています。他のユーザー名をご入力ください。と出るので、ログインしました。）

maruさん、かずまさん

ご説明をありがとうございました。
解決しました。

>かずまさん
ご丁寧に分かりやすい説明を感謝申し上げます。

Re: 作成したプログラムの失敗ケースはどんな場合か？

#3

by かずま » 5年前

HelloW さんが書きました: ↑
5年前
 考えましたが、どんな場合に上記の動作をしないのかがわからないので、教えてください。
char S[102]; は初期化されていないので、何が入っているかわかりません。

scanf("%s", S); で abc を入力すると、
S[0] = 'a', S[1] = 'b', s[2] = 'c', s[3] = '\0' と
先頭 4バイトだけ値が設定されます。
残りの S[5]～S[101] の 98バイトに 'I', 'i', 'l' が
入っている可能性があります。

その場合、sizeof(S) を使って配列全体を探索すると、
入力の abc に Iil が含まれていないにもかかわらず
caution が表示されてしまいます。

入力文字列の最後で探索を終了するために次のように変更してください。

コード: 

    for (int i = 0; S[i] != '\0'; i++)
また、最初の forループで Iil の有無が分かるので、
二度目の forループは要りません。

コード: 

#include <stdio.h>
#include <stdlib.h>  // system

int main(void)
{
	char S[102];
	scanf("%s", S);
	int i;
	for (i = 0; S[i] != '\0'; i++) {
		if (S[i] == 'i' || S[i] == 'l' || S[i] == 'I') {
			printf("caution");
			break;
		}
	}
	if (S[i] == '\0') printf("%s", S);
	printf("\nstop");
	system("pause");
	return 0;
}
さらに、<string.h> には strpbrk という関数があるので、
次のように書くこともできます。

コード: 

#include <stdio.h>   // scanf, puts, printf
#include <string.h>  // strpbrk
#include <stdlib.h>  // system

int main(void)
{
	char S[102];
	if (scanf("%101s", S) != 1) return 1;
	puts(strpbrk(S, "iIl") ? "caution" : S);
	printf("stop");
	system("pause");
	return 0;
}

Re: 作成したプログラムの失敗ケースはどんな場合か？

#2

by maru » 5年前

for文で回す回数が間違っています。sizeof(S)は102になり、文字列長ではありません。
そのため、文字列長が101未満の場合、配列Sに不定値が残ります。
オフトピック
文字列長102以上の場合、深刻な問題を引き起こす可能性があります。

作成したプログラムの失敗ケースはどんな場合か？

#1

by HelloW » 5年前

お世話になります。

すみません。「I」(大文字のアイ)と「l」(小文字のエル)と「i」(小文字のアイ)の文字がいずれか1つ以上含まれる場合は「caution」を出力し、そうでない場合は入力した文字列をそのまま表示するプログラムを作りたいのですが、あるケースの場合、上記の動作をしない場合があるみたいです。（作ったプログラムをテストするプログラムにかけたら失敗ケースが１つありました。）

考えましたが、どんな場合に上記の動作をしないのかがわからないので、教えてください。
どうかよろしくお願いします。

コード: 

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

int main()
{
	//・1 行目には大文字小文字を含む半角英字アルファベットで構成される文字列 S
	//・入力は 1 行となり、末尾に改行を１つ含みます。
	char S[102];
	scanf("%s", S);

	//テスト
	//printf("%s", S);

	//Iかiかlが入力されたらcaution	 
	for (int i = 0; i < sizeof(S); i++)
	{
		if (S[i] == 'i' || S[i] == 'l' || S[i] == 'I')
		{
			printf("caution");
			break;
		}
	}

	//Iもiもlも入力されなかったら、入力された文字列をそのまま表示
	int counter = 0;
	for (int i = 0; i < sizeof(S); i++)
	{
		if (S[i] != 'i' && S[i] != 'l' && S[i] != 'I')
		{
			counter++;
		}

		if (counter == sizeof(S))
			printf("%s", S);
	}

	printf("\nstop");
	system("pause");
	return 0;
}

ページトップ