for文やリスト等を用いてコードをシンプルにしたいです。

Question

前提・実現したいこと

コードをできるだけシンプルにしたいです。def以下のコードを他の方に教えていただいたため、それを上手く使えば簡単にできるかと思ったのですが、なかなか使いこなせていない状況です。また、最終的な保存はbom1～bom4だけでよいので、可能であれば他の保存の部分をなくしつつきれいにまとめたいと考えています。お力添えいただけると幸いです。参考のコードも教えていただけると助かります。お手数ですが、よろしくお願い致します。

発生している問題・エラーメッセージ

以下のようなエラーが生じています。
  in alpha_composite
    return im1._new(core.alpha_composite(im1.im, im2.im))

ValueError: images do not match

該当のソースコード

from PIL import Image

src1 = Image.open('img.png')

src = []
for i in range(1,4):
    src.append(src1.crop((i*125, i*125, 1000-i*125, 500)))

src2 = []
for i in src:
    src2.append(i.resize((1000,562)))
    
src2.insert(0,src1)
     
for i in range(len(src2)):
    src2[i].putalpha((i+7)*20)
    
src2[0].save('img21.png')
src2[1].save('img22.png')
src2[2].save('img23.png')
src2[3].save('img24.png')

def merge_save(infile1, infile2, outfile):
    image1 = Image.open(infile1).convert('RGBA')
    image2 = Image.open(infile2).convert('RGBA')
    outimage = Image.alpha_composite(image1, image2)
    outimage.save(outfile)

infiles1 = ['img7.png', 'img8.png', 'img9.png', 'img10.png']

infiles2 = ['img24.png','img23.png','img22.png','img21.png']

outfiles = ['bom1.png','bom2.png','bom3.png','bom4.png']

for i in range(len(infiles1)):
    merge_save(infiles1[i], infiles2[i], outfiles[i])

試したこと

補足情報（FW/ツールのバージョンなど）

PythonのPillowを用いて製作するように指示が出ています。よろしくお願い致します。

Comments

[quickquip]

該当のソースコード にエラーが出ているコードが含まれていないのはどういう意図なんでしょうか?

[quickquip]

何をしたいのか? も書かれていません。

[Shooo-8897]

何をしたいかは実現したいこととして最終的なものを記載したつもりです。ソースコードは、どの部分をどう変えたいかを伝えるために、処理が正常になされたものを書いています。どう変えたらエラーが生じたのかは発生している問題のところに書いたら伝わるのではないかと考え、このようにしています。分かり辛くてすみません。

Answer 1

for i in range(1,4):は「iが1から4までのあいだ繰り返す」という意味なので
i.cropだと「1をcropする」のを4まで繰り返すということになってしまい
「1は数字だからcropとか出来ないよ」というエラーが出ているものを思われます

やりたいことは画像を何枚か(i125, i125, 1000-i*125, 500)という数値でcropするということだと思いますので、
「まず画像を入れたリストをつくり、そのリストからforを使って画像を一枚ずつ取り出しcropしてscrリストに追加」
というのが正しい過程となると思います
filesというリストを上に持ってきて使ってみてはどうでしょうか

Comments

[Shooo-8897]

丁寧で詳細なご回答ありがとうございます。アドバイスを参考にコードを変更してみました。すると、エラーの出ていたfor文は上手く処理がなされたようです。しかし別のところで再度エラーが発生してしまいました。ご確認いただけると幸いです。よろしくお願い致します。

[pia]

まずひとつ、最初の質問のコードを書き換えてしまうと後で他の人が見た時に回答者が何を言っているのかわからなくなります

今の状態がまさにそれで、前回のやり取りを知らない人が見ると「ValueError: images do not matchというエラーが出ているという話に対して、どうしてint型にはcropというアトリビュートはないとか数字をcropする回答をしているんだろう」となってしまっています

AttributeErrorが出ていたコードに戻して一度質問を締め、新たに今回のコードでValueErrorが出てしまったという質問を立ち上げるのが良いかと思います

Answer 2

AttributeError: 'int' object has no attribute 'crop'

int型には、cropというアトリビュートはない、とおっしゃってますよ
そこでなにするつもりなんでしょうか

Comments

[Shooo-8897]

ご回答ありがとうございます。変数を1から3までにして、その変数の値によって切り取る領域を設定し、それをリストに追加したいと考えています。

[y_waiwai]

そのエラーメッセージのコードが提示のものにないってのは横に置いといて、
i って変数はなにかをよく考えてみては。

[Shooo-8897]

ありがとうございます。コードを更新しました。for文のところは取り敢えず処理できるようになったのですが、別のところでエラーが生じました。ご確認いただけると幸いです。よろしくお願い致します。