teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

3

見直し

2021/06/03 02:37

投稿

退会済みユーザー
answer CHANGED
@@ -27,4 +27,9 @@
27
27
  if($hoge3 ){'AND date=:date';
28
28
  }
29
29
  ```
30
- 何がしたいの? 文字列リテラルだけ置いても意味がない。
30
+ 何がしたいの? 文字列リテラルだけ置いても意味がない。
31
+
32
+ ダメ出しばかりしているのは、仕様や設計に関わる情報が質問文中にないために、
33
+ どう改善すればいいかを考える段階にないってことです。
34
+ 思考整理中に試しに実行したコードのまま、動作確認すら取ってないのかなって思えるほど、
35
+ phpコードからやりたいことが読み取れないので。

2

加筆

2021/06/03 02:37

投稿

退会済みユーザー
answer CHANGED
@@ -15,4 +15,16 @@
15
15
  // 条件が一つでも入力されていれば検索をかけれるようにする
16
16
  }elseif($sql=$pdo->query('SELECT * FROM items WHERE 1=1'));{
17
17
  ```
18
- elseifのところ、おかしいよな?
18
+ elseifのところ、おかしいよな?
19
+
20
+ ---
21
+
22
+ ```php
23
+ if($hoge1 ){'AND number=:number';
24
+ }
25
+ if($hoge2 ){'AND name=:name';
26
+ }
27
+ if($hoge3 ){'AND date=:date';
28
+ }
29
+ ```
30
+ 何がしたいの? 文字列リテラルだけ置いても意味がない。

1

加筆

2021/06/03 02:33

投稿

退会済みユーザー
answer CHANGED
@@ -5,4 +5,14 @@
5
5
  ```php
6
6
  if(empty($hoge1) && empty($hoge2) && empty($hoge3))
7
7
  ```
8
- くらいはパット見でわかった。
8
+ くらいはパット見でわかった。
9
+
10
+ ---
11
+
12
+ ```php
13
+ if(empty($hoge1 && empty($hoge2) && empty($hoge3)){ // 初期状態を空白
14
+ $sql = $pdo->query('SELECT * FROM items where id=0');
15
+ // 条件が一つでも入力されていれば検索をかけれるようにする
16
+ }elseif($sql=$pdo->query('SELECT * FROM items WHERE 1=1'));{
17
+ ```
18
+ elseifのところ、おかしいよな?