この会社辞めようと思ったソースコード＃18

[表示 : 全て最新50 1-99 101- 201- 301- 401- 501- 601- 701- 801- 901- 2chのread.cgiへ]
Update time : 02/23 23:29 / Filesize : 185 KB / Number-of Response : 948
[このスレッドの書き込みを削除する]
[＋板最近立ったスレ＆熱いスレ一覧 : ＋板最近立ったスレ／記者別一覧] [類似スレッド一覧]


↑キャッシュ検索、類似スレ動作を修正しました、ご迷惑をお掛けしました

1 名前：仕様書無しさん [2007/08/14(火) 23:48:45 ]: この会社辞めようと思ったソースコード。
プログラマとして幻滅するソースコード。
プログラマを悩ませるソースコード。
をつらつらと綴っていって頂戴。

ちなみにここは質問スレじゃないので
技術的な質問がしたいならム板 pc11.2ch.net/tech/ に逝って。

前スレ
この会社辞めようと思ったソースコード＃17
pc11.2ch.net/test/read.cgi/prog/1183700531/
713 名前：仕様書無しさん mailto:sage [2007/10/05(金) 14:36:27 ]: スレ違いっぽい話題を引きずることになるかも知れないけど
C++やVBだと不要メソッドのリストアップにどういう方法があるの？
714 名前：仕様書無しさん mailto:sage [2007/10/05(金) 14:59:20 ]: うちはVB厨だが、これを使ってる
ttp://www011.upp.so-net.ne.jp/flatsoft/soft/fs_about_vbcheck.html
715 名前：仕様書無しさん mailto:sage [2007/10/05(金) 16:09:54 ]: ＞Ｃ＃だからこそ何を削除したらいいか検索するのが不可能って流れになってるじゃん。
どこにそんな流れがあるんだ
716 名前：仕様書無しさん mailto:sage [2007/10/05(金) 16:13:01 ]: >>713
ググレカス
717 名前：仕様書無しさん mailto:sage [2007/10/05(金) 16:14:38 ]: つーか、リフレクションを駆使するようなチームだったら、コードをクリーンに保つくらいすると思うんだが。
718 名前：仕様書無しさん mailto:sage [2007/10/05(金) 16:51:09 ]: >>715
C#嫌いが嵩じて幻覚でも見てるんじゃ
719 名前：仕様書無しさん mailto:sage [2007/10/05(金) 17:00:20 ]: >>713
VBでこれ使ってる。
ttp://www.aivosto.com/vb.html

Cならsplint、C++は昔なんか使ってたような気がするけど忘れた。
720 名前：仕様書無しさん mailto:sage [2007/10/05(金) 22:33:55 ]: 不要社員のリストアップに
721 名前：仕様書無しさん mailto:sage [2007/10/05(金) 23:41:32 ]: >>708
この前、fxcopとかってツールをためしに使ったら、使われてないメソッドとかは指摘されてたような。うろおぼえ。
722 名前：仕様書無しさん mailto:sage [2007/10/05(金) 23:41:46 ]: >>711
C++はおろか、Javaや最近のVBですらリフレクションが実装されていることを知らないこと、
それ以前に問題の本質を見つけられない濁った目、そして文中に漂う全角アルファベット、

おまえ、もしかしてコボラだなww
723 名前：仕様書無しさん mailto:sage [2007/10/05(金) 23:47:35 ]: こぼらがいっぴき
こぼらがにひき…

この辺で…俺のはじめてのプロジェクトは火を噴いてあたりを転げまわりだした。
724 名前：仕様書無しさん mailto:sage [2007/10/06(土) 00:02:02 ]: ぇ、C++ってリフレクションあったっの? C++/CLIじゃなくて?
725 名前：仕様書無しさん mailto:sage [2007/10/06(土) 00:02:49 ]: >>707
関係ないが、そこで、上から目線で教えてやってるジッタってやつは、よそのスレで、ツッコミ入れられまくってたヤツだな。
このエントリで。
blogs.wankuma.com/jitta/archive/2007/07/26/87218.aspx
726 名前：仕様書無しさん mailto:sage [2007/10/06(土) 00:22:24 ]: 本当に関係ねえ
727 名前：仕様書無しさん mailto:sage [2007/10/06(土) 08:20:06 ]: >>725
先輩だか上司だかに、添削と言って、こんなソースにされたら、やめたくなるよ。
728 名前：仕様書無しさん mailto:sage [2007/10/06(土) 14:14:55 ]: dim strTemp(10) as string
const intTemp as integer = 10

for i as integer = 1 to intTemp
strTemp(i) = ""
'処理
next
729 名前：仕様書無しさん mailto:sage [2007/10/06(土) 14:17:15 ]: どの辺が気に入らないんだろう
730 名前：仕様書無しさん mailto:sage [2007/10/06(土) 14:30:39 ]: VBなところ
731 名前：仕様書無しさん mailto:sage [2007/10/06(土) 14:35:35 ]: for i as integer = 0 to intTemp
732 名前：仕様書無しさん mailto:sage [2007/10/06(土) 14:53:15 ]: for i as integer = LBound(strTemp) to UBound(strTemp)
733 名前：仕様書無しさん mailto:sage [2007/10/06(土) 15:03:54 ]: ＶＢの文字列って初期化されてることが保障されてなかったっけ？
734 名前：仕様書無しさん mailto:sage [2007/10/06(土) 15:06:03 ]: for i as integer = 0 to intTemp-1
735 名前：仕様書無しさん mailto:sage [2007/10/06(土) 15:12:03 ]: 綺麗なコーディングしてるだろ

バグってるんだぜ、これで…
736 名前：仕様書無しさん mailto:sage [2007/10/06(土) 15:30:29 ]: >>733
保障されてる
737 名前：仕様書無しさん mailto:sage [2007/10/06(土) 19:51:32 ]: VBって確か、
配列宣言時に指定する数が、
要素数じゃなくて添字の最大値なんだよな
738 名前：仕様書無しさん mailto:sage [2007/10/06(土) 20:15:39 ]: dim a(3) なら a(0) ～ a(3) だっけ？
昔のベーシックで a(0) を無しにしてメモリを節約するとかいう
セコいオプションがあったような記憶がある。
739 名前：仕様書無しさん mailto:sage [2007/10/06(土) 20:21:04 ]: VBでは、option base で配列の添字を 1～にするか0～にするか変更できる。
740 名前：仕様書無しさん mailto:sage [2007/10/06(土) 20:59:49 ]: もうすぐ64ビットが当たり前になる時代にそんなオプション意味あるんか？
741 名前：仕様書無しさん mailto:sage [2007/10/06(土) 21:32:07 ]: メモリの1バイトは血の一滴だ！
742 名前：仕様書無しさん mailto:sage [2007/10/06(土) 21:39:56 ]: おしい
血より精液のほうが説得力あったのに
743 名前：仕様書無しさん mailto:sage [2007/10/06(土) 21:40:28 ]: 初期化が保証されてても初期化、ということにはあまり文句は言わない
でも過保護言語のVBだし事情が違うのかな
744 名前：仕様書無しさん mailto:sage [2007/10/06(土) 21:43:52 ]: 初期化で思い出したけど
MFCとかでポインタ先が破棄済みかどうかに( pTest )ってやってるの困る
解放後の0割り当てなんてコード全体で徹底するのか？
いずれにせよMFC内部で delete this とかやるのあったらダメ
745 名前：仕様書無しさん mailto:sage [2007/10/06(土) 22:06:11 ]: メモリ管理もろくに出来ない無能な人間のための言語というアピールはひしひしと感じる
便利だから使うんじゃない，それしか使えないんだ，てな
746 名前：仕様書無しさん mailto:sage [2007/10/06(土) 22:26:48 ]: 管理以前にそもそもメモリの概念が分かっていません。
747 名前：仕様書無しさん mailto:sage [2007/10/06(土) 22:34:13 ]: メモリの節約、スレッドの節約、etc...
748 名前：仕様書無しさん mailto:sage [2007/10/07(日) 00:07:44 ]: 給料の節約、残業代の節約、etc...
749 名前：仕様書無しさん mailto:sage [2007/10/07(日) 00:10:42 ]: >>744
> 内部で delete this とかやるのあったらダメ

そうか?
750 名前：仕様書無しさん [2007/10/07(日) 03:47:54 ]: >>725
相当に疑わしい人間だな。
751 名前：仕様書無しさん [2007/10/07(日) 07:16:38 ]: > 解放後の0割り当てなんてコード全体で徹底するのか？
逆に delete して NULL 入れてないやつはぶっ殺したくなる
752 名前：仕様書無しさん mailto:sage [2007/10/07(日) 08:29:44 ]: 自所持ﾎﾟｲﾝﾀをNULLにしたところで
その参照先を他のやつもﾎﾟｲﾝﾀで参照してたら・・・・そっちが自動でNULLになってくれるわけじゃないから
NULL当てはめて安心するなんて無意味っちゃ無意味
753 名前：仕様書無しさん mailto:sage [2007/10/07(日) 09:22:50 ]: 意味不明。
自ポインタで必要なくなった参照先は、他のやつでも必要なくなるって決め付けてるのか？
そんなもん、プログラムのロジックによりけりだと思うが。他では保持する必要があるかもしれないし。
NULL当てはめて安心したいのは、処理が他の参照先に移ってるのに自ポインタが保持してるせいで
誤ってアクセスするというようなことを起こさないための防衛策もあるだろ。

「この会社辞めようと思った」の９０％くらいは真実だと思うが、１０％くらいは「辞めようと思った」奴の
レベルが低すぎるんじゃないかと思うことがある。
754 名前：仕様書無しさん mailto:sage [2007/10/07(日) 09:53:34 ]: 言ってることが全然分からんな。
よほど内部・詳細設計が腐っていると見える。
755 名前：仕様書無しさん mailto:sage [2007/10/07(日) 10:57:20 ]: >>751
そんなことするのヘタクソだけだろ。
756 名前：仕様書無しさん mailto:sage [2007/10/07(日) 10:59:49 ]: Javaで、1000行以上のメソッド（この時点でどうかとおもうが）を抜ける時に
一々オブジェクトにnullを入れて片付けているのを見たとき。
そんなどうでもいい気を使う前に、そのくそ長いメソッドを見直せと。
757 名前：仕様書無しさん mailto:sage [2007/10/07(日) 11:07:22 ]: Javaは変数スコープ明確にしてGCに任せるだけだから楽だな
758 名前：仕様書無しさん mailto:sage [2007/10/07(日) 12:32:51 ]: gcなんて訳の分からんもんによくお任せできるな
お前は通りすがりのオヤジに家の留守番を頼めるのか？
759 名前：仕様書無しさん mailto:sage [2007/10/07(日) 12:40:28 ]: いや、javaは別にgc依存でいいんじゃないか？
そういう言語なんだし。
760 名前：仕様書無しさん mailto:sage [2007/10/07(日) 12:50:25 ]: >>753
deleteした場合の話だろ？
761 名前：仕様書無しさん mailto:sage [2007/10/07(日) 13:00:12 ]: 同じアドレスを共有しているポインタがあるのに delete するって
それだけで重大な設計ミスだな

所有権移行なら、delete せずに権限渡して
自分のポインタは NULL 設定とかする(auto_ptr がこうなってるな)し、
複製ならオブジェクトのコピーするだろ、普通
複数箇所で共有するなら、いいからスマートポインタ使っとけって話だし
762 名前：仕様書無しさん mailto:sage [2007/10/07(日) 13:06:13 ]: >>757
スマートポインタも使わないのか？
参照カウンタだって立派なGCだが。
763 名前：仕様書無しさん mailto:sage [2007/10/07(日) 13:13:28 ]: >>758
それを言い始めたらコンパイラも実行環境も何もかも信用できなくなるよ
764 名前：仕様書無しさん [2007/10/07(日) 13:42:04 ]: ぬるぽ
765 名前：仕様書無しさん mailto:sage [2007/10/07(日) 13:48:08 ]: お前らはデンマークの2流都市出身の禿や
カナダの糞田舎で育ったキモいオッサンの作った言語なんて信用できるのか？
766 名前：仕様書無しさん mailto:sage [2007/10/07(日) 14:02:00 ]: malloc/freeのペアを使っていた時代は、freeの後にNULLは設定していたなー
論理的な異常処理とかでやむを得ず処理を終了する場合は、
ポインタ参照用変数がNULLじゃなけりゃ取りあえずfreeしまくってexitしていた。

まぁ、三つ子の魂百までじゃないけど、たまにそうコーディングして苦笑する事が最近はある。
767 名前：仕様書無しさん mailto:sage [2007/10/07(日) 14:07:25 ]: 最初に疑うべきはいつの時代でも自分のコーディングだけどな！
768 名前：仕様書無しさん mailto:sage [2007/10/07(日) 14:34:45 ]: ポインタをNULLクリアしなきゃならないコードの書き方を疑え。
769 名前：仕様書無しさん mailto:sage [2007/10/07(日) 15:01:04 ]: >>763
どんなコンパイラも実行環境も最初は通りすがりのおっさん程度の信頼性しかないものだ。
それが大勢に使われて、もまれて、信頼できる警察官になるんだよ。
770 名前：仕様書無しさん mailto:sage [2007/10/07(日) 16:00:39 ]: いまどき C++ で自分で delete 書いてる時点で何かおかしいと疑ったほうがいい。
std::auto_ptr, boost::scoped_ptr, boost::shared_ptr で全部済ませとけ。
771 名前：仕様書無しさん mailto:sage [2007/10/07(日) 16:12:58 ]: boost の利用は標準に取り込まれてから考える
772 名前：仕様書無しさん mailto:sage [2007/10/07(日) 16:52:43 ]: std::auto_ptr を delete 書くのを省くためだけに使うんだったら帯に短し襷に長しだな
773 名前：仕様書無しさん mailto:sage [2007/10/07(日) 17:17:30 ]: >>772
へ？その目的で使ったら十分だと思うよ。
本人の意思とは関係なく例外安全性が付いてきたりするけど、別に余計なもんじゃないし。
774 名前：仕様書無しさん mailto:sage [2007/10/07(日) 17:32:33 ]: まぁでもエラー処理とか例外安全とか一切考慮してないような自作スマートポインタクラスを使うのもなぁ～
775 名前：仕様書無しさん mailto:sage [2007/10/07(日) 17:33:38 ]: 普段ウォシュレットを使い慣れている奴はたまに駅の便所で拭かずにパンツを上げるらしい。
もしくは美味く拭けずに手にウンコ付けまくりんぐとか。

一番多いのが、ちゃんと拭いて流したつもりでも毛に残ったウンコでパンツが黄色いとかだな。
776 名前：仕様書無しさん mailto:sage [2007/10/07(日) 17:57:15 ]: エクセルが信用できないので計算機で出した値を手入力してます
ＧＣが信用できないので（ry
777 名前：仕様書無しさん mailto:sage [2007/10/07(日) 18:12:18 ]: GCはいつインスタンスが破棄されるか分からんのでデストラクタ使い辛いのがなぁ・・・
778 名前：仕様書無しさん [2007/10/07(日) 18:33:56 ]: ところでここはいつから"この会社辞めようと思った自分の一言"スレになったのですか?
779 名前：仕様書無しさん [2007/10/07(日) 18:41:25 ]: >>778
知るか。どうでもいい事書くな！ヴぉけ
780 名前：仕様書無しさん mailto:sage [2007/10/07(日) 18:41:26 ]: javaやC#も、スコープ抜けたら、デストラクタ(相当)がコールされる仕組みを作ってくれたらいいのに。
メモリ以外のリソースの開放がかえってめんどくさくなったな。
781 名前：仕様書無しさん mailto:sage [2007/10/07(日) 18:41:49 ]: 1メソッドだらだら1000行とか書くやつは害虫としか思えん。
自分が辞めるよりそいつを駆除したほうがいいだろ普通。
782 名前：仕様書無しさん [2007/10/07(日) 18:51:41 ]: >>780
javaならファイナライザがあるよ
783 名前：782 mailto:sage [2007/10/07(日) 18:52:39 ]: と思ったけど動くタイミングが微妙だな
784 名前：仕様書無しさん [2007/10/07(日) 19:52:52 ]: C++/CLIを使えばいいじゃない
785 名前：仕様書無しさん mailto:sage [2007/10/07(日) 20:00:32 ]: >>775
普通はウンコ吹いてからウォシュレット使うでしょ
786 名前：仕様書無しさん mailto:sage [2007/10/07(日) 20:01:12 ]: >>780
C#のusingでスコープ囲む奴じゃダメ？
787 名前：仕様書無しさん mailto:sage [2007/10/07(日) 20:04:38 ]: >>785
最初にウォシュレットで残留物を洗い流してから、
紙に色が付かなくなるまでふきふきしてる。
788 名前：仕様書無しさん mailto:sage [2007/10/07(日) 20:13:15 ]: >>786
usingでもいいけど、開放したいのが複数あったらネストが深くなるからね。

using (IDbConnection conn = new ・・・) {
  conn.Open();
  using (IDbCommand cmd = conn.CreateCommand()) {
    cmd.CommandText = "・・・";
    uding (IDbDataRedader reader = cmd.Execute・・・） {
      while (reader.Next()) {
        ・・・
      }
    }
  }
}

C++/CLI や D みたいにやれたらいいのに。
789 名前：仕様書無しさん mailto:sage [2007/10/07(日) 20:15:33 ]: >>787
それだと横に飛び散る様な気がするから俺は、>>785派
790 名前：仕様書無しさん mailto:sage [2007/10/07(日) 20:33:57 ]: >>788
順序は無視して、単純に開放したいのが複数あるときは
using( ... )
using( ... )
{}
って出来たと思う。的外してたらすまんす。
791 名前：仕様書無しさん mailto:sage [2007/10/07(日) 20:58:01 ]: exit(-1);
792 名前：仕様書無しさん mailto:sage [2007/10/07(日) 22:11:26 ]: >>777
デストラクタは破棄される自分自身が保持してるものをアボンヌする「のみ」・・・のはずだから
まぁタイミングなんて別にいいじゃん・・・・。いや実際処理入れられるけど、出来るけれど
793 名前：仕様書無しさん mailto:sage [2007/10/08(月) 00:04:26 ]: 自分の持っているもののみを開放するのみとは言え、

class CFile {
public:
　　CFile() : m_fileptr(NULL) { }
　　~CFile() { Close(); } // まさに開放するのみ
　　bool OpenForRead(const std::string &path);
　　bool OpenForReadWrite(const std::string &path);
　　void Close();
private:
　　FILE *m_fileptr;
};

CFile *file = new CFile();
file.OpenForReadWrite("/tmp/hoge");

// ここで GC の対象になったとして

CFile *file = new CFile();
file.OpenForRead("/tmp/hoge");

OpenForReadWrite, OpenForRead は
Reader-Writer Lock で flock かかるとして、
OpenForRead が成功するか失敗するかわからん、
というのが悩みどころなのでは。

いや、この場合は明示的に Close してやればいいんだけど。
# C/C++ しかわからんので C++ + GC という謎の物体でスマン
794 名前：仕様書無しさん mailto:sage [2007/10/08(月) 00:16:00 ]: 実処理の発動はともかく破棄権を与えるタイミングは把握できるはずだから問題ない・・・と思うんだけど
795 名前：仕様書無しさん mailto:sage [2007/10/08(月) 03:43:17 ]: ># C/C++ しかわからんので C++ + GC という謎の物体でスマン
途中まで読んで、最近はgc付きのC++なんてのがあるのかと思ったぞ
796 名前：仕様書無しさん [2007/10/08(月) 03:54:21 ]: // cnt が 0 より大きかったら処理する
if (cnt > 0) {
}
797 名前：仕様書無しさん mailto:sage [2007/10/08(月) 04:00:34 ]: if (cunt < dick) {
　　　// や～ん、こわれちゃう！
}
798 名前：仕様書無しさん mailto:sage [2007/10/08(月) 05:34:57 ]: デバックの結果、不等号の向きが逆、ってのはよくある話で
799 名前：仕様書無しさん mailto:sage [2007/10/08(月) 07:00:58 ]: // 余計な物がありますよ
if( a < 20 );
{
return hogehoge.detteiu( pero-n, a );
}
800 名前：仕様書無しさん mailto:sage [2007/10/08(月) 09:40:32 ]: >>796
見りゃわかるから、そんなコメントいらね！ってこと？
もしくは、 0 < cnt のほうが見やすいだろ！ってことか？
801 名前：仕様書無しさん mailto:sage [2007/10/08(月) 11:29:35 ]: >>800
そんなこともわからんのも
まずいと思う。
802 名前：仕様書無しさん mailto:sage [2007/10/08(月) 12:07:32 ]: >>gc付きのC++
C++/CLI　でもこれはやるのかな?
803 名前：仕様書無しさん mailto:sage [2007/10/08(月) 12:16:56 ]: >>801
や、すまん。俺も悩んだ。
804 名前：仕様書無しさん [2007/10/08(月) 12:21:43 ]: if (cnt > 0) {
}

何の問題もないだろ、
おれもこう書くが？
805 名前：仕様書無しさん mailto:sage [2007/10/08(月) 12:26:31 ]: ネタはコメントの方だろ
806 名前：仕様書無しさん mailto:sage [2007/10/08(月) 12:29:42 ]: 俺の部署のコーディングルールだと直値は禁止なので０じゃなくて
if(cnt > (int)NUMBER_ZERO){
}
とかにしないといけない。
807 名前：仕様書無しさん mailto:sage [2007/10/08(月) 12:31:03 ]: > NUMBER_ZERO
808 名前：仕様書無しさん [2007/10/08(月) 12:31:41 ]: > if(cnt > (int)NUMBER_ZERO){
> }
> とかにしないといけない。
まったく意味無しのウザイルールだな。
仕様が変わって 1 より大きくなったら、
if(cnt > (int)NUMBER_ONE){
}
ってすべて書き直すんだな。

あほだ
809 名前：仕様書無しさん mailto:sage [2007/10/08(月) 12:33:12 ]: 古典的な誤りだな
810 名前：仕様書無しさん [2007/10/08(月) 12:37:36 ]: 別にいいんだけど、NUMBER_ONEとかってセンスないっていうか、
ちょっとウケるな。
そもそもそこを閾値として、THRESHOLDなんたらとかにしておけば、仕様が変わった場合、
その定数の定義を変えるだけでいいが、NUMBER_ZEROだったら、
結局全部探して直さなきゃならないから、全く意味ないと思うが。
811 名前：仕様書無しさん mailto:sage [2007/10/08(月) 13:22:51 ]: > 仕様が変わって 1 より大きくなったら、

いやーこうじゃないか？

#if 0
#define NUMBER_ZERO 0
#endif
/* 2007/10/08 nanasi808 */
#define NUMBER_ZERO 1
812 名前：仕様書無しさん [2007/10/08(月) 13:34:14 ]: #if 0
#define NUMBER_ZERO 0
#endif
/* 2008/10/08 洋司 */
#define NUMBER_ZERO 1
813 名前：仕様書無しさん [2007/10/08(月) 13:35:56 ]: >>811
定数名と内容が一致してないだろ。
ダメダメなコードの典型だと思うぞ。
814 名前：仕様書無しさん [2007/10/08(月) 13:38:30 ]: #if 0
#define NUMBER_ZERO 0
/* 2008/10/08 洋司 */
#define NUMBER_ZERO 1
/* 2009/10/08 洋司 */
#define NUMBER_ZERO 2
/* 2010/10/08 洋司 */
#define NUMBER_ZERO 3
#endif

/* 2011/10/08 洋司 */
#define NUMBER_ZERO 4
815 名前：仕様書無しさん mailto:sage [2007/10/08(月) 13:39:10 ]: #defineでてきたか・・・、そんなことで#define使うなよ
その部分のソースのみを見て名前から動きが理解できないじゃないか・・・
816 名前：仕様書無しさん [2007/10/08(月) 13:40:25 ]: 洋司うるさいｗ
817 名前：仕様書無しさん mailto:sage [2007/10/08(月) 13:44:01 ]: 洋司はもうちょっと検討段階で頑張れよｗ
818 名前：仕様書無しさん [2007/10/08(月) 13:46:51 ]: #if 0
#define NUMBER_ZERO 0
/* 2008/10/08 洋司 35歳*/
#define NUMBER_ZERO 1
/* 2009/10/08 洋司 36歳*/
#define NUMBER_ZERO 2
/* 2010/10/08 洋司 37歳、リストラされる*/
#define NUMBER_ZERO 3
#endif

/* 2011/10/08 陳姪用 */
#define NUMBER_ZERO 4
819 名前：仕様書無しさん mailto:sage [2007/10/08(月) 13:48:28 ]: そういうところは仕様は絶対かわなんないんだよ。
もしNUMBER_ZERO で定義してるんだったらNUMBER_ONEってのもどっかにあるだろうし。
よほど変なコーディングでない限り、その部分が24になったり10456になったりとか
することはないよ。変わる仕様だったら別な名前でちゃんと定義してるだろ。
自分も>>808と似たようなところだが、
どんな数字も全部そう書くようにする規約があるからわかるんだが
１回しか登場しなくてどうやったって誰が書いてもどんなに仕様が変わっても
そこだけは必ず０から始まるようなものって結構あるんだよ。
たとえばゼロ割を回避するためだったり。
820 名前：仕様書無しさん [2007/10/08(月) 13:49:01 ]: 引き継ぎcomplete!!
821 名前：仕様書無しさん [2007/10/08(月) 13:49:36 ]: >>819
うまいことまとまった。サンクス
822 名前：仕様書無しさん [2007/10/08(月) 13:50:52 ]: >>819
元の書き込みはそうだろうが、俺らが文句言っているのは>>808に対してなので、
その辺、勘違いしないでよね！！
823 名前：仕様書無しさん [2007/10/08(月) 13:52:10 ]: >>822
>>808 も >>819 も同じ事いってるだろ、ヴぉけ
お前は洋司レベルだな
824 名前：仕様書無しさん mailto:sage [2007/10/08(月) 13:53:59 ]: >>819
NUMBER_ZEROってのが一般的に利用するcommon的なソースであって絶対に変化のない物なら理解できる

だが、そのシステムに特化したソース内では利用すべきじゃないと思う
それならば別の名前を付けることもできるだろうし、ゼロってのは数字であって名称ではないし
825 名前：仕様書無しさん [2007/10/08(月) 13:54:40 ]: >>823
褒め言葉として受け取っておこう！！
826 名前：仕様書無しさん [2007/10/08(月) 13:55:21 ]: おれは洋司とは仕事したくないぞ
827 名前：仕様書無しさん [2007/10/08(月) 13:56:00 ]: >>824
そうだね。
そのゼロがなんなのか分かる名前なら意味あるけど、
ただNUMBER_ZEROじゃ、打つのが面倒なだけで、0って直書きするのと変わらない。
828 名前：仕様書無しさん mailto:sage [2007/10/08(月) 14:20:27 ]: しかし、これがNUMBER_ZEROでなく(int)NULLだとさらに混沌の度が増す……
829 名前：仕様書無しさん [2007/10/08(月) 14:30:39 ]: 「FIVEは5か?」は昔からある教訓で、
>>819の考えは適切ではない。
830 名前：仕様書無しさん mailto:sage [2007/10/08(月) 14:43:47 ]: 定数ゼロを使う場合なんて大半は
マクロ定義改変でゼロ以外に変更してコードの理屈が通るとは考えられないしなぁ
必要かどうか、なら要らないわな
831 名前：811 mailto:sage [2007/10/08(月) 14:44:24 ]: 完全にネタのつもりで書いたのに。マジレスしてるやつは議論の余地があるとでも思ってるのか？恐ろしい。
832 名前：仕様書無しさん mailto:sage [2007/10/08(月) 14:45:08 ]: NUMBER_ZERO
この名前に誰も突っ込まないのがなあ。
SECTION_ZEROとかROW_ZEROとかのグループ的な意味を持つ場合ならいいけど
そうじゃないイミフな定数は定義した意味が半減するし
833 名前：仕様書無しさん mailto:sage [2007/10/08(月) 14:52:19 ]: >>832
みんな突っ込んでるだろ。
ZEROと書いといて、仕様変更で「0」以外の値になることは考えないのか？という話で。
代案もセンス悪い。
漏れだったら、COUNTER_START_NUM か何か、ZEROを含まず機能を類推できる名前にする。
834 名前：仕様書無しさん mailto:sage [2007/10/08(月) 14:55:06 ]: 現実に周囲を見回すと、物に名前を付けるって意味について二通りの意識を持つコーダーがいる気がするな

今後の拡張等を踏まえて、利便性を追求するのためにとりあえず名前を付ける人
ソースを分かりやすくするために適切な名前を付けようとする人

で、前者は大抵ワガママなソースになりがちで嫌われると思う
835 名前：仕様書無しさん mailto:sage [2007/10/08(月) 14:58:59 ]: ＞誰も突っ込まないのがなあ

突っ込んでんじゃん
文盲か？
836 名前：仕様書無しさん mailto:sage [2007/10/08(月) 15:02:18 ]: >SECTION_ZEROとかROW_ZEROとかのグループ的な意味を持つ場合ならいいけど

いいのかよｗ
837 名前：仕様書無しさん mailto:sage [2007/10/08(月) 15:07:02 ]: 釣れるのは雑魚ばかりか
838 名前：仕様書無しさん mailto:sage [2007/10/08(月) 15:19:47 ]: 関数は５０行ぐらい、多くても１００行までにしてくれ、と言っても
理解できない奴が多すぎて困る。
１０００行とか天才かよと。
839 名前：仕様書無しさん mailto:sage [2007/10/08(月) 15:25:22 ]: 関数分割って結構難しいと思う。
ノッて書き出すと１関数でずっと続けちゃうし、分割した方がいいと思っても、
バグ発生や見通しが悪くなるのが怖くてなかなか踏み切れない。
できるかできないかがプログラマのセンスの差なんだろう。
840 名前：仕様書無しさん mailto:sage [2007/10/08(月) 15:44:21 ]: そういう問題じゃねぇだろ
841 名前：仕様書無しさん mailto:sage [2007/10/08(月) 15:44:42 ]: > バグ発生や見通しが悪くなるのが怖くてなかなか踏み切れない。

釣りか？
842 名前：仕様書無しさん mailto:sage [2007/10/08(月) 15:48:08 ]: 関数分割で見通しが悪くなる、ってどんだけ
843 名前：仕様書無しさん mailto:sage [2007/10/08(月) 15:53:07 ]: 839だけど、俺くらい技術の低いプログラマだと、そういうレベルで
悩んでしまうってことです。
バグ発生は、同じ変数を他所でいじってるんじゃないかと取り越し苦労して、
見通しが悪くなるのは、機能を適切に分割できなくて、引数が複雑に
なったり、戻り値が凝ってる割にたいしたことやってなかったり拡張性が
悪い物だったりする。
844 名前：仕様書無しさん mailto:sage [2007/10/08(月) 16:04:02 ]: 分からんなら分かる奴とかPGの取りまとめとかに聞け。
分からんからといって問題を放置されるのが一番タマラン。
845 名前：仕様書無しさん mailto:sage [2007/10/08(月) 16:09:31 ]: 大きな関数をセンスで分割するというアプローチが既に誤り
目に見えない制約がたくさんあって、それを遵守すると機械的に
小さな関数群になる、という感じ
846 名前：仕様書無しさん mailto:sage [2007/10/08(月) 16:16:32 ]: こういうアクションで呼ばれた(たとえば、ボタン A が押された)ら、
「こうしてこうしてこうする」、というのを関数にして、
それぞれの「こうする」をさらに「こうしてこうする」にして…
というのを (OS なり何らかのライブラリなりの) API に達するまで繰り返す
というのを基本としたら、必然的に小さく収まるよなぁ

あー、何か俺自身感覚で分割してるような気になってきた
847 名前：仕様書無しさん mailto:sage [2007/10/08(月) 16:47:51 ]: >>844
すいません、今の職場、俺以外にPGいないんです。
社内間接部門で、他の人は仕様とりまとめて俺に卸してきます。コーディングは
Excel の自動生成マクロをちょっと手直しして使えるってれベルです。
プログラムは全部俺に任せるって言われてます。関数分割も含め、日々試行錯誤です。
OSSのソース読んだりして勉強してるつもりなんですが、外に出て実戦するのがいいのかなぁ。
848 名前：仕様書無しさん mailto:sage [2007/10/08(月) 17:04:15 ]: 初心者が自己流だけでいくのは将来危険。
いろいろ回って経験積んだ人にまかせた方がいいと思う。
その経験積んだ人にまかせるのだって、ほんとは危険。
一人にやらせるってのがそもそも危険。
849 名前：仕様書無しさん mailto:sage [2007/10/08(月) 17:21:53 ]: プログラミングは危険。超危険。
850 名前：仕様書無しさん mailto:sage [2007/10/08(月) 17:45:19 ]: 俺が今日作った関数500行ぐらいある。
疲れた。そして明日からももっと疲れるだろう。
851 名前：仕様書無しさん mailto:sage [2007/10/08(月) 17:45:51 ]: もはや釣りかネタとしか思えん
852 名前：仕様書無しさん mailto:sage [2007/10/08(月) 18:10:51 ]: >>801
本気でわからんので回答ください。

800 ：仕様書無しさん：2007/10/08(月) 09:40:32
>>796
見りゃわかるから、そんなコメントいらね！ってこと？
もしくは、 0 < cnt のほうが見やすいだろ！ってことか？

801 ：仕様書無しさん：2007/10/08(月) 11:29:35
>>800
そんなこともわからんのも
まずいと思う。

803 ：仕様書無しさん：2007/10/08(月) 12:16:56
>>801
や、すまん。俺も悩んだ。
853 名前：仕様書無しさん mailto:sage [2007/10/08(月) 18:17:47 ]: 見りゃわかるからだろ
どうでもいいとこにコメント書きまくってる奴に限って
本当に必要な所にはコメントなかったりするんだよな
854 名前：仕様書無しさん mailto:sage [2007/10/08(月) 18:22:39 ]: たいてい関数は短くするけど
ロジック的なものじゃなくて力任せにいろいろ構成を書いていく部分のコードだと長くなる　300とか
855 名前：仕様書無しさん mailto:sage [2007/10/08(月) 18:34:56 ]: ライブラリとか低位レイヤの関数は短いよね？
業務レイヤになってくると分割が意味を成さないような処理もあるから
ちょっとだけ長くなるよね？　ね？(一連動作を明示する意味もある！？)

短いとか長いとか両極端に走っちゃだめだよね？　ね？
856 名前：仕様書無しさん mailto:sage [2007/10/08(月) 18:50:47 ]: >>855
本来、業務ロジックなんか一番短くなってしかるべき
857 名前：仕様書無しさん mailto:sage [2007/10/08(月) 19:14:48 ]: 長い関数の中の一行を修正しなくちゃならんという時、
その一行で使われている変数が前後でどう使われているか
普通チェックするもんじゃない？
関数が短ければ追っかけなきゃならんロジックが減って
あとで楽できると思うんだが。

俺だったらhoge()っていう300行の関数作らず
100行ぐらいのhoge_a(),hoge_b(),hoge_c()を作って
それを順番に呼び出すだけのhoge()を作るなぁ。
858 名前：仕様書無しさん mailto:sage [2007/10/08(月) 19:43:39 ]: >>855
ログ吐く
処理1を呼び出す
結果を見てエラーがあればコードに応じて処理を書く
ログ吐く

ログ吐く
処理2を呼び出す
　・
　・
　・

みたいな感じで長くなることはあるな
859 名前：仕様書無しさん mailto:sage [2007/10/08(月) 20:16:01 ]: >>853
おｋ。納得した。
860 名前：wolf ◆8VH3XAqjlU mailto:sage [2007/10/08(月) 20:34:59 ]: >>852

// cnt が 0 より大きかったら処理する
if (cnt > 0) {

コメントがここに書いてある６番目の心掛けなければイケナイ事
www.ambysoft.com/javaCodingStandards.html

6. Document why something is being done, not just what
何をしているかをコメントにするのではなく、何故そうするのかを書きなさい
に抵触はしていますね
861 名前：仕様書無しさん mailto:sage [2007/10/08(月) 20:38:32 ]: 「他人に分からせる気はこれっぽっちもない」けど「懇切丁寧ぶったコードには見える」んだよ　ククク
862 名前：仕様書無しさん mailto:sage [2007/10/08(月) 20:45:29 ]: // cnt が 0 より大きかったら処理する
if (cnt > 0) {

ここの条件分岐が、実際の業務と直接リンクする場合は上のようなコメントを書くと思う。
そういうのはあり？たとえば「じいちゃんがココで死にそうだったら110番する」みたいな。
863 名前：仕様書無しさん mailto:sage [2007/10/08(月) 20:48:57 ]: でも0 ゼロだぞ
「○○がまだ残っている場合」とか
そういう書き方にならね
864 名前：仕様書無しさん mailto:sage [2007/10/08(月) 20:52:58 ]: >>862
それはありだと思う。
「（金額） cnt が 0 より大きかったら処理する」のように、cnt がもつ情報が類推できるような場合だよね。
ただし、それは他のコードやコメントと読み合わせてはじめて理解できるものじゃない？
コード例だけだと、類推のための手がかりがないから、論争になってるわけで。
865 名前：仕様書無しさん mailto:sage [2007/10/08(月) 20:55:34 ]: 変数名がcntだから、count=「個数」であることは確か(なはず)なので、
金額は例としてはまずいな。

しかし、個数である場合に、
「あれば～」とか「残っていれば～」
などでなく本当に
「0より大なら～」
というコメントにすべき例を思いつかない。
866 名前：仕様書無しさん mailto:sage [2007/10/08(月) 20:57:45 ]: カウンタ絡みのテキトー変数名でcntがついても不思議じゃない
867 名前：仕様書無しさん mailto:sage [2007/10/08(月) 20:58:05 ]: つまり、cntが何を表すか、処理ってどんな処理か書けっって言いたいんだろう。
>>862　でいう「じいちゃん」とか「110番」って情報が抜けてる。
868 名前：仕様書無しさん mailto:sage [2007/10/08(月) 21:01:41 ]: // まだ在庫があれば、賞味期限を確認する
if (cnt > 0) {
　/*賞味期限確認*/
}
869 名前：仕様書無しさん mailto:sage [2007/10/08(月) 21:20:24 ]: // cntが0以上だったら処理する
// 19XX.X.X. レビューで修正
//if (cnt > 0 || cnt == 0) {
if (!(cnt < 0)) {
870 名前：仕様書無しさん mailto:sage [2007/10/08(月) 21:21:05 ]: おまいら、なんか丁寧な議論をしてて好感が持てるぞ。
要は内部的な単なる処理に自明なコメント付けるなってことだよね。

i=i+1; //iに1を加算する　みたいな。
871 名前：仕様書無しさん mailto:sage [2007/10/08(月) 21:22:54 ]: howじゃなくてwhatを残すんだろ
コメントの基本
872 名前：仕様書無しさん mailto:sage [2007/10/08(月) 21:28:30 ]: >>868
コメントを書きたくなったら変数名や関数名を考え直すんだ。

こんなかんじで。
if (zaiko > 0) {
　shoumikigen_kakunin();
}
873 名前：仕様書無しさん mailto:sage [2007/10/08(月) 21:30:33 ]: >>869
わろす
874 名前：仕様書無しさん mailto:sage [2007/10/08(月) 21:39:53 ]: >>869
レビュワーのレベルも知れるな。
875 名前：仕様書無しさん mailto:sage [2007/10/08(月) 21:41:45 ]: 修正前の方がまだマシのような気が・・・
876 名前：仕様書無しさん mailto:sage [2007/10/08(月) 21:43:17 ]: 指摘した人間の想定の遙か上をいくアホであったのかも
877 名前：仕様書無しさん mailto:sage [2007/10/08(月) 21:44:09 ]: >>869
レビュワーの主義が知りたい。
878 名前：仕様書無しさん mailto:sage [2007/10/08(月) 21:45:30 ]: >>869
自分は今までIFは肯定文と否定文両方必要だと思ってたけど
今いるところで、無駄な文は作るなと直された。
SQLだと否定文を書くのはNGだけど
今の規約ではJavaの方は否定文なら否定文だけ書かされる（当たり前だが）

if (!methodA()) {
// 処理
}
879 名前：仕様書無しさん mailto:sage [2007/10/08(月) 22:11:53 ]: ちょい上のほうで関数分割ネタがあるんだが、
今漏れが見てるソースにおもろいのがあった。
VB6ですまんが

Sub calc()
処理１
　・
　・
　処理１０
　Call calc2
End Sub

Sub calc2()
処理１１
　・
　・
　処理２０
　Call calc3
End Sub

Sub calc3()
　略

これは関数分割といっていいのか？ｗ
880 名前：仕様書無しさん mailto:sage [2007/10/08(月) 22:43:55 ]: >>879
分割なのかと言われると確かに分割してるなｗ
881 名前：仕様書無しさん mailto:sage [2007/10/08(月) 22:49:44 ]: >>879
そんな感じで分割(?)されてて、しかも連番の振り方が処理内容ではなく改修の順番だったというクソコードのメンテさせられたことがある。
882 名前：仕様書無しさん mailto:sage [2007/10/08(月) 23:27:16 ]: >879
calc2がcalc以外からも呼び出されるなら、まあ許せる範囲にはいるかもしれん
883 名前：仕様書無しさん mailto:sage [2007/10/09(火) 00:26:20 ]: ぱっと見てどれほどの量の処理なのかわかんないから、
どんどん続きの処理が出てきてウヴォアァってなるなー
メンテでそんなコード見たくないねぇ
884 名前：仕様書無しさん mailto:sage [2007/10/09(火) 00:32:25 ]: >>877
条件式は!と<さえあれば表現できるので、
わざわざ==や!=や>のような余計なものを使って
初心者を混乱させるべきでない
とか？
885 名前：仕様書無しさん mailto:sage [2007/10/09(火) 00:34:16 ]: 4種類の単品マスターをセット組する処理で

単品マスターがカナとコードで検索が可能で
検索結果の並び順をカナ、コード、更新日時順で指定可能

24個のコピペ関数がKensaku1とかSearch2とかって名前で存在し、
cmdKensaku1で呼んでるのがKensaku4だった
ぐわーっとコメントアウトして書き直してしまった。
886 名前：仕様書無しさん mailto:sage [2007/10/09(火) 00:37:26 ]: >>885
俺、perlでそういうのに出会ったことがある。
正直あれは思い出したくない。
887 名前：仕様書無しさん mailto:sage [2007/10/09(火) 00:51:31 ]: あー。あるある。
888 名前：仕様書無しさん mailto:sage [2007/10/09(火) 00:58:45 ]: //if (cnt > 0 || cnt == 0) {

って結構見た目綺麗だよなぁ、と思ったりw
>=ってなんかキチャナイ気がするし
889 名前：仕様書無しさん mailto:sage [2007/10/09(火) 03:30:10 ]: 左辺がもっと長かったら２回書くのは鬱陶しいよ
評価するために何らかの処理が走る場合は２回走っちゃうし（一時変数使えばいいけどやっぱ手数が増える）
890 名前：879 mailto:sage [2007/10/09(火) 05:50:29 ]: >>880,882
一応きりのいいところできられてるっぽいが、
変数のスコープが狭くできているとかいうこともない。
calc2以下は前の関数から呼ばれることを前提に作られている。
関数分割のメリットはどこにあるのやらｗ

>>881
うっ・・・こっちのほうがまだましだったか

当時の事情はよくわかんねえんだが、たぶん、
関数の長さが長すぎるといけない、とかいうんで無理矢理ちぎった
というのがオチだろうと思います。
891 名前：仕様書無しさん mailto:sage [2007/10/09(火) 08:48:12 ]: case型構文があるときはどうすんの？　ちょんぎれないよね。
892 名前：仕様書無しさん mailto:sage [2007/10/09(火) 11:22:48 ]: ちょんぎるんじゃない、構造を見直すんだ
893 名前：仕様書無しさん mailto:sage [2007/10/09(火) 12:14:08 ]: case文ひとつで何百行にもなるのなら、その時点でおかしい。
ちょんぎるとかいうレベルじゃない。
894 名前：仕様書無しさん mailto:sage [2007/10/09(火) 12:53:47 ]: だって100とおりのcaseなら各中身が平均４行だって500行ぐらいすぐなるじゃない。
コマンドが何十種類とか。caseの数が多いこと＝設計ミスとは決めつけられないでしょ。
895 名前：仕様書無しさん mailto:sage [2007/10/09(火) 13:01:03 ]: 決め付けはできないけど、大抵の言語でダメだよ
896 名前：仕様書無しさん mailto:sage [2007/10/09(火) 13:03:01 ]: >>893
つWindowsアプリのソースコード
897 名前：仕様書無しさん mailto:sage [2007/10/09(火) 13:42:24 ]: >>894
それはcase文ひとつで数行である、ってことでは？
898 名前：仕様書無しさん mailto:sage [2007/10/09(火) 14:28:56 ]: >>896
WndProc か?
LRESULT CALLBACK WndProc(HWND hWnd, UINT msg, WPARAM wp, LPARAM lp) {
switch(msg) {
case WM_PAINT: return OnPaint(hWnd, wp, lp);
case WM_SIZE: return OnResize(hWnd, wp, lp);
default: return DefWndProc(hWnd, msg, wp, lp);
}
}
ってするな、俺は。
実際には WPARAM や LPARAM をそのまま渡したりはせずに
MSDNの記述に従って分割してから渡すけど。

仮に処理するウィンドウメッセージが多くて、
この case が 1000 個連なったとしても OK だろ、この場合は。

よくあるプログラミング講座みたいに case の中につらつら書くのはダメだ
899 名前：仕様書無しさん mailto:sage [2007/10/09(火) 18:35:36 ]: >>898
それって非常にありがちだけど、個人的にはそういう書き方は自己満足にしかなってないと思うなあ。
一度反省的に自分のそういうコードを見直してみて欲しいよ。

一箇所からしか呼ばれないコードを関数に括りだすことが本当に可読性に資するのか。
むしろ、コードを読む際にそこに飛ぶ手間を増やしているだけじゃないのか。

「caseの中につらつら書いた」コードが本当に読みにくいのか。
むしろ教条主義的にそう思い込んでるだけじゃないのか。
900 名前：仕様書無しさん mailto:sage [2007/10/09(火) 19:00:35 ]: 抽象ってどういう意味か考えてみて欲しいね
「読まければ意味がわからない関数」は括り出す意味は無い
可読性をあげるためには、「今まさに関心のある」レイヤ以外に存在する
関数について、「読む必要性自体」を消し去る必要があるのだが
901 名前：仕様書無しさん mailto:sage [2007/10/09(火) 19:10:34 ]: 「コードを読む際にそこに飛ぶ」のにそんなに手間がかかるのかｗ
902 名前：仕様書無しさん mailto:sage [2007/10/09(火) 19:13:13 ]: >>894
俺ならcaseマッチングのテーブルを挟んでループさせるな。
マッチングオブジェクトは当然処理関数への参照を持たせる。

マッチングしたら対象オブジェクトの処理を呼び出す。
C/C++なら普通に書ける関数テーブル。

処理部分はスマートになるし、定義部分ではコメント多用して
逆に分かり易く表現出来る。

VBはしらねーよw
903 名前：仕様書無しさん mailto:sage [2007/10/09(火) 19:17:03 ]: OOPLにおいては、
どうやって分岐するか、と分岐した先で何をやるか
は、分離しとかないと基本的にはOCP違反になる
904 名前：仕様書無しさん mailto:sage [2007/10/09(火) 19:34:57 ]: 関係の無いメッセージに対する処理を脳内でスキップする方が
関数1個参照に行くよりよほど時間かかるよな

何のサポートもしてくれないエディタ使ってる場合はどうか知らんが

長大な関数は変数のスコープなんかで
気にしないといけないことも爆発的に増えるし
905 名前：仕様書無しさん mailto:sage [2007/10/09(火) 20:53:01 ]: >>902
状態遷移表の実装なんかで俺もソレ良くやるけど、
関数ポインタ(Cなんで)使うと、静的解析で処理が追い辛くなるのが難点かねぇ。
906 名前：仕様書無しさん mailto:sage [2007/10/09(火) 21:03:31 ]: こういうときデリゲートとかラムダ式があれば…
boost禁止だしのぅ
907 名前：仕様書無しさん mailto:sage [2007/10/09(火) 21:11:55 ]: >>904
分かってないね。

「関係の無いメッセージに対する処理をスキップ」する必要性は
各メッセージの処理を関数に括り出そうがなくならない。

ただエディタ上で「読み飛ばす」行数が減るだけの効果しかないんだよ。

それに、ウィンドウプロシージャのような定型的で分かりきった処理は
行数が増えてもコードの見通しは悪くならない。

実際試せば実感できると思うけど、
こういう場合はむしろ関数呼び出しによる抽象化の方がコードの不透明性を高めるから
読んでイライラする蓋然性が上昇するよ。
908 名前：仕様書無しさん mailto:sage [2007/10/09(火) 21:23:47 ]: >>907
わかってないのはお前だ。だまされたままで良いから周囲にあわせとけ。
909 名前：仕様書無しさん mailto:sage [2007/10/09(火) 21:26:18 ]: んー・・・凄い個人的な主観的な話になるけど、
caseとcaseの間がエディタの半ページ以内なら気にならない。
1つのcaseでエディタ1ページを超えると途端に読み辛くなる。

開発環境によってエディタの行数変わるんだけど
(WindowsだとVCかサクラ、Linuxだとemacsとか)、
俺は大体こんな感じ。
910 名前：仕様書無しさん mailto:sage [2007/10/09(火) 21:33:48 ]: >>907
ガラクタを無秩序に押入れに押し込むことを抽象化と呼ぶのなら
お前のいうとおりだろうな
関数名に連番とか振ってないだろうなｗ？
911 名前：仕様書無しさん mailto:sage [2007/10/09(火) 21:48:45 ]: >>910
＞ガラクタを無秩序に押入れに押し込むことを抽象化と呼ぶのなら
そんなトンデモな前提に立った議論ではない。

君とか904は、まあダメなプログラマにありがちだけど
暗黙のうちに「プログラムを書いている時点の視点」に立っている。

確かにプログラムを書いている時点に限定すれば、ほとんどの場合は
関数に括り出した方が読みやすいし、見た目にもスッキリする。
「caseの中につらつら書いた」コードなんて醜いし見難いように思われる。

ところが同じコードを一年後に読むと評価が逆転するんだなこれが。
912 名前：仕様書無しさん mailto:sage [2007/10/09(火) 21:53:11 ]: なにそれ、全然理由になってない
913 名前：仕様書無しさん mailto:sage [2007/10/09(火) 21:54:05 ]: >>910
>関数名に連番とか振ってないだろうなｗ？
関数名、変数名がすべて [A-Z]{2,3}[0-9]{5} なコードの保守ならやらされたことがある。
それはともかく、case 文の中身が10行超えたら、普通 state か command か chain of responsibility あたりを使わね？
914 名前：仕様書無しさん mailto:sage [2007/10/09(火) 21:54:51 ]: case 1:
{

} break;

case 2
{
って最初に作っておいたらスコープ付くしbreak強制になるから便利
915 名前：仕様書無しさん mailto:sage [2007/10/09(火) 21:55:11 ]: >>911
オブジェクト指向にOCPってあるんだけど、きいたことある？
いろいろ勉強して、実践して、その上でいってるのか心配になっちゃうんだけど
916 名前：仕様書無しさん mailto:sage [2007/10/09(火) 21:58:03 ]: デリゲートを条件に使われる定数をインデックスにして連想配列にぶちこむのは一種のストラテジーパターンらしい
917 名前：仕様書無しさん mailto:sage [2007/10/09(火) 21:59:40 ]: >>915
そいつのレスはムダに長いが、主張したいことは最後の1-2行だけ。
そこ見たらこれ以上そいつにかまう必要が無いことがわかる。
918 名前：仕様書無しさん mailto:sage [2007/10/09(火) 22:11:46 ]: >>899
> >>898
> 一箇所からしか呼ばれないコードを関数に括りだすことが本当に可読性に資するのか。

Testability
919 名前：仕様書無しさん mailto:sage [2007/10/09(火) 22:22:07 ]: >918
君の一言を待っていた
920 名前：仕様書無しさん mailto:sage [2007/10/09(火) 23:55:45 ]: ( ﾟдﾟ )
921 名前：仕様書無しさん mailto:sage [2007/10/10(水) 00:47:25 ]: この類の話は
センスがない人にはいくら説明しても理解できないんだよね。
たぶん>>899,907,911は文系か元コボラ。
922 名前：仕様書無しさん mailto:sage [2007/10/10(水) 01:30:46 ]: >>921
今追いついたけど同意
923 名前：仕様書無しさん mailto:sage [2007/10/10(水) 01:38:13 ]: 厳しいこといえば、センスとか言ってるやつもヤバイ
基本中の基本だ
924 名前：仕様書無しさん mailto:sage [2007/10/10(水) 01:57:12 ]: こういうやつらが消火の悪いスパゲティ作るんだろうな
925 名前：仕様書無しさん mailto:sage [2007/10/10(水) 03:54:45 ]: 燃え盛ったときに消しにくいんだな
926 名前：仕様書無しさん mailto:sage [2007/10/10(水) 04:32:11 ]: 負け組PGの揚げ足取り乙
927 名前：仕様書無しさん mailto:sage [2007/10/10(水) 08:17:56 ]: >>899
逆でしょw
武道みたいにプログラミングでもスタイルや「型」は大事だし、
それらはほとんどの場合正しいが、いつでも正しいわけじゃない。

意味を持った処理は関数に括り出して抽象化することによって読みやすくなる、
という「型」はほとんどの場合正しいが、いつでも正しいわけじゃないんだが、
自分の頭で考えられない教条主義者（＝「センス」を欠いた人間）にはそれが
判断できないし、そういう発想が頭をよぎっても抑圧してしまうんだね。

王様は裸だ、と言えないのと同じ心理的メカニズムが作用するからねw
928 名前：仕様書無しさん mailto:sage [2007/10/10(水) 08:38:05 ]: 何が「逆」なのか、説明してくれ
929 名前：仕様書無しさん mailto:sage [2007/10/10(水) 10:11:44 ]: おまえら、当然ファウラーの「リファクタリング」を読んだ上で発言しているだろうな。
930 名前：仕様書無しさん mailto:sage [2007/10/10(水) 11:43:05 ]: >>929
教科書主義者乙
931 名前：仕様書無しさん mailto:sage [2007/10/10(水) 12:09:26 ]: > 教科書主義
932 名前：仕様書無しさん mailto:sage [2007/10/10(水) 12:15:19 ]: いくら教条主義はNGだからつって、そもそも教科書読んでねえやつは論外なわけだ
933 名前：仕様書無しさん mailto:sage [2007/10/10(水) 15:16:54 ]: 同意
934 名前：仕様書無しさん mailto:sage [2007/10/10(水) 21:36:24 ]: >>927
あいかわらずムダな長文だな。自分にレスして楽しいか？
935 名前：仕様書無しさん mailto:sage [2007/10/10(水) 23:40:23 ]: 俺には無意味な事を二度書いて、
最後に無関係なたとえ話をしている様に見える。

誰か >>927 をリファクタリングしてくれ。
936 名前：仕様書無しさん mailto:sage [2007/10/10(水) 23:49:49 ]: case毎の処理を関数に切り出すと読み辛くなるってのは、
caseの切り分け方がおかしい証拠に他ならんと思うのだが。

あと

>927
>意味を持った処理は関数に括り出して抽象化することによって読みやすくなる、
>という「型」はほとんどの場合正しいが、いつでも正しいわけじゃないんだが

の「正しくないケース」ってどんな時さ。
抽象化によって読み辛くなるのは、抽象化の仕方がおかしい時だけでしょ。
937 名前：仕様書無しさん mailto:sage [2007/10/10(水) 23:50:25 ]: >>935
#修正しておきました。
938 名前：仕様書無しさん mailto:sage [2007/10/11(木) 00:07:05 ]: >>937
全消しかいｗ
939 名前：仕様書無しさん mailto:sage [2007/10/11(木) 00:39:54 ]: >>936
>>898-を参照
940 名前：仕様書無しさん mailto:sage [2007/10/11(木) 00:45:20 ]: WTLのメッセージマップとか激しく便利だお
941 名前：仕様書無しさん mailto:sage [2007/10/11(木) 19:34:57 ]: >>936

正しくないケースを考えてみた。

1. >>927みたいな奴しかいない現場
2. 作成した関数を全て紙で管理している現場
3. 将来メンテする奴への嫌がらせ
4. リソースがカツカツ
5. そもそもサブロジックを切り出せない言語だ
6. コンパイラの限界に挑戦中

こんなもんか？
942 名前：936 mailto:sage [2007/10/12(金) 02:21:01 ]: >941
納得してしまった。w
943 名前：仕様書無しさん mailto:sage [2007/10/12(金) 09:24:34 ]: >>941
>5. そもそもサブロジックを切り出せない言語だ
そうかN88-BASICだったんだ
944 名前：仕様書無しさん mailto:sage [2007/10/12(金) 11:56:55 ]: N88-BASICじゃ仕方ないなｗ

N88-BASICでも、変数名と戻り値代わりに使う変数名を工夫すれば
GOSUB との組み合わせで何とかなるかもしれんが
本質じゃない不毛なコードが大量生産されそうだ
945 名前：仕様書無しさん mailto:sage [2007/10/12(金) 13:33:21 ]: いろんな火災現場を転々としてるダメPGが言うのもなんだけど
どちらの書き方でも、どんなに拘りの秘伝のソースでもいいが
可読性の無いソースを書くのはやめて欲しい。。

>>944
BASICの頃でも、ON (式) GOSUB *Label...でサブルーチン化はしてたよ。
946 名前：仕様書無しさん mailto:sage [2007/10/12(金) 13:59:28 ]: >945
>可読性の無いソースを書くのはやめて欲しい。。

無理。
自分の常識は他人の非常識である場合が多い。

# だからflgで３値を返すなとあれほど･･･
947 名前：仕様書無しさん mailto:sage [2007/10/13(土) 15:07:00 ]: いやそもそも可読性が無ければ保守不能だし。「可読性が低い」だろ。
じゃあその可読性の水準を向上させるためにはどうすれば良いか。
という話になるけど。これは99%以上が独学だろ。
スレの上のほうに出てきた本を読んでおくのは当然。
OSSという容易に読めるコードもあるじゃないか。

[ 新着レスの取得/表示 (agate) ] / [ 携帯版 ]

read.cgi ver5.27 [feat.BBS2 +1.6] / e.0.2 (02/09/03) / eucaly.net products.
担当:undef