- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
<?php
...
public function update() {
if($this->data['id'])
return false;
$values = array();
foreach($this->data AS $key => $value) {
if($key == 'id')
continue;
$values[] = "`{$key}` = " . ($value ? "'{$value}'" : "NULL");
}
$result = $this->sql->query("
UPDATE `users`
SET (".implode(', ' $values).")
WHERE `id` = '{$this->data['id']}'");
return (bool) $this->sql->affected_rows;
}
...
?>
mindfreakthemon 09.01.2011 11:58 # +1
А тут ! случаем не пропущен?
Иначе ведь запросы будут выполнятся только для id = 0. (для NULL, false будет ошибка)
Devzirom 09.01.2011 12:31 # 0
if(!$this->data['id'])
return false;
KirAmp 09.01.2011 12:10 # −1
Вы слишком самокритичны, беглый осмотр говна не выявил (разве только что это php) =)
Devzirom 09.01.2011 12:37 # 0
Lure Of Chaos 09.01.2011 12:40 # −2
думаю, стоило бы сразу лепить в строку в цикле
Devzirom 09.01.2011 12:45 # 0
Lure Of Chaos 09.01.2011 13:14 # 0
и без имплода
Devzirom 09.01.2011 13:26 # 0
striker 09.01.2011 17:13 # 0
и, кстати, не забудьте про квотинг переменных в sql запросах
Devzirom 09.01.2011 18:34 # 0
striker 10.01.2011 03:54 # 0
eth0 09.01.2011 19:19 # −1
Lure Of Chaos 09.01.2011 19:35 # −1
xXx_totalwar 09.01.2011 19:43 # +1
striker 10.01.2011 03:55 # 0