C言語5文字に1文字だけ別の文字を表示する

Question

実現したいこと

1.C言語を使って任意の値を入力します。
2.値のぶんだけ文字を表示します。
3.表示する4文字はハイフン（－）として5文字目はアスタリスク（＊）を表示します。

値：5
表示：－－－－＊

値：10
表示：－－－－＊－－－－＊

値：15
表示：－－－－＊－－－－＊－－－－＊

前提

visual studio2022を使用しています

発生している問題

意図したとおりに表示できないです。
なお、私が書いた下記2つのソースコードについて、どのように表示されるか、ソースコード末尾の実行結果をご覧ください。

該当のソースコード

#include<stdio.h>
int main(void) {
	int input = 0;
	printf("値：");	scanf_s("%d", &input);

	while (input > 0) {
		if (input % 5 == 0) {
			printf("－");

		}else{
			printf("＊");
		}
		input--;
	}
	return 0;
}
//実行結果
//値：10
//＊－－－－＊－－－－
//＊の表示位置が意図どおりでない

試したこと

コードを最初から書き換えてみましたが、意図した表示になりません

#include<stdio.h>
int main(void) {
	int input = 0, i = 0 , j = 0;
	printf("値：");	scanf_s("%d", &input);
	
	while (1) {
		while (1) {
			i++;
			if (i > 4)break;
			printf("－");

		}
		j = i;
		j++;
		printf("＊");
		if (j > input)break;
	}
	return 0;
}
//実行結果
//値：10
//－－－－＊＊＊＊＊＊
//＊の表示位置が意図どおりでない

Comments

[y_waiwai]

どういうふうに表示されるんでしょうか。
質問文を編集して説明を追記しましょう

[ceg34ry]

コメントありがとうございます。追記しました。

[y_waiwai]

勘違いなので削除

[YAmaGNZ]

for文で１～inputまで回して５の余りで判断すれば？

[ceg34ry]

YAmaGNZ様
コメントありがとうございます。
「該当のソースコード」と「試したこと」のソースコードがありますが、
「該当のソースコード」を書き換えたほうがよさそうでしょうか。

[jimbe]

どのコードでもですが、入力が 1 の時、どの行をどの順番に通って各変数がどう変化していくのかを考えているでしょうか。
同じ事を入力が 2 の時、 3 の時、 ４，５，６，…とコンピュータになったつもりでご自身がプログラムを"実行"してみてください、

Answer 1

whie文を使って、カウントダウンしてループを回してますが、
ここはカウントアップしてループを回せばお望みの結果となるかと。
まあそのためには、ループの条件のところも変える必要があります

んで、VisualStudioを使っているなら、コードの好きなところにブレークポイントを設定してそこで実行を止めて変数のナカミを見たり、１行づつ実行させてどういうふうにコードが動くのかを見れたりします
コードを組んで動かないからと思考停止してしまわないで、ご自分でどう動くのかを追いかけていきましょう

Comments

[ceg34ry]

ブレークポイントを設定する方法を教えてくださってありがとうございました。

Answer 2

皆さま、回答ありがとうございました。

Answer 3

「どんな方法がわかりやすいのか？」というのは人それぞれかと思うので，まだ出てないと見える方法を示しておく．

個人的にはこれが（表示内容をいろいろと変えて遊んだりできるという点も含めて）わかりやすいんじゃないかな，とか思うが，**PtrToNextOutputStr あたりが人によってはわかりにくい要素となるのかもしれない．

int main( void )
{
	//5パターンの文字列の繰り返し程度であれば，
	//表示パターンをこんな感じで配列として持ってしまうのがわかりやすいと思う．
	//※ここでは，配列の末尾要素として，パターンの終わりを判定する用の値としてNULLを持たせている．
	const char *Ptn[] = { "－","－","－","－","＊",NULL };

	//
	int input = 0;
	printf("値：");	scanf_s("%d", &input);

	//PtrToNextOutputStr は次に表示すべき文字列を指し示すポインタ．
	const char **PtrToNextOutputStr = Ptn;	//最初はパターンの先頭要素を指す
	while (input > 0)
	{
		//今現在 PtrToNextOutputStr が指している文字列を表示する
		printf( "%s", *PtrToNextOutputStr );

		//PtrToNextOutputStr の指す位置を次に進める．
		//ただし，NULLを指す位置に来た場合には，それは上記のようにパターンの終わりを示すので
		//その場合には先頭に戻す．
		++PtrToNextOutputStr;
		if( *PtrToNextOutputStr == NULL ){	PtrToNextOutputStr = Ptn;	}

		//
		input--;
	}

	return 0;
}

Comments

[ceg34ry]

ポインタを使ったやりたかた、理解をうながすためのソースコード中の//コメント、どちらもありがとうございました。

Answer 4

まずは、
1.C言語を使って任意の値を入力します。
2.値のぶんだけ文字を表示します。
のプログラムを書いてみましょう。10を入力したら-を10個、13を入力したら-を13個です。

それが期待通り動作するようになったら、カウンターを5で割った余りによって表示文字を*か-かを選択します。
ifとelseですね。ループの仕方は変えずに表示部分だけ変更します。

なお、入力10に対して、カウンターが0から9まで数えるのか、1から10まで数えるのか2パターン考えられますが、
0から9の場合は、4の時と9の時に*を表示するようにして、
1から10の場合は、5の時と10の時に*を表示するようにということですね。

Comments

[otn]

この方法だと例えば、「5,15,25,・・の時は"+"を表示する。10,20,30,・・の時は"*"を、それ以外は"-"」
みたいな場合も、表示のifの所をif else if else と3分岐にするだけで対応できますが、
「試したこと」に書いてあるような方針で行くと、ループ構造の作り直しです。

問題の本質的な構造を見極めることが大事です。

[ceg34ry]

回答ありがとうございます。

Answer 5

入力値は置いておいて、やりたいのは
＊と－を表示します。
表示ルールは、－をひたすら表示して行くが、5の倍数毎に表示したいのは＊。
ということ。
printfべた書き以外で、これをまず作ることを考える。
そうすると、一例として
1.何かしらのループを作る。(for or while？)
2.そのループ内で、－を表示させていく。5個目の表示を＊にする。
これの繰り返し。
ただこれだけだと分かる。

さて、これを実現する一つの手法として、
ループ内で－を表示していく。
文字表示させたカウンタを持っておき、それが5の倍数の場合には－ではなく＊を表示する。
まずこれが作れるか？無限ループでも構わないのでやってみる。
(VisualStudio使っていれば、止めるのも簡単)
これができたら、あとはループに上限を設ける。上限は入力値である。
入力値と比較するのは、上記のカウンタ。

注意としては
・プログラム内でのカウンタの扱い、
0始まりや1始まりでカウンタの判定が変わること。
・0は何で割っても、余りは0である。

Comments

[ceg34ry]

「考え方」を教えてくださってありがとうございました。

Answer 6

最小限の変更で済ませるなら、後者のコードの

	while (1) {
		while (1) {

を

	while (1) {
		i = 0;
		while (1) {

に変更すればよいでしょう。

しかし、

1.C言語を使って任意の値を入力します。

というこですから、input は5の倍数ではないのかもしれません。その場合は後者のアプローチでは少々面倒になるので、前者のアプローチでカウントアップするように変更するのがいいと思います。

	for (int i = 1; i <= input; ++i) {
		if (i % 5 == 0) {

Comments

[ceg34ry]

解決につながりました。感謝いたします。