Немного вопросов...
Форум / Архив / Немного вопросов...
Тема закрыта
нет авы 1) Olololik 29 Июля 2014 в 01:37

Почему вы делаете кучу запросов к БД при каждом обращении к скрипту? Все очистки старых сообщений и прочих штук. Почему бы не сделать это в файле и поставить на крон? Не надо насиловать мускул)
нет авы 2) Olololik 29 Июля 2014 в 01:39

Почему при регистрации не сделать так, чтобы все ошибки писались в переменную, потом ее проверять на пустоту и если она не empty - тогда вывести все ошибки сразу? и код там сократится в сотни раз за счет ваших шапок каждый раз xD
нет авы 3) Olololik 29 Июля 2014 в 01:42

Ах да, забыл самый интересное, зачем вы фильтруете цифровые значения всеми функциями, которые есть? Ну например:
$page = abs(intval(htmlentities($db -> real_escape_string($_GET['page']))));
$id = abs(intval(htmlentities($db -> real_escape_string($_GET['id']))));

тут будут только цифры, abs и intval хватит...
4) Павел 29 Июля 2014 в 15:46

Цитата: Olololik : Почему вы делаете кучу запросов к БД при каждом обращении к скри
Используются лишь самые необходимые.
5) Павел 29 Июля 2014 в 15:47

Цитата: Olololik : Почему при регистрации не сделать так, чтобы все ошибки писались
Это не dcms))
6) Павел 29 Июля 2014 в 15:48

Цитата: Olololik : Ах да, забыл самый интересное, зачем вы фильтруете цифровые знач
На 1 стадиях разработки скрипта быцли проблемы с безопасностью )) решал пока их-чудил. а изменить как-то всё руки не доходят. Да и не вижу в этом коде ни чего такого проблемного
нет авы 7) Olololik 29 Июля 2014 в 16:50

//////////////////Чистка мусора.....................
/////////Удаление старых писем
$db -> query("DELETE FROM `mail` WHERE `read`<>'save' and `time` < '".(time()-604800)."'");
/////////Удаление старых событий
$db -> query("DELETE FROM `lenta` WHERE `time` < '".(time()-604800)."'");
/////////Удаление старых запросов на восстановление пароля
$db -> query("DELETE FROM `report` WHERE `type`='lost' and `time` < '".(time()-259200)."'");



// вот это самое необходимое каждый раз при открытии любой страницы движка? угу, а крон бы раз в сутки запускал бы очистку...
8) Павел 29 Июля 2014 в 16:55

Цитата: Olololik : //////////////////Чистка мусора..................... <br />/////
да,как раз самое необходимое. При чем у каждого из них есть условие,что следовательно и не создаёт особой нагрузки на скрипт
нет авы 9) Olololik 29 Июля 2014 в 17:00

Лорд ,ну ладно, мне то все равно... Но еще скажите что запросы в цикле тоже не несут никакой нагрузки? счетчики в листинге категорий, например?
нет авы 10) Olololik 29 Июля 2014 в 17:02

и что еще за бред про условие, если вы даже индексы не выставили полям, по которым выполняется это условие (например, time)
На главную