アカウント名:
パスワード:
炎上していたプロジェクトだったわけではないのですが、途中参加したプロジェクトで「君の作ったメソッドはどれもこれも機能が単一すぎる」と言われました。どういう事だろう?と思って指摘した人に尋ねたところ、色々な機能がてんこ盛りになった単一のメソッド(決して各々のメソッドを呼び出しているわけではない)メソッドのソースを見せられながら「こういう関数を作っておけばこの関数さえ使えば全部に対応出来るでしょ」と教えられました。
あながち間違いではないのか? とも当時は思いましたが未だに納得できません。# 1メソッド・1機能というのが必ずしも正義ではないとは思いますが、1メソッドで全部やっちゃうっておかしいよね
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
「毎々お世話になっております。仕様書を頂きたく。」「拝承」 -- ある会社の日常
このメソッドには●●の機能しかないじゃないか! (スコア:0)
炎上していたプロジェクトだったわけではないのですが、途中参加したプロジェクトで
「君の作ったメソッドはどれもこれも機能が単一すぎる」と言われました。
どういう事だろう?と思って指摘した人に尋ねたところ、色々な機能がてんこ盛りになった
単一のメソッド(決して各々のメソッドを呼び出しているわけではない)メソッドのソースを見せられながら
「こういう関数を作っておけばこの関数さえ使えば全部に対応出来るでしょ」と教えられました。
あながち間違いではないのか? とも当時は思いましたが未だに納得できません。
# 1メソッド・1機能というのが必ずしも正義ではないとは思いますが、1メソッドで全部やっちゃうっておかしいよね
Re:このメソッドには●●の機能しかないじゃないか! (スコア:1)
私もメソッドはシンプルが良いと思います。
てんこ盛りなメソッドの良くないところは以下の3点かと。
1. 複雑ゆえに保守性が低い。
2. 再利用性が低い。
3. 遅い。
2については, パラメータが多く(推測), 気軽に呼び出すことが大変だから。
従って, 他のメソッドからは使いづらい。
また, 使っても3より遅いから全体として速度も落ちる。
あるいは, てんこ盛りメソッドの処理の途中で得られる結果が
欲しい場合もあるかも知れないのに, それを得る方法はない。
シンプルなメソッドを組み合わせて使うほうが, 全体として,
最終的に, より複雑な処理を効率よく実装できるものと思っています。
もちろん, 程度っつーものはあって, fopenなんかが
読み取り用, 書き込み用, 追加用で別々の名前に
なっていたら, ちょっと辟易する... かも。