「スッキリわかるPython入門」の練習問題レビュー

Question

前提

「スッキリわかるPython入門」の練習問題を解いています。
私がテキストの解答と異なる別解で解いた場合、コードレビューをしていただきたいです。
別解（私の書いたコード）の問題点を中心にご意見いただきたいです。

問題文

(1)〜(6)のように動作するプログラムをwhile文を用いて作成してください。

(1) 変数countを任意の値で初期化する。
(2) 画面に「カレーを召し上がれ」と表示する。
(3) 画面に「◯皿のカレーを食べました」と表示する（◯には食べた皿数が入る）。
(4) 画面に「おかわりはいかがですか？ (y/n)>>」と表示する。
(5) yが入力されたら、変数countの値を1増やして(3)へ戻る。
(6) nが入力されたら、「ごちそうさまでした」と表示して終了する。

該当のソースコード

#練習4-2　私の解答
count = 1
okawari = 'y'
print('カレーを召し上がれ')
while okawari == 'y':
    print(f'{count}皿のカレーを食べました')
    okawari = input('おかわりはいかかですか？ (y/n)>>')
    if okawari == 'y':
        count += 1
print('ごちそうさまでした')

#練習4-2 テキストの解答
count = 1
ans = True
print('カレーを召し上がれ')
while ans == True:
    print('{}皿のカレーを食べました'.format(count))
    key = input('おかわりはいかがですか？　(y/n)>>')
    if key == 'y':
        count += 1
    else:
        ans = False
print('ごちそうさまでした')

Comments

[Zuishin]

そもそも、基本的な構文を初心者に教えるという縛りのあるコードでレビューも何もありません。
どちらも大した違いはなく、重箱の隅をつつく程度のことにしかならないでしょう。
簡単な構文だけを使い、同じ動きをするのならばどちらも正解です。

print や input がロジックと密接に絡んでいるのが保守性を下げているので、「より良いコード」を目指すのであれば、依存性の注入やクラス分け、関数化、型ヒントの付与、ドキュメントコメントの記述、バリデーション、エラーハンドリングを行うことになります。

Answer 1

「入門」という本のタイトルを考えると、質問者さんのコードも模範回答のコードもどちらも良いコードだと思います。

ものすごく細かいことを言うと、「おかわりへの返答がy/n以外だったらどうするのが正しいのか」という疑問はありますが。
あとは好みの問題ですが、私なら以下のように書きます。

count = 1
print('カレーを召し上がれ')
while True:
    print(f'{count}皿のカレーを食べました')
    ans = input('おかわりはいかがですか? (y/n)>> ')
    if ans == 'y':
        count += 1
    if ans == 'n':
        print('ごちそうさまでした')
        break

Answer 2

個人的な見解になりますが…

質問者様のコードの方が優れていると思った部分

• f文字列を使用しており、可読性がいい。

# ✅ 質問者様のコード
print(f'{count}皿のカレーを食べました')

# 🔺 教材の回答
print('{}皿のカレーを食べました'.format(count))

教材の方が優れいると思った部分

• 一般的に、識別子に日本語（ローマ字）を使うのは避けた方がいいと言われています。

# 🔺 質問者様のコード
okawari

# ✅ 教材の回答
ans

• 非常に微妙なラインですが、一般的にフラグの管理はboolean型でされることが多いと思います。ただ質問者様のコードの方が行数が少なくてすみますので甲乙つけがたいです。

# 🔺 質問者様のコード
okawari = 'y'

# ✅ 教材の回答
ans = True

おまけコード

私だったら以下のようにコードを書きます。
基本的に教材のコードと同じです。

count = 1
responce = True

print('カレーを召し上がれ')

while responce == True:
    print(f'{count}皿のカレーを食べました')

    # .lower()により、ユーザーが大文字で入力したとしても、小文字に変換する
    key = input('おかわりはいかがですか？　(y/n)>>').lower()

    if key == 'y':
        count += 1
    elif key == 'n':
        responce = False
        
    # Y、y、N、n以外が入力されたら、このwhile文の先頭の処理からやり直す
    else:
        continue

print('ごちそうさまでした')

• 完全に個人の好みの問題なのですが、ansよりresponceが好きです…。または省略せずanswerの方が好きです。
• ユーザーが大文字で入力しても大丈夫なようにしました。（Y y N nok）
• Y y N n以外が入力された時の処理を追加しました。