ファイルの書き換えをする前とした後で出力の形が変わる

このコードを利用してファイルの中の在庫を変化させたいのですが。変化させる前とさせた後とのファイルへの出力の形が変わってしまいます。
そのままの形を保ち書き換えたいです。
partID = BR001

Python

file_name = "Parts_Inventory3.txt"
data_line = []

with open(file_name) as f:
     data_lines = f.read().splitlines()
     data_line.append(data_lines)
     SID = input("enter partID:")

for data_line in data_lines :
     if data_line.find(SID) >= 0:
         print(data_line)
         new = input ("enter new quantity:")
         data = "".join(data_line)
         lines = "".join(data_lines)
         lines = lines.replace(data, SID +" : "+  new)

with open(file_name ,"w") as f:
     f.write(lines)

変化前
下のpartIDのBR001はミスです。本当はBR003のつもりです。

変化後

toast-uz

2020/09/18 13:53

コードのインデントが統一していないのと、そもそもコードを実行しても変化後になりません。再確認して修正をお願いします。

kananananana

2020/09/18 13:59

すみません。誤ったコードを載せてしまいました。

toast-uz

2020/09/18 14:17

記載したコードをご自分で実行して、問題が再現するか、確認いただけますでしょうか？・そのままだと　NameError: name 'lines' is not defined　が出ます。・また上記を解決しても、BR00? を入力すると、for文直下のifが全てFalseなので、アウトプットは空ファイルになります。・インデントも合っていません。

kananananana

2020/09/18 14:35

・そのままだと　NameError: name 'lines' is not defined　が出ます。出ませんでした。・また上記を解決しても、BR00? を入力すると、for文直下のifが全てFalseなので、アウトプットは空ファイルになります。 BR00１と入れました

toast-uz

2020/09/18 14:40

partID = BR00? と質問文に記載されています。

kananananana

2020/09/18 14:48

すみません。？の部分にIDの数値パートを入れたという意味です。わかりづらくて申し訳ないです

行動規範の内容に同意します

回答2件

ベストアンサー

なんか、変数に意味を与えたのに、その記憶が1-2行書く間に失われてしまうような、つまり、記憶力が5秒くらいしか持たない人が書いたように見えるプログラムです。

とりあえず、改行をそのまま保ちたいなら、lines = "".join(data_lines)をlines = "\n".join(data_lines)に変えればいいです。

それだけの修正だと期待通りの動作にはならないと思いますが。

#追記
おそらく、こういうことがやりたいのでしょう。

Python
1file_name = "Parts_Inventory3.txt"
2
3with open(file_name) as f:
4    data = f.read()
5
6SID = input("enter partID:")
7
8for data_line in data.splitlines():
9    if data_line.find(SID) >= 0:
10        print(data_line)
11        new = input ("enter new quantity:")
12        data = data.replace(data_line, SID +" : "+  new)
13
14with open(file_name ,"w") as f:
15    f.write(data)