WAP Мастер Форум / PDO

WapStyle » 2011.09.22 17:19

Как правильно написать данный код на PDO

Код:

$online = $db->query("SELECT * FROM `online` WHERE `ip` = '$_SERVER[REMOTE_ADDR]'")->fetch(PDO::FETCH_NUM);

if(!$online[0]) $db->exec("INSERT INTO `online` (`ua`,`ip`,`time`) VALUES ('$_SERVER[HTTP_USER_AGENT]','$_SERVER[REMOTE_ADDR]','$_SERVER[REQUEST_TIME]');");

else $db->exec("UPDATE `online` SET `time` = '$_SERVER[REQUEST_TIME]' WHERE `id` = '$online[0]'");

$online = NULL;

$db->exec("DELETE FROM `online` WHERE '$_SERVER[REQUEST_TIME]' - `time` > 360");

Gemorroj » 2011.09.22 17:42

для начала нужно хотя бы правильно написать ключи в массивах ;)
$_SERVER[REQUEST_TIME] - php тут создаст константу REQUEST_TIME и присвоит ей строку "REQUEST_TIME". круто?

WapStyle » 2011.09.22 17:51

А ну да... поленился написать )) а больше вариантов как упростить код нет? Просто если на этом коде мне разобраться то на остальных соответственно тоже... ;)

Gemorroj » 2011.09.22 17:59

да есть что изменить, конечно. но тут переменные как бы доверенные, поэтому можно на фильтрацию забить. можно еще, например, обернуть это все в транзакцию.

WapStyle » 2011.09.22 18:03

т.е. как то так

Код:

1
2
3
4
5
6
7
8
9
10
11
12
13
14

try

{

$dbh -> setAttribute ( PDO :: ATTR_ERRMODE , PDO :: ERRMODE_EXCEPTION );

$dbh -> beginTransaction ();

$dbh -> exec ( "insert into staff (id, first, last) values (23, 'Joe', 'Bloggs')" );

$dbh -> exec ( "insert into salarychange (id, amount, changedate) values (23, 50000, NOW())" );

$dbh -> commit ();

}

catch ( Exception $e )

{

$dbh -> rollBack ();

echo "Шеф! Фсё пропало : " . $e -> getMessage ();

}

Gemorroj » 2011.09.22 18:06

типо того

WapStyle » 2011.09.22 18:16

Вот что получилось

Код:

1
2
3
4
5
6
7
8
9
10
11

$online = $db->query('SELECT * FROM `online` WHERE `ip` = "'.$_SERVER['REMOTE_ADDR'].'"')->fetch(PDO::FETCH_NUM);

$db->beginTransaction();

if($online[0]){

$db->exec('UPDATE `online` SET `time` = "'.$_SERVER['REQUEST_TIME'].'" WHERE `id` = "'.$online[0].'"');

} else {

$zapis = $db->prepare("INSERT INTO `online` (`ua`,`ip`,`time`) VALUES (?,?,?);");

$zapis->execute(array($_SERVER['HTTP_USER_AGENT'],$_SERVER['REMOTE_ADDR'],$_SERVER['REQUEST_TIME']));

}

$db->exec('DELETE FROM `online` WHERE "'.$_SERVER['REQUEST_TIME'].'" - `time` > 360');

$db->commit();

$online = NULL;