使用Playlist.new(params[:playlist])是不是不好？

Question

特别是在我的new/create操作中。我有@playlist = Playlist.new(params[:playlist])。问题是，我在attr_accessible中也有一些敏感数据，我不希望他们修改这些数据(播放列表上的监听次数，他们不应该能够更新)。

我试过Playlist.new(:title => params[:title], :description => params[:description], etc)，但不起作用。我假设是因为我需要执行params[:playlist][:title]，但这看起来相当混乱。我是不是做错了？

Answer 1

在模型中，您可以编写一个名为new_safe的函数，该函数创建具有所需参数的新对象，然后返回该对象

如下所示：

def new_safe(params)
   playlist = Playlist.new
   playlist.title = params[:title]
   playlist.description = params[:description]
   playlist.save
   playlist
end

我只是在想，类似地，你可以这样写，这样会更干净一些

控制器：

@playlist = Playlist.new
@playlist.input_params(params)

型号：

def input_params(params)
   playlist.title = params[:title]
   playlist.description = params[:description]
   playlist.save
end