Python Class 構文のエラー内容と見つけ方を教えてください。

Question

class Student:

    def __init__(self, name):
        self.name = name

    def calculate_avg(self, data):
        sum = 0

        for num in data:
            sum += num
       
       avg =sum/len(data)
       return avg

    def judge(self, avg):
        if(avg >= 60):
            result = "passed"
        else:
            result = "failed"
        return result

a001 = Student("sato")
data = [70,65,50,90,30]
avg = a001.calculate_avg(data)
judge_result = a001.judge

print(avg)
print(a001.name + judge_result)

visual studio code (windows 10)利用しています。
下記教えてください。

(1)
avg =sum/len(data)
の箇所で下記構文エラーが起きております。
”unindent does not match any outer indentation level (<unknown>, line 12)”

構文見直してもインデントなどに問題ないように感じますが、
原因おしえていただけますでしょうか。

別質問でクラスの理解についても進めているので、
この原因解決をしてから学習を進める予定です。確認お願いします。

(2)今回の構文エラーに気付くためのソースコードの追い方（確認方法）を教えてください。

Answer 1

avg =sum/len(data)
return avg
この2行だけ、インデントが少し浅くみえます。
tabでなくspaceとかで調整されてないでしょうか。

Comments

[mannah]

ありがとうございました！インデント直したらいけました。

Answer 2

”unindent does not match any outer indentation level (<unknown>, line 12)”
こちらはインデントがずれている場合に排出されるエラーです。
12,13行目の文頭のインデント半角スペース1つ分ずれていました。
インデントは半角スペース4つ分のTABを使用しましょう。

次に、再びスクリプトを実行すると27行目のavgは正しく出力できているが、28行目で再びエラーが生じるかと思います。
TypeError: can only concatenate str (not "method") to str

一つずつprint()を使用して紐解いて行きましょう。
まずはprint(a001.name)
→こちらは正しくsatoと出力されるはずです。

次にprint(judge_result)
→<bound method Student.judge of <__main__.Student object at 0x000001956EC92BB0>>と出力され、そもそもjudge_resultが正常に代入できていなかったことがわかります。

そうなると25行目で行った代入の構文にエラーが有るということになります。
25行目では、Studentクラス内で定義したjudge関数を用いており、judge関数では引数にavgを受け取る様に構成されていますが、実際に25行目では引数を与えず関数を実行してしまっています。
エラーの原因の大元はここにあります。

judge_result = a001.judge(avg)

と正しく引数を与えてあげることで、正常にコードは実行されます。

Comments

[mannah]

大変わかりやすかったです。ありがとうございました！

Answer 3

以下のエラーが起きている箇所とその下の行ですが、1マスだけ左にずれています

avg =sum/len(data)
return avg

また、これを修正しても以下のエラーが発生します

Traceback (most recent call last):
  File ".\test.py", line 27, in <module>
    print(a001.name + judge_result)
TypeError: can only concatenate str (not "method") to str

これは24行目で

judge_result = a001.judge

として、judge_resultに関数そのものを格納しているため、結果として27行目で文字列と関数を足している為発生しています

本来であれば

judge_result = a001.judge(avg)

とし、app1.judgeには引数を与えた上で関数の実行結果を変数に格納すべきです

一応、エラーなく動くことの確認できたソースを記載しておきます

class Student:

    def __init__(self, name):
        self.name = name

    def calculate_avg(self, data):
        sum = 0

        for num in data:
            sum += num
        avg =sum/len(data)
        return avg

    def judge(self, avg):
        if(avg >= 60):
            result = "passed"
        else:
            result = "failed"
        return result

a001 = Student("sato")
data=[70,65,50,90,30]
avg = a001.calculate_avg(data)
judge_result = a001.judge(avg)

print(avg)
print(a001.name + judge_result)

追記
書き忘れましたが、エラーの見つけ方についてです
まず、PCは嘘はつかないので、インデントがおかしいと言われたら、あっているように見えても注意深く探しましょう
また、vscodeではそういった些細なミスを見つけやすくするツールがたくさんあるので導入するのもおすすめです
インデントを色わけして見やすく教えてくれるindent-rainbowや
タイポを検知してくれるspellcheckerなどです
また、prettierなど、自動でソースを整形してくれるlintツールも導入するとより実践的です

Comments

[mannah]

indent-rainbow、spellchecker入れます！
回答もありがとうございました。