スラッシュドット・ジャパン

#define ZERO 1

　一発の破壊力でこれに勝るコードを見たことが無い。

ag·glom·er·ate [əglάmərèit]
  -n. [-rət, -rèit] 塊り, 《まとまりのない》 集団, 群
(研究社リーダーズプラス)

…きれいなコードになってるのかな？

猿コードは追わず、ただ作り直すのみ。

main()が8000行
switch文が6階層...

T/O

＃ 夜道で撃たれたくないので、AC

strutsのlogicタグ使いすぎによる最長不倒関数 [kojima-cci.or.jp]を見たことがあります。
簡略化して書くと

<logic:equal name="Form1" property="item1" scope="request" value="value1">
    // 色々と処理
    .......
    <logic:equal name="Form1" property="item1" scope="request" value="value2">
        （...色々と処理)
        <logic:equal name="Form1" property="item2" scope="request" value="value2">
            <logic:equal name="Form1" property="item3" scope="request" value="value3">
                <logic:equal name="Form1" property="item4" scope="request" value="value4">
                    <logic:iterate id="content1" property="item5" name="Form1" scope="request">
                    （...色々と処理)    <br>
                    </logic:iterate>
                </logic:equal>
            </logic:equal>
        </logic:equal>
    </logic:equal>
    <logic:equal name="Form1" property="item6" scope="request" value="value6">
        （...色々と処理)
        <logic:notEmpty name="Form1" property="item1" scope="request">
            （...色々と処理)
        </logic:notEmpty>
    </logic:equal>
     (...この後同じような調子でlogicタグによる条件文が数百行続く)
</logic:equal>

という具合に、logicタグを大量に使ってifとループが延々と続くようなものでした。
5層くらいのifが入り乱れてるところにループがちりばめられているのです。

C/Javaのエディターだとifやループの括弧に対する対応位置を自動的に検知してくれたりするので
ある程度長くても調べやすかったけれど、JSPにはまだそういうのが無くて
このファイルの修正作業にはブチ切れそうになりました。

昔、Cで最長不倒関数作ってたような人が、今はWeb開発に回されてるんでしょうねえ..。

C++ で見たものといえばこんな辺りでしょうか。

if (以下 1700 カラム)
    v = 1;
else if (以下 1300 カラム)
    v = 2;
else if (以下 600 カラム)
    v = 3;

……なんてのは、見たことがありますね。switch にできる単一変数評価で。
switch でもブチキレそうですが。

後はこんなのとかも。

class Foo {
public:
    char* p;
    void set_str(char* s) { strcpy(p, s); }
...

いずれも自称 C++ マスターさんの書かれたコードでしたが、何をどんな風にマスターしたのやら……。

キレイな結晶ができるんですね。

main()千行程度でwhileとgo toでプログラム作ってたのなら。
ここまでくるとむしろ若干感動した。

………いや、言ってみただけ。
汚い可読性の悪いけどバグなしのコードと可読性は高いけどバグありのコードのどちらがいいコードなんだ？

スパゲッティ大好きなのでAC

タブでインデントするのやめろよ！
どうせコード階層以外のとこでタブ使ってめちゃくちゃになるんだ！

一度しか通らない道だから実行結果に問題はないんだけど、
コードの途中に突然変数の宣言が現れてその変数を一時的に使ってたりとかなら・・・。
あとプロシージャの中でGotoとGosubとか日常的に使いまくりだったりとか・・・。

でもちゃんと動くからいいじゃない。
人に見せるわけでもないしいいよね。
どうせ見るのは自分だし・・・。
的な思いからそのままになってます。

だからといって、do - whileで囲んで、
breakでループ外に飛ばすような小技を使わなくても…。

＃最初1分ぐらい、何がしたいのかわからなかった

Javaの携帯アプリなんですが、

・クラスがエントリポイントを含むクラスとMainCanvasクラスのみ
・MainCanvasクラス一つのコードで一万三千行
・一つのメソッドで二千行
・フィールドの数は500個以上
・enum型代わりにstatic finalが宣言されているにもかかわらず数字直書き
・ほぼ同じコードが100回コピペされて並んでる
・フィールド/メソッド名が頭文字で短縮されており何の略なのかさっぱり
・数回の改造を経ているらしく命名法が三種類
・フィールドの使い方を途中から変えたらしく、命名法の役割とかけ離れている
・コメントはほぼ皆無。たまにあるコメントは日本語/英語/中国語の三種類で書かれる
・Javaなのに外部ツールで#define, #ifdefなどを使用している

などです。正直書き始めたらきりがありません。

# なにより怖いのは、これが目下移植中のコードだということ・・・

偏執的に一行ごとにエラーチェックと例外脱出が必要なケースで

　#define ERROR_OUT(e, g) do { push_error(__FILE__, __FUNCTION__, __LINE__, e); goto g; } while(0)

とかするのは別におかしくないと思うんだけどな。

自分だったら関数そのものをラップする形にして

　#define TRY(expr, tag) do { int rc; if ((rc = (expr)) != OK) { push_error(..., rc); goto tag; } while (0)

として

　 TRY((do_something()), fatal);
    　TRY((do_something_else()), fatal);
        return OK;
    fatal:
        task_abort();

とかまとめてしまうかも。

// 2009.11.1 modify start
#if 0
int a;
a = 0;
#endif
int a = 0;
// 2009.11.6 modify start
//int a = 1;
long a = 1;
// 2009.11.6 modify end
// 2009.11.1 modify end

みたいな変更履歴が7割を占めるコードがありました。
こんなのがコーディング基準で決まってるんですよね。
結構あちこちで採用されてたみたいなんですけど、はっきり言って読めたもんじゃない。

全文コメントアウトして、下に履歴削除したコードを書き直したのでAC :-P

客先に常駐すると・・・

main()
{
  funcA();
  funcB();
  funcC();
}

int funcA()
{

  /* TBD */

}

int funcB()
{

  /* TBD */

}

int funcC()
{

  /* TBD */

}

しかも戻り無し。

絶対「OpenSSLが書けるサルなら我が社は大歓迎だ！そのサルを連れてきてくれ！」があると思ったのに。