Rails 如何：如果项目有任务，则不应删除它：如何解决此问题？

Question

您好，我有一个项目，每个项目都有任务。任务属于项目。在删除项目之前，我想检查是否有相关任务。如果有任务我不想删除该项目。如果没有关联的任务，则应删除该项目。你能帮我看一下代码吗？我缺少什么？

class Project < ActiveRecord::Base  
  before_destroy :check_tasks    

  def check_tasks 
    if Project.find(params[:id]).tasks  
      flash[:notice] = 'This project has tasks.'
      redirect_to :action => 'list_projects'    
    end 
  end
end

Answer 1

从 before_destroy 方法返回 false 以防止实例被销毁。

该方法还应该返回一个有意义的错误以进行故障排除。

class Project < ActiveRecord::Base
  before_destroy :check_tasks

  def check_tasks
    if self.tasks.any?
      errors.add_to_base "Project has tasks and cannot be destroyed."
      return false
    end
  end
end

注意：flash[:notice] 和 params[:attr_name] 只能在控制器中使用。

Answer 2

你这里有几个问题。

1. 您没有（或不应该）访问 params 变量（它仅在控制器和视图中可用，除非您将其传递给模型，这可能不是您想要的） 。
2. 您的 if 检查 project.tasks 这是一个数组 - 即使是空数组也会评估为 true，因此您的其他代码分支永远不会发生项目是否有任务无关紧要。
3. 您可能应该从 ProjectsController#destroy 操作中设置视图的错误消息，而不是在模型中。

解决方案：

1. 将 Project.find(params[:id]) 更改为 self - 您想要检查项目每个实例的任务。
2. 将 if 语句中的检查从 if self.tasks 更改为 if self.tasks.any? ，它会返回您想要的值 (false 如果数组为空，true 否则）。
3. 将 flash[:notice] 从模型移至控制器，以及它们所属的redirect_to 调用。这意味着您的 check_tasks 方法可以更改为以下

代码：

def check_tasks
  return !self.tasks.any?
end

Answer 3

支票应该是self吗？ （不确定您从哪里获取 params[:id] ）。

不过还没有检查过 - 但由于我的用户模型需要类似的东西，我会看看它是如何工作的并回复你。

class Project < ActiveRecord::Base  
 before_destroy :check_tasks

 private

 def check_tasks
   #edited 
   if tasks.empty?  
     false
   end 
 end