PHP библиотека для работы с pdo

[error1001es]

Всем привет, год изучаю php, захотел написать свою библиотеку для облегчения работы с pdo.
Если вы сталкивались с этим или писали свою библиотеку и у вас есть желание помочь, прошу посмотреть мой код, потому что я почему-то не уверен, что в нем все сделано хорошо
Мой код находится на гитхабе:

https://github.com/error1001es/test/blob/master/

 

[fixxxer]

Тут плохо вообще все. даже не знаю с чего начать.

Вот почитай
http://phpfaq.ru/pdo

 

[error1001es]

Тут плохо вообще все. даже не знаю с чего начать.

Вот почитай
http://phpfaq.ru/pdo

это я и читал, чтобы написать

 

[fixxxer]

Тогда перечитай еще раз и особо обрати внимание на раздел про подготовленные выражения

 

[error1001es]

Тогда перечитай еще раз и особо обрати внимание на раздел про подготовленные выражения

у меня почти каждое выражение подготовленное

 

[c0dex]

Плейсхолдеры где?

 

[error1001es]

Плейсхолдеры где?

а как я там их буду использовать?

 

[error1001es]

Плейсхолдеры где?

и нужны ли вообще они там?

 

[c0dex]

Как все используют, так и ты. Любые данные в строку запроса попадают через подстановку плейсхолдеров.

 

[fixxxer]

и нужны ли вообще они там?

если тебя не беспокоят SQL-инъекции - то не нужны

 

[error1001es]

если тебя не беспокоят SQL-инъекции - то не нужны

ну, а как мне их там использовать? например в функции _setup?

 

[c0dex]

ну, а как мне их там использовать? например в функции _setup?

$this->query('INSERT INTO ' . $table . ' (' . implode(',', $keys) . ') VALUES (' . implode(',', $values) . ')');
Вот такого, как уже сказали, быть не должно.

 

[error1001es]

$this->query('INSERT INTO ' . $table . ' (' . implode(',', $keys) . ') VALUES (' . implode(',', $values) . ')');
Вот такого, как уже сказали, быть не должно.

то есть мне надо сделать сначала массив из $keys с добавленным двоеточием, а потом связать их с $values и выполнить запрос?

 

[error1001es]

$this->query('INSERT INTO ' . $table . ' (' . implode(',', $keys) . ') VALUES (' . implode(',', $values) . ')');
Вот такого, как уже сказали, быть не должно.

или лучше создать массив из '?' и отдать его вместо $values, а потом сделать ->execute($params);

 

[error1001es]

$this->query('INSERT INTO ' . $table . ' (' . implode(',', $keys) . ') VALUES (' . implode(',', $values) . ')');
Вот такого, как уже сказали, быть не должно.

function create($object) //создает строку (из объекта) (любые таблицы)
    {
        $table = $object->tablename;
        unset($object->tablename);
        $keys = [];
        $values = [];
        $columns = [];
        foreach ($object as $key => $value) {
            $keys[] = $key;
            $columns[$key] = ':' . $key;
            $values[$key] = $value;
        }
        $stmt = $this->prepare('INSERT INTO ' . $table . ' (' . implode(',', $keys) . ') VALUES (' . implode(',', $columns) . ')');
        foreach ($keys as $key) {
            $stmt->bindValue($columns[$key], $values[$key], is_integer($values[$key]) ? PDO::PARAM_INT : PDO::PARAM_STR);
        }
        $stmt->execute();
    }

теперь все хорошо?

 

[AnrDaemon]

Просто выброси этот ужас и никогда так не делай.

 

[c0dex]

Посмотри как это реализовано в других обертках. На память приходит http://phpfaq.ru/safemysql

 

[fixxxer]

$columns[$key] = ':' . $key

А если в key пробел и дальше чё угодно?
Ключи надо генерировать, вида p0 p1...

 

[error1001es]

А если в key пробел и дальше чё угодно?
Ключи надо генерировать, вида p0 p1...

но туда не может попасть пробел, ведь я сам ввожу код, а пользователь не сможет выбрать колонку, в которую вводить
да и у объектов свойства не могут начинаться с пробела

 

[AnrDaemon]

Имя колонки может содержать пробел. Легко.