用户和链接关联困难

Question

我有一个用户和链接MVC。我在下面列出了它们。本质上，当我创建一个新链接时，我希望它与我的用户绑定，然后在我的显示页面上显示带有报价的用户电子邮件，但当我进行身份验证时，我仍然会为用户获得零值，即使我有：

1. 用户和链接关联
2. 我的强参数中允许的:user_id
3. 有一个before_filter，它要求用户在发出新请求时登录
4. 在我的链接模式中有一个user_id

如果你看看我的show.html.erb和行<%= @link.user.try(:email) %>，这就是发布链接的用户电子邮件应该出现的地方，但它们都显示为零值。

我现在有点不知所措，为什么我不能让它发挥作用，任何帮助都将不胜感激！

型号：

class User < ActiveRecord::Base
  has_many :links
  acts_as_voter
  devise :database_authenticatable, :registerable,
    :recoverable, :rememberable, :trackable, :validatable
end
class Link < ActiveRecord::Base
  belongs_to :user
  acts_as_votable
  attr_accessor :avatar
  mount_uploader :avatar, AvatarUploader
end

控制器：

class LinksController < ApplicationController
  before_filter :authenticate_user!, except: [:index, :show]
  def index
    @links = Link.all
  end
  def show
    @link = Link.find(params[:id])
  end
  def new
    @link = Link.new
  end
  def edit
  end
  def create
    @link = Link.new(link_params)
    if @link.save
      redirect_to root_path
    else
      render 'new'
    end
  end
  private
  def link_params
    params.require(:link).permit(:title, :url, :avatar, :user_id)
  end
end

show.html.erb:

<p id="notice"><%= notice %></p>
<p>
  <strong>Author:</strong>
  <%= @link.title %>
</p>
<p>
  <strong>Quote:</strong>
  <%= @link.url %>
</p>
<small class="author">Submitted <%= time_ago_in_words(@link.created_at) %> ago by <%= @link.user.try(:email) %></small>

架构：

create_table "links", force: :cascade do |t|
    t.string   "title"
    t.string   "url"
    t.datetime "created_at",                     null: false
    t.datetime "updated_at",                     null: false
    t.integer  "user_id"
    t.integer  "cached_votes_total", default: 0
    t.integer  "cached_votes_score", default: 0
    t.integer  "cached_votes_up",    default: 0
    t.integer  "cached_votes_down",  default: 0
    t.string   "avatar"
  end
 create_table "users", force: :cascade do |t|
    t.string   "email",                  default: "", null: false
    t.string   "encrypted_password",     default: "", null: false
    t.string   "reset_password_token"
    t.datetime "reset_password_sent_at"
    t.datetime "remember_created_at"
    t.integer  "sign_in_count",          default: 0,  null: false
    t.datetime "current_sign_in_at"
    t.datetime "last_sign_in_at"
    t.string   "current_sign_in_ip"
    t.string   "last_sign_in_ip"
    t.datetime "created_at",                          null: false
    t.datetime "updated_at",                          null: false
  end

Answer 1

听起来好像从未设置过user_id。碰巧的是，您可能无论如何都不想通过params设置它（因为这些参数可以由用户操作）。

取而代之的是更换

 @link = Link.new(link_params)

带

 @link = current_user.links.build(link_params)

如果链接总是有一个用户，我也会将user_id列标记为非空，而不是在我的应用程序中到处调用try（通常我更喜欢try!而不是try）

Answer 2

1.据我所知，逻辑是只有登录后才能创建链接。用户只能为他/她创建链接（只有登录的用户才能是他创建的链接的所有者）。在这种情况下，您不应该在表单中公开user_id，而是在创建操作中添加它：

 def create
    @link = Link.new(link_params)
    if @link.save
      current_user.links << @link
      redirect_to root_path
    else
      render 'new'
    end
  end

2.此外，如果你不允许孤立链接（没有用户作为所有者），那么我建议不要使用user.try(:email)，而是使用@link.user.email

3. 如果您通过@link访问用户对象，那么您可以在显示操作中预加载它以保存一些DB查询

   def show   
    @link = Link.includes(:user).find(params[:id])  
   end
   

请进行这些更改，然后重试。此外，如果您使用默认的设备配置，则用户需要电子邮件，因此.try(:email)返回nil可能意味着找不到user对象，这意味着关联设置不正确（我的版本可能会修复此问题）