 |
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) |