os.walkで取得したファイル名でzipfileでフォルダ内ファイルを圧縮しようとしたとき、同じフォルダにサブフォルダがあると中身が空のzipファイルが出来てしまう

Question

一つのフォルダにあるファイルを全てzipファイルに圧縮しようとしています。
ところが、同じフォルダ内にサブフォルダが作られていると、出来上がったファイルの中身に何もないzipファイルが出来上がってしまいます。
サブフォルダが無いときには正常に動作します(サブフォルダがあるときはサブフォルダを無視してファイルだけを圧縮したい)。
何が間違えているのでしょうか？

コードは以下の通りです。

import zipfile
import os
os.chdir('c:\temporally')
des_folder = './'
for folder, subfolders, files in os.walk(des_folder):
    with zipfile.ZipFile('./archive.zip', 'w') as myZip:
        for file in files:
            full_name = os.path.join(des_folder, file)
            myZip.write(full_name, os.path.relpath(full_name, des_folder))

Answer 1

forループの中でzipファイルを作るのではなく、外で作るべきではないですか
そうでないと、同じファイル名でフォルダごとに上書きしてしまします。

import zipfile
import os
os.chdir('c:\temporally')
des_folder = './'
with zipfile.ZipFile('./archive.zip', 'w') as myZip:
    for folder, subfolders, files in os.walk(des_folder):
        for file in files:
            full_name = os.path.join(des_folder, file)
            if file_name != './archive.zip':
                myZip.write(full_name, os.path.relpath(full_name, des_folder))

Comments

[Aki1000]

始めはそうしたのですが、そうすると、作られたarchive.zipの中にarchive.zipが入ってしまいます。

[ppaul]

では、if文でそれだけ除外しましょう。
回答を修正しました。

[Aki1000]

ありがとうございます。やはりif文で除外するしかないですか……

[ppaul]

先ほどの修正では、サブフォルダ内にあるarchive.zipも除外していたので、再修正しました。

if文を使いたくなければ、別の場所に作ってからzipファイルを現在のフォルダに移動するとかですね。

[Aki1000]

なるほど。ありがとうございます。