2016-09-21 3 views
-1

私のアプリケーションには非常に簡単な方法があります。私はそれをさらに短くする方法があるのだろうかと思います。この簡単な方法をもっと短くするには?

def is_enough_words_to_play_game? 
    if Word.verified.size > 5 
     true 
    else 
     false 
    end 
end 
+3

'DEF is_enough_words_to_play_gameとしてこれを書きます?; Word.verified.size> 5;末尾 ' – Aetherus

+1

'def i; Word.verified.size> 5; end'。 –

答えて

5

あなただけ書くことができます。すでに

def is_enough_words_to_play_game? 
    Word.verified.size > 5 
end 

Word.verified.size > 5ため、戻りブール値を。

それとも私見より読み:

MIN_WORDS_FOR_GAME = 6 

def enough_words? 
    Word.verified.size >= MIN_WORDS_FOR_GAME 
end 
+1

このような簡単な方法を定義することは理にかなっていますか? –

+3

@HubertJakubiakこれは、きれいでシンプルでわかりやすいRubyのメソッドを定義する方法です。私がここで心配している唯一の事は、その意味が明確になるように、ある種の定数でなければならない魔法の数字「5」です。 – tadman

+0

@HubertJakubiakはい、 'is_enough_words_to_play_game? 'は読者にいくつかの文脈を与えてくれますが、あなたのメソッド名の選択を短くする可能性があります。 –

1

私は

def is_enough_words_to_play_game? 
    Word.verified.size > 5 ? true:false 
end 
+0

条件がすでにブール値を返すため、このコンテキストでは3項は無意味です。 – spickermann

+0

ありがとう – sureshvignesh

関連する問題