Question

我正在使用使用单例模式用PHP编写的CRUD类。这是我的CRUD文件和我的连接文件。

问题是：当我使用SELECT时（不需要特定的表选择），我可以选择多少个选择。像这样：

$pdo        = Connection::getInstance();
$crud       = Crud::getInstance($pdo);
# ----------------------------------------
$sql        = "SELECT * FROM images WHERE prod_id = ?";
$arrayParam = array($prod_id);
$data_img   = $crud->getSQLGeneric($sql, $arrayParam, true);

但是当我需要删除，插入或更新时，我必须将表设置在CRUD上，就像这样删除：

$pdo        = Connection::getInstance();
$crud       = Crud::getInstance($pdo,'images');
# ----------------------------------------
$arrayImg   = array('img_id=' => $img_id);
$return     = $crud->delete($arrayImg);

我无法一次执行这两个语句。假设我需要在同一代码块中进行插入和删除操作，但它只运行其中之一。

这时我陷入了困境，我的代码必须在DB中找到具有产品ID的所有图像，将其删除一个链接，然后再取消链接文件夹中的文件，然后从表中删除该产品。

$prod_id    = $_GET['prod'];
# -----
$pdo        = Connection::getInstance();
# ----------------------------------------
$crud       = Crud::getInstance($pdo);
# -----
$sql        = "SELECT * FROM images WHERE prod_id = ?";
$arrayParam = array($prod_id);
$data_img   = $crud->getSQLGeneric($sql, $arrayParam, true);
# -----
foreach($data_img as $img_info)
{
  unlink('../../img/'.$img_info->img_name);
  $arrayImg = array('img_id=' => $img_info->img_id);
  $return2  = $crud->delete($arrayImg);
}
# ----------------------------------------
$crud       = Crud::getInstance($pdo,'products');
# -----
$arrayDel   = array('prod_id=' => $prod_id);
$return     = $crud->delete($arrayDel);
# ----------------------------------------
echo 'Deleted';

有什么办法可以做到这一点？是正确的方法吗？

欢迎任何帮助！

谢谢！

Answer 1

我希望您以此来修改Singleton逻辑。

class Crud
{
    private $pdo         = null; # Storing PDO connection
    private $table       = null; # Storing table name
    private static $crud = []; # Static attribute that contains a self instance

  # ----------------------------------------
    # Class constructor -> PUBLIC method
    # ----------------------------------------
    public function __construct($connection, $table = 'default')
    {
        if (!empty($connection)) {
            $this->pdo = $connection;
        } else {
            echo 'Conexão inexistente!';
            exit();
        }

        if (!empty($table) && $table !== 'default') {
            $this->table =$table;
        }
    }

    # ----------------------------------------
    # Static public method that returns a Crud class instance
    # ----------------------------------------
    public static function getInstance($connection, $table = 'default')
    {
        # Verifying if there's a class instance
        if (!isset(self::$crud[$table])) {
            try {
                self::$crud[$table] = new Crud($connection, $table);
            } catch (Exception $e) {
                echo 'Error '.$e->getMessage();
            }
        }
        return self::$crud[$table];
    }
}

我已将您的私有静态属性$crud转换为数组，并按表名存储了每个实例。

重要的变化在这里突出显示：

        if (!isset(self::$crud[$table])) {
            try {
                self::$crud[$table] = new Crud($connection, $table);
            } catch (Exception $e) {
                echo 'Error '.$e->getMessage();
            }
        }

因此以下代码将像这样工作：

Crud::getInstance($pdo); // Create new instance and store it on key 'default'
Crud::getInstance($pdo); // Just return 'default' instance

Crud::getInstance($pdo,'foo'); // Create new instance and store it on key 'foo'
Crud::getInstance($pdo,'foo'); // Just return 'foo' instance

Answer 2

您使用的Crud类仅允许在其中存储一个实例。此后无修改。这是通过代码的这一部分完成的：

public static function getInstance($connection, $table=null)
  {
    # Verifying if there's a class instance
   if (!isset(self::$crud))
    {
      try
      {
        self::$crud = new Crud($connection, $table);
      }
      catch (Exception $e)
      {
        echo 'Error '.$e->getMessage();
      }
    }
    return self::$crud;
  }

第一次调用getInstance时，它将实例设置在静态类中，并且在其后的所有后续调用都返回从第一次调用中已设置的实例。不管您是否提供不同的连接或表。

在这种情况下，您将不得不将Crud的新实例用于不同的表/连接。不要使用'getInstance'，而是使用new Crud(...)创建一个新实例。

PHP CRUD上的两个实例为单例

2 个答案: