質問編集履歴

4

情報追加

2018/10/27 08:34

投稿

tomohiroo
tomohiroo

スコア19

test CHANGED
File without changes
test CHANGED
@@ -28,13 +28,15 @@
28
28
 
29
29
  ここで質問が2つあるのですが、
30
30
 
31
- 1. current_userのクリップ(保存したレストラン)一覧を取得するようなモデルを跨ぐようなメソッドをモデル層に記述したいときはuser.rbやclip.rb、restaurant.rbなどのクラスメソッド、またはインスタンスメソッド作るべきなのでしょうか?
31
+ 1. current_userのクリップ(保存したレストラン)一覧を取得するようなモデルを跨ぐようなメソッドをモデル層に記述したいときはuser.rbやclip.rb、restaurant.rbなどどのモデルに、また、クラスメソッド、もしくはインスタンスメソッドのどちらで作るべきなのでしょうか?
32
32
 
33
33
  現状は、
34
34
 
35
- Clip.search(params[:query])のようにクリップクラスからメソッドを生やすか
35
+ Clip.search(params[:query], current_user.id)のようにクリップクラスからメソッドを生やすか
36
36
 
37
- current_user.search_clips(params[:query])のようにユーザーのインスタンスメソッドを作るかで迷い、後者を採用しています。
37
+ current_user.search_clips(params[:query])のようにユーザーのインスタンスメソッドを作るか
38
+
39
+ の2つで迷い、後者を採用しています。
38
40
 
39
41
 
40
42
 

3

情報追加

2018/10/27 08:33

投稿

tomohiroo
tomohiroo

スコア19

test CHANGED
File without changes
test CHANGED
@@ -28,7 +28,13 @@
28
28
 
29
29
  ここで質問が2つあるのですが、
30
30
 
31
- 1. current_userのクリップ(保存したレストラン)一覧を取得するようなモデルを跨ぐようなメソッドをモデル層に記述したいときはuser.rbやclip.rb、restaurant.rbなどのどこに作るべきなのでしょうか?
31
+ 1. current_userのクリップ(保存したレストラン)一覧を取得するようなモデルを跨ぐようなメソッドをモデル層に記述したいときはuser.rbやclip.rb、restaurant.rbなどのどこのクラスメソッド、またはインスタンスメソッドに作るべきなのでしょうか?
32
+
33
+ 現状は、
34
+
35
+ Clip.search(params[:query])のようにクリップクラスからメソッドを生やすか
36
+
37
+ current_user.search_clips(params[:query])のようにユーザーのインスタンスメソッドを作るかで迷い、後者を採用しています。
32
38
 
33
39
 
34
40
 

2

誤字

2018/10/27 08:32

投稿

tomohiroo
tomohiroo

スコア19

test CHANGED
File without changes
test CHANGED
@@ -32,7 +32,7 @@
32
32
 
33
33
 
34
34
 
35
- 2. 今は、あるモデルについて検索するメソッドをモデルに書き、それをcontrollerで使っていますが、そもそもClip.searchのようなメソッドを作り、現在controllerで行っているような処理を全てcontrollerで行ってしまった方がいいのでしょうか?
35
+ 2. 今は、あるモデルについて検索するメソッドをモデルに書き、それをcontrollerで使っていますが、そもそもClip.get_with_queryのようなメソッドを作り、現在controllerで行っているような処理を全てmodelで行ってしまった方がいいのでしょうか?
36
36
 
37
37
  controllerとmodelをどう使い分けるべきかがよくつかめていません。
38
38
 

1

情報追加

2018/10/27 08:29

投稿

tomohiroo
tomohiroo

スコア19

test CHANGED
File without changes
test CHANGED
@@ -28,7 +28,7 @@
28
28
 
29
29
  ここで質問が2つあるのですが、
30
30
 
31
- 1. current_userのクリップ(保存したレストラン)一覧を取得するようなモデルを跨ぐようなメソッドをモデル層に記述したいときはuser.rbclip.rbのどちらに作るべきなのでしょうか?
31
+ 1. current_userのクリップ(保存したレストラン)一覧を取得するようなモデルを跨ぐようなメソッドをモデル層に記述したいときはuser.rbclip.rb、restaurant.rbなどのどに作るべきなのでしょうか?
32
32
 
33
33
 
34
34