>>141450685 (OP)
Уже знаю как тебя атаковать. Жди через месяцок. Схоронил ссырцы для дальнейшего нагугливания.

>>141450885
умничка, кашу доесть не забудь :*

БАМП

БАМП

БАМП

Чо такое suspendUser($order->id)?

Я еще всегда пишу LIMIT 1 в запросах (см. 4 и 5 строки)

>>141450685 (OP)
В твоем коде ужасно всё от первой до последней строчки.

>>141452476
Вроде все норм.

>>141450685 (OP)
Да

>>141452394
это функция, которая блокирует клиента в панеле, она возвращает ответ, если успешно и пустоту, если ошибка

>>141452394
> Я еще всегда пишу LIMIT 1
раньше тоже писал, но зачем?

>>141452476
аргументируй

>>141453094
Ну, если дохуя записей, и одна найдена, зачем дальше искать? Ну, и для апдейта это вообще почти обязательное правило, иначе рискуешь оказаться в ситуации, когда он наобновляет тебе за щеку хуй знает сколько.

>>141450685 (OP)
Сколько учился кодингу?

Вдобавок name и email можно получить склейкой таблиц в самом верхнем запросе.

>>141453311
пишу второй месяц

>>141453380
в смысле, можно подробней, не очень понял

>>141453264
Так он итак дальше и не будет искать или будет???
> когда он наобновляет тебе за щеку хуй знает сколько
Ну как бы это никак невозможно или возможно???
--
Поясните развёрнутее, пожалуйсьа

>>141453433

SELECT `orders_host.id`,`orders_host.srv_name` FROM `orders_host`, `users` WHERE .... AND orders_host.id_user = users.id

> //отправили на почту письмо
> //массив для лога
Всегда угорал с таких капитанских комментов.

>>141453544
и что мне это даст? Мне же нужно по id_user вытащить из users name и email

>>141453584
я написал для двощей это, в своём коде обхожусь без комментариев, ибо работаю с ним один

Ах да, возможно ли это:
$uquery=$mysqli->query("SELECT `name`, `email` FROM `users` WHERE `id` = '$order->id_user'");
$user = $uquery->fetch_object();
объединить в одну строку?
Интересует именно ООП, а не процедурный

>>141453589
А, виноват:

SELECT `orders_host.id`,`orders_host.srv_name`, users.name, users.email FROM `orders_host`, `users` WHERE .... AND orders_host.id_user = users.id

>>141453662
$user = $mysqli->query("SELECT `name`, `email` FROM `users` WHERE `id` = '$order->id_user'")->fetch_object();

Наркоман?

>>141453703
поправочка, вот так действительно работает:
SELECT orders_host.id,orders_host.srv_name, users.name, users.email FROM `orders_host`, `users` WHERE `status` = '1' AND `time_end` <= '".$time."' AND orders_host.id_user = users.id

Мужик, ты гений. Вот только вопрос, это лучше, чем отдельно делать запрос? Я понимаю, что по сути получается 1 запрос вместо двух, но то, что запрос стал длинным, это ничего?

>>141450885
>>141450936
SQL Injection.

>>141453764
вот это я тупой, спасибо

>>141453424
Не пугай людей
хотя бы time фильтруй в первой строке

>>141453864
ты лол, там все параметры задаются скриптом, ни одно поле пользователем не вбивается, нет там SQL Injection

>>141453904
$time=time();
Зачем фильтровать?

>>141450685 (OP)
Куда оффер высылать?
Тимлид из гугл

>>141453970
Жесть, вот это индексируемость у двоща

Обновил код с учётом всех замечаний, есть ещё какие-то недочёты?
Аргументировано, пожалуйста :)