Rails视图调用控制器对象报错：undefined method domain for String

这个问题我太熟了！你遇到的本质问题是控制器传给视图的不是ScrapedPage对象，而是字符串类型的URL值，所以当你试图调用domain方法时，自然会报错——毕竟字符串本身没有这个方法。

先拆解下错误信息：

undefined method `domain' for "https://www.lookagain.co.uk/":String

这句话直白说就是：你把一个URL字符串当成了ScrapedPage对象来调用domain方法，字符串当然没有这个方法啦。

而视图里直接写<%@savedHTML = ScrapedPage.all%>能正常工作，是因为ScrapedPage.all返回的是完整的ScrapedPage对象集合，每个对象都应该在模型里定义了domain方法（比如从url字段提取域名的逻辑），所以调用起来没问题。

下面是具体的解决步骤：

第一步：修正控制器的变量传递代码 #

你大概率是在控制器里把变量赋值错了，比如不小心用了pluck(:url)或者直接取了某个对象的url字段，而不是整个对象集合。

举个错误的反例：

# 错误的控制器代码（返回的是字符串数组，不是对象）
def index
  @scraped_pages = ScrapedPage.pluck(:url)
end

正确的写法应该是传递完整的对象集合：

# 正确的控制器代码
def index
  @scraped_pages = ScrapedPage.all # 返回ScrapedPage对象的集合
end

第二步：确保视图里正确遍历对象 #

在视图里遍历@scraped_pages时，要针对每个ScrapedPage对象调用domain方法，而不是直接对集合或字符串操作：

<% @scraped_pages.each do |page| %>
  <tr>
    <td><%= page.domain %></td>
    <!-- 其他需要展示的字段，比如page.url、page.created_at等 -->
  </tr>
<% end %>

第三步：验证模型里的domain方法定义 #

虽然你视图直接用对象时能正常调用，但还是确认下ScrapedPage模型里的domain方法是否正确定义：

# app/models/scraped_page.rb
class ScrapedPage < ApplicationRecord
  def domain
    # 示例：从url字段提取域名的逻辑，可根据实际需求调整
    URI.parse(url).host if url.present?
  end
end

额外排查点（如果是单个对象的情况） #

如果控制器是传递单个对象，要确保你取的是对象本身，而不是它的url字段：

# 错误：取了url字符串
def show
  @scraped_page = ScrapedPage.find(params[:id]).url
end

# 正确：取完整对象
def show
  @scraped_page = ScrapedPage.find(params[:id])
end

这样调整后，既符合MVC的设计原则（控制器负责准备数据，视图负责展示），也不会再出现那个错误啦。

内容的提问来源于stack exchange，提问作者Padu