If/Else 作为{代码块}？

Question

所以我试图定义“#titleize”，一种将字符串中所有单词的首字母大写的方法，除了诸如“the”、“and”和“if”之类的绒毛词之外。 '

到目前为止我的代码：

def titleize(string)
words = []
stopwords = %w{the a by on for of are with just but and to the my had some in} 

string.scan(/\w+/) do |word|
    if !stopwords.include?(word) 
        words << word.capitalize
    else 
        words << word 
    end

words.join(' ')
end

我的问题在于 if/else 部分 - 当我在字符串上运行该方法时，我收到“语法错误，意外 $end，期待关键字_end”。

如果我使用 if/else 的简写版本，我认为代码会起作用，它通常会进入 {花括号} 内的代码块中。我知道这个速记的语法看起来像

string.scan(/\w+/) { |word| !stopwords.include?(word) words << word.capitalize : words       
    <<  word }

...与

words << word.capitalize 

如果 !stopwords.include?(word) 返回 true 并且

words << word

在 !stopwords.include?(word) 返回 false 时发生。但这也不起作用！

它也可能看起来像这样（这是一种不同/更有效的方法 - 没有实例化单独的数组）：

string.scan(/\w+/) do |word|
    !stopwords.include?(word) word.capitalize : word
end.join(' ')

（来自Calling methods within methods to Titleize in Ruby） ...但是当我运行此代码时，我也会收到“语法错误”消息。

所以！有谁知道我所指的语法？你能帮我记住吗？ 或者，您能否指出这段代码不起作用的另一个原因？

Answer 1

我想你错过了end：

string.scan(/\w+/) do |word|
    if !stopwords.include?(word) 
        words << word.capitalize
    else 
        words << word 
    end
end #<<<<add this

对于速记版本，请执行以下操作：

string.scan(/\w+/).map{|w| stopwords.include?(w) ? w : w.capitalize}.join(' ')

Answer 2

Active Support 有titleize 方法，它可以作为一个有用的起点，因为它会将字符串中的单词大写，但它并不完全智能；它浪费了停用词。不过，进行一些后期处理以恢复它们可以很好地解决这个问题。

我会这样做：

require 'active_support/core_ext/string/inflections'

STOPWORDS = Hash[
  %w{the a by on for of are with just but and to the my had some in}.map{ |w| 
    [w.capitalize, w]
  }
]


def my_titlize(str)
  str.titleize.gsub(
    /(?!^)\b(?:#{ STOPWORDS.keys.join('|') })\b/,
    STOPWORDS
  )
end
# => /(?!^)\b(?:The|A|By|On|For|Of|Are|With|Just|But|And|To|My|Had|Some|In)\b/

my_titlize('Jackdaws love my giant sphinx of quartz.')
# => "Jackdaws Love my Giant Sphinx of Quartz."

my_titlize('the rain in spain stays mainly in the plain.')
# => "The Rain in Spain Stays Mainly in the Plain."

my_titlize('Negative lookahead is indispensable')
# => "Negative Lookahead Is Indispensable"

我这样做的原因是构建 YAML 文件或数据库表来提供停用词列表非常容易。从这个单词数组中，很容易构建一个散列和一个正则表达式，将其馈送到gsub，然后使用正则表达式引擎来修改停用词。

创建的哈希是：

{
  "The"=>"the",
  "A"=>"a",
  "By"=>"by",
  "On"=>"on",
  "For"=>"for",
  "Of"=>"of",
  "Are"=>"are",
  "With"=>"with",
  "Just"=>"just",
  "But"=>"but",
  "And"=>"and",
  "To"=>"to",
  "My"=>"my",
  "Had"=>"had",
  "Some"=>"some",
  "In"=>"in"
}

创建的正则表达式是：

/(?!^)\b(?:The|A|By|On|For|Of|Are|With|Just|But|And|To|My|Had|Some|In)\b/

当gsub 命中正则表达式模式中的单词时，它会在哈希中查找并将值替换回字符串中。

代码可以使用downcase 或其他计算方法来反转大写单词，但这会增加开销。 gsub 和正则表达式引擎非常快。部分原因是散列和正则表达式避免循环遍历停用词列表，因此该列表可以很大而不会大大降低代码速度。当然，引擎在不同版本的 Ruby 上发生了变化，所以旧版本的效果并不好，所以运行 Ruby

Answer 3

很难在次优代码中寻找错误。以规范的方式执行此操作，并使可能的错误易于发现。

class String
  SQUELCH_WORDS = %w{the a by on for of are with just but and to the my had some in}

  def titleize
    gsub /\w+/ do |s|
      SQUELCH_WORDS.include?( s ) ? s : s.capitalize
    end
  end
end

"20,000 miles under the sea".titleize #=> "20,000 Miles Under the Sea"

Answer 4

您不仅缺少end（关闭方法），而且您的words.join(' ') 位于scan 块内，这意味着words is joining every time you iterate throughscan`。

我想你想要这个：

def titleize(string)
  words = []
  stopwords = %w{the a by on for of are with just but and to the my had some in} 

  string.scan(/\w+/) do |word|
      if !stopwords.include?(word) 
          words << word.capitalize
      else 
          words << word 
      end
  end

  words.join(' ')
end

虽然可以清理您的代码，但此时基本流程是合理的。