Rails视图无法调用控制器方法求助：附相关代码

嘿，我来帮你梳理下这个问题的核心和解决办法～

问题根源 #

你现在的问题在于把change_artist方法嵌套定义在了mylist_artist方法内部——这种嵌套方法在Ruby里属于局部方法，只能在mylist_artist内部调用，视图根本访问不到它。而且这种写法本身就不符合Ruby的代码规范，也违背了MVC的设计逻辑。

另外还有个小问题：你在change_artist里直接用了params[:link]，但方法明明接收了link参数，这会导致逻辑混乱，应该使用传入的参数而不是直接取请求参数。

修正方案 #

1. 调整控制器代码，把嵌套方法抽出来

把change_artist改成控制器的私有实例方法，同时修正参数使用：

class YourController < ApplicationController # 记得替换成你的实际控制器名称
  def mylist_artist
    @songs = current_user.songs.all
    @songartists = @songs.pluck(:artist).uniq
    
    # 推荐：提前在控制器处理所有艺术家的歌曲数量，传给视图
    @artist_song_counts = @songartists.each_with_object({}) do |artist, result|
      result[artist] = change_artist(artist)
    end
  end

  private

  def change_artist(link)
    require 'uri'
    require 'open-uri'
    require 'nokogiri'
    
    encoded_link = URI.encode(link)
    linkurl = "https://www.ufret.jp/artist.php?data=#{encoded_link}"
    
    doc = Nokogiri::HTML(open(linkurl))
    # 直接返回计数结果，不需要赋值给实例变量
    doc.xpath("//a[@class='list-group-item list-group-item-action normal-chord']/strong").children.count
  end
end

2. 优化视图代码，避免直接调用控制器方法

按照MVC的设计，视图应该只负责展示数据，不要包含业务逻辑。我们可以直接使用控制器提前处理好的@artist_song_counts：

<% @songartists.each do |artist| %>
  <%= artist %>: <b><%= @artist_song_counts[artist] %></b> total songs.<br>
<% end %>

额外提醒 #

• 生产环境中使用open-uri时，一定要添加异常处理（比如捕获OpenURI::HTTPError、网络超时等），避免因为请求失败导致页面崩溃。
• 如果change_artist的逻辑比较耗时，建议考虑缓存结果，减少重复请求，提升页面加载速度。

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