Блог

Я пытаюсь создать пункт назначения, но он продолжает сообщать мне в моем браузере, что ‘name’ равно нулю, когда перенаправляет перенаправления на мое представление ‘show’.

Я получаю ошибку

 undefined method `name' for nil:NilClass
  

Вот мои действия контроллера для new, create и show:

 def show
        @destination = Destination.find_by(id: params[:id])
    end

def new
        @destination = Destination.new
    end

    def create
        @destination = Destination.create(dest_params)
        redirect_to user_destination_path(current_user, @destination.id )
        
    end

    private

    def dest_params
        params.require(:destination).permit(:name,:user_id)
    end
  

Моя новая форма, в которой я ввожу имя назначения:

 <h2>Add a destination</h2>

<div>
<%= form_for @destination do |f|%>
  <%= f.label :name %>
  <%= f.text_field :name %><br>
  <%= f.submit %>
<% end %>
</div>
  

вот мое представление для чтения / показа:

 <h3>Added destination</h3>
<div>
    <p><%= @destination.name %></p>
</div>
  

Перед всем этим я получал ошибки с отсутствующими обязательными ключами [: id], но я, казалось, исправил это, но по какой-то причине я подозреваю, что это может быть как-то связано с проблемой, с которой я сталкиваюсь сейчас. Дайте мне знать, если вы сможете обнаружить проблему

Обновлена ошибка

 No route matches {:action=>"show", :controller=>"destinations", :id=>nil, :user_id=>"1"}, missing required keys: [:id]
  

Основной проблемой здесь является полное отсутствие обработки ошибок. Вы вообще не проверяете, правильно ли пользователь ввел входные данные или запись вообще была сохранена в вашем методе create.

 def create
  @destination = Destination.create(dest_params)
  redirect_to user_destination_path(current_user, @destination.id )
end
  

Если запись не сохранена, например, из-за неудачной проверки @destination.id , значение равно нулю.

В вашем show методе, который вы используете find_by вместо find , который просто пропускает ошибку вместо того, чтобы вызывать ActiveRecord::RecordNotFound ошибку.

Ваш контроллер на самом деле должен выглядеть как:

 class DestinationsController
  def show
    # will raise if the record is not found and return a 404 not found response 
    # instead of just exploding
    @destination = Destination.find(params[:id]) 
  end

  def new
    @destination = Destination.new
  end

  def create
    # never just assume that the record is created unless you want 
    # to get kicked in the pants.
    @destination = Destination.new(dest_params)
    if @destination.save
      # this route really does not need to be nested.
      # see https://guides.rubyonrails.org/routing.html#shallow-nesting
      redirect_to user_destination_path(current_user, @destination)
    else
      # re-render the form with errors
      render :new 
    end
  end
  
  private

  def dest_params
    params.require(:destination).permit(:name,:user_id)
  end
end