Francisの日記: ネスト10段の恐怖
日記 by
Francis
本日は愚痴を吐きます。
たのむから、作業領域の初期化とメモリ確保の確認だけで7段もネストしないでください。殺すよ?
ふつークラス化して2段初期化関数で、メモリ確保できたかどうか一発確認でしょー。
それに、びっくりするようなデータの参照の仕方してるし。
記憶力のいい奴ががんばって書いちゃったプログラムぐらい手におえないものはありません。
コメントは1行も書いてないしね。趣味プロじゃないんだからさー。
あんな珍妙なソートの仕方、私始めて見ましたよ。
やたら
GetParent()->myinstanse.flag=3;
みたいな使い方もやめて。1つや2つならいいけど10回も20回も呼びやがって、しまいには2段でよんでる。GetParent()->GetParent()->m_size=100;とか
どうせ使い方が.DoModal()なんだから、子供ダイアログ作るときにデータ渡してよ!終了してから引きづり出しなさいよ!
プライベートにしてゲッタ・セッタとおしてね、とかそういう問題以前よコレ。
珍妙な書き方、最初は速度向上とかループ処理中のメモリ確保回避とか狙ってるのかと思ったけど、
「明らかに無駄に見えるループだけど実は処理が進む謎めいたループ」とか、
「インスタンス生成を回避しているように見えて影でバリバリnewしてる」とか。
もうね、
私も大したプログラマじゃないけどね、修行しなおして来なさいあなた。
ネスト10段の恐怖 More ログイン