回答編集履歴

2

加筆

2021/03/04 09:55

投稿

退会済みユーザー
test CHANGED
@@ -51,3 +51,53 @@
51
51
  ```
52
52
 
53
53
  で、search()に該当する処理ももし長く感じるなら区切ってもいいかもしれませんし。
54
+
55
+
56
+
57
+ 関数として切り離すと、変数のスコープの影響も出てくるため、
58
+
59
+ 必要に応じて関数の引数として与えることも忘れずに。
60
+
61
+
62
+
63
+ あと、この箇所は私ならこう直す。
64
+
65
+
66
+
67
+ ```php
68
+
69
+ $sql = 'SELECT product.id, name, price, img, status, stock
70
+
71
+ FROM product
72
+
73
+ JOIN item_stock
74
+
75
+ ON product.id = item_stock.stock_id
76
+
77
+ WHERE name like ?';
78
+
79
+ ```
80
+
81
+
82
+
83
+ ```php
84
+
85
+ $sql = <<<EOT
86
+
87
+ SELECT product.id, name, price, img, status, stock
88
+
89
+ FROM product
90
+
91
+ JOIN item_stock ON product.id = item_stock.stock_id
92
+
93
+ WHERE name like ?
94
+
95
+ EOT;
96
+
97
+ ```
98
+
99
+ [ヒアドキュメント構文](https://www.php.net/manual/ja/language.types.string.php#language.types.string.syntax.heredoc)と言います。
100
+
101
+ 変更前の`$sql`には、インデントしたスペースやタブコードも混ざってしまっちゃうので。
102
+
103
+ 別に混ざっていても動くことは動くんですが、全角空白の紛れ込みの予防にもなるので。

1

見直し

2021/03/04 09:55

投稿

退会済みユーザー
test CHANGED
@@ -7,6 +7,26 @@
7
7
  ```php
8
8
 
9
9
  <?php
10
+
11
+
12
+
13
+ function search()
14
+
15
+ {
16
+
17
+ // ~
18
+
19
+ }
20
+
21
+
22
+
23
+ function list()
24
+
25
+ {
26
+
27
+ // ~
28
+
29
+ }
10
30
 
11
31
 
12
32
 
@@ -28,22 +48,6 @@
28
48
 
29
49
 
30
50
 
31
- function search()
51
+ ```
32
52
 
33
- {
34
-
35
- // ~
36
-
37
- }
38
-
39
-
40
-
41
- function list()
53
+ で、search()に該当する処理ももし長く感じるなら区切ってもいいかもしれませんし。
42
-
43
- {
44
-
45
- // ~
46
-
47
- }
48
-
49
- ```