pythonで線形探索を行いたい

Question

pythonで線形探索を行っています。データには、番号と名前プロフィール情報が書かれていて、ユーザーが探したい番号の人を探し、０が入力されたら、エラーメッセージを表示させたいです。しかし、次のようなエラーが出てしまいます。対処法を教えていただきたいです。よろしくお願いいたします。

class Player:
	def __init__(self,number,name,profile,next):
		self.number = number
		self.name = name
		self.profile = profile
		self.next = next

def print_info(p:Player):
    while (p != None):
        print("{:>2d}".format(int(p.number)),p.name,p.profile)
        p = p.next
    print("")
    
def search(x,p:Player):
    for p in data:
        if int(p.number) ==x:
            print("{:>2d}".format(int(p.number)),p.name,p.profile)
            print("")
            break
        else:
            print("該当する選手はいません。")
            print("")
            
        

data = []

f = open('zac_japan.txt','r',encoding = 'utf-8')
data = f.readlines()
f.close

p=None
for n in data:
    new = (n.split())
    p = Player(int(new[0]),new[1],new[2],p)
    
print_info(p)

while True:
    print("探索する登録番号は？")
    x=int(input())
    for n in data:
        new = (n.split())
        p = Player(int(new[0]),new[1],new[2],p)
        if x == 0:
            print("プログラムを終了します。")
        else:
            search(x,p)

エラーメッセージ

 if int(p.number) ==x:
AttributeError: 'str' object has no attribute 'number'

Answer 1

class Player:
    def __init__(self,number,name,profile,next):
        self.number = number
        self.name = name
        self.profile = profile
        self.next = next

def print_info(p:Player):
    while (p != None):
        print("{:>2d} {} {}".format(p.number, p.name, p.profile))
        p = p.next
    print()

def search(x, p:Player):
    while p != None:
        if p.number == x:
            print("{:>2d} {} {}\n".format(p.number, p.name, p.profile))
            return
        p = p.next
    print("該当する選手はいません。\n")

f = open('zac_japan.txt','r',encoding = 'utf-8')
data = f.readlines()
f.close

p = None
for n in data:
    new = n.split()
    p = Player(int(new[0]), new[1], new[2], p)

print_info(p)

while True:
    print("探索する登録番号は？")
    x = int(input())
    if x == 0:
        print("プログラムを終了します。")
        break
    search(x, p)

動きました、ありがとうございました、で済ませないでください。

このコードで分からなないところは何か、よく考えて、ちゃんと調べて
それでも分からないところは、質問してください。

それから、質問にある元のプログラムの何が悪いのかが分かったなら、
それを説明してください。分からいないなら質問してください。

プログラムは論理的思考によって作成します。

例えば、data = f.readlines() で上書きされてしまうのに。
data = [] は何のためにあるのですか?
不要だと思いませんか？

読み込んだデータの最初の項目は int(new[0]) で int に変換しています。
それは Player の number に代入しますから、number は int です。
表示の時に int(p.number) とする必要はありません。

def search(x,p:Player)
    for p in data:
        if int(p.number) ==x:

自分で作った線形リスト p のなかにすべてのデータがあるのに、
どうしてまた入力データの data の中身を見ようとするのですか？
for の p は線形リストの p ではありません。
data が str のリストで、各str は各行の内容です。
p はその 1行の内容で、str です。
str に number という attribute はありません。

追記

print_info の中の print で余計な "\n" があったので削除しました。
search の中で、print() により空行を出していたのをその直前の
print の書式を "{:>2d} {} {}\n" にして、print() を削除したとき、
print_info の print も同じ書式にしようとコピペしたため
このバグが入ってしまいました。

Comments

[Q71]

ああ、なるほど。
https://teratail.com/questions/197418
こっちから続いているんですね。

Answer 2

def search(x,p:Player):
    for p in data:

原因はこの辺りでしょうね。
for ブロックの中で p は何を指すか、考えてみましょう。

Answer 3

pはstrなので、number というメンバーがない、といってます。
そこらへんしらべてみては