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

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

ただいまの
回答率

88.57%

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

解決済

回答 3

投稿 編集

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

退会済みユーザー

(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列目、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
#最後の行が出力されない


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

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • 退会済みユーザー

    退会済みユーザー

    2018/11/07 13:08

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

    キャンセル

  • can110

    2018/11/07 13:15

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

    キャンセル

  • ikedas

    2018/11/07 13:18

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

    キャンセル

回答 3

+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() # 書込先ファイルを閉じる

# 以下同様に修正

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

check解決した方法

0

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

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/11/07 13:30

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

    キャンセル

  • 2018/11/07 13:35

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

    キャンセル

0

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

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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