Ruby on rails RubyonRails5-Resource在检查它是否已经存在时给出了错误信息——不管它是否存在_Ruby On Rails_Ruby_Ruby On Rails 5

Ruby on rails RubyonRails5-Resource在检查它是否已经存在时给出了错误信息——不管它是否存在

ruby-on-rails ruby

Ruby on rails RubyonRails5-Resource在检查它是否已经存在时给出了错误信息——不管它是否存在,ruby-on-rails,ruby,ruby-on-rails-5,Ruby On Rails,Ruby,Ruby On Rails 5,我正在尝试检查一个产品是否已经存在——如果已经存在，就会出现一条闪光消息，并带有指向已经存在的产品的链接。如果没有，则应创建产品问题在于，无论产品是否已经存在，闪存错误都会出现。我哪里做错了 products_controller.rb： def create @product = Product.new(product_params) if @product.product_code.present? @existing_product = Product.

我正在尝试检查一个产品是否已经存在——如果已经存在，就会出现一条闪光消息，并带有指向已经存在的产品的链接。如果没有，则应创建产品

问题在于，无论产品是否已经存在，闪存错误都会出现。我哪里做错了

products_controller.rb：

def create
    @product = Product.new(product_params)

    if @product.product_code.present?
        @existing_product = Product.where(product_code: "#{@product.product_code}").ids.first.to_s
        flash.now[:error] = %Q[This product already exists: <a href="/products/#{@existing_product}">#{@product.product_code}</a>. Please either rename this product or view/edit the existing one.].html_safe
        render 'new'

    elsif @product.save
        redirect_to @product
    else
        render 'new'
    end

end

1234是我正在尝试的产品代码。

您的条件检查错误。这：

if @product.product_code.present?

仅检查您刚制作的

@product

中是否存在

product\u code

，而不检查数据库中是否存在与该产品代码相关的内容

我想你想要更像这样的东西：

@existing_product = Product.find_by(product_code: @product.product_code)
if @existing_product
  flash.now[:error] = ...
  render 'new'
elsif @product.save
  redirect_to @product
else
  render 'new'
end

您可以使用

find_by

来查找您正在寻找的东西，这将为您提供产品或

nil

（如果没有匹配的产品）

您可能还需要研究其他几件事情：

使用URL助手而不是手动构建链接会更好

您的模型应该检查产品代码的唯一性，而不是控制器

正如评论中提到的，flash消息中的HTML并不是最好的主意。您最好在视图中检查

@现有的\u产品

，然后将链接和您需要的任何“复杂”错误消息完全放在视图中

你的条件是检查错误的东西。这：

if @product.product_code.present?