質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.75%

  • Python

    6807questions

    Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

  • Python 3.x

    5278questions

    Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

  • Tkinter

    123questions

    Tkinterは、GUIツールキットである“Tk”をPythonから利用できるようにした標準ライブラリである。

globalを使っているのに変数が更新されない

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 129

ryota_adr

score 9

 前提・実現したいこと

tkinterとBeautifulSoupを使って、入力したURLからリンクを取得し、listboxに表示させ、URLをクリックするとそのページに飛べるプログラムを作っています。

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

def urlgetterでglobalを使ってurl = []を更新しようと思っているのですが、urlgetterの外でprint(urls)をすると、[]を返します。
urlgetterの中でprint(urls)をするとリンクのリストが返ってくるのでこちらはちゃんと動いでいます。
def urlgetterのurlsを外のurls = []に代入するにはどうすればいいでしょうか?

 該当のソースコード

from bs4 import BeautifulSoup
from tkinter import *
from tkinter import ttk
import urllib.request
import webbrowser

urls = []

def urlgetter(oneurl):
    global urls
    try:
        oneurl = str(oneurl)
    except ValueError:
        return entry.delete(0, END)
    html = urllib.request.urlopen(oneurl)
    soup = BeautifulSoup(html, "lxml")
    hrefs = []
    for link in soup.find_all("a"):
        hrefs.append(link.get("href"))
    hrefs = filter(None, hrefs)
    pat = "http"
    for links in hrefs:
        if pat in links:
            urls.append(links)
    return urls

def urljump(evt):
    wgt = evt.widget
    print(wgt)
    idx = wgt.curselection()[0]
    url = wgt.get(idx)
    webbrowser.open_new(url)

root = Tk()
root.title(u'Url Getter')
root.geometry("300x300")

frame = Frame(root)
frame.grid()

entry = Entry(frame)
entry.bind("<Return>", lambda event:urlgetter(entry.get()))
entry.grid(row=0)

listbox1 = Listbox(frame, width=30)
for url in urls:
    listbox1.insert(END, url)
listbox1.bind("<<ListboxSelet>>", urljump)    
listbox1.grid(row=1, column=0)

scrollbar = Scrollbar(frame, orient=VERTICAL, command=listbox1.yview)
listbox1['yscrollcommand'] = scrollbar.set
scrollbar.grid(row=1, column=1, sticky=(N,S))

root.mainloop()

 試したこと

ここに問題に対して試したことを記載してください。

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

ここにより詳細な情報を記載してください。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+3

まず誤字<<ListboxSelet>>"<<ListboxSelect>>"に修正しないとリストボックスのイベントが呼ばれません。
また、リストボックス構築時に項目を追加する処理において、まだurlsは空なので無意味です。
urlgetter内で行う必要があります。

以上を踏まえて修正したソースです。
細かい動作は確認していませんが、ひととおり動作します。

from bs4 import BeautifulSoup
from tkinter import *
from tkinter import ttk
import urllib.request
import webbrowser

listbox1 = None

def urlgetter(oneurl):
    listbox1.delete(0,END) # ListBoxの項目を全削除
    try:
        oneurl = str(oneurl)
    except ValueError:
        return entry.delete(0, END)
    html = urllib.request.urlopen(oneurl)
    soup = BeautifulSoup(html, "lxml")
    hrefs = []
    for link in soup.find_all("a"):
        hrefs.append(link.get("href"))
    hrefs = filter(None, hrefs)
    pat = "http"
    for links in hrefs:
        if pat in links:
            listbox1.insert(END, links)

def urljump(evt):
    wgt = evt.widget
    print(wgt)
    idx = wgt.curselection()[0]
    url = wgt.get(idx)
    webbrowser.open_new(url)

root = Tk()
root.title(u'Url Getter')
root.geometry("300x300")

frame = Frame(root)
frame.grid()

entry = Entry(frame)
entry.bind("<Return>", lambda event:urlgetter(entry.get()))
entry.grid(row=0)

listbox1 = Listbox(frame, width=30)
listbox1.bind("<<ListboxSelect>>", urljump) # "<<ListboxSelet>>" 誤字修正
# ここでのURLリスト追加は無意味
listbox1.grid(row=1, column=0)

scrollbar = Scrollbar(frame, orient=VERTICAL, command=listbox1.yview)
listbox1['yscrollcommand'] = scrollbar.set
scrollbar.grid(row=1, column=1, sticky=(N,S))

root.mainloop()

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/03/28 15:11

    ありがとうございました!

    キャンセル

+2

まずぶっちゃけると、global宣言はこの場合あってもなくても動きます。

>>> lst = []
>>> def hoge():
...     lst.append("hoge")
... 
>>> lst
[]
>>> hoge()
>>> lst
['hoge']


理由は、別に代入はしてなくてメソッド呼んでるだけだからです。そんな感じ。

恐らく、動かない原因はそこではないでしょう。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.75%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Python

    6807questions

    Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

  • Python 3.x

    5278questions

    Python 3はPythonプログラミング言語の最新バージョンであり、2008年12月3日にリリースされました。

  • Tkinter

    123questions

    Tkinterは、GUIツールキットである“Tk”をPythonから利用できるようにした標準ライブラリである。