teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

1

ソースと表示される内容がずれているので修正してみたが…

2017/04/13 09:59

投稿

suzukis
suzukis

スコア1449

answer CHANGED
@@ -10,6 +10,6 @@
10
10
 
11
11
  AccountsControllerの中での話であれば`Account.find(params[:id])`は複数のアクションで行われるでしょうから、メソッドにまとめた方がよいです(scaffoldのコードを参考にしてください)。代替主キーを使うような時にも変更が1カ所ですみます。
12
12
 
13
- `Task`を直接検索する回答がついていますが、同様に適不適がありますので慎重に判断してください。キーが変わるようなこともありますが、そもそも`Account`モデルや`AccountsController`の中に`Task.where`...のようなコードが現れるのはよろしくないです。
13
+ `Task`を直接検索する回答がついていますが、同様に適不適がありますので慎重に判断してください。そもそも`Account`モデルや`AccountsController`の中に`Task.where`...のようなコードが現れるのはよろしくないです。
14
14
 
15
15
  また、`update_all`はバリデーションが呼ばれないとかタイムスタンプの絡むが更新されないとか癖があります。普通は避けた方がよいとされています。