Q&A
試したこと欄にあるコードを平文でなくコードブロックでお書きください、for文の階層が見えないです
前提・実現したいこと
エラーを解決したいです。
pythonでゲームを作っている最中に以下のエラーメッセージが発生しました。
発生している問題・エラーメッセージ
line 232, in <module> main() line 209, in main fieldMap = Map(screen, 'field.map', player) line 119, in __init__ self.loadMap(filename) line 157, in loadMap idx, movable = [int(tok) for tok in fi.readline().split(',')] line 157, in <listcomp> idx, movable = [int(tok) for tok in fi.readline().split(',')] ValueError: invalid literal for int() with base 10: ''
該当のソースコード
python3.9
1if __name__ == '__main__': #231.232 2 main() 3 4fieldMap = Map(screen, 'field.map', player) #209 5 6self.loadMap(filename) #119 7 8idx, movable = [int(tok) for tok in fi.readline().split(',')] #157 9 10 11
試したこと
python
1 2 for mapchipDefFile in mapchipDefFiles: 3 with open(mapchipDefFile, 'r') as fi: 4 png_f = fi.readline().strip() 5 data = MapchipData() 6 data.mapchipFile = png_f 7 self.mapchipDatas.append(data) 8 data.sheet = load_image(os.path.join('data', png_f)) 9 data.ncol, data.nrow = [int(tok) for tok in fi.readline().split(',')] 10 for row in range(data.nrow): 11 for col in range(data.ncol): 12 idx, movable = [int(tok) for tok in fi.readline().split(',')] 13 data.mapchipData[idx] = movable
問題の起きている部分の抜粋です。
strip()も試してみましたが、解決しませんでした。
field.mapも確認しましたが、','などの異常は見られませんでした。
field.mapを転記しておきます。
3
pipo-map001_at-umi.mapchip
pipo-map001.mapchip
mapchip-base.mapchip
0, 4
20, 15
Bottom
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 1:0, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 1:0, 1:0, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 1:0, 1:0, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
Top
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
0:4, 1:0, 1:14, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 0:4
0:4, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:38, 1:39, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 0:4
0:4, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:46, 1:47, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 0:4
0:4, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 0:4
0:4, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 0:4
0:4, 1:0, 1:0, 1:0, 1:0, 1:0, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 1:0, 1:0, 1:0, 1:0, 0:4
0:4, 1:0, 1:0, 1:0, 1:0, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 1:0, 1:0, 1:0, 0:4
0:4, 1:0, 1:0, 1:0, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 1:0, 1:0, 1:0, 0:4
0:4, 1:0, 1:0, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 1:0, 1:0, 1:0, 0:4
0:4, 1:0, 1:0, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 1:0, 1:0, 1:0, 0:4
0:4, 1:0, 1:0, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 1:0, 1:0, 1:0, 0:4
0:4, 1:0, 1:0, 1:0, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 1:0, 1:0, 1:0, 1:0, 0:4
0:4, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 1:0, 0:4
0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4, 0:4
補足情報（FW/ツールのバージョンなど）
ここにより詳細な情報を記載してください。
初めての利用で分かりづらい表記になってしまいすみません。
階層が分からないのはごもっともでした。以後気を付けます。
コメントありがとうございます。
data.ncol, data.nrow はいくつになっていますか？
data.ncol * data.nrow 回、readlineが呼ばれているのですが、それだけの行数がありますか？
readlineはEOFになると '' (空文字) を返し続けるので、それが怪しい気がします。
プログラムの抜粋とのことですが、抜粋によって見えなくなっている処理がいくつかあります
たとえば、「data = MapchipData()」の部分でどのようなデータが生成されているのかは読み取れません
その後の処理はこのdataにまつわる物ですので、ここで生成されているdataの内部仕様などが分かればより詳細なアドバイスが受けられるかと思われます
回答3件
0
''.split(',')は['']を返します、readline()で読み込む範囲に空行などはありませんか？for row in range(data.nrow):、for col in range(data.ncol):によって二重ループになっていますがnrow*ncolは読み込みファイルの行数を超えていませんか？ 終端行が空行として読まれている可能性があります
一度以下のようなコードでテストすることをお勧めします。
Python
1print("nrowの値：",end = "") 2print(data.nrow) 3print("ncolの値：",end = "") 4print(data.ncol) 5for i in range(data.nrow): 6 print("{}行目の処理".format(i)) 7 for j in range(data.ncol): 8 print("{}列目の処理".format(j)) 9 print(fi.readline())
(追記)
forの内部がどれだけ呼び出されているか、ということを知るためのprint文を追記しました
空文字をprintした際も改行は行われますが念のため
投稿2021/03/17 00:15
編集2021/03/18 02:07総合スコア65
0
内包表現ってのは記述量も少ないので使いたい気持ちがわかりますが、こういうトラブったときにどうしてもわかりづらくなります
ここはforで１行づつ読み出して、コンマで分割して、変換する、とぢみちに実行していきましょう。
そして、どうなったらどういうエラーが出るのかを見ていきましょう
投稿2021/03/16 23:38
総合スコア88038
0
空文字列''を数字にしようとしてできませんでしたというエラーメッセージです。
fi.readline()で読み込んだ文字列が'10,20,'のように','で終わっているのかもしれません。その場合は、以下のように修正すれば動きます。
idx, movable = [int(tok) for tok in fi.readline().strip(',').split(',')]
そうでなければ、fi.readline()で読み込んだ文字列が'10,,20'のようになっています。
入力を修正しましょう。
投稿2021/03/16 22:13
総合スコア24670
回答ありがとうございます。
ppaul様に指摘頂いた部分は確認させていただきましたが異常が見られませんでした。
デバックを行うとやはり'field.map'読み込み時のみエラーが発生しているようです。
コメントありがとうございました。
あなたの回答
tips
プレビュー
