python classのエラー文

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
pythonでclassの機能を使って4つの変数を使って情報を扱うことができるプログラムを作っています。それに加えて5つ目の変数を作成し，4つの変数に入力されたものを5つ目に記憶させて好きなタイミングでプログラムを終了し，まとめて出力させるプログラムを作ろうとしていました。
上記の機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

Traceback (most recent call last):
File "C:\Users\無題-72.py", line 6, in <module>
while x.name != "end":
AttributeError: name

該当のソースコード

class	P(object):
	__slots__=["name","phonenumber","zipcode","address","link"]
x=P()
top=x
x.name=0
while x.name != "end":
	x.name=input('Name>')
	x.phonenumber=input('Phonenumber>')
	x.zipcode=input('Codenumber>')
	x.address=input('MOJI>')
	x.link=P()
	x=x.link
x.link=None
while x.link!=None:
		print(x.name,x.phonenumber,x.zipcode,x.address)
		x=x.link
x=top
while x.link!=None:
	print(x.name,x.phonenumber,x.zipcode,x.adress)
	x=x.link

Phonenumber>342
Codenumber>124
MOJI>g
python

試したこと

出てきたエラー文を検索にかけたが類似するエラー例が見つからなかった為どのように対処すればいいのかわからず手が止まっています。

補足情報（FW/ツールのバージョンなど）

pythonの3.6のバージョンのものを使用しています。

Answer 1

問題点について

同じエラーを出力する単純なソースコード

class P(object):
    __slots__ = ["name"]

x = P()
print(x.name)

を用いて説明します。x = P()でクラスPのインスタンスが作成されますが、この時点ではnameというアトリビュートは作成されていません。したがって元々のソースコードにあるwhile x.name != "end":でx.nameが文字列endかどうかを調べていますが、x.nameというアトリビュートが予め作成されている必要があります。

1回目はwhileの直前にx.name=0とありますので、ここでインスタンスxに対してnameというアトリビュートが作成されています。しかし2回目以降は

x.link=P()
x=x.link

となっていて、x.nameが作成されていません。そのためAttribute errorとなります。

解決方法について

x.nameが予め作成されていれば問題解決します。オリジナルのソースコードを以下のように2箇所変更すれば動作します。

class    P(object):
    __slots__=["name","phonenumber","zipcode","address","link"]
x=P()
top=x
x.name=input('Name>')  # x.name=0ではなくここで入力してしまう
while x.name != "end":
    x.phonenumber=input('Phonenumber>')
    x.zipcode=input('Codenumber>')
    x.address=input('MOJI>')
    x.link=P()
    x=x.link
    x.name=input('Name>')  # 次のループのためにnameアトリビュートを作成
x.link=None

x=top
while x.link!=None:
    print(x.name,x.phonenumber,x.zipcode,x.adress)
    x=x.link

Comments

[waddle_dee19]

なるほど，やはり二回目のための準備がちゃんとできていないのが問題だったのですね
ありがとうございます！

Answer 2

slots でクラス変数を宣言することができますが、宣言した変数をコンストラクタで
初期化してあげないとアクセス時にエラーになるようです。

質問のコードだと、入力1周目のときは、直前で x.name=0 としているので while x.name='end' を判定できますが、2周目以降は同様の処理がないので、 whileの判定時に初期化されていない name にアクセスしようとしてエラーになってしまいます。

修正したコードでは、クラスPにコンストラクタを追加して、そこで中身を初期化するようにしてみました。
あと、endの判定タイミングの問題で、最後に name = end な P が追加されてしまっていたのでそこも修正してあります。

class P(object):
    __slots__=["name","phonenumber","zipcode","address","link"]

    def __init__(self):
        # print("P instance created.")
        self.name = ""
        self.phonenumber = ""
        self.zipcode = ""
        self.address = ""
        self.link = None
    
x=P()
top=x
x.name=0
while True:
    x.name=input('Name>')
    if (x.name == 'end'): 
        break

    x.phonenumber=input('Phonenumber>')
    x.zipcode=input('Codenumber>')
    x.address=input('MOJI>')
    x.link=P()
    x=x.link

x.link=None
while x.link!=None:
        print(x.name,x.phonenumber,x.zipcode,x.address)
        x=x.link

x=top
while x.link!=None:
    print(x.name,x.phonenumber,x.zipcode,x.address)
    x=x.link

Comments

[waddle_dee19]

なるほど！情報が処理されていない状態で無理押し込もうとしているのが良くなかったのですね！
そのうえ他にも改善した方が良い点を教えて頂きありがとうございます