Rails控制器DoubleRenderError问题排查求助

问题代码与错误提示 #

控制器代码 #

before_action :set_blacklisted_nfts, only: %i[blacklisted_nfts]
def blacklisted_nfts
   if  @blacklisted_nft.present?
    redirect_to destroy and return
   end
end
def destroy
  success = false
  message = 'Unable to delete NFT!'
  if !current_user.admin?
     message = 'This action require admin privileges!'
  elsif @nft.destroy
     success = true
     message = 'NFT deleted successfully!'
  end
  render layout: false, locals: {
  type: success ? 'success' : 'error',
  message: message,
  nft_id: @nft.id
  }
end
private

def set_blacklisted_nfts
@blacklisted_nft = BlacklistedNft.create(contract_address: @nft.contract_address,   contract_type: @nft.contract_type, token_id: @nft.token_id, chain: @nft.chain)
end

错误提示 #

错误：在同一个action中多次调用了Render和/或redirect。请注意，每个action中只能调用一次render OR redirect，且二者只能选其一。另外需要注意的是，redirect和render都不会终止action的执行，所以如果你想在redirect后退出action，需要使用类似"redirect_to(...) and return"的写法。

核心问题 #

1. redirect_to参数错误：redirect_to destroy写法完全错误，redirect_to需要接收路由路径（比如destroy_nft_path(@nft)），而非直接写控制器方法名。这种错误写法会导致Rails无法正确跳转，甚至触发额外方法调用，引发多次响应冲突。
2. @nft未初始化：代码中直接使用@nft，但没有任何初始化逻辑（比如@nft = Nft.find(params[:id])），会导致@nft为nil，BlacklistedNft.create执行失败，后续逻辑混乱。
3. 黑名单判断逻辑错误：set_blacklisted_nfts里用create会直接创建新记录，而非检查是否已存在，导致@blacklisted_nft.present?永远为true，每次都会触发错误的跳转逻辑。
4. 响应触发逻辑冗余：错误的跳转写法会导致blacklisted_nfts action无法正常终止，加上destroy方法必然执行render，同一请求中会同时触发redirect和render，触发错误提示。

修复方案 #

1. 修正redirect_to路由路径
   确保路由配置正确（比如routes.rb里有resources :nfts），将跳转代码改为：

   redirect_to destroy_nft_path(@nft), status: :see_other and return
   

2. 添加@nft初始化逻辑
   新增before_action初始化@nft：

   before_action :set_nft, only: %i[blacklisted_nfts destroy]
   
   private
   def set_nft
     @nft = Nft.find(params[:id])
   end
   

3. 修正黑名单检查逻辑
   将set_blacklisted_nfts改为查询是否已存在，而非直接创建：

   def set_blacklisted_nfts
     @blacklisted_nft = BlacklistedNft.find_by(
       contract_address: @nft.contract_address,
       contract_type: @nft.contract_type,
       token_id: @nft.token_id,
       chain: @nft.chain
     )
   end
   

4. 确保响应唯一性
   保留redirect_to ... and return的写法，确保跳转后立即终止当前action执行，避免后续代码触发额外响应。

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