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

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

ただいまの
回答率

88.63%

No route matches [GET] "/users/user.id"のエラーについて

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,114
退会済みユーザー

退会済みユーザー

 やりたいこと/困っていること

参考書等を参考にしてrailsを勉強しています。
参考書を基に所々アレンジしたらエラーが起きてしまいました。
エラーメッセージで結ばれていないと警告されています。私としては結べていると思っているのですが何がいけないのでしょうか?

<div class="main users-index">
  <div class="container">
    <h1 class="users-heading">ユーザー一覧</h1>
    <!-- 以下の<% %>を使ってeach文を追加してください -->
        <% @users.each do |user| %>
      <div class="users-index-item">
        <div class="user-right">
          <!-- 以下の<%= %>を使ってユーザー名を表示してください -->
          <%= link_to(current_user.username, "/users/#{user.id}") %>
        </div>
      </div>
      <% end %>
    <!-- 以下の<% %>を使ってeach文のendを追加してください -->
  </div>
</div>
Rails.application.routes.draw do
  get "users/index" => "users#index"
  get "users/id" => "users#show"
  #get 'users/index'
  #get 'pages/home'
  get "/" => "home#top"
  get "about" => "home#about"
  root to: 'pages#home'
  devise_for :users
  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
end
//uesr.rb
class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
  # nameカラムに関するバリデーションを作成してください
  #validates :name, {presence: true}

  # emailカラムに関するバリデーションを作成してください
  #validates :email, {presence: true, uniqueness: true}

  def username
    return self.email.split('@')[0].capitalize
  end

  def index
    @users = User.all
  end

  def show
    @user = User.find_by(id: params[:id])
  end

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

+1

コードに書いてある通りeach文追加すれば良いだけと思います。
user.idだけだとuserがローカル変数になっているので、ブロック作って変数定義しない限り使えないですね。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/11/19 12:51

    そうなんですね。
    参考書等を記述を真似てeach文を書いてみたのですが案の定エラーが出てしまいました。
    エラーメッセージはundefined method `each' for nil:NilClassと表示されています。
    メソッドの中に何もないから実行できないと言われているのですが、user.rb内の@userの後のコードに問題があるのでしょうか?

    キャンセル

checkベストアンサー

0

Modelで @users を定義してもModel内部からしか参照できませんよ
Modelで定義するのは自身だけで完結するものです
Viewとの仲介はControllerで行いましょう

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
  # nameカラムに関するバリデーションを作成してください
  #validates :name, {presence: true}

  # emailカラムに関するバリデーションを作成してください
  #validates :email, {presence: true, uniqueness: true}

  def username
    return self.email.split('@')[0].capitalize
  end
end

class UsersController < ApplicationController
  def index
    @users = User.all
  end

  def show
    @user = User.find_by(id: params[:id])
  end
end

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/11/20 07:05

    すみません。コードを全部見てませんでした。Ighrsさんの言う通り、@usersをusers_controllerに定義すればOKだと思います。

    キャンセル

  • 2018/11/20 20:44

    回答していただきありがとうございました。
    お二人のおかげで解決することができました。

    キャンセル

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

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

関連した質問

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