ヘタなコードの書き方

ヘタなコードの書き方 ..

697:693
08/08/07 01:09:32
>>694
その辺は彼なりになんかあったのかも。
まぁ、そもそもまともな思考があれば、このマクロが及ぼす弊害は簡単に想像つくわけで。

>>695
多分大丈夫だったと思う。しかしこれは氷山の一角で、
そのLOOP_CANCELERが本当に役に立ってるんじゃないかと信じてしまいたくなるほど
壊滅的なコードだった。ベテランも含め誰もが直接触るのは避け、
山ほどバグを出し、それらは場渡り的に修正され、そのままリリースされた。

698:デフォルトの名無しさん
08/09/24 22:26:25
void function()
{

for (int i = 0; i < HOGEMAX; i++)
{
.
.
fuga:
.
.
}

699:デフォルトの名無しさん
08/09/24 22:27:12
途中で投入してしまった。

void function()
{
・
・
goto fuga;
・
・
}

700:デフォルトの名無しさん
08/10/23 01:44:39
今仕事でメンテしてるのはコメントが疑問形で書いてあって辛い
//このループでXXXがXXXXとなった場合のXXXXをしているのか？

か？って言われても…

701:デフォルトの名無しさん
08/10/25 00:29:53
// はい、やってます。

って、書いとけ。

702:デフォルトの名無しさん
08/10/25 01:00:19
しかし、実際本当にわからないのがヘタコードクオリティ

703:デフォルトの名無しさん
08/10/25 01:25:41
引き継いだコードがマジックナンバーの嵐だった…

704:デフォルトの名無しさん
08/10/25 03:59:33
何か問題あるの？

705:デフォルトの名無しさん
08/11/05 08:45:35
全ての行にコメント書いてます

706:デフォルトの名無しさん
09/01/31 03:07:45
for(int i=0;i<max;i++)
{
　　　HOGE hoge = new HOGE();
}

707:デフォルトの名無しさん
09/01/31 11:23:32
>>706
＊がないからＣ＃か？
だったら問題ないか？

708:デフォルトの名無しさん
09/01/31 11:36:09
ヘタってのは、セキュアじゃないとか、リークするとかとはまた別の要素なんだな。
なんというか、正しい動作はするんだけど素人臭いというか、そんな感じ。

709:デフォルトの名無しさん
09/01/31 19:00:28
無駄なコメント

710:デフォルトの名無しさん
09/02/02 19:43:16
// これはコメントです

711:デフォルトの名無しさん
09/02/07 07:00:28
a = 0; // aにゼロを代入する

712:デフォルトの名無しさん
09/02/08 00:15:27
// 2009/02/08 変更開始
//a = 0; // aにゼロを代入する
a = 1; // aにゼロを代入する
// 2009/02/08 変更終了

713:デフォルトの名無しさん
09/02/14 16:19:32
procedure unko{
ittekuru();
}