From: sora <sora.hs@...>
Date: 2008-02-11T11:31:48+09:00
Subject: [ruby-list:44606] Re: アーティストのニュースを自動取得


To：るびきちさん

どーも。sora です。

早速の有益な情報ありがとうございます！！

> 疑問に思った場合は思い切って質問してみましょう。
> もしかしたらかっこいい答えが返ってくるかもしれません。

速攻で、来ました(^^;
これぞ、rubylist！！みたいなかっこ良いコードを
教えていただき、ありがとうございました。

頂いた情報を元に、クラスなど書き直してみます。

さ、もっともっと勉強しなくちゃ！


こういうMLとかに参加するのが初めてで、こういった
やりとりなどすべて感動しまくりです。。。

今後ともよろしくお願いいたしますm(_ _)m

++--------++--------++--------++--------++--------++
Keep making things, that's my project for life.

e-mail : sora.hs@gmail.com
blog：http://sora2hs.blog70.fc2.com/
++--------++--------++--------++--------++--------++


On 2008/02/11, at 1:26, rubikitch@ruby-lang.org wrote:

> From: sora <sora.hs@gmail.com>
> Subject: [ruby-list:44602] アーティストのニュースを自動取得
> Date: Mon, 11 Feb 2008 00:11:33 +0900
>
> るびきちです。
>
>> 一人で勉強するより、やっぱこういうの良いですね。
>
> いいですね＾＾
> プログラミングは自分で調べるのが基本ですが、もっといい書き 
> 方がないかと
> 疑問に思った場合は思い切って質問してみましょう。
> もしかしたらかっこいい答えが返ってくるかもしれません。
>
>> んで、調子こいて僕も便乗させてください。
>
> どうぞ。ではさっそく。
>
>>    @engine = engine
>>    if @engine == 0 then
>>      @load = MNewsLoad.new('http://www.excite.co.jp/music/news/')
>>    else
>>      @load = MNewsLoad.new('http://music.yahoo.co.jp/music_news/')
>>    end
> (snip)
>> s_engine = 0  # 0=excite, 1=yahoo
>> News.new(s_engine).start
>
> News#get_newsでも@engineの分岐が出ているた 
> め、MNewsLoad#print_newsを新設して
> News#initializeの引数で直接MNewsLoadオブジェクトを渡 
> したほうがすっきりするでしょう。
>
> 0とか1を使うくらいならば:exciteとか:yahoo等の 
> Symbol使いましょう。
> そうすれば、「if @engine == :excite〜」などと書けますし。
>
>>    print Kconv.kconv(@track.name, Kconv::UTF8) + "\n"
>
> 「Kconv.kconv(@track.name, Kconv::UTF8)」は 
> 「@track.name.toutf8」と書けます。
> この場合はputsを使ったほうがいいですね。
>
>>    puts @track.date_added.to_s
>
> putsやprintで非文字列を表示しようとすると内部で 
> to_sを呼んでくれるので「.to_s」は不要です。
>
>>      if self.is_changed? then
>>        break
>>      end
>
> 僕なら「break if is_changed?」と書きます。メソッド呼 
> び出しのselfは省略できます。
>
>>  def is_changed?
>>    track = @itunes.get_current_track
>>    if @track.artist != track.artist then
>>      true
>>    else
>>      false
>>    end
>>  end
>
> 条件式は真偽値を返すので「@track.artist !=  
> @itunes.get_current_track.artist」
> の一行で済みます。
> また、メソッド名も「artist_is_changed?」にすると
> 「break if artist_is_changed?」と書けて英語っぽくなり 
> ます（笑）
>
> --
> rubikitch
> Blog: http://d.hatena.ne.jp/rubikitch/
> Site: http://www.rubyist.net/~rubikitch/
>