【发布时间】:2013-09-26 22:30:49
【问题描述】:
我有一个包含 4 个模型的 Rails 应用程序。我只在一个控制器操作中访问这 4 个模型。我目前有 4 个不同的控制器来处理这些模型。我想知道将这 4 个动作塞到一个控制器中是否是一种不好的做法。
当前设置:
class GmDataController < ApplicationController
def dashboard
@data = GmData.all
end
end
class GmRetentionController < ApplicationController
def dashboard
@data = GmRetention.all
end
end
class GsDataController < ApplicationController
def dashboard
@data = GsData.all
end
end
class GsRetentionController < ApplicationController
def dashboard
@data = GsRetention.all
end
end
建议的设置:
class DashboardController < ApplicationController
def gm_data_dashboard
@data = GmData.all
end
def gm_retention_dashboard
@data = GmRetention.all
end
def gs_data_dashboard
@data = GsData.all
end
def gs_retention_dashboard
@data = GsRetention.all
end
end
【问题讨论】:
-
从这里很难说,我看不到你出轨会得到什么
-
将要关闭的问题标记为“主要基于意见”,因为这实际上取决于在这些操作中是否不仅包含此信息。
标签: ruby-on-rails ruby ruby-on-rails-3 rails-activerecord