0

私は (主に python と json を学ぼうとしていますが、また) Twitter からトレンド トピックのリストを定期的に取得してフォーマットしようとしています。私はこれをまとめて、さまざまなチュートリアルをざっと読みました。それは私の目的を果たします-標準出力に必要なHTMLを出力しますが、オブジェクトについて別の方法で考えたり、より適切に構造化したりすることができたのではないかと思っています。支援する？

class trend:
        #these are the fields that Twitter provides, so they make up one trend.
        def __init__(self, query, name, promoted_content, events, url):
                self.query = query
                self.name = name
                self.promoted_content = promoted_content
                self.events = events
                self.url = url 

        def listitem(self):
                print "\t <li><a href=\"%s\">%s</a></li>\n" %(self.url, self.name)

class trending:
        def __init__(self,api_url,title):
                self.api_url = api_url
                self.title = title

        def get_trending(self):
                import simplejson as json
                import urllib2

                trends_all = json.loads(urllib2.urlopen(self.api_url).read())
                # test print
                # print trends_all[0]['trends']
                print "<p>%s</p> \n <ol>" % self.title

                #I'm initializing an array, though I don't actually use it. That's next.
                trends = []
                for x in trends_all[0]['trends']:
                    thistrend = trend(x['query'], x['name'], x['promoted_content'], x['events'], x['url'])
                    thistrend.listitem()
                    trends.append(thistrend) 
                print "</ol>\n"
                return trends

usa = trending("http://api.twitter.com/1/trends/23424977.json","Trending nationally")
usa.get_trending()

フィードバック？

4

1 に答える 1

1

あなたの例では、機能が1つしかないため、トレンドがクラスである必要がある理由がまったくわかりません。get_trendingこれは、api_url と title を引数として受け取るスタンドアロン関数として記述できます。

于 2011-08-31T20:56:28.477 に答える