tkinter 画像

Question

tkinterでwebから画像をスクレイピングする部分を関数化してキャンパスに描写できない

tkinterでボタンをクリックしたらwebから画像を習得したいのですが、毎回webからスクレイピングを
するのは負担がかかると思い↓のように関数化したのですがなぜか関数化すると画像がキャンパスに描写されませんでした。普通に書くと問題なく描写されるのですが、それだと毎回通信をして画像をwebからスクレイピングしているのかと思いボタンの中で関数化しました。
そもそもtkinterのループの仕組みがわかっていないからなのか、とりあえずリファレンスを読みましたが手抜きのあまりにもひどいリファレンスだったので全く理解できず・・・

##関数化の部分
def imageprint(url):
   
    headers = {
        "User-Agent": "Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:47.0) Gecko/20100101 Firefox/47.0",
        }

    request = urllib.request.Request(url=url, headers=headers)


    f = io.BytesIO(urllib.request.urlopen(request).read())
    
    img = Image.open(f)
        
    img.thumbnail((200, 200), Image.ANTIALIAS)
    img = ImageTk.PhotoImage(img)  # 表示するイメージを用意  
    
    return img 

##こちらが全ソース
import tkinter as tk
from PIL import Image, ImageTk
import urllib.request 
import io

from PIL import Image 
'''
web上にある画像を保存します
'''
img =""
url = "http://imgcc.naver.jp/kaze/mission/USER/20180519/35/3012705/157/500x718xedab7ba7e203cd7576d12004.jpg"
root = tk.Tk()
root.geometry('800x560')
root.title('IMG')
canvas = tk.Canvas(
    root, # 親要素をメインウィンドウに設定
    width=500,  # 幅を設定
    height=500, # 高さを設定
    #relief=tk.RIDGE  # 枠線を表示
    # 枠線の幅を設定
    bg = "black"
    )
canvas.place(x=0, y=0)  # メインウィンドウ上に配置

def btn_click_1_1():
    global url
    global img
    img = imageprint(url)
    print(img)
# ボタンを表示 command=処理
button= tk.Button(text="button",command=btn_click_1_1)
button.place(x=0, y=200)


def imageprint(url):
   
    headers = {
        "User-Agent": "Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:47.0) Gecko/20100101 Firefox/47.0",
        }

    request = urllib.request.Request(url=url, headers=headers)


    f = io.BytesIO(urllib.request.urlopen(request).read())
    
    img = Image.open(f)
        
    img.thumbnail((200, 200), Image.ANTIALIAS)
    img = ImageTk.PhotoImage(img)  # 表示するイメージを用意  
    
    return img 
 
    
canvas.create_image(  # キャンバス上にイメージを配置
            0,  # x座標
            0, # y座標
            image=img,  # 配置するイメージオブジェクトを指定
            tag="illust",  # タグで引数を追加する。
            anchor=tk.NW  # 配置の起点となる位置を左上隅に指定
    ) 

root.mainloop()

Comments

[hayataka2049]

関数化していないコードも掲載してください

[ads3hcgff]

全ソースと書いてあるのが関数化していないコードに当たります

[hayataka2049]

？　　imageprintを使っているので関数化したコードだと思ったのですが、違うのですか？

[ads3hcgff]

imageprintが、スクレイピングを関数化してそれをボタンが押されたらコールバック関数としたんです。

[ads3hcgff]

いまさらですが、関数化していないコードの意味が分かりました。すいませんもう関数化してしまったので元のコードはないです。ただ、もう一つtkinterで画像を表示させるときに注意しなければならないことが見つかったのでそれを回答に載せておきます。

Answer 1

btn_click_1_1関数にcanvas.create_imageの呼び出し文がないです。
そのため、変数:imgに値を格納しても画面の更新が行わなれないかと。

def btn_click_1_1():
    global url
    global img
    img = imageprint(url)
    # canvas.create_imageの行を追加root.mainloopのcanvas.create_imageは不要です。
    canvas.create_image(  # キャンバス上にイメージを配置
            0,  # x座標
            0, # y座標
            image=img,  # 配置するイメージオブジェクトを指定
            tag="illust",  # タグで引数を追加する。
            anchor=tk.NW  # 配置の起点となる位置を左上隅に指定
    ) 
    print(img)

---

毎回webからスクレイピングをするのは負担がかかる

pathlibを使って画像をローカルに保存するという手(※)もありますが。
※hashlibを使って、urlからハッシュ関数の出力結果を求め、保存ファイル名として使用すればよいかと。

同じURLに対してプログラム起動中はキャッシュを行いたいだけならば。
functools.lru_cacheが使えます。

# -*- coding: utf-8 -*-
"""
web上にある画像を保存します
"""
import tkinter as tk
from PIL import Image, ImageTk
import urllib.request
from io import BytesIO
from functools import lru_cache
from PIL import Image


@lru_cache(maxsize=None)
def get_content(url: str):
    headers = {
        "User-Agent": "Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:47.0) Gecko/20100101 Firefox/47.0",
        }
    request = urllib.request.Request(url=url, headers=headers)
    print("content")
    return BytesIO(urllib.request.urlopen(request).read())


def imageprint(url):
    img = Image.open(get_content(url))
    img.thumbnail((200, 200), Image.ANTIALIAS)
    img = ImageTk.PhotoImage(img)  # 表示するイメージを用意
    return img


img =""
url = "http://imgcc.naver.jp/kaze/mission/USER/20180519/35/3012705/157/500x718xedab7ba7e203cd7576d12004.jpg"
root = tk.Tk()
root.geometry('800x560')
root.title('IMG')
canvas = tk.Canvas(
    root, # 親要素をメインウィンドウに設定
    width=500,  # 幅を設定
    height=500, # 高さを設定
    #relief=tk.RIDGE  # 枠線を表示
    # 枠線の幅を設定
    bg = "black"
    )
canvas.place(x=0, y=0)  # メインウィンドウ上に配置

def btn_click_1_1():
    #global url # 再代入を伴わないので、globalはコメントアウト
    global img
    img = imageprint(url)
    print(img)
    canvas.create_image(  # キャンバス上にイメージを配置
        0,  # x座標
        0,  # y座標
        image=img,  # 配置するイメージオブジェクトを指定
        tag="illust",  # タグで引数を追加する。
        anchor=tk.NW  # 配置の起点となる位置を左上隅に指定
    )
# ボタンを表示 command=処理
button= tk.Button(text="button",command=btn_click_1_1)
button.place(x=0, y=200)

root.mainloop()

Comments

[umyu]

かぶった。

[ads3hcgff]

回答ありがとうございます。最初のコードおっしゃる通りです。
Tk()でインスタンス化してからloopのなかのwigetを毎回読み込んでいると思っていましたが違うようですね。すいません。把握漏れです。2つ目のソースはあまり理解していません。取り合えずローカルには保存せず、メモリに保存して展開したかったのですが・・・url.openでメモリ上にロードしまくるとGCとかしてくれないんでしょうか。urllibの仕組みがよく分かっておらず・・・

[umyu]

@ads3hcgffさんへ
>メモリに保存して展開
urllibは毎回HTTPリクエストをしますよー。
https://docs.python.jp/3/library/urllib.request.html#module-urllib.request
というかHTTPリクエストを勝手にキャッシュしてレスポンスを返すものは他言語でも少ないような。

[ads3hcgff]

ありがとうございます。精進します。

Answer 2

すいません。できました
create_imageを関数化したコードの中にいれればよかったみたいです。

tk()でインスタンス化してからloopのなかは実は一回しか読み込まれてない？
試しにcreate_imageを関数の外に出したら無理でした。

追記

勝手ながらもう一つ重要な項目が見つかったので載せておきます。
こちらの記事

create_canvasの引数 imageにはローカル変数ではだめで、グローバル変数を入れなきゃだめらしい
ここに入れるのはImageTk.PhotoImage()で作成したオブジェクトなんですが
どうやらこれをローカル変数に入れたままcanvas_create関数に入れるとうまく動きません。
理由はImageTk.PhotoImage()をローカルに入れると一瞬でGCが起きるらしいです。
意味不明ですね、この仕様・・・・
これはtkinterが悪いのかpythonの仕様上そうなってしまうのかそれともPILが悪いのか謎・・・
根本的な理由はtkinterにあるらしいのですが・・・やれやれｗ

    canvas.create_image(  # キャンバス上にイメージを配置
                0,  # x座標
                0, # y座標
                image=img,  ここをローカル変数に入れたまま代入すると動きませんｗ無能すぎる・・
                anchor=tk.NW  # 配置の起点となる位置を左上隅に指定
                tag="illust", 
        )