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

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

ただいまの
回答率

90.34%

  • Python 3.x

    7367questions

    Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

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

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 139
退会済みユーザー

退会済みユーザー

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

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

123 山田 10
456 高橋 20
789 鈴木 0
321 田中 30
#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


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

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • LouiS0616

    2018/06/07 18:03

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

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2018/06/07 18:04

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

    キャンセル

回答 3

checkベストアンサー

+1

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

  • 逆に、なぜ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

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/06/07 19:25

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

    キャンセル

  • 2018/06/07 19:25

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

    キャンセル

+1

ファイル読込処理はおいといて。
クラス定義部分での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()

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/06/07 19:28

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

    キャンセル

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

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/06/07 19:27

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

    キャンセル

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

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

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

  • Python 3.x

    7367questions

    Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。