4

我正在尝试构建一些东西，它会从 Ruby 中的列表中随机选择两个对象（使用 Rails），然后对这些对象执行基本操作。我目前的设置涉及在开始时随机选择数字 - 使用这些数字从数据库中获取对象，然后将这些对象渲染到索引。但是，现在我正在尝试使用 AJAX 编辑这些对象，但遇到了问题。

似乎，当我单击一个链接时，随机数正在重新计算并导致我的 AJAX 函数不起作用。

我已经在下面包含（我相信是）所有相关代码，但如果您需要查看其他内容以了解详情，请告诉我。我正在寻找旧动物的标题和（最好是它们以前的评级和）它们的新评级出现在页面底部。

谢谢，

马特

controllers/static_pages.rb

class StaticPagesController < ApplicationController
respond_to :html, :js
before_filter :pickanimals

def pickanimals
    @allAnimals = Animal.all
    @random_no = rand(@allAnimals.length)
    @animal = @allAnimals[@random_no]
    @allAnimals.delete_at(@random_no)
    @newRandom = rand(@allAnimals.length)
    @animal2 = @allAnimals[@newRandom]
end

  def index
    respond_to do |format|
        format.html
        format.js
    end
end

  def help
  end

  def about
  end

  def contact
  end

def league
end

def voting
    @votedAnimal = Animal.find(params[:id])
    if @votedAnimal == @animal
        @animal.rating += 1
        @animal2.rating -= 1
    else
        @animal.rating -= 1
        @animal2.rating += 1    
    end
    Animal.transaction do
        @animal.save!
        @animal2.save!
    end
    respond_to do |format|
        format.html { redirect_to root_path }
        format.js  
    end
end
end

app/views/static_pages/index.html.erb

<div class="center hero-unit">
<h1>Animal Attack</h1>
<p> Who will win when nature collides? </p>
<div class="animalcontainerright">
<h2> <%= @animal.name %> </h2> 
<%= link_to image_tag(@animal.attachment.url(:large)), voting_path(@animal.id), :remote => true, :confirm => "Rating:  "+@animal.rating.to_s  %>
</div>
<div class="animalcontainerleft">
<h2> <%= @animal2.name %> </h2>
<%= link_to image_tag(@animal2.attachment.url(:large)), voting_path(@animal2.id), :remote => true, :confirm => "Rating:  "+@animal2.rating.to_s  %>
</div>
<div id="animalfacts"></div>

routes.rb

AnimalAttack::Application.routes.draw do

resources :animals

root to: 'static_pages#index'
match '/help' => 'static_pages#help'
match '/about' => 'static_pages#about'
match '/contact' => 'static_pages#contact'
match '/league' => 'static_pages#league'
match '/voting/:id' => 'static_pages#voting'

end

rake routes

    animals GET    /animals(.:format)          animals#index
        POST   /animals(.:format)          animals#create
 new_animal GET    /animals/new(.:format)      animals#new
edit_animal GET    /animals/:id/edit(.:format) animals#edit
 animal GET    /animals/:id(.:format)      animals#show
        PUT    /animals/:id(.:format)      animals#update
        DELETE /animals/:id(.:format)      animals#destroy
   root        /                           static_pages#index
   help        /help(.:format)             static_pages#help
  about        /about(.:format)            static_pages#about
contact        /contact(.:format)          static_pages#contact
 league        /league(.:format)           static_pages#league
               /animals(.:format)          animals#new
               /voting/:id(.:format)       static_pages#voting
4

1 回答 1

8

我猜@animal是因为你的:before_filter. 除非您另有说明，否则它将在所有操作之前执行（因此在每个控制器操作@animal之前重新分配）。您可能希望将其限制为仅需要新生成的随机数的方法。例如：

before_filter :pickanimals, :only => :index

还。你知道Array#sample吗？这是一种从数组中选择随机元素的内置方式。您可能会发现它在这里很有用：

@animal, @animal2 = @allAnimals.sample(2)

此外

像这样依赖控制器中的实例变量是在函数之间传递信息的一种非常脆弱的方式。如果您的控制器需要知道id两个模型的 s 以根据用户选择在每个模型上增加或减少计数器，为什么不使用表单将这两个ids 发送到此控制器操作？到目前为止，您只需要两个字段：chosen_animal_id和reject_animal_id.

于 2012-11-03T00:06:55.277 回答