29

我是 Ruby on Rails 的新手，Michael Hartl 的优秀书籍《Ruby on Rails 教程》给了我极大的帮助。我已经读到了第 8 章，现在正在学习该章的练习。我在练习 1 中遇到了（我假设是一个典型的“新手”）问题。在本练习中，要求“1.重构登录表单以使用 form_tag 代替 form_for”。我已经尝试在 Stackoverflow、Google、Railscast 和许多其他“网络搜索”中搜索这两天的帮助，但我似乎没有找到回答这个问题所需的帮助。我试图用 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>

我在这个应用程序中使用 Rails 3.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>

我正在使用 Rspec 2.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和text_field代替 label_tag 和 text_field_tag 工作正常，您不必更改控制器代码（这会产生与原始 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 回答