アカウント名:
パスワード:
炎上していたプロジェクトだったわけではないのですが、途中参加したプロジェクトで「君の作ったメソッドはどれもこれも機能が単一すぎる」と言われました。どういう事だろう?と思って指摘した人に尋ねたところ、色々な機能がてんこ盛りになった単一のメソッド(決して各々のメソッドを呼び出しているわけではない)メソッドのソースを見せられながら「こういう関数を作っておけばこの関数さえ使えば全部に対応出来るでしょ」と教えられました。
あながち間違いではないのか? とも当時は思いましたが未だに納得できません。# 1メソッド・1機能というのが必ずしも正義ではないとは思いますが、1メソッドで全部やっちゃうっておかしいよね
> 1メソッドで全部やっちゃう
メソッドにする必要ってあるの???
行数を稼げます。
#クレバーなコードよりも、可読性が悪くて大規模なコードのほうがお金になるんです。
ケースバイケースだなぁ。イメージを描画する場合、ファイル名を渡すだけで自分自身に描画までしてしまうメソッドはあってもいい。当然、そのメソッドを構成する各要素も、それぞれ個別のメソッドになっているべきだけれど。
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
ハッカーとクラッカーの違い。大してないと思います -- あるアレゲ
このメソッドには●●の機能しかないじゃないか! (スコア:0)
炎上していたプロジェクトだったわけではないのですが、途中参加したプロジェクトで
「君の作ったメソッドはどれもこれも機能が単一すぎる」と言われました。
どういう事だろう?と思って指摘した人に尋ねたところ、色々な機能がてんこ盛りになった
単一のメソッド(決して各々のメソッドを呼び出しているわけではない)メソッドのソースを見せられながら
「こういう関数を作っておけばこの関数さえ使えば全部に対応出来るでしょ」と教えられました。
あながち間違いではないのか? とも当時は思いましたが未だに納得できません。
# 1メソッド・1機能というのが必ずしも正義ではないとは思いますが、1メソッドで全部やっちゃうっておかしいよね
Re:このメソッドには●●の機能しかないじゃないか! (スコア:1)
> 1メソッドで全部やっちゃう
メソッドにする必要ってあるの???
Re: (スコア:0)
行数を稼げます。
#クレバーなコードよりも、可読性が悪くて大規模なコードのほうがお金になるんです。
Re: (スコア:0)
ケースバイケースだなぁ。
イメージを描画する場合、ファイル名を渡すだけで自分自身に描画までしてしまうメソッドはあってもいい。
当然、そのメソッドを構成する各要素も、それぞれ個別のメソッドになっているべきだけれど。