👇コードレビューやアドバイスを投稿してみよう! (要ログイン)
PHP

結婚式の2次会フォーム

PHPも投稿してみる

1
2019-08-19 11:11:58
コードレビュー一覧 (3)
0

勝手に改行された疑惑?

0

改行は元ファイルが悪ござんした・・・。

0
  • 恐らくこの後に通常のHTMLが続くものと思われるので、その場合die()は不適切。(その場で処理を打ち切るので、その後のHTMLが出力されなくなる・・・ハズ。)
  • is_successは最初はfalseにしておき、成功時にtrueにするように修正。(こうしないと、他のエラー処理を挿入するような修正が入った時のフラグ下ろし忘れによる不具合の原因になる。)
  • 処理に失敗した時、executeの失敗なのかExceptionの発生なのかが区別できずデバッグし辛いので「お手数ですが~」の前に区別用の文言を追加。(ついでに、成功時は末尾に改行が含まれているのに失敗時は改行されていなかったので改行するように統一。)