我应该尽可能避免覆盖rails模型方法吗？

Question

更具体地说，如果我有两个选择：在控制器中检查或覆盖模型中的association method，我应该选择哪一个？

编辑：

class Book < ApplicationRecord
  belongs_to :author

  def author
    super || build_author
  end
end

上面的代码是正确的还是我更喜欢下面的其他解决方案？

class BooksController < ApplicationController
  def update
    set_author
    if @book.update_attributes(params[:book])
      #redirect
    else
      #render show page - unprocessable entity
    end
  end

  def set_author
    a = Author.where(fields)
    @book.author = a || @book.build_author
  end
end