編集履歴

回答編集履歴

修正

2019/03/28 08:04

投稿

スコア80850

test CHANGED Viewed

@@ -366,6 +366,8 @@
 ```
 ３:
 ```php
@@ -394,6 +396,12 @@
 ```
+正常な場合のみにフォーカスを当てがちでですが、異常な方がプログラムとしては大事です。
+先に異常なほうを書きましょう。正常なほうは後回しでも問題ないです。
 入力チェックの処理を外だしして関数呼び出してメインの処理はエラーメッセージの有無だけにしたほうがスッキリします。
 「本筋の処理と直接関係がないのなら外出し」「同じようなコードは変数で分岐かけて共通化」が基本です。

修正

2019/03/28 08:04

投稿

スコア80850

test CHANGED Viewed

@@ -300,7 +300,9 @@
 ５．ifのスコープが長すぎる
-フォーマットかけたらよく分かると思います。特にelseが長いのは致命的です。可読性が低く、それはメンテナンス性の低下も招きます。
+フォーマットかけたらよく分かると思います。特に短い方が後に来るコードは読みづらいです。
+可読性が低く、それはメンテナンス性の低下も招きます。
 elseを使うにしても以下のどちらが読みやすいか考えてみてください。

修正

2019/03/28 08:02

投稿

スコア80850

test CHANGED Viewed

@@ -395,3 +395,33 @@
 入力チェックの処理を外だしして関数呼び出してメインの処理はエラーメッセージの有無だけにしたほうがスッキリします。
 「本筋の処理と直接関係がないのなら外出し」「同じようなコードは変数で分岐かけて共通化」が基本です。
+６：PDOExceptionほぼ握りつぶしてる
+```php
+            } catch (PDOException $e) {
+                $errorMessage = header("Location: 500.php");
+            }
+```
+何のために例外捕捉するようにしたのか。これでは何か異常を検知したときに原因を調べられない。
+セキュリティ観点から画面に出力する必要はないので、せめてログに出力してください。オブジェクトなので$eをjson_encode()かけたものでもいいから。
+- [本当にあった怖い(Javaアンチパターンの)話:例外の抹殺](https://qiita.com/Shrimpman/items/56bfbfc44454e3096da5#no3-%E4%BE%8B%E5%A4%96%E3%81%AE%E6%8A%B9%E6%AE%BA)
+- [「知らない」では怖い例外処理](https://www.techscore.com/blog/2016/12/23/%E3%80%8C%E7%9F%A5%E3%82%89%E3%81%AA%E3%81%84%E3%80%8D%E3%81%A7%E3%81%AF%E6%80%96%E3%81%84%E4%BE%8B%E5%A4%96%E5%87%A6%E7%90%86/)
+- [
+例外処理について](https://donow.jp/skillup/?p=291)
+※Javaの記事ばかりですが考え方は同じ。握りつぶしちゃダメです。何のための例外をキャッチさせてるのか考えましょう。

修正

2019/03/28 07:52

投稿

スコア80850

test CHANGED Viewed

@@ -302,6 +302,96 @@
 フォーマットかけたらよく分かると思います。特にelseが長いのは致命的です。可読性が低く、それはメンテナンス性の低下も招きます。
+elseを使うにしても以下のどちらが読みやすいか考えてみてください。
+1:
+```php
+if($test){
+  /*
+   長い処理
+  */
+}else{
+  echo 'error!';
+}
+```
+2:
+```php
+if(!$test){
+  echo 'error!';
+}else{
+  /*
+   長い処理
+  */
+}
+```
+３:
+```php
+if(!$test){
+  die('error!'); //関数ならここでreturn
+}
+/*
+   長い処理
+*/
+```
 入力チェックの処理を外だしして関数呼び出してメインの処理はエラーメッセージの有無だけにしたほうがスッキリします。
 「本筋の処理と直接関係がないのなら外出し」「同じようなコードは変数で分岐かけて共通化」が基本です。

修正

2019/03/28 07:41

投稿

スコア80850

test CHANGED Viewed

@@ -295,3 +295,13 @@
 パスワード何桁とかわざわざチェックするのは悪意あるユーザーのヒントになります。
 どっちも必須チェックくらいにとどめるべき。
+５．ifのスコープが長すぎる
+フォーマットかけたらよく分かると思います。特にelseが長いのは致命的です。可読性が低く、それはメンテナンス性の低下も招きます。
+入力チェックの処理を外だしして関数呼び出してメインの処理はエラーメッセージの有無だけにしたほうがスッキリします。
+「本筋の処理と直接関係がないのなら外出し」「同じようなコードは変数で分岐かけて共通化」が基本です。