如何在 Ruby 中的方法中实现 for 循环

Question

我的任务是创建一个pig_latin 方法。

猪拉丁语是一种虚构的儿童语言，旨在成为 令人困惑。它遵守一些简单的规则（如下），但是当它被说出来时 很快，对于非儿童（和非本地人）来说真的很难 演讲者）来理解。

规则 1：如果单词以元音开头，则添加“ay” 词尾。

规则 2：如果单词以辅音开头，则将其移至末尾 词尾，然后在词尾加上“ay”音。

（对于边缘情况还有一些规则，并且有区域性 变体也是如此，但这应该足以理解测试。）

我所有的测试都通过了保存一个，翻译很多单词。

这是我的错误：

#translate
  translates a word beginning with a vowel
  translates a word beginning with a consonant
  translates a word beginning with two consonants
  translates two words
  translates a word beginning with three consonants
  counts 'sch' as a single phoneme
  counts 'qu' as a single phoneme
  counts 'qu' as a consonant even when it's preceded by a consonant
  translates many words (FAILED - 1)

Failures:

  1) #translate translates many words
     Failure/Error: expect(s).to eq("ethay ickquay ownbray oxfay")

       expected: "ethay ickquay ownbray oxfay"
            got: "ethay"

       (compared using ==)
     # ./spec/04_pig_latin_spec.rb:70:in `block (2 levels) in <top (required)>'

Finished in 0.00236 seconds (files took 0.10848 seconds to load)
9 examples, 1 failure

Failed examples:

rspec ./spec/04_pig_latin_spec.rb:68 # #translate translates many words

这是我的方法：

def translate(str)

  def add_ay(str)
   return str + 'ay'
  end

  def word_begins_with_vowel(str)
    if (!(str.match(' '))) && $vowels[str[0]]
      return add_ay(str)
    end     
  end

  def begins_with_consonant(str)
    if ((!$vowels[str[0]]) && (!$vowels[str[1]]) && (!$vowels[str[2]])) 
      first_three = str.split('').slice(0, 3).join('');
      str = str.slice(3, str.length - 1)
      return str +  first_three + 'ay'
    end
    if ((!$vowels[str[0]]) && (!$vowels[str[1]])) 
      first_two = str.split('').slice(0, 2).join('');
      str = str.slice(2, str.length - 1)
      return str +  first_two + 'ay'
    end
    if ((!$vowels[str[0]]))
     first_char = str.split('').slice(0);
     str = str.slice(1, str.length - 1)
     return str + first_char +'ay'
    end
  end

  def translates_two_words(str)
    if (str.match(' '))
      str = str.split(' ');
      first_char = str[1].split('').slice(0);
      str[1] = str[1].slice!(1, str[1].length - 1);

      return str[0] + 'ay' + ' ' + str[1] + first_char + 'ay'
    end     
  end

  def translates_many_words(str)
      str = str.split(' ');
      if str.length > 2
        str.each do |item| 
          return begins_with_consonant(item) || word_begins_with_vowel(item)
        end
      end
  end

  $vowels = {
    'a' => add_ay(str),
    'e' => add_ay(str), 
    'i' => add_ay(str), 
    'o' => add_ay(str), 
    'y' => add_ay(str)
  }
    return translates_many_words(str) || word_begins_with_vowel(str) || begins_with_consonant(str) || translates_two_words(str)
end

我认为这会处理很多单词：

def translates_many_words(str)
      str = str.split(' ');
      if str.length > 2
        str.each do |item| 
          return begins_with_consonant(item) || word_begins_with_vowel(item)
        end
      end
  end

但事实并非如此。

Answer 1

正如@theTinMan 所说，return - 将拒绝下一次迭代并只返回第一次迭代中的第一个值，从我的评论来看，我认为这对你有用（只需对你的代码进行最少的编辑）：

def translates_many_words(str)
  str = str.split(' ');

  if str.length > 2
    str.map do |item| 
      begins_with_consonant(item) || word_begins_with_vowel(item)
    end.join(' ')
  end
end

UPD 另外，我建议您重构代码以使其更具可读性，它可以在将来帮助您。 我的这种方法的变体是：

def translates_many_words(str)
  str = str.split

  # line under - is a shortcut from `return nil if str.size <= 2`
  # `#size` is more relative to this context if you will count elements of array 
  return unless str.size > 2 

  # Now, when we excluded possibility of work with array that have less then 2 elements, 
  # we can continue with our iteration
  str.map do |item| 
    begins_with_consonant(item) || word_begins_with_vowel(item)
  end.join(' ')
end