快速幫助重構Ruby類

Question

我寫過這個類，它返回Feed更新，但我認爲它可以進一步改進。這不是出問題或任何東西，但作爲一個新的Ruby開發者，我認爲這是一件好事，提高:-)

class FeedManager 
    attr_accessor :feed_object, :update, :new_entries 

    require 'feedtosis' 

    def initialize(feed_url) 
    @feed_object = Feedtosis::Client.new(feed_url) 
    fetch 
    end 

    def fetch 
    @feed_object.fetch 
    end 

    def update 
    @updates = fetch 
    end 

    def updated? 
    @updates.new_entries.count > 0 ? true : false 
    end 

    def new_entries 
    @updates.new_entries 
    end 
end 

正如你可以看到，這是相當簡單的，但我看到的是，事情並不很正確的是：

1. 每當我通過終端調用fetch時，它會打印一個包含更新的列表，當它真的假定返回一個對象時。

因此，作爲一個例子，在終端，如果我這樣做：

client = Feedtosis::Client.new('http://stackoverflow.com/feeds') 
result = client.fetch 

然後我得到：

<Curl::Easy http://stackoverflow.com/feeds> 

這也正是我所期待的。但是，這樣做與「inniting」級同樣的事情時有：

FeedManager.new("http://stackoverflow.com/feeds") 

我得到恢復與對飼料的所有項目的數組對象。

當然，我做錯了什麼，所以任何幫助重構這個類他將不勝感激。

此外，我希望看到有關我的實施以及任何形式的評論，以使其更好的意見，歡迎。

在此先感謝

Answer 1

試試這個：

class FeedManager 

    require 'feedtosis' 

    attr_accessor :feed_object  

    def initialize(feed_url) 
    self.feed_object = Feedtosis::Client.new(feed_url) 
    end  
    def fetch 
    feed_object.fetch 
    end  
    def updates (reload = true) 
    @updates = reload ? fetch : @updates 
    end  
    def updated? 
    updates(false).new_entries.count > 0 
    end  
    def new_entries 
    updates(false).new_entries 
    end 
end 

現在你可以更新如下：

result = FeedManager.new("http://stackoverflow.com/feeds").updates 

PS：我已經刪除了attr_accessor for：update和new_entries。

編輯

我加入的代碼，使有條件的緩存重新加載。

feed = FeedManager.new("http://stackoverflow.com/feeds") 
updates = feed.updates # reloads the updates 
# do something 

updates = feed.updates(false) # get the updates from cache. 

Answer 2

看來您所期望的初始化方法返回調用更新的結果。 Initialize基本上是Ruby中的一個構造函數，所以它會返回新的FeedManager對象。

在類定義的中間放置require語句也很「不尋常」。

Answer 3

1. :update，@updates

2. count > 0 ? true : false可以只是count > 0