回答編集履歴

1

改善点の追加

2016/02/27 02:47

投稿

shi_ue
shi_ue

スコア4437

test CHANGED
@@ -15,3 +15,67 @@
15
15
  $stmt->execute();
16
16
 
17
17
  ```
18
+
19
+
20
+
21
+ さらに、
22
+
23
+ ```PHP
24
+
25
+ // Loginボタンが押されたら
26
+
27
+ if (isset($_POST['Login'])) {
28
+
29
+ // エラー
30
+
31
+ if (empty($_POST['name'])) {
32
+
33
+ $errorMessage = "名前が無いです";
34
+
35
+ } else if (empty($_POST['password'])) {
36
+
37
+ $errorMessage = "パスワードが無いです";
38
+
39
+ }
40
+
41
+
42
+
43
+ // エラーが無かったら
44
+
45
+ if (!empty($_POST['name']) && !empty($_POST['password'])) {
46
+
47
+ ```
48
+
49
+ ここは、
50
+
51
+ ```PHP
52
+
53
+ // Loginボタンが押されたら
54
+
55
+ if (isset($_POST['Login'])) {
56
+
57
+ $errorMessage = '';
58
+
59
+ // エラー
60
+
61
+ if (empty($_POST['name'])) {
62
+
63
+ $errorMessage = "名前が無いです";
64
+
65
+ } else if (empty($_POST['password'])) {
66
+
67
+ $errorMessage = "パスワードが無いです";
68
+
69
+ }
70
+
71
+
72
+
73
+ // エラーが無かったら
74
+
75
+ if (empty($errorMessage)) {
76
+
77
+ ```
78
+
79
+ としましょう。変数は必ず初期化するように癖をつけてください。
80
+
81
+ また、同じような条件文が出てくる場合は、他と統合できないか、もっと簡単な方法はないか探しましょう。