2015-09-10 62 views
-3

你能不能給我我怎樣才能使下面的代碼更加不久的提示和美麗:如何讓代碼更漂亮?

Persons.with_tour.map(&:father).map(&:first_name).to_sentence 
+2

你也可以瓦納檢查這個https://github.com/styleguide/ruby – zee

+1

順便說一句:當你的問題是完全切合主題的這裏StackOverflow,它也是* [CodeReview.SE](http://codereview.stackexchange.com/)上的主題,它可能會得到比這裏更好的答案。但是,請不要交叉發帖。如果你希望你的問題出現在那裏,你應該詢問該網站的版主,或者在本網站上將你的問題標記爲版主注意並要求它被遷移。 –

+0

@JörgWMittag儘管在代碼審閱中這是主題討論還是偏離主題可能存在爭議,但對於存根代碼或假設代碼來說,它可能會被關閉。 –

回答

0

這個怎麼樣?

Persons.with_tour.collect{|x| [x.father, x.first_name]}.to_sentence 
+3

你確定要做同樣的事嗎? 'to_sentence'甚至不知道如何處理數組數組? –

+0

Dawn Green,非常感謝!不知道那個) – ezelen42

2

我不知道我有你的模型是正確的,但這是主意。不要選擇人物並循環獲取父親,從一開始就選擇父親,然後根據需要進行篩選。而且,如果您不打算使用整個父對象(您只需要名字),則可以使用pluck來獲取數組。

Father.of_tour_persons.pluck(:first_name).to_sentence 

和father.rb

class Father < ActiveRecord::Base 

    scope :of_tour_persons, -> { joins(:persons).where(:persons => { :with_tour => true }) } 

end 
+0

非常感謝!我也沒有深入這個模型。但我相信它將在未來幫助我!謝謝! – ezelen42