float(変数)の行でプログラムの実行が止まる

Question

競技プログラミングの問題にて
if,elseで選手の体重ごとにボクシングの階級を出力していたところ、下記コードではprint('hoge')が実行されませんでした。なぜでしょうか？また、入力した文字列を一行でstr→int→floatに変換するとしたらどのように書くべきでしょうか。

while True:
    try:
        w = int(input())
        w = float(w)
        print('hoge')
        if w > 91.00:
            print('heavy')
        elif w > 81.00:
            print('light heavy')
        elif w > 75.00:
            print('middle')
        elif w > 69.00:
            print('light middle')
        elif w > 64.00:
            print('welter')
        elif w > 60.00:
            print('light welter')
        elif w > 57.00:
            print('light')
        elif w > 54.00:
            print('feather')
        elif w > 51.00:
            print('bantam')
        elif w > 48.00:
            print('fly')
        else:
            print('light fly')
    except:
        break

Answer 1

まず int(input()) でエラーを吐いています。

>>> int('3.14')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: invalid literal for int() with base 10: '3.14'

intを介さずにfloatに直接渡せば良いでしょう。

>>> float('3.14')
3.14

---

try-except で握り潰してしまうと、エラーメッセージから原因を知る機会を失います。
特に except: なんて広域な捉え方をしているコードは、絶対に参考にしてはいけません。

できるだけ狭い範囲を、できるだけ狭く適切な例外クラスで捕捉するようにします。

>>> try:
...     float('')
... except ValueError:
...     print('break')
...
break

例外を使わずとも処理を簡易に表現できるときは、避けた方が賢明です。(註)

>>> if '':
...     pass
... else:
...     print('break')
...
break

---

註： 例外は避けるべき？

意見が分かれるかもしれませんが、例外はただの『(比較的)安全なgoto』だと思っています。
むしろ上位に伝播されるぶん悪質かも。

もちろん『一切使うべきでは無い』という主張ではありません。

Comments

[esklia]

ご回答くださりありがとうございます。直接floatでいいのですね。以前str→floatでエラーを吐いた記憶があったのですが、のちに試してみます。exceptについても仰るとおり適切な例外処理を施すようにいたします。教えていただきありがとうございます。