ぱっと見て「ヘタだなぁ」と思うコード その7
■ このスレッドは過去ログ倉庫に格納されています
0768デフォルトの名無しさん
2007/11/04(日) 21:24:55switch文にしろif文にしろそうだけど。
数千行にも及ぶのはカンベンして欲しい。
tokenの種類に応じた、分割統治とかそういうコーディングを
おぼえて欲しい。どうしても分割統治しづらいなら、
せめて処理を関数に分けて以下のようにしてくれ。
関数呼び出しのオーバヘッドが気になるなら、inlineに
でもすればい。
if ((handler = GetHandler(token)) == NULL) {
return ERROR;
}
ret = handler(args);
■ このスレッドは過去ログ倉庫に格納されています