アカウント名:
パスワード:
炎上していたプロジェクトだったわけではないのですが、途中参加したプロジェクトで「君の作ったメソッドはどれもこれも機能が単一すぎる」と言われました。どういう事だろう?と思って指摘した人に尋ねたところ、色々な機能がてんこ盛りになった単一のメソッド(決して各々のメソッドを呼び出しているわけではない)メソッドのソースを見せられながら「こういう関数を作っておけばこの関数さえ使えば全部に対応出来るでしょ」と教えられました。
あながち間違いではないのか? とも当時は思いましたが未だに納得できません。# 1メソッド・1機能というのが必ずしも正義ではないとは思いますが、1メソッドで全部やっちゃうっておかしいよね
ルールじゃないけど、昔引き継いだあるVBのプログラム、画面パーツが全部コントロール配列になっていて、当然イベントハンドラはひとつだけ、その中で
if( Index = 0 )then...数百行elseif( Index = 1 )then...数百行elseif( ...
相互に何の関係も無いコードがひとつのサブルーチンに全部収まってるのを見たときは目がくらみました。
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
犯人はmoriwaka -- Anonymous Coward
このメソッドには●●の機能しかないじゃないか! (スコア:0)
炎上していたプロジェクトだったわけではないのですが、途中参加したプロジェクトで
「君の作ったメソッドはどれもこれも機能が単一すぎる」と言われました。
どういう事だろう?と思って指摘した人に尋ねたところ、色々な機能がてんこ盛りになった
単一のメソッド(決して各々のメソッドを呼び出しているわけではない)メソッドのソースを見せられながら
「こういう関数を作っておけばこの関数さえ使えば全部に対応出来るでしょ」と教えられました。
あながち間違いではないのか? とも当時は思いましたが未だに納得できません。
# 1メソッド・1機能というのが必ずしも正義ではないとは思いますが、1メソッドで全部やっちゃうっておかしいよね
Re:このメソッドには●●の機能しかないじゃないか! (スコア:0)
ルールじゃないけど、昔引き継いだあるVBのプログラム、画面パーツが全部コントロール配列になっていて、当然イベントハンドラはひとつだけ、その中で
if( Index = 0 )then
...数百行
elseif( Index = 1 )then
...数百行
elseif( ...
相互に何の関係も無いコードがひとつのサブルーチンに全部収まってるのを見たときは目がくらみました。