Question

我有一个简单的脚手架项目来测试这个功能。它有两个模型，User模型和Fruit模型。

我正在尝试实现一个功能，用户可以使用数据库中的collection_select选择现有的水果，然后将其与他的名字一起列出，也可以根据水果查询用户（EG：列出所有苹果用户））

注意：我在水果桌上装满了样品水果来测试它。不确定它是否是正确的方法，但我让collection_select工作。

我的问题

问题是双重的。一个水果没有保存到特定用户（即使params有必要的属性），第二个如果我试图调用User.fruits.name我得到“水果”和User.fruits给我一个意想不到的结果“ ＃”。

我一直在阅读并尝试我的主要项目的解决方案，该项目具有确切的结构，但它似乎无法工作。如果有更高效的版本，我也在寻找其他选择。

以下是代码段：

参数哈希

class User < ActiveRecord::Base
  has_many :fruits
end

用户/ index.html.erb

<p id="notice"><%= notice %></p>

<h1>Listing Users</h1>

<table>
  <thead>
    <tr>
      <th>Name</th>
      <th colspan="3"></th>
    </tr>
  </thead>

  <tbody>
    <% @users.each do |user| %>
      <tr>
        <td><%= user.name %></td>
        <<td><%= user.fruits%></td>
        <td><%= link_to 'Show', user %></td>
        <td><%= link_to 'Edit', edit_user_path(user) %></td>
        <td><%= link_to 'Destroy', user, method: :delete, data: { confirm: 'Are you sure?' } %></td>
      </tr>
    <% end %>
  </tbody>
</table>

<br>

<%= link_to 'New User', new_user_path %>

用户/ _form.html.erb

<%= form_for(@user) do |f| %>
  <% if @user.errors.any? %>
    <div id="error_explanation">
      <h2><%= pluralize(@user.errors.count, "error") %> prohibited this user from being saved:</h2>

      <ul>
      <% @user.errors.full_messages.each do |message| %>
        <li><%= message %></li>
      <% end %>
      </ul>
    </div>
  <% end %>

  <div class="field">
    <%= f.label :name %><br>
    <%= f.text_field :name %>
  </div>

  <div class="field">
    <%= f.label :fruits %>
    <%= f.collection_select(:fruits, Fruit.all, :id, :name, :include_blank => "Please select") %>
  </div>
  <div class="actions">
    <%= f.submit %>
  </div>
<% end %>

users_controller

class UsersController < ApplicationController
  before_action :set_user, only: [:show, :edit, :update, :destroy]

  # GET /users
  # GET /users.json
  def index
    @users = User.all
  end

  # GET /users/1
  # GET /users/1.json
  def show
  end

  # GET /users/new
  def new
    @fruit = Fruit.new
    @user = User.new
  end

  # GET /users/1/edit
  def edit
  end

  # POST /users
  # POST /users.json
  def create
    debugger
    @user = User.new(user_params)

    respond_to do |format|
      if @user.save
        format.html { redirect_to @user, notice: 'User was successfully created.' }
        format.json { render :show, status: :created, location: @user }
      else
        format.html { render :new }
        format.json { render json: @user.errors, status: :unprocessable_entity }
      end
    end
  end

  # PATCH/PUT /users/1
  # PATCH/PUT /users/1.json
  def update
    respond_to do |format|
      if @user.update(user_params)
        format.html { redirect_to @user, notice: 'User was successfully updated.' }
        format.json { render :show, status: :ok, location: @user }
      else
        format.html { render :edit }
        format.json { render json: @user.errors, status: :unprocessable_entity }
      end
    end
  end

  # DELETE /users/1
  # DELETE /users/1.json
  def destroy
    @user.destroy
    respond_to do |format|
      format.html { redirect_to users_url, notice: 'User was successfully destroyed.' }
      format.json { head :no_content }
    end
  end

  private
    # Use callbacks to share common setup or constraints between actions.
    def set_user
      @user = User.find(params[:id])
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def user_params
      params.require(:user).permit(:name, :fruit_ids)
    end
end

fruit.rb

class Fruit < ActiveRecord::Base
  belongs_to :user
end

user.rb

class User < ActiveRecord::Base
  has_many :fruits
end

schema.rb

ActiveRecord::Schema.define(version: 20151215214130) do 

create_table "fruits", force: :cascade do |t|
    t.string   "name"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.integer  "user_id"
  end

  create_table "users", force: :cascade do |t|
    t.string   "name"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

end

老实说，我的智慧结束了。我觉得我错过了一些非常基本和明显的东西。

Answer 1

似乎是对user.fruits的误解。用户的fruits集合本质上是一个Fruit对象的数组。你不想这样做

= @user.fruits

或者

= User.fruits # I don't think this method on the `User` class exists

也许你想这样做

= @user.fruits.map{|f| f.name}

或者

= render @user.fruits

后者将调用_fruit.erb partial并根据其中的代码进行渲染。

# _fruit.erb
= fruit.name

要设置user.fruits，您需要在db中设置它。有两种方法可以做到这一点

1）将水果ID发送给用户＃fruit_ids = method

# for example
@user.fruit_ids = [1,2,3,4]

2）在水果对象上设置user_id

@fruit.user = @user # or @fruit.user_id = @user.id

看起来您想将fruit_ids传递给用户。要做到这一点，执行此操作的常用方法是设置复选框网格。他们需要有一个参数名称“user [fruit_ids] []”。最后的额外[]是向rails表明它是一个数组。

另一种方法，也就是你想要的，是设置一个多选框

<%= f.collection_select(:fruit_ids, Fruit.all, :id, :name, { :include_blank => "Please select")}, { :multiple => true } %>

在控制器中，确保强params接受此参数 - 语法如下

params.require(:user).permit(:name, :fruit_ids => [])

然后你可以将id数组传递给控制器，它会在所有水果对象上正确设置user_id。

最后 - 我不确定你的意思是做什么的。 has_many，belongs_to关系通常是父子关系。我没有看到你提前设置水果然后将其分配给用户的情况。也许这只是一个练习？另一种在水果上设置user_id的方法 - 您可以在用户下创建水果。

@user.fruits << Fruit.new(:name => "Orange")

如果您希望拥有一个水果对象（系统中只有一个橙色或一个Apple），用户可以在其中收藏它或选择它而不编辑水果记录，您可以这样做。

有另一张表代表喜欢的水果的表格如下：

class UserFruit < ActiveRecord::Base

  belongs_to :user
  belongs_to :fruit

end

使用此模型，您将能够代表所有权或喜爱的水果并重复使用水果对象，以便其他用户也可以喜欢或拥有它。

您可以这样设置

class User < ActiveRecord::Base
  has_many :user_fruits, :dependent => :destroy
  has_many :fruits, :through => :user_fruits
end

并更改fruit.rb

class Fruit < ActiveRecord::Base
  has_many :user_fruits, :dependent => :destroy
  has_many :users, :through => :user_fruits
end

您仍然可以使用@user.fruit_ids=(array)方法创建/销毁联接表记录，您的用户表单根本不会发生变化。

不要忘记迁移

create_table :user_fruits do |t|
  t.references :user
  t.references :fruit
end

collection_select with has_many和belongs_to

1 个答案: