【C言語の質問です】偶数には★を奇数には●を表示するプログラムを作成するには・・

Question

＊たくさんの方よりご指摘をいただけてとてもありたいです。
引き続き、よろしくお願い致します。

C言語を勉強している初心者のものです。

今課題をいただき、その作成をしております。
先日提出(全18問）したのですが、大部分が間違っているので、
再度問題を見なおした上で提出してくれと言われてしまいました。

今回は課題なので、間違っている点を教えていただけませんでした。
個人的には、どの辺りが間違っているのかわからないため、
客観的な視点が欲しく、今回ご相談させていただきました。

18問の中の1つですが、
ソースが問題文との趣旨と外れているようでしたら
ご指摘いただけますと幸いです。

【問題】
1-20までのうち、「偶数」には★を、「奇数」には●をつけて表示するプログラムを作成せよ。

【回答】

#include <stdio.h>

int main(void)
{	
	int i;

	for(i = 1 ; i <= 20 ; i++)
	{
		if(i % 2 == 0)
		{
			 printf("%2d★\n",i);
		}
		if(i % 2 == 1)
		{
			 printf("%2d●\n",i);
		}
	}

	return 0;

}

Answer 1

処理の共通化、分岐の省略を考え、自分ならこうします。

#include <stdio.h>

int main(void)
{    
    int i;
    char *mark[] = { "★", "●" };
    
    for(i = 1 ; i <= 20 ; i++)
    {
        printf("%2d%s\n", i, mark[i % 2]);
    }

    return 0;
}

Comments

[退会済みユーザー]

ご回答ありがとうございます。
こちらはポイントに★と●を読み込ませて、
i % 2の計算結果次第で、分けて表示するというものでしょうか。
すいません、理解が追いついておりませんでしたので、
質問させていただきました。

[ttyp03]

i % 2 の結果は 0 か 1 しかありませんから、それをマーク文字列の配列の添え字に利用しています。
例としてソースを書きましたが、初歩の課題レベルでは、
if( i % 2 == 0 ){
…
} else {
…
}
でも良いと思いますよ。

[WoodenHamlet]

たしかにこれは綺麗ですね！

mark[0] = "★"（型: char*）
mark[1] = "●" (型: char*)

を用意して mark[i%2]... 0:偶数、1:奇数 で分けて表示しているということですね

[退会済みユーザー]

ttyp03さんご回答ありがとうございます。
WoodenHamletさんも追記ありがとうございました。
理解できました。
配列の認識が甘かったので、
再度勉強しなおしてみます！

Answer 2

間違いというほどではないと思いますが、無駄があるとはいえます。私なら以下の通り修正します。

(1) 整数iを2で割った余りは0か1しかないので、ifを2回書く必要はない(elseを使う)
(2) 数値を表示するprintfはifで分ける必要はない(「★」と「●」の出力のみ分ける)

「(2)」はいま修正しても大した違いではないでしょうが、“共通化できるところはする”という考え方としてとらえていただければと思います。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
おっしゃるとおり、無駄があると処理が遅くなることもありますので、
共通化できるところはしていくように意識していきます。

ご指摘ありがとうございました！

Answer 3

マルチバイト文字というのが煩わしい…
足す文字がO,Eとかならもっと簡単なんですが…

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

int main(void)
{    
    int i;

#if 1
    char mark[3];
    for(i = 1 ; i <= 20 ; i++)
    {
        if(i % 2 == 0)
        {
			strcpy(mark, "★");
        } else {
			strcpy(mark, "●");
        }
        printf("%3d %s\n", i, mark);
    }
#else 
    char mark;
    for(i = 1 ; i <= 20 ; i++)
    {
        mark = (i % 2)? 'O': 'E';    /* 解らなかったら三項演算子でググってください */
        printf("%3d %c\n", i, mark);
    }
#endif
    return 0;
}

#else 以降は mark へ E, O を代入して出力という形ですね。'●'、'★'がマルチバイト文字なのでこの方法そのままでは実装できません（１文字で char 数個を使う）。
そこでstrcpyを使ったのですが…あんまり美しくないですね。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
すいません、理解が追いついていないので、
早速三項演算子ググって理解できるようにします！
ありがとうございました。

Answer 4

return 0

↑結果を返すならここでは？

printf文を使うこと自体は分岐の外に出せますね。
分岐内でやるべきは、

• 偶数なら数字に★をつける
• 奇数なら数字に⚫︎をつける

という操作のはずです。

最終的な結果を返す処理の部分は、共通で書けるはずです。
まず、resultという変数を作り、そこにあるべき結果を代入する形にしてみてはどうでしょうか。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
代入する形を考えてみたのですが、イメージとしてはこのような形でしょうか。

int i,result;

for(i = 1 ; i <= 20 ; i++)
{
if(i % 2 == 0)
{
result = ;
}
else
{
result =;
}
}

printf("%d\n",result);
return 0;

すいません、形はなんとなく理解できたのですが、
このresultになにを入れればいいのか思いつきませんでした・・・
パッとイメージした感じresult = ("%d★",i);になったのですが、いかがでしょうか。

[WoodenHamlet]

むしろ
int i;
**** result;

for(i = 1 ; i <= 20 ; i++)
{
if(i % 2 == 0)
{
result = ****;
} else {
result =****;
}
printf(****);
}

return 0;

の形だとおもいますよ

[退会済みユーザー]

ありがとうございます。
for文の最後にprintf関数を入れるってことだったんですね！
たしかにそのほうがしっくり来るのは理解できました。

ちなみにresultの数値はどのような数値が入りますか？
考えてはいるのですが、思いつかなくて・・