书籍大写类错误

Question

我正在自学如何编码，但似乎无法理解此类如何调用输入的标题。争论首先在哪里？ attr_accessor 或 def 标题 capital_it(@title) 结束？

class Book

    attr_accessor :title

    def title
        capital_it(@title)
    end

    def capital_it(title)
        word_arr = @title.capitalize.split(" ")
        word_arr.map do |word|
            word.capitalize! unless little_words.include?(word)
            end
        word_arr.join(" ")
    end

    def little_words
        ["the", "a", "an", "and", "in", "of"]
    end

end

Answer 1

论点首先在哪里？ attr_accessor 或 def title ... ?

attr_accessor :title 为您定义了两种方法：title（“读取器”）和title=（“写入器”）。当您在 attr_accessor 之后执行 def title 时，您将完全用您的新方法替换 title 方法。

换句话说，attr_accessor 生成的方法永远不会被调用，因为你已经用自己的方法覆盖了它。

由于您正在编写自己的“阅读器”方法，因此您应该只使用attr_writer：

class Book
  attr_writer :title

  def title
    capital_it(@title)
  end

  # ...
end

Answer 2

首先，您的def capital_it(title) 方法存在问题。

1. .map 方法将返回一个您没有分配给任何东西的新数组。如果您想将其重新分配给局部变量 word_arr，请改用 .map!。
2. 您可能应该将条件word.capitalize! unless little_words.include?(word) 重写为little_words.include?(word) ? word : word.capitalize

附：在您的情况下，它可以工作，因为您在每个元素字符串上使用 capitalize!，但这样的写作风格并不好。

所以我建议你像这样写你的课程：

class Book
  SKIP = %w(the a an and in of)

  attr_writer :title

  def initialize(title)
    @title = title
  end

  def title
    capital(@title)
  end

  private

  def capital(title)
    array = @title.split(/\s+/)
    array.map! do |word|
      SKIP.include?(word) ? word: word.capitalize
    end
    array.join(" ")
  end
end