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

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

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

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

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

Q&A

解決済

2回答

1263閲覧

Ruby on Rails:うまくルーティングを設定できていないらしい?

S.N.

総合スコア21

Ruby

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

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

0グッド

0クリップ

投稿2017/09/14 12:15

ご覧いただきありがとうございます。
気になることがあり質問いたします。

前提条件

  • Ruby on Rails初学者です
  • 複数人でバージョン管理しながらアプリケーションをつくっています

困っていること

user_contorollerがうまく挙動していないです。
具体的には、「/user」で始まるページすべてで

ActiveRecord::RecordNotFound in UserController#top Couldn't find User with 'id'=

と表示されます。
無理にバージョンを統合したのでモデルだったりルーティングだったり
で混乱したのかなと思いました。
モデルは新しく作ろうとすると重複していると言われたので可能性としては
ないかなと感じています。

ソースコード

Ruby

1# 以下ルーティング 2#ログイン後のユーザトップページ 3 get 'user/:id' => 'user#top' 4 5#ログインのルーティング 6 post 'user/:id/login' => 'user#login' 7 get 'user/login_form' => 'user#login_form' 8 9#編集ページのルーティング 10 post 'user/:id/edit' => 'user#update' 11 get 'user/:id/edit' => 'user#edit' 12 13#新規投稿ページのルーティング 14 get 'user/new' => 'user#new' 15 post 'user/new' => 'user#create' 16 17#トップページのルーティング 18 get 'user/home' => 'user#home' 19 20 21# 以下コントローラ 22def home 23 end 24 25 def top 26 @user = User.find(params[:id]) 27 @post = Post.where(params[:id]) 28 @comment = Comment.where(params[:id]) 29 end 30 31 def new 32 @user = User.new 33 end 34 35 def create 36 @user = User.new( 37 user_name: params[:user_name], 38 email: params[:email], 39 password: params[:password] 40 ) 41 session[:user_id] = @user.id 42 @user.save 43 redirect_to user_top_path(user_params[:user_id]) 44 end 45 46 def edit 47 @post = Post.where(id: params[:id]) 48 @comment = Comment.where(id: params[:id]) 49 end 50 51 def update 52 @user = User.find_by(id: params[:id]) 53 @user.update_attributes( 54 user_name: params[:user_name], 55 email: params[:email], 56 password: params[:password] 57 ) 58 redirect_to('/user/:id') 59 end 60 61 def login_form 62 end 63 64 def login 65 puts params 66 @user = User.find_by( 67 email: params[:email], 68 password: params[:password] 69 ) 70 if @user 71 session[:user_id] = @user.id 72 redirect_to('/user/:id') 73 else 74 render('user/login_form') 75 end 76 end 77 78 def logout 79 @user = User.find_by(id: session[:user_id]) 80 if @user 81 session[:user_id] = nil 82 redirect_to(user_top_path) 83 end 84 end 85 86 private 87 def user_params 88 params.require(:user).permit(:user_name, :email, :password) 89 end 90

長くなってしましましたが、困っていることといたしましては
冒頭に記載したエラーがユーザ回りのビューすべてで出てしまうことです。
原因、対応策等ご教示いただけますでしょうか。

よろしくお願いいたします。

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

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

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

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

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

guest

回答2

0

ベストアンサー

ルーティングは、先に書いたほうが優先されます。そのため、真っ先に get 'user/:id' => 'user#top'と書いてしまうと、user/以下のルートはすべてこれが回収してしまうことになってしまいます。

なお、Rails wayとしてはgetpostの単品で書くより、resourcesなどでRESTfulなルーティングを使う方がいい、とのことです。

投稿2017/09/14 13:21

maisumakun

総合スコア145183

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

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

S.N.

2017/09/14 23:27

ご回答ありがとうございます。 リソースベースで書き換えてみましたが、うまくいかなかったのでアプリを作り直しました。
guest

0

routes.rb

のuserを全てusersにしてみてください。

コントローラー名もuser_controller.rbになっていそうなので、そうならusers_controller.rbにしてみてください。

投稿2017/09/14 13:21

glvty83

総合スコア135

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

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

S.N.

2017/09/14 23:28

ご回答ありがとうございます。 やってみましたが、変化が見られずでした… ちなみに、複数形にするとどんな変化が期待できるのでしょうか?
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問