Почему в базу добавляется два значения?

Question

[Orbb]

Привет.

Есть простенькая функция, которая принимает аргументы названия базы и массив значений :

function insertDbValue($db, $values, $where = null)
{
    $bindAr = prepareBindParams($values, true, true);
    $paramsString = prepareBindParams($values, true, false);
    include DB;
    $sql = "
        INSERT INTO $db ($paramsString)
        VALUES ($bindAr)
        ";
    $result = $pdo->prepare($sql);
    foreach($values as $key=>$val){
        $result->bindValue(":{$key}", $val);
    }
    $result->execute();
}

Вызываю в коде:

$values = ['status_text' => 'Тестовый статуз', 'status_source' => 'site.ru'];
insertDbValue('statuses', $values) ;

prepareBindParams возвращает определенного рода массив, она не при делах, ее не пишу.

Я грешил на foreach, но записав на бумаге алгоритм с него сомнения отпали. Мб неправильно понимаю механизм работы bindValue: мне казалось что foreach должен сам сделать из такого

$s->bindValue(':order_message', $order_message);
        $s->bindValue(':comment', $comment);
        $s->bindValue(':checkout', $checkout);
        $s->bindValue(':items', $items);
        $s->bindValue(':id', $id);
        $s->execute();

Такое

foreach($values as $key=>$val){
        $result->bindValue(":{$key}", $val);
    }
    $result->execute();

Не могу понять в чем косяк и почему 2 значения уходят в базу, а не одно.
Вопрос: за каждый вызов функции insertDbValue($db, $values, $where) строк статуса добавляется в базу 2, а не 1. Почему 2?


п.с. говоря про bindValue - смутило что при отлове массива $result в цикле каждую итерацию вылезает полный запрос.

Answer 1

[Orbb]

небольшой ап после курения стака и мануалов:

function insertDbValue($db, $values, $where = null)
{
    $bindAr = prepareBindParams($values, true, true);
    $paramsString = prepareBindParams($values, true, false);
    include DB;
    $sql = "
        INSERT INTO $db ($paramsString)
        VALUES ($bindAr)
        ";
    $result = $pdo->prepare($sql);
    $result->execute($values);
}

форич оказался избыточен, но код по прежнему добавляется 2 раза.

Answer 2

[Никита Кудинов]

Была подобная проблема. Справлялся только так:
include_once 'dbconnect.php';
Где файл dbconnect.php в виде:

<?php
error_reporting( E_ALL & ~E_DEPRECATED & ~E_NOTICE );
if(!mysql_connect("localhost","root",""))
{
	die('oops connection problem ! --> '.mysql_error());
}
if(!mysql_select_db("YOUR_DB"))
{
	die('oops database selection problem ! --> '.mysql_error());
}

?>

Comments

[Orbb]

спасибо за ответ, я отказался от include_once в своем проекте с файлами бд. Т.к. на одной странице может быть 20 запросов и всем нужна бд, но функции разные и первая обрубит все остальные.

[Orbb]

oxidmod: вообще меня удивляет больше то, что читая статьи в интернете "как надо делать", "лучшие практики", а затем внимая советам других пользователей, создается впечатление, будто все им следуют. Я за 2 года работы в вебе, от SEOшника до верстальщика, а сейчас пхп-джуна не видел ни одного идеально вылизанного сайта, без костылей и со внятными структурами, вне зависимости от бюджета - делали ли его за 350т.р. или за 1.5 млн. рублей, везде 50% говнокода и регулярно что-то отваливается. Это так, оффтоп. А в своем приложении я использовал такой подход, потому что мне удобней так:)