回答編集履歴
5
調整
answer
CHANGED
|
@@ -13,8 +13,12 @@
|
|
|
13
13
|
PDO::queryはPDOが持っているので。
|
|
14
14
|
それだけラッパーしたとしても記述が冗長になるだけでなく、newが二度発生するのでむしろ余計なコストが発生します。
|
|
15
15
|
そのコストを補ってあまりある恩恵を受けられる設計を考えられるのであれば是非やってみてください。
|
|
16
|
+
きちんとやればできますし、ちょっとしたフレームワークくらいのボリュームにはなります。
|
|
16
|
-
現状では無駄なコストだけが悪目立ちする言わば「使えないクソコード」です。
|
|
17
|
+
ただ現状では無駄なコストだけが悪目立ちする言わば「使えないクソコード」です。
|
|
17
18
|
なにも考えてないというのが手に取るようにわかる実装です。
|
|
19
|
+
再利用性など加味されているか？使いやすいか？汎用性はあるか？
|
|
20
|
+
など、オブジェクト指向の利点がなにも活かされていません。
|
|
21
|
+
これならPDOそのままと全くかわりません。
|
|
18
22
|
|
|
19
23
|
オブジェクト指向をもっときちんと学んで使いこなさないことにはあなたが作ったであろうDbObjectというクラスはただのお荷物のままになるでしょう。
|
|
20
24
|
|
4
コストの件を勘違いしている人のための説明追記
answer
CHANGED
|
@@ -14,7 +14,7 @@
|
|
|
14
14
|
それだけラッパーしたとしても記述が冗長になるだけでなく、newが二度発生するのでむしろ余計なコストが発生します。
|
|
15
15
|
そのコストを補ってあまりある恩恵を受けられる設計を考えられるのであれば是非やってみてください。
|
|
16
16
|
現状では無駄なコストだけが悪目立ちする言わば「使えないクソコード」です。
|
|
17
|
-
なにも考えてないという
|
|
17
|
+
なにも考えてないというのが手に取るようにわかる実装です。
|
|
18
18
|
|
|
19
19
|
オブジェクト指向をもっときちんと学んで使いこなさないことにはあなたが作ったであろうDbObjectというクラスはただのお荷物のままになるでしょう。
|
|
20
20
|
|
3
調整
answer
CHANGED
|
@@ -12,6 +12,9 @@
|
|
|
12
12
|
その使い方だとDbObjectというクラスの存在はほぼ無意味です。
|
|
13
13
|
PDO::queryはPDOが持っているので。
|
|
14
14
|
それだけラッパーしたとしても記述が冗長になるだけでなく、newが二度発生するのでむしろ余計なコストが発生します。
|
|
15
|
+
そのコストを補ってあまりある恩恵を受けられる設計を考えられるのであれば是非やってみてください。
|
|
16
|
+
現状では無駄なコストだけが悪目立ちする言わば「使えないクソコード」です。
|
|
17
|
+
なにも考えてないという意味ですね。
|
|
15
18
|
|
|
16
19
|
オブジェクト指向をもっときちんと学んで使いこなさないことにはあなたが作ったであろうDbObjectというクラスはただのお荷物のままになるでしょう。
|
|
17
20
|
|
2
調整
answer
CHANGED
|
@@ -13,4 +13,12 @@
|
|
|
13
13
|
PDO::queryはPDOが持っているので。
|
|
14
14
|
それだけラッパーしたとしても記述が冗長になるだけでなく、newが二度発生するのでむしろ余計なコストが発生します。
|
|
15
15
|
|
|
16
|
-
オブジェクト指向をもっときちんと学んで使いこなさないことにはあなたが作ったであろうDbObjectというクラスはただのお荷物のままになるでしょう。
|
|
16
|
+
オブジェクト指向をもっときちんと学んで使いこなさないことにはあなたが作ったであろうDbObjectというクラスはただのお荷物のままになるでしょう。
|
|
17
|
+
|
|
18
|
+
ちなみに気になったのでコメント拾います
|
|
19
|
+
|
|
20
|
+
> 業務で完成されたものばかり触っていたので、忘れてしまいました。
|
|
21
|
+
|
|
22
|
+
忘れたとかそういうレベルのコード書いてません。
|
|
23
|
+
基本です。その程度分からないのに元々知っていたかのような口ぶりは笑止千万。
|
|
24
|
+
PHPマニュアルすら確認しないで何を言ってるんでしょうか。
|
1
調整
answer
CHANGED
|
@@ -4,7 +4,7 @@
|
|
|
4
4
|
> __construct ([ mixed $args = "" [, $... ]] ) : void
|
|
5
5
|
|
|
6
6
|
返却値 voidですのでreturn書いたところでその情報は返しません。
|
|
7
|
-
newの結果、つまりオブジェクトです。
|
|
7
|
+
newの結果、つまりDbObjectオブジェクトです。
|
|
8
8
|
|
|
9
9
|
|
|
10
10
|
コンストラクタでDB接続して保持したいのでしたらオブジェクトのメンバー変数にPDOをnewした結果を保持させておく必要があります。
|