回答編集履歴
2
修正
answer
CHANGED
|
@@ -6,7 +6,7 @@
|
|
|
6
6
|
設定でpkeyのリストを持っておいて必須チェックかけるようにすれば良いと思います。
|
|
7
7
|
|
|
8
8
|
今後構成が変わってキーが増減する場合のことを考えて命名すべきと思います。
|
|
9
|
-
メソッド名を変えて引数も増やすのか
|
|
9
|
+
メソッド名を変えて(呼びもともメソッドにも)引数も増やすのか
|
|
10
10
|
呼びもとの引数を増やしてコメント追記するだけにするのか
|
|
11
11
|
影響範囲次第だと思います。
|
|
12
12
|
|
1
懸念点
answer
CHANGED
|
@@ -3,4 +3,13 @@
|
|
|
3
3
|
PhpDocの様式でコメントでparamとreturnの補足をつければ良いかと。
|
|
4
4
|
モデルであればどのデータのgetIdかはクラス名辿ればかりますし。
|
|
5
5
|
固定の引数がたくさんあるのは拡張性に欠けるので配列でドンと渡して
|
|
6
|
-
設定でpkeyのリストを持っておいて必須チェックかけるようにすれば良いと思います。
|
|
6
|
+
設定でpkeyのリストを持っておいて必須チェックかけるようにすれば良いと思います。
|
|
7
|
+
|
|
8
|
+
今後構成が変わってキーが増減する場合のことを考えて命名すべきと思います。
|
|
9
|
+
メソッド名を変えて引数も増やすのか
|
|
10
|
+
呼びもとの引数を増やしてコメント追記するだけにするのか
|
|
11
|
+
影響範囲次第だと思います。
|
|
12
|
+
|
|
13
|
+
IDEを使っていてきちんとPhpDoc書いてればコメントは開発時に確認できるので
|
|
14
|
+
長すぎるメソッド名は開発を進めてメンテナンスまで考えた場合、
|
|
15
|
+
リスク高いと思います
|