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