Pythonでjpg画像を二枚表示し、ボタンでそれぞれ異なる画像を表示したいのです。

Question

前提・実現したいこと

Pythonでjpg画像を二枚表示し、ボタンでそれぞれ異なる画像を表示したいのです。

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

コマンドボタンをクリックしたら上下の画像を違う画像に切り替えたいのでが、現在は上側の画像のみが変わります。

また一度使用した画像はリストから削除したのですが、うまく行きません。　
　flower　= flowers.pop() で取り出してその画像ファイルは削除したいのですが、codingが良く分かりません。

一枚だけの画像だと上手く切り替わりますが、二画像だと画像fileがreturnで上手く引き渡せていないか、廃棄されてしまうのかもしれません。


宜しくお願い致します。

該当のソースコード

import tkinter as tk
import random
from PIL import Image, ImageTk

root = tk.Tk()
root.geometry('300x500')

flowers = ['カタバミ.jpg', 'カンパニューラ.jpg', 'クレオメ.jpg',
           'クンシラン.jpg', 'ジキタリス.jpg', 'シューカイドウ.jpg', ]


def select_img_file():
    flower = random.sample(flowers, 2)
    photo_file = flower
    photo_file = photo_file[0]
    photo_file = flower
    photo_file = photo_file[1]

    return photo_file


def push_next_button(event):
    img_next = Image.open(select_img_file())
    img_next = ImageTk.PhotoImage(img_next)
    label1.configure(image=img_next)
    label1.photo = img_next


# run時に一回だけ動作する
img = Image.open(select_img_file())
img = ImageTk.PhotoImage(img)

label1 = tk.Label(root, image=img)
label1.place(x=40, y=30)

label2 = tk.Label(root, image=img)
label2.place(x=40, y=250)


btn1 = tk.Button(root, text='次へ')
btn1.place(x=50, y=450)
btn1.bind("<1>", push_next_button)

def destroy():
    root.destroy()

btn2 = tk.Button(root, text='終わる', command=destroy)
btn2.place(x=150, y=450)

root.mainloop()

試したこと

変数をglobalにしたり、select_img_file(): を二つ作ったりしましたが、私の実力では解決できませんでした。

codeを変更したりしても、上下が入れ替わるくらいで、うまく行きませんでした。　

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

開発環境はW10　x64、Visual Sutudio Codeを使用しております。

出来上がり図と必要画像を添付致します。画像はpyファイルと同じholderにあります。

![

Answer 1

以下のコードですが、同じ変数に対して値を複数回代入した場合、一番最後に代入したものが残ります。

def select_img_file():
    flower = random.sample(flowers, 2)
    photo_file = flower
    photo_file = photo_file[0]
    photo_file = flower  # すでに記録されていた photo_file[0] の情報が消える
    photo_file = photo_file[1]

    return photo_file

つまり、このコードは以下と同じであり、2枚の画像パスを返すという処理になっていません。

def select_img_file():
    flower = random.sample(flowers, 2)
    photo_file = flower
    photo_file = photo_file[1]

    return photo_file

a = 1
a = 2  # 2を代入したら、すでに記録されていた1という値の情報は消える
print(a) # 2

---

リストから値を一つ取り出しすというのは list.pop() が利用できます。
この関数はリストの末尾の値を1つ取り出して、それをリストから削除します。
なので、pop() を何回も呼び出せば、1度使った値はリストから削除していくということが実現できます。

lst = [1, 2, 3]

v = lst.pop()  # 末尾から1つ取り出す
print(v, lst)  # 3 [1, 2]
v = lst.pop()  # 末尾から1つ取り出す
print(v, lst)  # 2 [1]
v = lst.pop()  # 末尾から1つ取り出す
print(v, lst)  # 1 []

サンプルコード

上記の点を踏まえて以下のように修正してみました。

import tkinter as tk
from tkinter import messagebox
import random
from PIL import Image, ImageTk

root = tk.Tk()
root.geometry("300x500")

flowers = [
    "カタバミ.jpg",
    "カンパニューラ.jpg",
    "クレオメ.jpg",
    "クンシラン.jpg",
    "ジキタリス.jpg",
    "シューカイドウ.jpg",
]

# シャッフルしておく。
random.shuffle(flowers)


def get_img():
    # 画像を1枚取り出す。取り出した画像は flower から削除される。
    img_path = flowers.pop()
    # 画像を読み込む。
    img = Image.open(img_path)
    # リサイズする。
    img.thumbnail((200, 200))
    # ImageTk.PhotoImage オブジェクトに変換する。
    img = ImageTk.PhotoImage(img)

    return img


def setup_images():
    """2枚の画像をラベルに設定する。
    """
    img = get_img()
    label1.configure(image=img)
    label1.photo = img

    img = get_img()
    label2.configure(image=img)
    label2.photo = img


def push_next_button(event):
    """「次へ」ボタンが押されたときに呼ばれる関数
    """

    # 画像が空の場合
    if not flowers:
        # メッセージボックスを表示
        messagebox.showinfo("警告", "これ以上表示する画像がありません")
        return
    
    setup_images()


# run時に一回だけ動作する
label1 = tk.Label(root)
label1.place(x=40, y=30)

label2 = tk.Label(root)
label2.place(x=40, y=250)

# 画像を設定する。
setup_images()

btn1 = tk.Button(root, text="次へ")
btn1.place(x=50, y=450)
btn1.bind("<1>", push_next_button)


def destroy():
    root.destroy()


btn2 = tk.Button(root, text="終わる", command=destroy)
btn2.place(x=150, y=450)

root.mainloop()

Comments

[退会済みユーザー]

tiitoriさん

いつも素晴らしい解決案を教えて頂き有難うございます。

def get_img():を呼び出すようにするのですね、分かりました。

実はブラックジャックを作ってまして、トランプを親と子に配布するところで停滞しておりました。
今回のアドバイスを基にチャレンジしてみます。　その時にまたお世話になるかもしれませんが、宜しくお願い致します。

今日は良い夢を見られそうです、有難うございました。