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

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

ただいまの
回答率

90.34%

  • Ruby

    8234questions

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

  • Ruby on Rails 5

    2193questions

データの呼び出しができない(ActionController::UrlGenerationError)

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 32

前提・実現したいこと

現在、Ruby on rails を勉強中で練習用としてオリジナルでアプリを開発中。
ユーザーがデータを保存するところまではうまくいったんですが、それを呼び出して画面に表示するというところで今不具合が生じているところです。

試したこと

アクション内で.lastメソッドを用いて最新のデータのみ取り出したいのですがエラーが出てしまいうまくいきません。
ただ、同じコードのままlastメソッドをfirstメソッドに置き換えるだけでそのエラーは解消されます(目的と違うので解決には至ってない)。

また、保存データを一つだけ残して他を全て削除(.lastでも.firstでも同じデータを引き出せるように)してもやはりfirstメソッドではエラーはなく、lastメソッドの場合は同じエラーが出るので保存データそのものに問題がないように思えます。

また、別のモデルのデータは同様にlastメソッドを使っても呼び出せています。

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

エラーメッセージ
ActionController::UrlGenerationError in Toppages#index
Showing /home/ec2-user/environment/trainary/app/views/toppages/index.html.erb where line #32 raised:

No route matches {:action=>"edit", :controller=>"diaries", :id=>nil}, missing required keys: [:id]
Extracted source (around line #32):


30        <% if @diary_last %>
31        <div style="margin-bottom:8px; margin-left:8px;">
32          <button type="button" class="btn btn-success"><%= link_to "編集", edit_diary_path(@diary_last),:class => "href-btn" %></button>
33          <button type="button" class="btn btn-danger"><%= link_to "削除", diary_path(@diary_last), method: :delete, :class => "href-btn", data: {confirm: "削除してもよろしいですか??"} %></button>
34        </div>
35        <% else %>

Rails.root: /home/ec2-user/environment/trainary

該当のソースコード

toppages_controller.rb

class ToppagesController < ApplicationController
  def index
    if logged_in?
      @user = current_user
      @diary = @user.diaries.build

      @diary_last = @user.diaries.last #エラーが出るところ
      #@diary_last = @user.diaries.firstだとエラーなし
      @goal = @user.goals.last
      #Goalモデルのデータの場合は.lastでもエラーは出ない
    else
      @user = User.new
    end
  end
end
toppages/index.html.erb

<% if logged_in? %>
<h1 class="head-line">Home</h1>
  <div class="row">
    <div class="col-sm-3">
      <div class="panel panel-info">
        <div class="panel-heading">
          <h2 class="text-center">前回の記録</h2>
        </div>
        <div class="panel-body">
          <div>
            <h4>日付</h4>
            <p><%= @diary_last.month %><%= @diary_last.date %></p>
          </div>
          <div>
            <h4>記録内容</h4>
            <p><%= simple_format(@diary_last.content) %></p>
          </div>
          <div>
            <h4>体重</h4>
            <p><%= @diary_last.weight %>kg</p>
          </div>
          <div>
            <h4>プロテイン量</h4>
            <p><%= @diary_last.protein %>g</p>
          </div>

        </div>
        <div style="margin-bottom:8px; margin-left:8px;">
          <button type="button" class="btn btn-success"><%= link_to "編集", edit_diary_path(@diary_last),:class => "href-btn" %></button>
          <button type="button" class="btn btn-danger"><%= link_to "削除", diary_path(@diary_last), method: :delete, :class => "href-btn", data: {confirm: "削除してもよろしいですか??"} %></button>
        </div>
      </div>
    </div>
routes.rb

Rails.application.routes.draw do
  root to: "toppages#index"

  get "login"=>"sessions#new"
  post "login"=>"sessions#create"
  delete "logout"=>"sessions#destroy"

  get "signup" => "users#new"
  resources :users, :only => [:show, :create, :destroy, :edit, :update]


  get "analysis" => "diaries#analysis"
  resources :diaries, :only => [:new, :create, :destroy, :edit, :update, :index]

  resources :goals, :only =>[:new, :create, :edit, :update]



end
rails routesの結果

analysis GET    /analysis(.:format)                                                                      diaries#analysis
                  diaries GET    /diaries(.:format)                                                                       diaries#index
                          POST   /diaries(.:format)                                                                       diaries#create
                new_diary GET    /diaries/new(.:format)                                                                   diaries#new
               edit_diary GET    /diaries/:id/edit(.:format)                                                              diaries#edit
                    diary PATCH  /diaries/:id(.:format)                                                                   diaries#update
                          PUT    /diaries/:id(.:format)                                                                   diaries#update
                          DELETE /diaries/:id(.:format)                                                                   diaries#destroy
自分の力のみでは解決できないようなのでどなたかよろしければアドバイスをいただけたらと思います。
それと、lastメソッドとfirstメソッドは根本的に違う機能を持ってたりするのでしょうか?その辺もお聞かせいただけたらと思います。
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

再現してみました。

@user = current_user
@diary = @user.diaries.build

@diary_last = @user.diaries.last
=> #<Diary id: nil, user_id: 1>

@user.diaries
=> #<ActiveRecord::Associations::CollectionProxy [#<Diary id: 1, user_id: 1>, #<Diary id: nil, user_id: 1>]>

buildしたレコードがテーブルの最後のレコードになっており、
@user.diaries.lastでそのレコードを取得しています。
なのでidnilとなっているためNo route matchesと言われるのでしょう。
おそらくですが、取得したいのはその一つ前のレコードでしょうか。
とりあえずの解決としてはbuildする前に@diary_lastを取得する事ですかね。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/12/07 16:28

    順序を入れ替えることで見事解決しました!!

    .buildとすることで.saveを行う前でも "データの入ってないひとつのレコード" として扱われるんですね!!
    これはとてもいい勉強になります!

    ありがとうございました!!!

    キャンセル

同じタグがついた質問を見る

  • Ruby

    8234questions

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

  • Ruby on Rails 5

    2193questions