29

私はRubyonRailsを初めて使用し、MichaelHartlの優れた本であるRubyonRailsチュートリアルに大いに助けられました。私は第8章に到達し、現在その章の演習を行っています。演習1で(典型的な「初心者」と思われる)問題が発生しています。この演習では、「1.form_forの代わりにform_tagを使用するようにサインインフォームをリファクタリングする」ように求められます。Stackoverflow、Google、Railscast、およびその他の多くの「Web検索」で2日間、これに関する支援を検索しようとしましたが、この問題に答えるのに必要な支援が見つからないようです。form_tagで変更しようとしているファイルは次のとおりです。

<% provide(:title, "Sign in") %>
<h1>Sign in</h1>

<div class="row">
  <div class="span6 offset3">
    <%= form_for(:session, url: sessions_path) do |f| %>

      <%= f.label :email %>
      <%= f.text_field :email %>

      <%= f.label :password %>
      <%= f.password_field :password %>

      <%= f.submit "Sign in", class: "btn btn-large btn-primary" %>
     <% end %>

    <p>New user? <%= link_to "Sign up now!", signup_path %></p>
  </div>
</div>

このアプリケーションではRails3.2.3を使用しています。誰かが私を正しい方向に向けることができますか?誰かがこの問題で私を助けることができますか?私は最も感謝するでしょう。

これは、form_tagを使用する実装です。

<% provide(:title, "Sign in") %>
<h1>Sign in</h1>

<div class="row">
  <div class="span6 offset3">
    <%= form_tag( url: sessions_path ) do  %>

      <%= label_tag :email %>
      <%= text_field_tag :email %>

      <%= label_tag :password %>
      <%= password_field_tag :password %>

      <%= submit_tag "Sign in", class: "btn btn-large btn-primary" %>
    <% end %>

    <p>New user? <%= link_to "Sign up now!", signup_path %></p>
  </div>
</div>

私はRspec2.9.0を使用していますが、以下は失敗したテストです。

describe "signin page" do
    before { visit signin_path }

    it { should have_selector('h1',    text: 'Sign in') }
    it { should have_selector('title', text: 'Sign in') }
  end 

describe "with invalid information" do
            before { click_button "Sign in" }

            it { should have_selector('title', text: 'Sign in') }
            it { should have_selector('div.alert.alert-error', text: 'Invalid') }

            describe "after visiting another page" do
              before { click_link "Home" }
              it { should_not have_selector('div.alert.alert-error') }
            end
      end

describe "with valid information" do
            let(:user) { FactoryGirl.create(:user) }
            before do
              fill_in "Email",    with: user.email
              fill_in "Password", with: user.password
              click_button "Sign in"
            end

            it { should have_selector('title', text: user.name) }
            it { should have_link('Profile', href: user_path(user)) }
            it { should have_link('Sign out', href: signout_path) }
            it { should_not have_link('Sign in', href: signin_path) }

            describe "followed by signout" do
                    before { click_link "Sign out" }
                    it { should have_link('Sign in') }
            end
      end

これが私のルートファイルです:

SampleApp::Application.routes.draw do
  resources :users
  resources :sessions, only: [:new, :create, :destroy]


  get "users/new"

  root to: 'static_pages#home'

  match '/signup',  to: 'users#new'
  match '/signin',  to: 'sessions#new'
  match '/signout', to: 'sessions#destroy', via: :delete

  match '/help',    to: 'static_pages#help'
  match '/about',   to: 'static_pages#about'
  match '/contact', to: 'static_pages#contact'
end
4

5 に答える 5

31

私もこの演習を完了したばかりなので、私は決して専門家ではありません。ただし、これは私のために機能し、すべてのテストに合格したコードです。

../app/views/sessions/new.html.erb

<% provide(:title, "Sign in") %>
<h1>Sign in</h1>

<div class="row">
  <div class="span 6 offset 3">
    <%= form_tag sessions_path do %>

      <%= label_tag :email %>
      <%= text_field_tag :email, params[:email] %>

      <%= label_tag :password %>
      <%= password_field_tag :password %>

      <%= submit_tag "Sign in", class: "btn btn-large btn-primary" %>
    <% end %>

    <p>New User?<%= link_to "Sign Up Now", signup_path %> </p>
  </div>
</div>

../app/controllers/sessions_contollerも変更する必要がありました

class SessionsController < ApplicationController

  def new
  end

  def create
    user = User.find_by_email(params[:email])
    if user && user.authenticate(params[:password])
      session[:user] = user.id
      sign_in user
      redirect_to user
    else
      flash.now[:error] = 'Invalid email/password combination'
      render 'new'
    end  
  end

  def destroy
    sign_out
    redirect_to root_path
  end
end

これは機能しますが、なぜ機能するのか完全にはわかりません。コントローラーの変更が必要な理由を誰かが説明できれば、大いにありがたいです。これは別の質問として提起される可能性があることは知っていますが、OPと密接に関連しており、これを機能させる方法だけでなく、なぜこのように機能するのかを理解するのに非常に役立つと確信しています。以下は、元のビューとコントローラーファイルです。

元の'form_for'new.html.erb:

<% provide(:title, "Sign in") %>
<h1>Sign in</h1>

<div class="row">
  <div class="span6 offset3">
    <%= form_for(:session, url: sessions_path) do |f| %>

      <%= f.label :email %>
      <%= f.text_field :email %>

      <%= f.label :password %>
      <%= f.password_field :password %>

      <%= f.submit "Sign in", class: "btn btn-large btn-primary" %>
    <% end %>

    <p>New user? <%= link_to "Sign up now!", signup_path %></p>
  </div>
</div>

および元のsessions_controller:

class SessionsController < ApplicationController

  def new
  end

  def create
    user = User.find_by_email(params[:session][:email])
    if user && user.authenticate(params[:session][:password])
      sign_in user
      redirect_to user
    else
      flash.now[:error] = 'Invalid email/password combination'
      render 'new'
    end  
  end

  def destroy
    sign_out
    redirect_to root_path
  end
end
于 2012-05-13T12:57:08.817 に答える
15

私はチュートリアルの同じステップに取り組んでいます、そしてあなたの質問は私が私の道を見つけるのを助けました。

label_tagとtext_field_tagの代わりにlabeltext_fieldを使用すると正常に機能し、コントローラーコードを変更する必要はありません(これにより、元のform_forメソッドと同じHTMLコードが生成されます)。

<%= form_tag(sessions_path) do %>
  <%= label :session, :email %>
  <%= text_field :session, :email %>

  <%= label :session, :password %>
  <%= password_field :session, :password %>

  <%= submit_tag("Sign in", class: "btn btn-large btn-primary") %>
<% end %>

詳細については、 http: //guides.rubyonrails.org/form_helpers.html#dealing-with-model-objectsをご覧ください。

于 2012-10-22T15:23:34.333 に答える
2

RoRガイドでは、form_tagがどのように機能するかについて説明します。

http://guides.rubyonrails.org/form_helpers.html

于 2012-05-07T21:24:47.477 に答える
1

実装で使用していないSession_Helperがあります。

ヘルパーメソッドを編集します。

  def sign_in(user)
    cookies.permanent[:remember_token] = user.remember_token
    session[:user_id] = user.id
  end

  def sign_out
    cookies.delete(:remember_token)
    session[:user_id] = nil    
  end

次に、セッションコントローラで適切なメソッドを使用するだけです。これは、モジュール式の設計アプローチに従った、より優れた実装です。rspecにも問題があるようです。

describe "with valid information" do
      let(:user) { FactoryGirl.create(:user) }
      before do
        fill_in "Email",    with: user.email.upcase
        fill_in "Password", with: user.password
        click_button "Sign in"
      end

      it { should have_selector('title', text: user.name) }
      it { should have_link('Profile', href: user_path(user)) }
      it { should have_link('Sign out', href: signout_path) }
      it { should_not have_link('Sign in', href: signin_path) }

このページにアクセスするとすべてが正しい場合でも、これらのテストにはフラグが付けられます。私の推測では、変更する必要があるのはbefore do、それがあなたが望むことをしていれば、テストに合格するからです。

とにかく、誰かがアイデアを持っているなら、私はrspecの問題が何であるか興味があります!

問題はにあると思いますuser.email.upcase.upcaseあなたはそれを削除するときにテストに合格する必要はありません。

于 2013-07-22T08:58:19.023 に答える
1

フィールドに正しい名前を付けていないことに気づきました。オリジナルのソースを表示すると、命名スキームが表示されます。

<%= form_tag(sessions_path) do %>

  <%= label_tag 'session_email', 'Email' %>
  <%= text_field_tag 'session[email]' %>

  <%= label_tag 'session_password', 'Password' %>
  <%= password_field_tag 'session[password]' %>

  <%= submit_tag "Sign in", class: "btn btn-large btn-primary" %>
<% end -%>
于 2013-09-12T03:38:44.113 に答える