質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
Cloud9

Cloud9は、クラウドからのプログラミングが可能になるWebサービス。IDEとしての機能が搭載されており、GitHubやHerokuなど他ツールとの連携も可能です。ブラウザ上で動くため、デバイスに関係なく開発環境を準備できます。

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails 6

Ruby on Rails 6は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

1回答

3240閲覧

Couldn't find Address without an IDのエラー(削除)を解決したい

pi-nattu

総合スコア61

Cloud9

Cloud9は、クラウドからのプログラミングが可能になるWebサービス。IDEとしての機能が搭載されており、GitHubやHerokuなど他ツールとの連携も可能です。ブラウザ上で動くため、デバイスに関係なく開発環境を準備できます。

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails 6

Ruby on Rails 6は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2022/10/19 13:45

編集2022/10/19 13:48

前提

課題でショッピングアプリを作成しています。
今回のエラーは配送先を登録し、一覧の中から一件のみ削除する際に起きています。
自力で解決できないためよろしくお願いいたします。

実現したいこと

  • 削除機能を実装すること

発生している問題・エラーメッセージ

ActiveRecord::RecordNotFound in Public::AddressesController#destroy Couldn't find Address without an ID

該当のソースコード

ruby_on_rails

1 def destroy 2 @addresses = Address.find(params[:address_id]) 3 @addresses.destroy 4 redirect_to addresses_index_path 5 end 6

試したこと

idの取得ができていないと思い、最初

ruby_on_rails

1@addresses = Address.find(params[:id])

としていたものから現在のように変えてみたのですが、エラー内容は変わらずidの取得ができていないようです。
※current_idなどを入れてみたりしましたがそれも同じくエラーが出てしまいました。

補足情報(FW/ツールのバージョンなど)

ruby_on_rails

1class Public::AddressesController < ApplicationController 2 def index 3 @address_new = Address.new 4 @addresses = current_customer.addresses.all 5 end 6 7 def create 8 @address_new = Address.new(address_params) 9 @address_new.customer_id = current_customer.id 10 @address_new.save 11 redirect_to addresses_index_path 12 end 13 14 def update 15 end 16 17 def edit 18 end 19 20 21 def destroy 22 @addresses = Address.find(params[:address_id]) 23 @addresses.destroy 24 redirect_to addresses_index_path 25 end 26 27 def address_params 28 params.require(:address).permit(:customer_id, :name, :postal_code, :address) 29 end 30end 31

↓は自分の住所や名前の表示と新規配送先登録を兼ねたものになります。
indexとnewを同じビューで扱っており、念のため全体のコードを貼っています。

ruby_on_rails

1<h1>自分の住所</h1> 2<h4>氏名</h4><%= current_customer.last_name %> <%= current_customer.first_name %><br> 3 <%= current_customer.last_name_kana %> <%= current_customer.first_name_kana %><br> 4 <h4>住所</h4><%= current_customer.postal_code %><br><%= current_customer.address %><br> 5<%= link_to "編集する", addresses_edit_path %> 6 7<%= form_with model: @address_new, url: addresses_create_path, method: :post do |f| %> 8 <%= f.label :郵便番号(ハイフンなし)%><%= f.text_field :postal_code %><br> 9 <%= f.label :住所 %><%= f.text_field :address %><br> 10 <%= f.label :氏名 %><%= f.text_field :name %><br> 11 <%= f.submit "新規登録" %> 12<% end %> 13 14<h4>お届け先一覧</h4> 15<div> 16 <table> 17 <thead> 18 <tr> 19 <th>郵便番号</th> 20 <th>住所</th> 21 <th>氏名</th> 22 <th></th> 23 </tr> 24 </thead> 25 <tbody> 26 <% @addresses.each do |addresses| %> 27 <tr> 28 <td><%= addresses.postal_code %></td><br> 29 <td><%= addresses.address %></td><br> 30 <td><%= addresses.name %></td><br> 31 <td> 32 <%= link_to "編集する", addresses_edit_path %> 33 <%= link_to "削除する", addresses_destroy_path, method: :delete, 34 "data-confirm" => "本当に削除しますか?" %> 35 </td> 36 </tr> 37 <% end %> 38 </tbody> 39 </table> 40</div>

ruby_on_rails/routing

1 addresses_index GET /addresses/index(.:format) public/addresses#index 2 addresses_edit GET /addresses/edit(.:format) public/addresses#edit 3 addresses_create POST /addresses/create(.:format) public/addresses#create 4 addresses_update PATCH /addresses/update(.:format) public/addresses#update 5 addresses_destroy DELETE /addresses/destroy(.:format) public/addresses#destroy

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

エラー内容は変わらずidの取得ができていないようです。

はい、ルーティングを/addresses/destroyと引いてしまっているので、ID番号を入れる余地がありません。

ルーティングをどう設定したのかが書かれていないのでなんとも言いづらいですが、resourcesを使った標準的なルーティングを使うことをおすすめします。

投稿2022/10/19 13:50

maisumakun

総合スコア145183

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

pi-nattu

2022/10/19 14:47 編集

素早い回答ありがとうございます! 回答をいただいてidの入る余地がない、ということを初めて認識しました。resourcesを使っていなかったので、回答の通りresourcesにroutingを書き替えurlも <%= link_to "削除する", address_path(current_customer.id), method: :delete, "data-confirm" => "本当に削除しますか?" %> と修正してみたところ動くようになりました。 ですが、新しく NoMethodError (undefined method `[]' for nil:NilClass @addresses = Address.find(params[:current_customer][:id]) ^^^^^): のエラーが出てしまいました。 ログイン中のcustomerに紐づくaddressのidを取得ヲ1件取得するために上記のように書き換えてみたのですがidがnillになってしまいます。 どうしたらidを取得できるでしょうか? @addresses = Address.find(params[:addresses][:id]) や[:id]だけでもnilになってしまいます。 ※一部重複していたので修正しました
maisumakun

2022/10/19 14:47

ルーティングを確認してみましょう。ふつうにresources :addressesで作ったのならparams[:id]だけで問題ないかと思います。
pi-nattu

2022/10/20 14:44

返信が遅くなり申し訳ありません。 @cart_item = CartItem.find(params[:id]) としてみたところ Couldn't find Address with 'id'=30 というエラーが起きてしまいました。 cloud9のターミナルでcustomer_idとaddressのid?が30で同じ値になっているような気がするのですが、こちらも何か関係あるでしょうか? 以下cloud9のターミナルでの挙動です。 createの時 INSERT INTO "addresses" ("customer_id", "name", "postal_code", "address", "created_at", "updated_at") VALUES (?, ?, ?, ?, ?, ?) [["customer_id", 30], ["name", "y"], ["postal_code", "y"], ["address", "y"], ["created_at", "2022-10-20 13:22:27.279862"], ["updated_at", "2022-10-20 13:22:27.279862"]] destroyの時 Started DELETE "/addresses/30" for Processing by Public::AddressesController#destroy as HTML Parameters: {"authenticity_token"=>"[FILTERED]", "id"=>"30"} (0.1ms) SELECT sqlite_version(*) ↳ app/controllers/public/addresses_controller.rb:22:in `destroy' Customer Load (0.1ms) SELECT "customers".* FROM "customers" WHERE "customers"."id" = ? ORDER BY "customers"."id" ASC LIMIT ? [["id", 30], ["LIMIT", 1]] ↳ app/controllers/public/addresses_controller.rb:22:in `destroy' Completed 404 Not Found in 34ms (ActiveRecord: 1.6ms | Allocations: 13245)
maisumakun

2022/10/20 14:50

質問本体になく突然登場したCartItemは何者ですか?
pi-nattu

2022/10/22 04:52

失礼いたしました。 @addresses = Address.find(params[:id]) の間違いです。
maisumakun

2022/10/22 04:56

エラーメッセージにcustomersとありますが、本当にAddress.findを行っての結果でしょうか?
pi-nattu

2022/10/22 07:08

申し訳ありません、少々混乱してしまったのでもう一度記述を見直してから再度新しく質問しようと思います。 最初の質問の『id』が取得できないエラーについてはルーティングの見直しをするなど糸口になったのでひとまずベストアンサーにさせていただきますね。 時間を取らせてしまい申し訳ありませんでした。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問