selfオブジェクトについて

Question

class Human:
age = 0
lastname = ''
firstname = ''
height = 0.0
weight = 0.0

def get_bmi(self):
    return (self.weight) / (self.height **2)

nakata = Human()
nakata.lastname = '中田'
nakata.firstname = '敦彦'
nakata.height = '1.7'
nakata.wight = '68'

bmi = nakata.get_bmi()
print(bmi)
とコードを打ったら
line9とline17に
unsupported operand type(s) for ** or pow() 'str' and 'int'
とでました。どうすれば改善するのでしょうか？

Answer 1

nakata.heightに文字列を設定しているからです。

エラーの再現

>>> height = '1.7'
>>> height ** 2
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: unsupported operand type(s) for ** or pow(): 'str' and 'int'

このようなエラーを防ぐためにも、Human.__init__の定義・利用をお勧めします。

---

また、プログラム内でなんらかの量を扱う際には、
単位を変数名に含めるか、あるいは基本単位以外用いないようにした方が良いです。

今回の場合は前者の方が自然にコードを書けるはずですが。

コードの書き方について

teratailには、上記のようにコードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押してください。
特にPythonの場合、インデントが崩れるとコードの意味が変わってしまいます。

Answer 2

nakata.height = '1.7'
nakata.wight = '68'

この2行でメンバー変数が文字列になってしまっているので、これを数値にしてやればよい。

nakata.height = 1.7
nakata.wight = 68