パスワードを忘れた? アカウント作成
この議論は賞味期限が切れたので、アーカイブ化されています。 新たにコメントを付けることはできません。

#俺が見たクソコード選手権、Twitterで盛り上がる」記事へのコメント

  • クソコードというかクソコーディング規約ですが。
    本気でどうしようかと思った。

    #関数名が通し番号とかじゃないだけマシだったと思うほかない

    • 【1関数50ライン、1モジュール500ライン、1ライン80カラム】
      これでもProCの規約だったりする。
      ほぼ全てのモジュールでカラム数以外超えまくりでしたがね。
      • by Anonymous Coward on 2012年04月03日 20時48分 (#2129015)

        >【1関数50ライン、1モジュール500ライン、1ライン80カラム】
        杓子定規に従うのはまぁ、なんだと思うが、言っていることは割とまともだと思うが。
        これらを越えると、正直読みづらくて頭に入らない。

        親コメント
        • by wolf03 (39616) on 2012年04月03日 21時29分 (#2129062) 日記
          いやいや、面倒なコーディングが非常に多いかつ、変数の命名規約で変数名もかなり長めなんですよ。
          で、自分が担当した物で一番小さかった関数でも100ラインを軽く超え、複数行にまたがっての記述
          だらけになり、見た目の行数は非常に多くなるんですよ。
          親コメント
          • by Anonymous Coward

            まあ3000行くらいのループが3つくらい入っているswitch文で、
            1万行くらいの関数があるよりマシですね。

            1メソッドなんて20行もあれば十分でしょう。>Smalltalk
            50行もあったら、コードレビュー以前の問題でしょう。

          • Pro*C書いたことは無いですが見たことはあります。
            あの言語は、SQLとかプログラム中に普通に登場するから行数が増えるってことですよね?

            普通の言語(JavaにしろPHPにしろ)なら真っ当なコーディング規約でも、それを違う言語にそのまま持ってくると糞規約に早変わり、というのは有り勝ちなことです。
            どちらかというと、COBOLとかCとかの規約をJavaやらに持ってこられて泣かされるケースの方が多いですが・・・。

            • ついでに、データ保持用の変数とホスト変数は必ず分ける規約で、数値なら代入、文字ならstrncpyが大量に・・・。
              初期値に1を入れる数値変数であっても、必ず0で初期化を入れろとか・・・。

              各関数に入れるお決まりのロジックだけで10行超えるのに、これ入れて50行でしたし。

              #変数名・関数名の命名規則はまともだったのになぁ・・・
              親コメント
          • by Anonymous Coward

            50行 -> 25Line x2
            80カラム -> 半角80文字

            もしかして、dos時代に作られた規約なんじゃ....

開いた括弧は必ず閉じる -- あるプログラマー

処理中...