Немного вопросов... Тема закрыта
 |
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)
|
|