2016-06-16 12 views
-3

コメントレビューごとにコメントを移動しました。コードレビューをコメントごとに移動

+0

コードは解析されません。 Rubyは、関数定義のようなブロックを終了させるために 'end'を必要とします。 Array#selectを使って調べてみましょう。 –

+0

"[ask]"をお読みください。サンプルコードが要件を満たしているかどうかを確認し、必要な処理を行い、構文的に正しいかどうかを確認し、[codereview.se]を見て、それがあなたの質問に適しているかどうかを確認します。つまり、それは要件を満たしておらず、構文的に正しいものでもなく、あなたが望むことをしません。 –

+0

ありがとう@ChrisHeald – throwaway0524

答えて

0

ではなく、明示的な繰り返しをやって、最善の方法は、このように可能性がArray#selectです:

配列 numbers内の各要素に与えられたブロックを実行してブロックが返されるすべての要素を含む配列を生成します
even_numbers = numbers.select { |n| n.even? } 

本当。

+0

提案ありがとうございました – throwaway0524

0

したり、問題の規則次の代替ソリューション:selectメソッドのために行く

def get_even_numbers(array) 
     even_num = [] 
     array.each do |n| 
      even_num << n if n.even?  
     end 
    even_num 
end 

そしてもちろんのは常に好ましいです。

+0

ありがとうございました。私のコードの改善が必要な箇所がわかります。 – throwaway0524

関連する問題