Python 変数の扱い

Question

前提・実現したいこと

pythonの学習を始めたばかり者なのですが、最初の学習として様々な変換式を扱っています。
購入するものに消費税を掛けて、支払金額を表示するプログラムを作成したいのですが、入力する変数の値がマイナスの時に'入力金額が不適切な値です'と表示させ、入力金額が数字でない場合（例えば漢数字など）、'入力金額が不適切です'と表示させたいです。この二つを同時に満たし、なおかつこの両者の場合は税率計算をしないでそこでプログラムがおわるものを作りたいのでご助力お願いします。

発生している問題・エラーメッセージ

不適切な数値の場合に不適切と表示させ、そうでない場合は計算結果を表示させたい。

該当のソースコード

import sys
try:
x = input('品物の価格を入力してください:')
x.isdecimal()
x = int(x)
if x <= 0:
print('正しい金額が入力されていません')
sys.exit()
except SystemExit:
pass
except:
print('整数以外が入力されました')
else:
print('支払金額は', round(x * (1.1)), '円です')

試したこと

try exceptでエラーを発生・回収するプログラム（不要なエラーを発生させてるため美しくない）

補足情報（FW/ツールのバージョンなど）

jupyter notebook

Comments

[meg_]

コードを掲載してください。

[new_lemonade]

失礼しました、編集します

[meg_]

既に回答が付いているので、もし次回質問することがありましたら、コードは「コードの挿入｣で記入してください。※インデントを表示するため

[new_lemonade]

ご迷惑をおかけしました。今後は必ず気を付けます。

Answer 1

関数を定義すると良いです。returnで脱出できます。

def main(tax_rate=1.1):
    x = input('品物の価格を入力してください:')
    try:
         x = int(x)
    except ValueError:
         print('整数以外が入力されました')
         return

    if x < 0:
        print('正しい金額が入力されていません')
        return

    print(f'支払金額は{round(x * tax_rate)}円です')

なお、数値の判定は正規表現を使っても簡潔でしょう。

import re

def main(tax_rate=1.1):
    x = input('品物の価格を入力してください:')
    if not re.fullmatch(r'-?[1-9][0-9]*', x):
        print('整数以外が入力されました')
        return

    x = int(x)
    if x < 0:
        print('正しい金額が入力されていません')
        return
    
    print(f'支払金額は{round(x * tax_rate)}円です')

コードの貼り方について

teratailには、コードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押して下さい。

特にPythonの場合、インデントが崩れるとコードの意味が変わってしまいます。

Comments

[new_lemonade]

解答及びコードの件ありがとうございます。関数定義でやるという方法もあるのですね...とても励みになりました。今後も学習を頑張りたいと思います！！

Answer 2

try-exceptによる方法が美しくないという感性，よくわかります．

私なら，という方法ですが，

import sys

x = input("品物の価格を入力してください:")
if not x.isdigit():
  print('整数以外が入力されました')
  exit(1)
print(f"支払金額は{round(int(x) * 1.1)}円です")

とします．isdigitはisdecimalと異なりハイフンも認めません．10進数数値で用いられる数字のみです．よって数値であることと同時に正の数であることを保証できると考えられます．

if文で除外条件を確認し，該当するなら終了．違うならば処理を継続する，というスタイルです．
プログラムの構造としてはelse節に税率計算処理を記述するほうが正しいのでしょうが，Pythonの場合，インデントを深くすると面倒なのでelseを使っていません．

Comments

[new_lemonade]

ご回答ありがとうございます！！
環境がJupyter notebookなのでそのせいなのもあるかもしれませんが、これを実行すると、
　整数以外が入力されました
　支払金額は-330円です
と返ってきます。exitがifの操作を終わらせただけで、結果的に計算をしてしまうのですがこれはその他の環境であれば支払金額の計算をせずに終わりますか？できれば不適切な値の場合は金額計算をせずにコメントだけ表示するプログラムを作りたいです。

[ebal]

それならばifで場外したあとelse節で金額表示，あるいはその逆ですね．
exit(1)はpython インタプリタならば異常終了で制御を端末に戻しますがJupyterの場合は確かにそうなりますか……

例:
```python
import sys

x = input("品物の価格を入力してください:")
if x.isdigit():
print(f"支払金額は{round(int(x) * 1.1)}円です")
else:
print('整数以外が入力されました')
```

[LouiS0616]

str.isdigitもだいぶ癖があって、例えば '①'.isdigit() がTrueを返したりします。
(細かいところでアレですが)