Q&A
わたしが、答えられるかはわかりませんが、ファイル名、ファイルの中身に使用している文字コードが何かは、記述しておいた方がいいかと思います。
前提・実現したいこと
pythonにて東京２３区の区名と駅名を入力すると、入力した区名が含まれるtxtファイルを検索し、当該のtxtファイルを読み込んで、入力した駅名が記載されている行数を返す関数を作成したいと考えております。
エラーは出力されずに終了するプログラムを作成できたのですが、駅名を変えても入力した区名が含まれるtxtファイルの行数＋１を返すプログラムになっており、入力した駅名の検索が出来ていないことがわかりました。
(下記の例ですと、35行目までのtxtファイルを検索しきって、36行目を返してくる。)
print関数を用いて、途中のデータの状態を確認したところ、open関数でtxtファイルを読み込むところで、txtファイルの中身ではなく、「io.TextIOWrapper name=、、、」を読み込んでいることがわかりました。
open関数でtxtファイルを読み込むところで、txtファイルの中身ではなく、「io.TextIOWrapper name=」を読み込んでいる原因と解消方法についてご教示いただけると幸いです。
＊open関数で読み込まれたファイルオブジェクト自体を文字列として読み込んでいるのかと思うのですが、何故そうなるのかと解消方法がわからず、ご教示いただけると幸いです。。。
発生している問題・エラーメッセージ
--------------------------------------------------------------------------- 港区 六本木駅 ./input/港区の駅一覧.txt <_io.TextIOWrapper name='./input/\udce6\udcb8\udcaf\udce5\udc8c\udcba\udce3\udc81\udcae\udce9\udca7\udc85\udce4\udcb8\udc80\udce8\udca6\udca7\udcef\udcbc\udc88\udce8\udca6\udcb3\udce5\udc85\udc89\udcef\udcbc\udc89- \udce3\udc81\udc98\udce3\udc82\udc83\udce3\udc82\udc89\udce3\udc82\udc93net.txt' mode='r' encoding='utf-8'> serch 六本木駅 36 ---------------------------------------------------------------------------
該当のソースコード
python
1def getstationid(args1,args2): 2 for f in glob('./input/*.txt'): 3 if args1 in f: 4 file3 = str(f) 5 print(file3) 6 with open(file3, 'r', encoding='utf-8') as f1: 7 a = 1 8 x1 = 0 9 f2 = str(f1) 10 print(f2) 11 print("serch") 12 print(args2) 13 for src_data in f1: 14 if args2 in src_data: 15 break 16 else: 17 a = a +1 18 x1 = a 19 return x1
XXnettxt
1例）港区の駅一覧.txt 2青山一丁目駅 3赤坂駅 4赤坂見附駅 5赤羽橋駅 6麻布十番駅 7お台場海浜公園駅 8御成門駅 9表参道駅 10神谷町駅 11外苑前駅 12汐留駅 13品川駅 14芝浦ふ頭駅 15芝公園駅 16白金高輪駅 17白金台駅 18新橋駅 19泉岳寺駅 20高輪ゲートウェイ駅 21高輪台駅 22竹芝駅 23田町駅 24溜池山王駅 25台場駅 26大門駅 27虎ノ門駅 28虎ノ門ヒルズ駅 29乃木坂駅 30浜松町駅 31日の出駅 32広尾駅 33三田駅 34モノレール浜松町駅 35六本木駅 36六本木一丁目駅
回答1件
0
ベストアンサー
f2 = str(f1) を f2 = f1.read() にするのが簡単です。
ただし後続のコードをそのまま使いたいならば f1 = list(f1) が良いかもしれませんね。
＊open関数で読み込まれたファイルオブジェクト自体を文字列として読み込んでいるのかと思う
文字列化しただけで勝手に読み込む仕様にはなっていないのです。
書いてみた
もうちょっと読み易くなるかなと思ったので。
Python
1def get_station_id(region_name: str, station_name: str) -> int: 2 # 該当するファイルあるいは行が無い場合の返り値 3 RET_LINE_NOT_FOUND = -1 4 5 # 該当するファイルを取得 6 file_names = [ 7 f_name 8 for f_name in glob('./input/*.txt') 9 if region_name in f_name 10 ] 11 print(file_names) 12 13 # 該当するファイルが無い場合 14 if not file_names: 15 return RET_LINE_NOT_FOUND 16 17 # 先頭のファイルだけ取り出し 18 file_name, *_ = file_names 19 with open(file_name, mode='r') as fin: 20 for i, line in enumerate(fin, start=1): 21 if station_name in line: 22 return i 23 24 # 該当する行が無い場合 25 return RET_LINE_NOT_FOUND
仕様を簡潔にするため、globで発見したファイルの内先頭しか見ていません。
投稿2021/02/01 10:56
編集2021/02/01 11:13
総合スコア35668
ありがとうございます。
「with open(file3, 'r', encoding='utf-8') as f1」では文字列化しただけで、for文で初めて１行目から読み込まれていることが理解出来ました！
コードの書き方も大変参考になりました！
あなたの回答
tips
プレビュー
