pythonでのファイルマージツールの作り方について。

Question

pythonでのファイルマージツールの作り方を教えてください。初心者です。
二つのテキストファイルがあり、そこから１行ずつ読み込んで新しいファイルを作りたいです。

二つのファイルをファイルAとファイルBとし、ファイルAは６行、ファイルBは8行あります。

ファイルAの1行目
ファイルBの1行目
（空行）
ファイルAの2行目
ファイルBの2行目
（空行）
：
：
：
のように組み合わせていき、この場合だとファイルAが先に終わってしまいますが

:
:
:
ファイルAの6行目
ファイルBの6行目
（空行）
ファイルBの7行目
（空行）
ファイルBの8行目

のようにファイルBが終わるまで、空行と交互に出力をしたいです。

その後、ファイルがオープンできない場合や、ファイルを2つ以上指定してしまったときのエラー処理を行いたいのですが、どのようにコードを入れればいいのか全く分かりません。画像は途中まで自分でやってみたものです。詳しい方、教えて頂けると幸いです。よろしくお願いします。

Comments

[y_waiwai]

その途中までやったというコードをテキストで提示しましょう
そして、そのコードのどこがまずいのか、それをどうしたいのかという説明とともに、質問に追記しましょう

[hanachan.]

import sys

fa = open(sys.argv[1], 'r')
fb = open(sys.argv[2], 'r')

while True:
linea = fa.readline()
lineb = fb.readline()
if linea == '':
break

print(linea,lineb,'\n', end ='')

fa.close()
fb.close()

エラー処理はやり方がわからなくてしていません。ifのあたりからやり方がわかりません。わからないところがわからない　という困った状況です。どうしたいのかという点に関しては質問と同じです。すみません。

[udon-ken]

そんなに難しですかね？
少なくとも、
> ファイルを2つ以上指定してしまったときのエラー処理
については日本語と既存のコードでだいたい言い表せているのでコードにできませんかね？

・ファイル3つ指定は入力値として話しにならないので冒頭でエラー処理
・sys.argvで指定されたファイルを受け取っているので、「sys.argv[3]以降が存在する＝3つ以上指定されている」を判定してエラー処理。
リストの要素がいくつあるかはlen()で取得できます。

でいいのでは？

[ppaul]

入力ファイルの行数をどういうように定義しているのかがあいまいですので、明確にしてください。
具体的には、
'aaa\nbbb\nccc'は3行だと解釈するのだと思いますが、'aaa\nbbb\n\n'は3行ですか。それとも4行ですか。

[hanachan.]

ppaulさん
4行です。

[BeatStar]

画像での質問はやめたほうがいいです。

例えば「半角スペースにすべきところを全角スペースにしている」のような、反転させないとわからないような場合もあります。

そうなると再現率が低くなるため回答がつきにくくなります

Answer 1

こういうプログラムというのは、一見簡単そう見えるのですが、何を１行と見なすかによって、アルゴリズムやコードが全然違うものにしなければなりませんので、結構難しいです。

hanachan.さんの行の定義は、改行で区切られたファイルの各部分なので、readlinesで返ってくる行の数とは違っています。
出力の方も、一番最後に改行が入ると、hanachan.さんの行の定義に合わないので最後の行だけ特別扱うが必要になります。

readlineは、EOFに達すると空文字列''を返すため、最後の空白行との区別を付けるのが難しくなります。readでファイル全体を読み込み、'\n'でsplitするのが、hanachan.さんが期待する行の定義を実現するには適しているでしょう。

ファイルが巨大でなければ、まずデータを作るという方法が確実ですのでそういうコードを書いてみました。

import sys

if len(sys.argv) != 3:
    print('error')   #ここはちゃん場合分けしてメッセージを出しましょう。
    exit(-1)

fileA = sys.argv[1]
fileB = sys.argv[2]
fileOUT = 'output.txt'

try:
    with open(fileA, 'r') as fA:
        listA = fA.read().split('\n')
except FileNotFoundError:
    print('error {fileA}')   #ここはちゃんメッセージを書き換えましょう。
    exit(-1)

try:
    with open(fileB, 'r') as fB:
        listB = fB.read().split('\n')
except FileNotFoundError:
    print('error {fileB}')   #ここはちゃんメッセージを書き換えましょう。
    exit(-1)

maxlen = max(len(listA), len(listB))
if len(listA) < maxlen:
    listA.extend(['']*(maxlen - len(listA)))
elif len(listB) < maxlen:
    listB.extend(['']*(maxlen - len(listB)))
else:
    pass

try:
    with open(fileOUT, 'w') as fOUT:
        for i in range(maxlen - 1):
            fOUT.write(f'{listA[i]}\n{listB[i]}\n\n')
        fOUT.write(f'{listA[maxlen - 1]}\n{listB[maxlen - 1]}')
except Exception:
    print('error {fileOUT}')   #ここはちゃんメッセージを書き換えましょう。
    exit(-1)

本当は、

if __name __ == '__main__':

を使って書くべきなのですが、初心者ということで何もかも一度に変更すると混乱するかもしれないので止めておきました。

これだけの変更でも大変だとは思いますが、動かしてみてください。

Comments

[hanachan.]

ppaulさん
とても分かりやすかったです。自分のものと比較することでtry exceptやlenの使い方のイメージがつかめました。ありがとうございます！