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

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

ただいまの
回答率

90.03%

NoMethodError in Users::Configurations#indexのエラーの直し方

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,214

culuculu

score 52

現在複数のWebページでrailsの勉強をしています。
そこでユーザの名前の変更ができる機能とユーザを一覧表示させるのではなく、個人の名前やメールアドレスは個人しか見えないようにしたいと思っています。
現在のエラーは、NoMethodError in Users::Configurations#index
と出ており、undefined method `name' for nil:NilClass と言われています。

そして、ユーザの一覧表示は出来るのですがこのままでは自分自身以外のユーザの情報や編集、削除もできてしまうので個人の情報しか見えないようにしたいです。自分の書いたコードのコントローラーのindexメソッドであるUser.find_by_id(params[:user_id])にしています。 どのようにすればいいでしょうか? ご回答お願いします。 以下にソースコードを載せておきます。

<h1>Users::Configurations#edit</h1>
<p>Find me in app/views/users/configurations/edit.html.erb</p>
<h1>Editing User</h1>

<%= render 'form' %>

<%= link_to 'Show', @user %> |
<%= link_to 'Back', users_path %>
<h1>Users::Configurations#index</h1>
<p>Find me in app/views/users/configurations/index.html.erb</p>
<p id="notice"><%= notice %></p>

<h1>Listing Users</h1>

<table>
  <thead>
    <tr>
      <th>Name</th>
      </tr>
  </thead>

  <tbody>
    <% @user %>
      <tr>
        <td><%= @user.name %></td>
        <td><%= link_to 'Show', @user %></td>
        <td><%= link_to 'Edit', edit_@user_path(user) %></td>
        <td><%= link_to 'Destroy', @user, method: :delete, data: { confirm: '本当に退会してもよろしいですか?' } %></td>
      </tr>
  </tbody>
</table>
class Users::ConfigurationsController < ApplicationController
before_action :set_user, only: [:show, :edit, :update, :destroy]
  def index
    @user = User.find_by_id(params[:user_id])
  end

  def edit
  end

  def new
    @user = User.new
  end

  def show
  end

  def destroy
    @user.destroy
    respond_to do |format|
      format.html { redirect_to users_url, notice: 'User was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

check解決した方法

0

ブラウザに表示はできたので一応解決したという形で結果を載せます。
以下のコードのように修正すると直りました。

class Users::ConfigurationsController < ApplicationController
  def index
    @user = User.find_by_id(params[:user_id])
  end

  def edit
  end

  def destroy
    @user.destroy
    redirect_to users_path
  end
end
<h1>Users::Configurations#index</h1>
<p>Find me in app/views/users/configurations/index.html.erb</p>
 <ul>
<% @user %>
  <li>
   <%= link_to "[Edit]", users_configurations_edit_path %>
   <%= link_to "[Delete]", users_configurations_destroy_path, method: :delete, data: {
       confirm: "本当に削除してもよろしいですか?" } %>
  </li>
 </ul>

ですが、edit.html.erbとdestroy.html.erbがうまく書けていないので動作しませんがブラウザにはうまく表示できました。
回答してくださったyh1126さん。
そして、この課題に取り組んでいただいた方々、ありがとうございました。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

私も初心者ですが...
editは以下のようにしてみてはいかがでしょうか?おそらくidが渡っていないのでuserが見つけられず怒られているのではないでしょうか?コードを拝見しましたが、Showdeletelink_toは正常に動きますか?

 <td><%= link_to 'Edit', edit_user_path(user.id) %></td>

後者の問題ですが、個人の名前やメールアドレスは個人しか見えないようにしたい、とのことなので、そのページではログイン機能があると推測しました。そうすると、ログイン中のユーザのidをとってくることで、そのidを用いてwhere文やfind_byなどでユーザを探せばよろしいのではないでしょうか?
そうして探したuserの情報を元に、各種情報を表示すれば良いと思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/01/23 04:27 編集

    回答ありがとうございます。
    Showと Deleteのlink_toは正常に動いています。
    ログイン認証機能を説明していませんでした。申し訳ありません。
    whereやfind_byについて学習してみたところ、find_by_idがエラーの対策良さそうだったのでこれを選択しました。
    早速試してみたいと思います。

    キャンセル

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

  • ただいまの回答率 90.03%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる