for文の処理が最後まで行われない

Question

(file1)
C  301   12.89   1.00
N  301    7.54   1.00
O  301   34.76   1.00
CD 301    8.42   1.00
CA 302   15.64   1.00
C  302   11.15   1.00
O  302   10.76   1.00

file1の0列目と1列目、1列目と2列目、２列目と3列目の間に空白を増やすために以下のスクリプトを書きました。

path = 'file1'
f = open(path).readlines()
for line in f:
    array1 = line[3:6]
    array_1 = array1.rjust(3)
    tmp = line[:3] + array_1 + line[5:]
    fileobj = open("file2","a")
    print >> fileobj, tmp.rstrip()

path2 = 'file2'
f2 = open(path).readlines()
for line2 in f2:
    array2 = line2[12:17]
    array_2 = array2.rjust(9)
    tmp = line2[:12] + array_2 + line2[17:]
    fileobj = open("file3","a")
    print >> fileobj, tmp.rstrip()


path3 = 'file3'
f3 = open(path).readlines()
for line3 in f3:
    array3 = line3[24:28]
    array_3 = array3.rjust(6)
    tmp = line3[:24] + array_3 + line3[28:]
    fileobj = open("file4","a")
    print >> fileobj, tmp.rstrip()

その結果、途中までは期待された結果が得られましたが、最後の行だけ出力されていませんでした。

(file2)
C     301   12.89   1.00
N     301    7.54   1.00
O     301   34.76   1.00
CD    301    8.42   1.00
CA    302   15.64   1.00
C     302   11.15   1.00
O     302   10.76   1.00

(file3)
C     301       12.89   1.00
N     301        7.54   1.00
O     301       34.76   1.00
CD    301        8.42   1.00
CA    302       15.64   1.00
C     302       11.15   1.00
O     302       10.76   1.00

(file4)
C     301       12.89     1.00
N     301        7.54     1.00
O     301       34.76     1.00
CD    301        8.42     1.00
CA    302       15.64     1.00
C     302       11.15     1.00
#最後の行が出力されない

どこを変えればいいのか教えてください。
よろしくお願いします。

Comments

[ikedas]

スクリプトが動きません。実際に実行した通りのスクリプトを提示して下さい。

[can110]

line3が未定義エラーとなります。他人が検証できる最小限の実行コードを提示ください。

[can110]

現象を再現できません（全行出力される）。詳細（OSや実行手順など）を追記ください。

[退会済みユーザー]

実行の流れとしては、その他の列も同様にfor文で空白を増やしていたところ、問題が発生しました。for文1回目で1列目を編集→2回目で2列目を編集（ここまでは全行出力されていました）→3回目で3列目を編集（最後の1行がでない）という結果です。

[can110]

要するに提示コードを３回そのまま実行すると現象が発生するということでよろしいでしょうか？

[ikedas]

問題の起こらないスクリプトだけでなく、問題の起こるスクリプトも提示してください。

[退会済みユーザー]

より実際のスクリプトに近い形で提示したので、よろしくお願いします。

[can110]

path2、path3変数は使われていませんが、正しいでしょうか？

[ikedas]

度々ですみませんが、提示されている結果とスクリプトの実行結果が異なります (file4の表示幅が違います)。**実際に実行したスクリプトと同じもの**を提示して下さい。

Answer 1

完全な再現コードは依然として不明ですが…
同じファイルに対して複数回open(～)しているようです。
その結果、以前の書込がバッファに残り、ファイルに書き出されないまま読込みされる場合があるなど、分かりにくい不具合の原因となりえます。

ファイルを利用し終わったら必ずclose()するようにしましょう。

fp1 = open(～) # 後で閉じることができるようにfp1にファイルを保持しておく
f = fp1.readlines() # 読込
fp1.close() # 読込元ファイルを閉じる
for line in f:
	# 略
    fileobj = open("file2","a")
    print >> fileobj, tmp.rstrip()
    fileobj.close() # 書込先ファイルを閉じる

# 以下同様に修正

Answer 2

提示したスクリプトで実行したところ、最終行まで表示されました。
実際のスクリプトと比較して確認を行います。
回答していただきありがとうございます。

Comments

[can110]

質問で提示されたコードで、本来の目的は達成できているのでしょうか？
最初から「実際のスクリプト」を提示いただけると良かったのですが…

[退会済みユーザー]

f.close()やfileobj.close()をきちんと表記するとうまくいきました！
ありがとうございます。

Answer 3

fileobj = open("file2","a") をfor文の上に出して
最後に fileobj.close() を追加してみたら如何でしょうか？