AttributeErrorが出て、結びつきを確認してみたのですが、原因が分かりません

Question

問題点

以下のコードを実行すると、
下から10番目の
sixHundred.addStudent(ug1)
部分で停止し、
self.grades[student.getIdNum()] = []
AttributeError: 'Grades' object has no attribute 'grades'
というエラーが出てしまいます。
エラー文を見る限り、gradesとGradesが結びついていないみたいで、
class Grades(object):の
def addStudent(self, student):
の部分に問題があると思い、確認してみたのですが、インデントレベルなどを確認しても特に問題はないように思えます。

コードの不備にお気づきの方がいらっしゃいましたら、ご教授の程宜しくお願いします。

該当のソースコード

class Person(object):

    def __init__(self, name):
        """"「人間」を形成する"""
        self.name = name #selfにname(入力)を代入する
        try:
            lastBlank = name.rindex(' ') #lastBlankにnameの空白の後ろからの位置番号を代入
                                         #rindex()は()を後ろから探したときの出現位置を返す
            self.lastName = name[lastBlank + 1:] #姓をself.lastNameに代入
        except: #空白が見つからないとき
            self.lastName = name
        self.birthday = None #初期化

    def getName(self):
        """selfの名前(フルネーム)を返す"""
        return self.name

    def getLastName(self):
        """selfの姓を返す"""
        return self.lastName

    def setBirthday(self, birthdate):
        """birthdateをdatetime.date型とする
           selfの生年月日をbirthdateと設定する"""
        self.birthday = birthdate

    def getAge(self):
        """selfの現在の年齢を日単位で返す"""
        if self.birthday==None:
            raise ValueError
        return (datetime.date.today() - self.birthday).days #今日の日付-誕生日 (.daysで日に換算)

    def __lt__(self, other):
        """selfの名前がotherの名前と比べて
           アルファベット順で前ならばTrueを、
           そうでなければFalseを返す
           比較は、姓について行われるが、
           姓が同じであれば名前(フルネーム)が比較される"""
        if self.lastName==other.lastName: #selfの姓がotherの姓とは同じ場合
            return self.name < other.name #フルネームの比較に入る
        return self.lastName < other.lastName #selfの名前がotherの名前と比べてアルファベット順で前
    
    def __str__(self):
        """selfの名前(フルネーム)を返す"""
        return self.name


class MITPerson(Person): #属性の継承

    nextIdNum = 0 #個人識別番号

    def __init__(self, name):
        Person.__init__(self, name) #Personの中のinitを参照
        self.idNum = MITPerson.nextIdNum #selfのidNumに、MITPersonのnextIdNumを代入する
        MITPerson.nextIdNum += 1 #MITPersonのnextIdNumを1増やす

    def getIdNum(self):
        return self.idNum

    def __lt__(self, other):
        return self.idNum < other.idNum

    def isStudent(self):
        return isinstance(self, Student) #instance(引数1, 引数2)
                                         #1番目の引数が2番目の引数で示された方の
                                         #インスタンスであるときTrue




class Student(MITPerson): #MITPerson/Student
    pass

class UG(Student): #Student/UG ,UGはStudentのサブクラス
    def __init__(self, name, classYear):
        MITPerson.__init__(self, name)
        self.year = classYear
    def getClass(self):
        return self.year

class Grad(Student): #Student/Grad, メソッド
    pass


class TransferStudent(Student): #編入生クラスを追加

    def init(self, name, fromSchool):
        MITPerson.__init__(self, name)
        self.fromSchool = fromSchool

    def getOldSchool(self):
        return self.fromSchool


class Grades(object):
    def __init__(self):
        """空の成績ブックを生成する"""
        self.students = []
        self.grade = {}
        self.isSorted = True

    def addStudent(self, student):
        """studentをStudent型とする
           studentを成績ブックへ追加する"""
        if student in self.students:                                    #AttributeError
            raise ValueError('Duplicate student') #重複した生徒
        self.students.append(student)
        self.grades[student.getIdNum()] = []
        self.isSorted = False

    def addGrade(self, student, grade):
        """gradeをfloat型とする
           gradeをstudentの成績リストへ追加する"""
        try:
            self.grades[student.getIdNum()].append(grade)
        except:
            raise ValueError('Student not in mapping')

    def getGrades(self, student):
        """studentの成績リストを返す"""
        try:#studentの成績リストのコピーを返す
            return selg.grades[student.getIdNum()][:]
        except:
            raise ValueError('Student not in mapping')

    def getStudents(self):
        """成績ブックに収められた学生の、ソートされたリストを返す"""
        if not self.isSorted: #ソートされていない場合
            self.students.sort()
            self.isSorted = True
        return self.students[:] #学生のリストのコピーを返す


    def gradeReport(course):
        """courseをGrades型とする"""
        report = ''
        for s in course.getStudents():
            tot = 0.0
            numGrades = 0
            for g in course.getGrades(s):
                tot += g
                numGrades += 1
            try:
                average = tot/numGrades
                report = report + '\n'\
                         + str(s) + '\'s mean grade is ' + str(average)
            except ZeroDivisionError:
                report = report + '\n'\
                        +str(s)+' has no grades'
        return report

ug1 = UG('Jane Doe', 2014)
ug2 = UG('John Doe', 2015)
ug3 = UG('David Henry', 2003)
g1 = Grad('Billy Buckner')
g2 = Grad('Bucky F. Dent')
sixHundred = Grades()
sixHundred.addStudent(ug1) 
sixHundred.addStudent(ug2)
sixHundred.addStudent(g1)
sixHundred.addStudent(g2)
for s in sixHundred.getStudent():
    sixHundred.addGrade(s, 75)
sixHundred.addGrade(g1, 25)
sixHundred.addGrade(g2, 100)
sixHundred.addStudent(ug3)
print(gradeReport(sixHundred))

Answer 1

class Grades(object):
def init(self):
"""空の成績ブックを生成する"""
self.students = []
self.grade = {}
self.isSorted = True

class Grades(object)
...
def addStudent(self, student):
"""studentをStudent型とする
studentを成績ブックへ追加する"""
if student in self.students: #AttributeError
raise ValueError('Duplicate student') #重複した生徒
self.students.append(student)
self.grades[student.getIdNum()] = []
self.isSorted = False

片やgradeで、片やgradesだからです。統一してください。

---

また、次の部分ですが...

class Grades(object)
...
def addGrade(self, student, grade):
"""gradeをfloat型とする
gradeをstudentの成績リストへ追加する"""
try:
self.grades[student.getIdNum()].append(grade)
except:
raise ValueError('Student not in mapping')

try-exceptが物凄く気になります。
例外は確かに強力ですが、この用法だと自らの首を絞めます。

>>> try:
...     haoigheowahfawfa
... except:
...     pass
...
>>> # こんなコードでもエラーが出ないで通ってしまう

しっかりと用法を把握するまでは、できるだけif文で対処した方が良いです。
例外を用いるにしても、せめて例外オブジェクトの種類を限定してください。

class Grades(object)
    ...
    def addGrade(self, student, grade):
        """gradeをfloat型とする
           gradeをstudentの成績リストへ追加する"""
        try:
            self.grades[student.getIdNum()].append(grade)
        except KeyError:
            raise ValueError('Student not in mapping')

これなら幾分かマシです。

Comments

[epipy]

もう一度、全体のコードを見返してみるとコードミスが、所々に見つかりました。　以後注意深くコードを見返すようにします。
try-exceptより、できるだけif文を使った方がいいとのことでしたので、該当箇所を書き換えてみると、コードが格段に見やすくなりました。
迅速な回答に加え、助言までありがとうございました！