while構文を使う際のbreakの位置と使い方

Question

プログラムの無駄な部分や、直したら見やすくなる部分を教えてほしい

pythonにて、商品の価格を入力すると小計・消費税・計の３つを表示するプログラムを初心者なりに書いてみました。価格に"q"を入力すると価格の入力が終了するようにしてみました。

プログラムがごちゃごちゃしてしまい、ほかにもう少しシンプルな書き方がないのか気になりました。加えて、なんとなくでbreakを使っていたのですが、breakの正しい使い方がイマイチ良く分かっていないため、どのように使うのかを教えていただきたいです。

result = 0
i = 1
while True :
    x = input( str(i) + "個目の商品の価格を入力してください:")
    i += 1
    if x != "q":
        result += int(x)
    else :
        tax = int(result*0.1)
        print("小計:" , result)
        print("消費税:" , tax)
        print("計:", result + tax )
        break

Comments

[TakaiY]

pythonはインデントがわからないとプログラムが読めないので、質問を編集して、ソースコードは </>ボタンを押し、指示にしたがって```と```の枠の間に入れてください。

[Ficus]

編集しました。

[meg_]

> プログラムの無駄な部分や、直したら見やすくなる部分を教えてほしい
プログラムは正しく動作する、という前提で良いでしょうか？

[Ficus]

プログラムは正しく動きました！分かりづらく申し訳ありません。

[meg_]

ループ処理の話ではありませんが、「価格に"q"を入力すると価格の入力が終了する」という仕様の説明を冒頭にした方が良いのでは？と思いました。

[Ficus]

ご指摘ありがとうございます！確かに仕様は最初に書いておくべきだと思いました…。次回から気をつけて見ようと思います！

Answer 1

特に冗長な部分などは無いと思うので言うことはないのですが、コメント入れたこともあり、自分の趣味全開で書いてみます。

・入力処理と計算処理は分離したい
・そのまま「入力がqなら止める」というロジックにする
・合計をtotalと明示する
・出力の書式をprintの仕様に任せない

result = 0
i = 1
while True :
    x = input( f"{i}個目の商品の価格を入力してください:")
    if x == "q":
        break
    result += int(x)
    i += 1

tax = int(result*0.1)
total = result + tax

print(f"小計: {result}")
print(f"消費税: {tax}")
print(f"計: {total}")

やってませんが、消費税の小数点以下の扱いをどう処理すべきかについても明示的に書きたいところ。

Comments

[Ficus]

入力処理と計算処理を分けるだけでもかなり分かりやすくなりそうで参考になりました…！
詳しく教えて下さりありがとうございました！