PHP -SQL语句错误

Question

我尝试使用我的网站数据库实现视图计数器。

我的代码：

函数，创建查询：

public function addRowViewCompany($arrElement) {
        $query = "INSERT INTO views_company (";
        $columns = "";
        $values = "";

        // columns
        foreach ($arrElement as $column => $value) {

            if ($value === end($arrElement)) {
                // if last column
                $columns .= $column . ") VALUES(";
                $values .= ":" . $column . ");";
            } else {
                $columns .= $column . ", ";
                $values .= ":" . $column . ", ";
            }
        }

        $this->setQuery($query . $columns . $values);
        $this->prepareStatement();
        $this->executeStatement($arrElement);
    } 

调用方法：

            $database->addRowViewCompany(array(
                "view_id" => null,
                "view_date" => date("Y-m-d"),
                "company_id" => intval($companyID),
                "views" => 1
            ));

创建的查询：查询

INSERT INTO views_company (view_id, view_date, company_id) VALUES(views) VALUES(:view_id, :view_date, :company_id);:views);

在此位置是错误的：插入tovie_company（view_id，view_date，company_id）values（view> ）valut（：view_id，view_id， ：view_date，：company_id ）;：视图）;

但是我不明白为什么它会被创建 对于其他用例，此功能有效。

编辑：

public function __construct() {
    $this->connection = new PDO("mysql:host=localhost;dbname=ref;charset=utf8", "***", "***");
}

public function setQuery(string $query) {
    $this->query = $query;
}

public function prepareStatement() {
    $this->statement = $this->connection->prepare($this->query);
}

public function executeStatement(array $args = null) {
    if ($args == null) {
        $this->statement->execute();
    } else {
        $this->statement->execute($args);
    }
} 

Answer 1

如果有一个值重复数组中的最后一个值的值，则您的代码将无法正常工作，因为$ value === end（$ arrelement）对重复是正确的。这将使您在查询中添加额外的值。

不要使用循环，请使用umpode（）。那么，您无需特别治疗最后的迭代。

$columns = implode(', ', array_keys($arrElement);
$values = implode(', ', array_map(function($col) { return ':' . $col; }, array_keys($arrElement)));

$this->setQuery("$query ($columns) VALUES ($values)");

Answer 2

那对您的代码的修改如何？

public function addRowViewCompany($arrElement) {
    $query = "INSERT INTO views_company (";
    $columns = "";
    $middle = ") VALUES(";
    $values = "";
    $end = ")";

    // columns
    foreach ($arrElement as $column => $value) {
        $columns .= ((strlen($column)==0)?"":", ") . $column;
        $values .= ((strlen($values)==0)?"":", ") . ":" . $column;
    }

    $this->setQuery($query . $columns . $middle . $values . $end);
    $this->prepareStatement();
    $this->executeStatement($arrElement);
}