JPG画像が入ったフォルダからのGIF画像作製を複数フォルダ一括で行いたい

Question

前提・実現したいこと

カレントディレクトリにJPG画像を含んだフォルダがあり、そのフォルダごとのGIF画像を一括で作りたいです。
一つのフォルダ内のJPG画像群からGIF画像を創る関数を定義して、カレントディレクトリ内の全部のフォルダでforループすることを考えたのですが、うまくいきません。
エラーは出ないですが、ループしてるはずなのに、”{cap}”という一つのGIFファイルしか出力されません。

初歩的な質問で申し訳ありません…
よろしくお願いいたします。

該当のソースコード

# GIFアニメーションを作成
def create_gif(in_dir, out_filename):
    path_list = sorted(glob.glob(os.path.join(*[in_dir, '*']))) # ファイルパスをソートしてリストする
    imgs = []                                                   # 画像をappendするための空配列を定義
 
    # ファイルのフルパスからファイル名と拡張子を抽出
    for i in range(len(path_list)):
        img = Image.open(path_list[i])                          # 画像ファイルを1つずつ開く
        imgs.append(img)                                        # 画像をappendで配列に格納していく
 
    # appendした画像配列をGIFにする。durationで持続時間、loopでループ数を指定可能。
    imgs[0].save(out_filename,
                 save_all=True,
                 append_images=imgs[1:], 
                 optimize=False, 
                 duration=50, 
                 loop=0)

cur_dir = os.getcwd()  # カレントディレクトリ取得
dirs = os.listdir(cur_dir)    # 指定ディレクトリ内に存在するファイル、フォルダをリスト化

for cap in dirs:
    create_gif(in_dir = cap, out_filename = '{cap}.gif')

Answer 1

まずは複数フォルダ一括で、ということは考えないで、一つのフォルダだけでGIFの作成を行いましょう。
それがきちんとできてから、複数フォルダでするようにしていけばよろしい。

Comments

[hisui-mo]

記述不足ですみません。それはできました。
しかし、データが膨大なのでステップアップとして勉強しております。
よろしくお願いいたします。

[y_waiwai]

> out_filename = '{cap}.gif'

'{cap}.gif'という名前のファイルに上書きしていってますが、これどーにかしましょう
そりゃファイル一つしかできませんよ

[hisui-mo]

なるほど、同じ名前のファイルに何度も上書きしてしまっていたわけですね。
命名の一部を変数にすることはできないのでしょうか…？
下記の書き方で大括弧やフォーマット文字列を試してみましたがうまくいきませんでした…

> out_filename = '[cap].gif'
> out_filename = 'f"{cap}.gif"'

[y_waiwai]

変数にそのファイル名いれて、その上で関数に渡せば。

[y_waiwai]

out_filename = f"{cap}.gif"
でしょう

[hisui-mo]

シングルクオーテーションが要らなかったんですね。
ありがとうございます。

変数としての記述の仕方がどう良くなかったかは分かったのですが、以下のような新たなエラーが出るようになってしまいました…
もう一度forループなしで一つのフォルダに対して関数を実行してみたところ、うまく動いたので、関数はこのままでfor文を直せばいいのかなの思うのですが、どう直せばよいのかわかりません、ご助言いただけないでしょうか？？

for文ではループの一つ一つが独立してないのでしょうか？？

---------------------------------------------------------------------------
IndexError Traceback (most recent call last)
<ipython-input-3-b0759ecbb285> in <module>
4
5 for cap in dirs:
----> 6 create_gif(in_dir = cap, out_filename = f"{cap}.gif")

<ipython-input-2-14889f7c3f9c> in create_gif(in_dir, out_filename)
10
11 # appendした画像配列をGIFにする。durationで持続時間、loopでループ数を指定可能。
---> 12 imgs[0].save(out_filename,
13 save_all=True,
14 append_images=imgs[1:],

IndexError: list index out of range

[y_waiwai]

そこのimgsが空なのでは
そこで止めて各変数のナカミをチェックしてみては

[hisui-mo]

for文を途中で止めるとは、ループを抜けるといういみでしょうか？

[y_waiwai]

pythonの開発環境を探せば、任意の行で実行を止めて変数のナカミを確認できるようなものもあります
エキスパートはそういうツールで開発を行ってます。
って、いきなりそれは無理なようなので、

とりあえずそこで、print(imgs)　でもおいて、どういう表示のときにコケてるのか、とかで調べてみては

[hisui-mo]

print(imgs)やimgsで表示させてみたんですが、not definedでした…
関数内の定義って関数内でしか反映されない仕組みなのでしょうか…

もう一つ質問させてくださいm(__)m
dirsは定数ではないのでしょうか？
>　dirs = os.listdir(cur_dir)
の定義はfor文の前ですが、forループでcur_dirの中身が改変されていくとすでに定義されているdirも変わっていくのでしょうか？？

[y_waiwai]

ならこんどは、forのなかでiをプリントしてみたらどうなるでしょう

こんなことになってる！で思考停止せずに、なにがどうなったらそんな結果になるのかを考えていきましょう。
プログラムは、書いたひとの思うように動くのじゃありません。
書いているとおりにしか動きません

[hisui-mo]

なるほど、そうやって確かめるんですね。
勉強になりました。

質問していた件ですが、
> IndexError: list index out of range
のエラーは最後まで消えなかったですが、何回かやっていたら出力してくれました。
ありがとうございました。

処理を実行するフォルダにGIFを出力していたので、出力したGIFに対して処理ができないという意味のエラーだったのかもしれませんし、
無線のオンライン上のＨＤＤにカレントディレクトリを置いていたのでそことのコネクトが一時的に上手くいっていなかったのかもしれません。

こんな風になってるのかなとか、いろいろと仮説を立ててみるのですが、プログラムでそれを実証する方法が分からないことが多々あり、今後もこのサービスに教えを乞うことがあるかと思います。
あまり難しい質問はまだまだ投げれないと思いますので、また今回のように助言を頂けましたら幸いです。
今後ともよろしくお願いいたします。