Как правильно перебрать свойства?

Question

[4sadly]

Есть функция получения свойств объекта со строки через pdo:

function dispense($t){
        $r = $this->query('DESCRIBE '.$t)->fetch();
        $r->tablename = $t;
        return $r;
    }

и функция вставки объекта:

function insert($o){
        $t = $o->tablename;
        unset($o->tablename);
        $keys = null;
        $values = null;
        foreach ($o as $k => $v){
            $keys .= $k.', ';
            $values .= $v.', ';
        }
        $this->query('INSERT INTO '.$t.' ('.$keys.') VALUES('.$values.')');
    }

Как мне правильнее это реализовать? чтобы не было лишних запятых и строки оборачивались в кавычки?

Answer 1

[twobomb]

implode

Comments

[4sadly]

а как string обернуть в кавычки? получилось пока так:

function insert($o){
        $t = $o->tablename;
        unset($o->tablename);
        $keys = array();
        $values = array();
        foreach ($o as $k => $v){
            $keys[] = $k;
            $values[] = $v;
        }
        $this->query('INSERT INTO '.$t.' ('.implode($keys).') VALUES('.implode($values).')');
    }

Answer 2

[Радость моя]

<?php

function insert($o)
{
	$t = $o->tablename;
	unset($o->tablename);
	$keys = array();
	$values = array();
	foreach ($o as $k => $v) {
		$keys[] = $k;
		$values[] = is_string($v) ? "'{$v}'" : $v;
	}
	$keys = implode(',', $keys);
	$values = implode(',', $values);
	$this->query("INSERT INTO {$t} ({$keys}) VALUES({$values})");
}

Comments

[4sadly]

Я здесь не за кодом, а за саморазвитием.
Что делает {}? и обязательны ли двойные кавычки?

[Радость моя]

4sadly,
1. я поправил изначальный вариант, чтобы строковые $v были в апострофах и не было ошибки sql.
2. {} это просто визуально отделяет переменные пхп в строках. можно без них обойтись в ряде случаев. но лучше привыкать делать с ними, чтобы без сюрпризов.
3. двойные кавычки обязательны, т.к. только в них работает подстановка переменных внутри строки. Если использовать обычные апострофы то пхп выведет строку "как есть". т.е. как вы ее ща видите, без подстановки.
4. если используете пхп версию выше 5.5+ то можно сократить

$keys = array();
$values = array();

на

$keys = [];
$values = [];

[4sadly]

Радость моя, а можно как-то без двойных кавычек? или надо как-то побороть их боязнь?
везде где возможно я их убираю и делаю что-то типа
$text = 'текст: '.$var;

[Радость моя]

4sadly, можно и точками соединять, если вам там больше нравится. все от ситуации зависит.
иногда удобнее (более читаемо) когда двойные кавычки, иногда апострофов хватит.

[4sadly]

Радость моя, а если значение поля = null, как подставить сюда null?
сделал так:

function insert($o)
    {
        $t = $o->tablename;
        unset($o->tablename);
        $keys = [];
        $values = [];
        foreach ($o as $k => $v) {
            $keys[] = $k;
            if (empty($v)): $values[] = 'null';
            else: $values[] = is_string($v) ? "'{$v}'" : $v; endif;
        }
        $keys = implode(',', $keys);
        $values = implode(',', $values);
       $this->query('INSERT INTO ' . $t . ' (' . $keys . ') VALUES (' . $values . ')');
    }

[Радость моя]

4sadly, если у вас в бд поле поддерживает null для пустой то этот вариант подойдет.
если нет, то надо
if (empty($v)): $values[] = "''";

[4sadly]

Радость моя, поддерживает, а зачем нужно двоеточие после условия, если оно и так до точки с запятой отработает?

[Радость моя]

4sadly, вы сами зачем то такой формат выбрали. используйте {} это вроде как по стандартам правильнее.

[4sadly]

Радость моя, переделал на

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();
    }

[Радость моя]

4sadly, можно и так :)