機能要望 > その他 > SC_DbConn を削除しました |
その他
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | 下へ |
投稿者 | スレッド |
---|---|
seasoft |
投稿日時: 2010/8/12 13:54
対応状況: −−−
|
神 登録日: 2008/6/4 居住地: 投稿: 7367 |
Re: SC_DbConn を削除しました 引用:
「SC_Utils の使用は比較的限定されている」という前提でしたら、「SC_Query や SC_DB_DBFactory の中から使用する SC_Utils のメソッドは、他の EC-CUBE クラスに依存させない」といった方針はいかがでしょうか?
|
nanasess |
投稿日時: 2010/8/12 15:46
対応状況: −−−
|
神 登録日: 2006/9/9 居住地: 投稿: 2313 |
Re: SC_DbConn を削除しました 引用:
賛成です. ゆくゆくは, SC_Utils は他のクラスに依存せず, できるだけ単独で動作することを目標にしましょう! |
nanasess |
投稿日時: 2010/8/12 20:07
対応状況: −−−
|
神 登録日: 2006/9/9 居住地: 投稿: 2313 |
Re: SC_DbConn を削除しました 引用:
1番目の対応も含め, SC_Query を singleton にして, 異常な循環参照を防ぐよう対応してみました. http://svn.ec-cube.net/open_trac/changeset/18784 SC_DB_DBFactory_Ex::getInstance(); で SC_Query のインスタンスを渡すアイディアは, Ex のインスタンスで SC_Query のインスタンスを参照できないため断念しました. その代わり, SC_Query::getSingletonInstance() で, SC_Query のインスタンスを Singleton で取得できるようにし, 無駄に SC_Query のインスタンスを生成しないようにしてみました. その他, PHPDoc コメントを書いたり, setXXX 関数で Chain of Responsibility パターンをサポートしてみたりしましたので, 興味のある方は是非試してみてください! |
« 1 (2) |
スレッド表示 | 新しいものから | 前のトピック | 次のトピック | トップ |