trueまたはfalseの数値0または1を返すmap関数

Question

文字列の母音の数を文字の配列に分割し、母音の文字を1にマップして配列を合計してみようとしています。

def count_vowels(string) 
    vowels = ['a','e', 'i', 'o', 'u'] 
    return string.split("").map{ |n| vowels.include? n ? 1 : 0}.inject(0,:+) 
end 

include?一部が正しく1または0を返しません。これはなぜ飛ばないのですか？

は私が働くこのバージョンにそれをハッキング、愚かなのようなものを探します：

def count_vowels(string) 
    vowels = ['a','e', 'i', 'o', 'u'] 
    return string.split("").map{ |n| vowels.include? n}.inject(0) do |mem,x| 
     x ? mem + 1 : mem 
    end 
end 

Answer 1

理由：

string.split("").map{ |n| vowels.include? n ? 1 : 0}.inject(0,:+) 

n ? 1 : 0ための引数として評価され、渡されている動作しません。 nの代わりにinclude?あなたはinclude?にいくつかの括弧を追加する必要があります。

string.split("").map{ |n| vowels.include?(n) ? 1 : 0}.inject(0,:+) 

あなたは、単にあなたがinclude?メソッドのパラメータの括弧を必要とする。この場合、

def count_vowels(string) 
    vowels = ['a','e', 'i', 'o', 'u'] 
    string.split(//).select { |x| vowels.include? x }.length 
end 

Answer 2

を行うことができます。だから、

return string.split("").map{ |n| vowels.include?(n) }.inject(0) do |mem,x| 

とにかく、あなたのコードが良いかもしれない

• VOWELS = %w(a e i o u) # string's array
• あなたの方法でリターンを必要としない、それが最後の文
• string.split("") =>string.chars

です

あなたの方法はそうかもしれないことに注意してください：

def count_vowels(string) 
    string.count "aeiou" 
end 

Answer 3

mapは必要ありません。

def count_vowels(string) 
    vowels = %w[a e i o u] 
    string.chars.select{|n| vowels.include? n}.size 
end