アカウント名:
パスワード:
炎上していたプロジェクトだったわけではないのですが、途中参加したプロジェクトで「君の作ったメソッドはどれもこれも機能が単一すぎる」と言われました。どういう事だろう?と思って指摘した人に尋ねたところ、色々な機能がてんこ盛りになった単一のメソッド(決して各々のメソッドを呼び出しているわけではない)メソッドのソースを見せられながら「こういう関数を作っておけばこの関数さえ使えば全部に対応出来るでしょ」と教えられました。
あながち間違いではないのか? とも当時は思いましたが未だに納得できません。# 1メソッド・1機能というのが必ずしも正義ではないとは思いますが、1メソッドで全部やっちゃうっておかしいよね
1メソッドで全部やるってことは
methodAll( int flag , ....) { switch( flag ) { case 1: method1(....); brek; case 2: method1(....); brek; case 3: method1(....); brek; case 4: method1(....); brek; .... case 10: method1(....); default: /* エラー処理 */ }}
みたいになってるだけということを理解してない故の発言かも。
呼び出しもとがmethod1の機能を呼び出すのが目的なら、methodALL()を呼ぶよりmethod1()を呼び出した方が安全確実。
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
弘法筆を選ばず、アレゲはキーボードを選ぶ -- アレゲ研究家
このメソッドには●●の機能しかないじゃないか! (スコア:0)
炎上していたプロジェクトだったわけではないのですが、途中参加したプロジェクトで
「君の作ったメソッドはどれもこれも機能が単一すぎる」と言われました。
どういう事だろう?と思って指摘した人に尋ねたところ、色々な機能がてんこ盛りになった
単一のメソッド(決して各々のメソッドを呼び出しているわけではない)メソッドのソースを見せられながら
「こういう関数を作っておけばこの関数さえ使えば全部に対応出来るでしょ」と教えられました。
あながち間違いではないのか? とも当時は思いましたが未だに納得できません。
# 1メソッド・1機能というのが必ずしも正義ではないとは思いますが、1メソッドで全部やっちゃうっておかしいよね
Re:このメソッドには●●の機能しかないじゃないか! (スコア:1)
1メソッドで全部やるってことは
methodAll( int flag , ....) {
switch( flag ) {
case 1: method1(....); brek;
case 2: method1(....); brek;
case 3: method1(....); brek;
case 4: method1(....); brek;
....
case 10: method1(....);
default: /* エラー処理 */
}
}
みたいになってるだけということを理解してない故の発言かも。
呼び出しもとがmethod1の機能を呼び出すのが目的なら、methodALL()を
呼ぶよりmethod1()を呼び出した方が安全確実。