我的控制器中有一个动作,我遇到了麻烦.这是我的第一个rails应用程序,所以我不确定有关rails的最佳实践.
我有一个名为Group的模型和一些进入它控制器的动作.
我编写了一个测试,该测试应该导致控制器因为组ID无效而在JSON中呈现错误.它不是渲染和退出,而是看起来控制器正在渲染并继续执行.
测试
- test 'should not remove group because of invalid group id' do
- post(:remove,{'group_id' => '3333'})
- response = JSON.parse(@response.body)
- assert_response :success
- assert_equal 'Success',response['message']
- end
控制器动作
- # Post remove
- # group_id
- def remove
- if((@group = Group.find_by_id(params[:group_id])) == nil)
- render :json => { :message => "group_id not found" }
- end
- @group.destroy
- if(!Group.exists?(@group))
- render :json => { :message => "Success" }
- else
- render :json => { :errors => @group.errors.full_messages }
- end
- end
在控制器中,第一个if语句执行：render：json => {：message => “找不到group_id”}
但@ group.destroy仍在执行中.这对我来说似乎是违反直觉的,我认为渲染方法应该退出控制器.
解决方法
就像@ user1022209所说,你可以添加返回退出动作：
- render(:json => { :message => "group_id not found" }) and return
关于你的代码,我想我会这样写：
- def remove
- if(!Group.exists?(params[:group_id])
- render :json => { :message => "group_id not found" }
- else
- @group = Group.find(params[:group_id]
- @group.destroy
- if @group.destroyed?
- render :json => { :message => "Success" }
- else
- render :json => { :errors => @group.errors.full_messages }
- end
- end
- end