Рефакторинг

Тема для сбора примеров рефакторинга.

Для булевого toggle значения в БД не требуется дополнительная выборка.

$element = $db->GetOne('SELECT * FROM work_theme WHERE id = ' . $id);
$element['active'] = $element['active'] ? '0': '1';
$db->Query('UPDATE work_theme SET active = '.$element['active'].' WHERE id = ' . $id);

Заменяем на

$db->Query('UPDATE `work_theme` SET `active` = 1 - `active` WHERE id = ' . $id);

Добавить комментарий

Ваш адрес email не будет опубликован. Обязательные поля помечены *