この会社辞めようと思ったソースコード#18at PROGこの会社辞めようと思ったソースコード#18 - 暇つぶし2ch■コピペモード□スレを通常表示□オプションモード□このスレッドのURL■項目テキスト899:仕様書無しさん 07/10/09 18:35:36 >>898 それって非常にありがちだけど、個人的にはそういう書き方は自己満足にしかなってないと思うなあ。 一度反省的に自分のそういうコードを見直してみて欲しいよ。 一箇所からしか呼ばれないコードを関数に括りだすことが本当に可読性に資するのか。 むしろ、コードを読む際にそこに飛ぶ手間を増やしているだけじゃないのか。 「caseの中につらつら書いた」コードが本当に読みにくいのか。 むしろ教条主義的にそう思い込んでるだけじゃないのか。 900:仕様書無しさん 07/10/09 19:00:35 抽象ってどういう意味か考えてみて欲しいね 「読まければ意味がわからない関数」は括り出す意味は無い 可読性をあげるためには、「今まさに関心のある」レイヤ以外に存在する 関数について、「読む必要性自体」を消し去る必要があるのだが 901:仕様書無しさん 07/10/09 19:10:34 「コードを読む際にそこに飛ぶ」のにそんなに手間がかかるのかw 902:仕様書無しさん 07/10/09 19:13:13 >>894 俺ならcaseマッチングのテーブルを挟んでループさせるな。 マッチングオブジェクトは当然処理関数への参照を持たせる。 マッチングしたら対象オブジェクトの処理を呼び出す。 C/C++なら普通に書ける関数テーブル。 処理部分はスマートになるし、定義部分ではコメント多用して 逆に分かり易く表現出来る。 VBはしらねーよw 次ページ最新レス表示レスジャンプ類似スレ一覧スレッドの検索話題のニュースおまかせリストオプションしおりを挟むスレッドに書込スレッドの一覧暇つぶし2ch