2

目前，我有一个名为 的数据库访问类DB，它使用PDO. 然后我有一些子类来访问每个表：

  • Users
  • Images
  • Galleries
  • Text
  • Videos

当我第一次开始我的项目时，这很好，但现在我不确定它是否那么好，因为我在每个类中使用的每个数据库查询都有一个方法。例如：

Images::insertNew($filename)
Images::getTotalInGallery($galleryId)
Images::getAllInGallery($galleryId, $fetchStyle)
Images::updateDescription($imageId, $description)
Images::updateGallery($imageId, $galleryId, $orderNum)
Images::getSingle($imageId)
Images::getFilename($imageId)
Images::getImageIdByFilename($filename)


Galleries::getNameById($galleryId)
Galleries::getAll()
Galleries::getMaxImages($galleryId)
Galleries::checkIfExists($galleryId)
Galleries::createNew($galleryName)
Galleries::getById($galleryId)
Galleries::delete($galleryId)

反正你懂这个意思。随着对它们的需求出现，我一直在添加这些方法，并且在开发中，我首先使用DB类：

//Execute a query
DB::query($query);

//Get a single row
$row = DB::getSingleRow($query);

//Get multiple rows
$rows = DB::getMultipleRows($query);

所以，我用我的 DB 类测试查询，然后当它们工作时，我将它们包装在与之相关的类的方法中（图像表的图像类，画廊表的画廊类等）。

我觉得这会随着我稍后添加新功能而不断增长（这可能没问题，但我不确定）。有人可以批评我的方法和/或提供替代解决方案吗？

谢谢！

4

1 回答 1

1

不，这实际上听起来不错。您似乎在业务逻辑和数据访问之间有一个坚实的抽象层（这称为数据映射器模式）。

这个变大的唯一问题是，您最终可能会使用重叠的方法。您还应该尝试在两个类之间保持标准命名约定。

在 Images 中，方法是 Images::insertNew，在 Galleries 中是 Galleries:createNew。

你真的有模特吗？因为看起来你有很多查询来组合单个值，而不是整个对象。

于 2010-08-04T16:06:05.480 回答