Ticket #564 (assigned 改善提案) — at Version 10
PEAR::DB から PEAR::MDB2 or ADOdb への変更
| Reported by: | nanasess | Owned by: | nanasess |
|---|---|---|---|
| Priority: | 高 | Milestone: | EC-CUBE2.11.0 |
| Component: | その他 | Version: | コミュニティ (eccube-comu) |
| Keywords: | Cc: | ||
| 修正済み: |
Description (last modified by nanasess) (diff)
現在, DB の抽象化ライブラリとして PEAR::DB を使用しているが, 以下の問題がある.
- PEAR::DB は PHP ライセンスであり, GPL と矛盾する
- もうメンテナンスされてない
上記の理由により PEAR::MDB2 若しくは ADOdb へ変更する
導入要件
- GPL と矛盾しないライセンスで配布されていること
- Sequence 取得の抽象化メソッドがあること
- Sequence に関して PostgreSQL 及び MySQL を同一のコードベースで記述可能なこと
- データベースのメタデータを扱うメソッドがあること
- 活発にメンテナンスされていること
利点
- PEAR::MDB2
- 2.5.x では, PDO 用のドライバも有り
- エラーハンドリングを, 他の PEAR パッケージと統一可能
- ADOdb
- 速いというウワサ
- キャッシュ機構あり
- adodb-errorpear.inc.php を include すれば, PEAR::Error も使用可能
- adodb-errorhandler.inc.php で, 任意のエラー定数で trigger_error を使用可能
欠点
- PEAR::MDB2
- PHP の include_path に, 他の PEAR クラスが存在すると, 関数の多重定義エラーになる
- EC-CUBE のエラーハンドリングが貧弱なため, DBエラーが発生すると無限ループする
- ADOdb
- DB::last_query に相当する機能が無い(EC-CUBE 本体では未使用)
- adodb-errorpear.inc.php を使用した場合, 出力するエラー文字列が膨大になり, memory_limit を越える場合がある
その他
- DBエラーが発生した場合, PHP エラーではなく EC-CUBE のエラー画面を表示するのが望ましい
参考
Change History
comment:1 Changed 16 years ago by nanasess
- Component changed from フロント to その他
- Description modified (diff)
- Summary changed from PEAR::DB から PEAR::MDB2 への変更 to PEAR::DB から PEAR::MDB2 or ADOdb への変更
comment:2 Changed 16 years ago by nanasess
- Status changed from new to assigned
- Description modified (diff)
現状は, どちらも導入要件を満たしていそうなので, サンプルコードを書いてみることにします.
comment:4 Changed 16 years ago by nanasess
- Description modified (diff)
PEAR::DB から PEAR::MDB2 へ移行のための参考サイト追加.
現状は, PEAR::DB への依存が激しすぎるので MDB2 の方が妥当かも...
comment:6 Changed 16 years ago by nanasess
- Description modified (diff)
説明を更新.
PEAR::MDB2 の欠点の影響が強すぎ, 簡単に動いてくれないので, ADOdb を使用する方向で検討.
comment:10 Changed 16 years ago by nanasess
- Description modified (diff)
エラー表示に関する説明を追記
PEAR::DB 版 SC_DbConn は下記のような実装になっている
- DB 接続エラーの場合
- SC_DbConn::getAll() 関数の場合のみ 0 を返し, 「DB接続に失敗しました」という文言を表示する. その他の関数は SC_DbConn::send_err_mail() 関数を呼び exit する
- SQL エラーの場合
- SC_DbConn::send_err_mail() 関数を呼び exit する
PEAR::DB 版 SC_DbConn の実装の問題点は, 接続エラーの場合は文言が表示されるものの, それ以外の場合は真っ白な画面になってしまう.
また, エラー画面を表示しようとしても, 表示するまでに DB 接続を行うため, 多重でエラーが発生する.
本来であれば, DB 関連のエラーはエラーコードを付与して EC-CUBE のエラー画面を表示するのが望ましい.
Note: See
TracTickets for help on using
tickets.
