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

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

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

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

Ruby on Rails 4

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

Q&A

解決済

1回答

2792閲覧

【rails】login.html.erbにアクセスするがshow.html.erbの画面が現れる

sayama

総合スコア28

Ruby on Rails

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

Ruby on Rails 4

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

0グッド

0クリップ

投稿2016/08/17 15:11

編集2016/08/17 22:48

###発生している問題
railsの勉強を始めて数週間の初心者です。
ブラウザのURLの場所に

http://localhost:3000/members/login

と直接アドレスを打ち込むと、アクセスされるビューは

app/views/members/login.html.erb

だと思うのですが、違うビューの画面が現れてしまいます。

login.html.erbを以下に示しますが、特にredirect_toなど他のページに飛ぶような指示は入れていません。

それに、違うビューの画面が現れたときのURLは

http://localhost:3000/members/login

であり、ページが移動した訳ではないと思っています。

コマンドプロンプト上には、

Start GET "/members/login" for ::1 at 2016-08-18 00:05:07 +0900
Processing by MembersController#show as HTML
Parameters: {"id"=>"login"}
Rendered members/show.html.erb within layouts/application (0.0ms)
Complited 200 OK in 27ms (…

などと表示されています。

ちなみに、上記のURLをブラウザに入力したとき現れる、違うビューの画面はshow.html.erbの画面であり、中身は

<h1>Show</h1>

これだけです。

以上、よろしくお願い致します。

###該当のソースコード

【login.html.erb】 <h1>ログイン</h1> <%= form_for(@member, :url => {:action => 'login'}) do |f| %> <% if @member.errors.any? %> <div id="error_explanation"> <h2>エラーがあります。</h2> <ul> <% @member.errors.full_messages.each do |msg| %> <li><%= msg %></li> <% end %> </ul> </div> <% end %> <div class="field"> <%= f.label :user, "ユーザ名" %><br /> <%= f.text_field :user %> </div> <div class="field"> <%= f.label :pass, "パスワード" %><br /> <%= f.text_field :pass %> </div> <div class="actions"> <%= f.submit :value => "ログイン" %> </div> <% end %> 【routes.rb】 Rails.application.routes.draw do resources :messages resources :members get 'members/login' post 'members/login' get 'members/logout' post 'members/logout' root :to => 'members#login' コメントアウト文 end 【members_controller.rb】 # -*- coding: utf-8 -*- class MembersController < ApplicationController # GET /members # GET /members.json def index @members = Member.all end def logout p "check" sleep 5 session[:login] = nil redirect_to "/" return end # GET /members/1 # GET /members/1.json def show end # GET /members/new def new @member = Member.new end # GET /members/1/edit def edit end # POST /members # POST /members.json def create @member = Member.new(member_params) respond_to do |format| if @member.save format.html { redirect_to @member, notice: 'Member was successfully created.' } format.json { render :show, status: :created, location: @member } else format.html { render :new } format.json { render json: @member.errors, status: :unprocessable_entity } end end end # PATCH/PUT /members/1 # PATCH/PUT /members/1.json def update respond_to do |format| if @member.update(member_params) format.html { redirect_to @member, notice: 'Member was successfully updated.' } format.json { render :show, status: :ok, location: @member } else format.html { render :edit } format.json { render json: @member.errors, status: :unprocessable_entity } end end end # DELETE /members/1 # DELETE /members/1.json def destroy @member.destroy respond_to do |format| format.html { redirect_to members_url, notice: 'Member was successfully destroyed.' } format.json { head :no_content } end end def login params.permit! # これは本当はstrong parameterなるものに変えたが良いらしい @msg = "get" @member = Member.new if request.post? then @member = Member.new(params[:member]) record = Member.find_by_user(params[:member][:user]) if record == nil then @msg = "そのようなユーザはいません。" @member.errors.add('user', ":ユーザ名が違います。") else if record.pass != params[:member][:pass] then @msg = "パスワードが違います。" @member.errors.add('pass', ":パスワードが違います。") else @msg = "good!" @member = record session[:login] = record redirect_to '/messages' end end end end private # Use callbacks to share common setup or constraints between actions. def set_member @member = Member.find(params[:id]) end # Never trust parameters from the scary internet, only allow the white list through. def member_params params.require(:member).permit(:user, :pass, :name, :mail, :memo, :admin) end end 【rake routes の結果】 Prefix Verb URI Pattern Controller#Action messages GET /messages(.:format) messages#index POST /messages(.:format) messages#create new_message GET /messages/new(.:format) messages#new edit_message GET /messages/:id/edit(.:format) messages#edit message GET /messages/:id(.:format) messages#show PATCH /messages/:id(.:format) messages#update PUT /messages/:id(.:format) messages#update DELETE /messages/:id(.:format) messages#destroy members GET /members(.:format) members#index POST /members(.:format) members#create new_member GET /members/new(.:format) members#new edit_member GET /members/:id/edit(.:format) members#edit member GET /members/:id(.:format) members#show PATCH /members/:id(.:format) members#update PUT /members/:id(.:format) members#update DELETE /members/:id(.:format) members#destroy members_login GET /members/login(.:format) members#login POST /members/login(.:format) members#login members_logout GET /members/logout(.:format) members#logout POST /members/logout(.:format) members#logout root GET / members#login ![rake routesの実行結果](86fcac9db81b25154b581c876193bb23.png)

###補足情報(言語/FW/ツール等のバージョンなど)
windows 7
Ruby 2.1.9p490
Rails 4.2.1

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

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

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

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

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

退会済みユーザー

退会済みユーザー

2016/08/17 15:15

memberscontrollerか、routes.rbに恐らく原因があるのでファイル提示してもらってもいいでしょうか。
tkmtmkt

2016/08/17 15:42

rake routes の結果を見た方が確実かも。
guest

回答1

0

ベストアンサー

/members/loginより先に,/members/:id(.:format) にurlがマッチしてしまうためです。
直す方法は,複数あります。

1,resource :membersより前に

get 'members/login'
post 'members/login'
get 'members/logout'
post 'members/logout'

を書く

2.loginをactionにするのをやめて,showの中でprivateメソッドログインを呼び出すようにする。

3.login機能を他のコントローラの処理に分離する。

1が最も簡単ですが,また同じミスをすると,ミスに気づきずらくなります。
他にも方法はあると思いますが3がよくやられる手法です。

投稿2016/08/17 23:49

退会済みユーザー

退会済みユーザー

総合スコア0

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

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

sayama

2016/08/18 01:11

ありがとうございます。 resourceの意味が分かっていませんでしたので、書く順番が関係あるとは思ってなかったです。 2,3の方法も勉強として確かめたいと思います。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問