3

使用我的 Rails 应用程序,我可以成功创建一个对象(称为工作;将它们视为博客文章)作为 current_user。用户 has_many 工作。我可以通过使用我的 postgresql 浏览器检查数据库来验证对象是否已成功创建。该表还包含创建工作的正确 user_id,因此我知道我的 create 函数在我的控制器中工作。

但是,问题是当我尝试查看作品时,我收到以下错误:

WorksController#show 中的 ActiveRecord::RecordNotFound

找不到 id=23 的用户

app/controllers/works_controller.rb:43:in `show'

奇怪的是,我仍然可以查看我几周前创作的作品。该错误仅出现在我最近创作的作品中。

这是 Works 控制器:

class WorksController < ApplicationController
   #before_filter :current_user,   only: [:edit, :update]

  def index
    @works = Work.all

respond_to do |format|
  format.html # index.html.erb
  format.xml  { render :xml => @works }
end
end
def create
    @work = current_user.works.create(params[:work])
    redirect_to current_user
  end

def edit
    @work = current_user.works.find(params[:id])
end

def new
  @work = current_user.works.new
end

def destroy
  @work = current_user.works.find(params[:id]).destroy
  flash[:success] = "Work deleted"
  redirect_to current_user
end

 def update
    @work = current_user.works.find(params[:id])
    if @work.update_attributes(params[:work])
      flash[:success] = "updated"
      redirect_to @work
    else
      render 'edit'
    end
  end


  def show
    @user = User.find(params[:id])
    @work = @user.works.find(params[:id])
    @activities = PublicActivity::Activity.order("created_at DESC").where(trackable_type: "Work", trackable_id: @work).all

    respond_to do |format|
      format.html # show.html.erb
      format.xml  { render :xml => @work }
    end
  end
end

我假设错误在 Works 控制器中。为了修复“显示”错误,我需要在控制器中编辑什么?

编辑:如果我使用上面的当前代码,我只能查看其他人的作品(查看我自己的会引发错误)。但是,如果我将 current_user 添加到查询中(例如 @works= current_user.works),我就只能看到我自己的作品。查看其他人的作品会引发错误。我该如何解决这个问题,以便我可以查看自己的作品和他人创作的作品?

编辑2:

@work = Work.find(params[:id]) 如果我从控制器中删除 @user 并且作品 show.html 视图文件中的“@user”引用有效。但是,我需要控制器中的“@user”引用,因为我想显示创建作品的用户的名称。我该怎么做呢?

编辑 3:

固定的!再次感谢所有贡献答案的人!这是我为修复它所做的:

  1. 我从控制器中删除了@user 引用(“显示”操作)。正如 Fred 在下面提到的,不需要 @user 引用并且需要将其删除,因为我使用 :id 两次来引用两个单独的对象。

  2. 将“@work”变量编辑为@work = Work.find(params[:id]). 这将根据 id 查找正确的工作项,无论是哪个用户创建的。

  3. 当我需要在工作页面上显示用户数据时,只需<%= @work.user.name %>在 show.html.erb 视图页面上使用即可。不需要“@user = User.find(params[:id])”,因为我已经使用“belongs_to :user”在工作模型上定义了外键关系。

再次感谢所有的帮助!-j

4

4 回答 4

4

在你的表演动作中,一起摆脱 @user 并使用:

@work = Work.find(params[:id])

这将允许任何人查看任何作品。

您的其他操作也是如此。通过说:

@work = current_user.works.find(params[:id])

您正在所有 current_user 的作品中搜索与id == params[:id].

于 2013-03-29T16:51:28.840 回答
1

我怀疑您在查找用户时使用的是work_id而不是,因此它有时可以工作,但 ID 错误。user_id

您在工作控制器中。params[:id] 是工作。

所以这:

@user = User.find(params[:id])
@work = @user.works.find(params[:id])

应该是这样的:

@user = User.find(current_user)
@work = @user.works.find(params[:id])

如果您只想使用所有作品

@works = Work.all

因为您可以使用与 te id 相同的 params[:id] 来查找用户和工作。如果他们是同一个ID,那只是巧合。

于 2013-03-28T22:59:27.183 回答
1

我发现使用 current_user.works.find_by(id: params[:id]) 可以在找不到任何东西时返回 nil 。

我认为这会更好一些,因为您现在可以在查找中使用 current_user 会更安全一些,因为您不必依赖视图来显示它不应该显示的数据,因为它永远不会得到它。

于 2016-08-05T21:25:49.313 回答
0

看起来您正在删除一个用户,并留下悬空引用works

通过查询作品

@work = @user.works.find(params[:id])

您正在寻找属于@user 的所有@user 作品(冗余)

如果您改为使用@work = @user.works那些悬空的作品,则不会触及参考,因为 @user.works 将返回一个空响应。

不过,这确实不是最好的解决方案,您需要在删除用户时更好地处理依赖关系的破坏。

编辑:对不起,我在应该使用@user 的地方使用了 current_user,上面的评论有同样的错误。

于 2013-03-28T22:53:48.050 回答