【发布时间】:2015-09-28 03:53:47
【问题描述】:
在我们的 Rails 应用程序中,我们有一个 CalendarsController:
class CalendarsController < ApplicationController
def create
@calendar = current_user.calendars.create(calendar_params)
current_user.add_calendar_and_role(@calendar.id, 'Owner')
if @calendar.save
current_user.total_calendar_count += 1
current_user.owned_calendar_count += 1
current_user.save
flash[:success] = "Calendar created!"
redirect_to dashboard_path
else
render 'static_pages/home'
end
end
def show
@calendar = Calendar.find(params[:id])
@posts = @calendar.posts
@post = Post.new
end
def index
end
def edit
end
def destroy
Calendar.find(params[:id]).destroy
flash[:success] = "Calendar deleted"
redirect_to dashboard_path
end
private
def calendar_params
params.require(:calendar).permit(:name)
end
end
在create动作中,当一个新的@calendar被创建时,我们运行@calendar.save来检查新实例是否真的被创建,然后执行一些动作。
我们希望在 destroy 操作中实施类似的流程。
我们正在考虑更新destroy方法如下:
def destroy
@calendar = Calendar.find(params[:id])
@calendar.destroy
if @calendar.delete
flash[:success] = "Calendar deleted"
current_user.total_calendar_count -= 1
if @calendar.administrations.role == "Owner"
current_user.owned_calendar_count -= 1
end
end
redirect_to dashboard_path
end
这段代码的语法是否正确,尤其是if @calendar.delete 和if @calendar.administrations.role == "Owner"?
而且,最重要的是,这个destroy 操作的代码是否有意义?
【问题讨论】:
标签: ruby-on-rails ruby-on-rails-4 model-view-controller destroy