最近すっかりWord/Excel/Visio/Outlookがお友達で、自分でコーディングする時間が取れなくなりました…悲しい。頼んだ人のプログラムをレビューするんですが、もう少しがんばれと。
Cで変数の初期化をしないのに、条件判定で有効かどうか判定しているとか。
HANDLE hHandle; if(hoge == 0) { goto End; } hHandle = CreateFile(...); End: if(hHandle) { CloseHandle(hHandle); }
条件を一個足してとお願いすると、まるまるコピーしてくれるとか。
//修正前 char strLog[200]; sprintf(strLog, "%s %s ...", foo,bar); WriteFile(....);
//修正後 if(flag == 0){ char strLog[200]; sprintf(strLog, "%s %s ...", foo,bar); WriteFile(....); } else { char strLog[200]; sprintf(strLog, "%s %s ...", foo,bar); WriteFile(....); }
いやいや…どうも「頼まれたところできっちりと切り出して責任分担する」という考えがあるのか知りませんが、オフショアで頼んでもこういう傾向がしばしばみかけられます。私は直させますけどね。