Classを用いたときにエラーがでる

Question

生徒番号、名前、テストの点数が書かれたテキストファイルを引数として読み込んで、ある書式で出力するというコードを書いています。

テキストファイルは以下のような例です。

１２３　山田　１０
４５６　高橋　２０
７８９　鈴木　０
３２１　田中　３０

#coding:utf-8

import sys
file = open(sys.argv[1], "r")

class Record():
    def __init__(self, num:int, name:str, score:int):
    	self.num = num
    	self.name = name
    	self.score = score
    	
data = [Record(num,name,score)]
		
def printdata():
	for r in data[0:]:
		print("{:4d},{:16s},{:3d}".format(r.num, r.name, r.score))
			
printdata()

これで実行した場合、

Traceback (most recent call last):
  File "ex07-1.py", line 12, in <module>
    data = [Record(num,name,score)]
NameError: name 'num' is not defined

以上のようなエラーが出ていまいち対処法がわかりません。
定義されていないということはどういうことでしょうか。

Comments

[LouiS0616]

『テキストファイルからデータを読み込む』とは、何を意味しているのでしょうか。詳しく追記して下さい。

[退会済みユーザー]

字足らずで申し訳ございません。修正いたします。

Answer 1

numが定期されているであろうRecordクラスの__init__のスコープから抜けているので、もう触れません、という話なのですが・・・（だからって中に入れようとか思わないでくださいね）。
他にもいろいろ懸念が生じてしまうコードだったので、ツッコミ入れます。

#coding:utf-8

import sys
file = open(sys.argv[1], "r")  # せっかくopenしたのに以下で一回も使ってない（あとcloseもしてください）

class Record():  # そもそもこれだけの処理にクラス作る意味って。この使い方なら精々tupleかdictで十分なのでは
    def __init__(self, num:int, name:str, score:int):
        self.num = num
        self.name = name
        self.score = score

data = [Record(num,name,score)]  # 本来はファイルを読み込む処理などが入るであろう場所、せめてリスト内包表記か何かであってほしかったのに要素数1のリストって・・・（numの件にはもう触れません）

def printdata():
    for r in data[0:]:  # [0:]とか要ります？
        print("{:4d},{:16s},{:3d}".format(r.num, r.name, r.score))

printdata()

Comments

[退会済みユーザー]

ご回答誠にありがとうございました。

Answer 2

ファイル読込処理はおいといて。
クラス定義部分でのnumとメインの処理部分でのnumは別物です。
よって提示エラー（定義されていません）が発生しています。

# これはクラスの定義
class Record():
    def __init__(self, num:int, name:str, score:int):
        self.num = num
        self.name = name
        self.score = score

# ここからがメインの処理

# 以下で使われる変数を用意（define）する
num = 123
name = '山田'
score = 10

data = [Record(num,name,score)] # 変数が定義されているのでOK

# Recordに実値を渡すほうが分かりやすい
data.append(Record(456,'高橋',20))

# これは関数の定義
def printdata():
    for r in data[0:]:
        print("{:4d},{:16s},{:3d}".format(r.num, r.name, r.score))

# これもメインの処理
printdata()

Comments

[退会済みユーザー]

クラス部分とメイン部分の違い理解できました。。
ご回答有り難うございました。

Answer 3

なんというか、いろいろとツッコミどころがあります。

• 逆に、なぜnumなどが既に定義されていると仮定できるのか？
• テキストファイルの数字がなぜ全角？
• printdataを関数として定義する意義がいまいちわからない

たぶん

こういうコードを書きたかったのかと推測します。

#coding:utf-8

import sys

file = open(sys.argv[1], "r")

class Record():
    def __init__(self, num:int, name:str, score:int):
        self.num = num
        self.name = name
        self.score = score

data = []
for row in file:
    num, name, score = row.split()
    num = int(num)
    score = int(score)
    data.append(Record(num, name, score))

file.close()
    
    
def printdata():
    for r in data[0:]:
        print("{:4d},{:16s},{:3d}".format(r.num, r.name, r.score))

printdata()

実行結果 Wandbox

 123,山田              , 10
 456,高橋              , 20
 789,鈴木              ,  0
 321,田中              , 30

比べやすくするために、変更は最小限にしています。
どこが変なのか見比べてみてください。

---

なお、私ならこんな風に書きます。Wandbox

Comments

[退会済みユーザー]

遅くなってしまい申し訳ありませんでした。
ご回答ご指摘誠にありがとうございました。

[退会済みユーザー]

遅くなってしまい申し訳ありませんでした。
ご回答ご指摘誠にありがとうございました。