>>771
if (デバッグモード) {
// DB書き込みをしない
}
else {
// DB書き込みをする
}
という形にすればいいよね。とだけ言うと詭弁に思われるかもしれないから、
if (デバッグモード === false) {
// DB書き込みをする
}
というブロックが唐突に現れるより意図が明確だと思うんだけどどうかな。
else { DB書き込みをしない }ってつけるのはもっと不自然だと思う。

Exceptionの件については例外ブロックの作り方がおかしいような気がするけれど、
こちらの考えが及んでないだけならごめん。