質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.58%

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

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 803

epipy

score 17

 問題点

以下のコードを実行すると、 
下から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))
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+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')

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/11/23 18:47

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.58%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る