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

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

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

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

Ruby on Rails

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

Q&A

解決済

1回答

2748閲覧

railsでログインしたユーザーと1:1の関係にある投稿(アカウント)を表示させたい

zendendo

総合スコア43

Ruby

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

Ruby on Rails

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

0グッド

0クリップ

投稿2017/11/08 02:08

編集2017/11/08 02:17

###実現したいこと
ruby(ruby on rails)で、銀行みたいなアプリをつくっており、
ユーザーは一人につき一つだけ口座を保有しています。
ログインしたユーザーが、自分の口座情報だけを見れる
(他のユーザーはアクセスできないし、自分も他のユーザーの口座情報にアクセスはできない)
ようにしたいのですが、うまくできずに困っています。

ログインしたユーザーが自分の口座画面を見るためにヘッダーにある「口座」というメニューをクリックするのですが、
このときのリンクの設定はどうすればいいのか?

ログインしているユーザーを識別したり、口座情報の詳細を取得する際のコントローラの記述はどうすればいいか?

教えていただければ幸いです。
よろしくお願いします。

###前提と現状のコード内容
deviseでユーザー認証等をしています。
userモデルとbasic_income_account(口座)モデルは1:1で関連付けをしています。

header.html.erb(リンク設定付近のみ記述)

・・・ <ul class="nav navbar-nav navbar-right"> <li><%= link_to "Home", root_path %></li> <li><%= link_to "口座", basic_income_account_path %></li> </ul> ・・・

config/routes.rbの記述です。
ユーザーは口座を一つしか持てないので単数リソースにしました。

Rails.Application.routes.draw do devise_for :users, :controllers => { :registrations => 'users/registrations' } #単数リソース(resouce).一人のユーザーが一つだけもつ自分のアカウントを管理するとき用。 #作成されるURLを絞り込む。今回の場合はshowのみ作成。 resource :basic_income_account, :only => [:show] devise_scope :user do root :to => "devise/sessions#new" end end

app/controllers/basic_income_accounts_controller.rbの記述です。
現状では、http://localhost:3000/basic_income_accountにアクセスすると
ActiveRecord::RecordNotFound in BasicIncomeAccountsController#show
Couldn't find BasicIncomeAccount with 'id'=
というエラーが発生します。

class BasicIncomeAccountsController < ApplicationController #ログイン済みユーザーのみアクセスを許可する before_action :authenticate_user! def show @basic_income_account = BasicIncomeAccount.find(params[:id]) end private def user_params #paramsがuserというキーを持ち、params[:user]は、 #:passwordと:emailというキーをもつハッシュであることを検証する params.require(:user).permit(:password, :email) end def basic_income_account_params #paramsが:basic_income_accountというキーをもち、params[:basic_income_account]は、 #:account_numberと:user_idというキーをもつハッシュであることを検証する params.require(:basic_income_account).permit(:account_number, :user_id) end end

app/views/basic_income_accounts/show.html.erb(口座情報表示画面)の記述です。

<h1>BasicIncomeAccounts#show</h1> <p>app/views/basic_income_accounts/show.html.erb</p> <p>ここでは口座の詳細(口座番号とユーザー名と残高)を表示します</p> <table class ="table-striped table-hover"> <thead> <tr> <th>口座番号</th> <th>ユーザー名</th> <th>残高</th> </tr> </thead> <tbody> <tr> <td><%= @basic_income_account.account_number %></td> <td><%= @basic_income_account.user_id %></td> <td><%= @basic_income_account.balance %></td> </tr> </tbody> </table>

###補足情報(言語/FW/ツール等のバージョンなど)
Rails 5.1.3
ruby 2.4.1
devise (4.3.0)

ここまで読んで下さりありがとうございます。
もし足りない部分を指摘して頂ければ追記します。

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

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

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

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

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

guest

回答1

0

ベストアンサー

エラーはparams[:id]を渡していないため、表示する口座が特定できないことが原因です。

ruby

1 def show 2 @basic_income_account = BasicIncomeAccount.find(params[:id]) 3 end

deviseを使っているなら current_user でログインしているユーザーを取得できるので
current_userの関連から持ってくれば、口座を特定できます。

ruby

1 def show 2 @basic_income_account = current_user.basic_income_account 3 end

投稿2017/11/08 02:41

hellomartha

総合スコア329

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

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

zendendo

2017/11/08 07:43

hellomarthaさん、ありがとうございます!解決することができました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問