Как разослать всем уведомление, не циклом?

Question

[Аннушка ❤️]

Доброго времени суток!! У меня есть сайт на котором уже зарегистрировано более 100 человек, я сделала рассылку уведомлений, но циклом while, в этом цикле присутствует много запросов и долго отправляются запросы. Как можно сделать так, чтобы это было намного быстрее и менее затратно по ресурсам хостинга?
Вот код который я использую на данный момент:

$result = mysql_query('SELECT `id` FROM `users` WHERE `id` != 2 ORDER BY `id` ASC');
while ( $row = mysql_fetch_assoc($result) ) {
    $time = time();
    mysql_query('INSERT INTO `mail` (`user`, `to`, `text`, `time`) VALUES (2, '.$row['id'].', "'.$text.'", '.$time.')');
    $contacts = mysql_fetch_assoc(mysql_query('SELECT * FROM `contacts` WHERE `user` = 2 AND `ho` = '.$row['id'].' AND `delete` = 0 OR `user` = '.$row['id'].' AND `ho` = 2 AND `delete` = 0'));
    if ( !$contacts ) {
        mysql_query('INSERT INTO `contacts` (`user`, `ho`, `time`) VALUES (2, '.$row['id'].', '.$time.'), ('.$row['id'].', 2, '.$time.')');
    } else {
        mysql_query('UPDATE `contacts` SET `time` = '.$time.', `delete` = 0 WHERE `user` = '.$row['id'].' AND `ho` = 2');
        mysql_query('UPDATE `contacts` SET `time` = '.$time.', `delete` = 0 WHERE `user` = 2 AND `ho` = '.$row['id']);
    }
}

Comments

[AlexVWill]

для ста записей этот код должен выполняться за доли секунды, если нет еще чего то

[mayton2019]

А почему так?

mysql_query('UPDATE `contacts` SET `time` = '.$time.', `delete` = 0 WHERE `user` = '.$row['id'].' AND `ho` = 2');
mysql_query('UPDATE `contacts` SET `time` = '.$time.', `delete` = 0 WHERE `user` = 2 AND `ho` = '.$row['id']);

[Akina]

Сначала надо забыть о существовании PHP и решить задачу чисто в рамках MySQL. Получится или два, или три запроса (логика происходящего в показанном коде мне непонятна). Всего, а не "для каждого пользователя".

В скобках отмечу - выполняемая в этом коде куча запросов выглядит как осознанное мероприятие по постановке сервера раком.

Answer 1

[Rsa97]

Делаете составной уникальный ключ (`user`, `ho`) , после чего всё сводится к двум запросам

INSERT INTO `mail` (`user`, `to`, `text`, `time`)
  SELECT 2, `id`, :text, :time
    FROM `users`
    WHERE `id`  != 2;

INSERT INTO `contacts` (`user`, `ho`, `delete`, `time`)
  SELECT *
    FROM (
      SELECT `id`, 2, 0, :time
        FROM `users`
        WHERE `id` != 2
      UNION ALL SELECT 2, `id`, 0, :time
        FROM `users`
        WHERE `id` != 2
    ) AS `t`
  ON DUPLICATE KEY UPDATE `delete` = 0, `time` = VALUES(`time`);

А вообще, какой-то жуткий легаси у вас. Библиотека mysql_ уже давно почила в бозе.

Comments

[Аннушка ❤️]

Заменить мои запросы на ваши в цикле? Правильно понимаю?

[Rsa97]

miss_allegory, Цикл вообще не нужен. Это два запроса полностью заменяют ваш код. Только подставьте значения вместо :text и :time.

[Rsa97]

miss_allegory, И не забудьте, что пара (`user`, `ho`) в таблице `contacts` должна быть уникальным составным ключом.

[Аннушка ❤️]

Rsa97,

И не забудьте, что пара (`user`, `ho`) в таблице `contacts` должна быть уникальным составным ключом.

Это как? Объясните пожалуйста

[Rsa97]

miss_allegory, Покажите результат запроса SHOW CREATE TABLE `contacts`

[Аннушка ❤️]

Rsa97,

[Rsa97]

miss_allegory, Не кусок картинки, а полный текст результата

[Аннушка ❤️]

Rsa97, ну я пыталась понажимать чтобы полный текст взять, но ничего не открывает и это весь текст который показало

[Ипатьев]

Папа-программист сидит за компьютером. Рядом стоит маленький сын и жует яблоко. Вдруг он спрашивает:
— Папа, а почему, если укусить яблочко, оно потом становится коричневым?
Отец, не поворачивая головы от компьютера:
— Видишь ли, в яблоке содержится железо. После того, как плод лишается кожуры, железо окисляется под воздействием кислорода, образуется оксид железа, который имеет темную окраску.
Пауза минут на десять, после чего сын робко спрашивает:
— Пап, а с кем это ты сейчас разговаривал?

[Rsa97]

miss_allegory, Ну, не видя текущего состояния не стоит создавать ключ, иначе может оказаться дубль.

[Аннушка ❤️]

Rsa97, Или может попробовать скачать таблицу, и там уже посмотреть?

[Rsa97]

miss_allegory, С PHPMyAdmin практически не работал, ничего по его интерфейсу сказать не могу.
Нужен либо результат CHOW CREATE TABLE либо ищите, где у вас показывает список ключей/индексов таблицы

[Аннушка ❤️]

Rsa97,

[Rsa97]

miss_allegory, Ну, если это всё, что связано с таблицей `contacts`, то ключа нет и его надо создать.

ALTER TABLE `contacts`
  ADD UNIQUE KEY `uk_user_ho` (`user`, `ho`);

[Аннушка ❤️]

Rsa97, сделала, что нужно дальше делать?

[Rsa97]

miss_allegory, Заменить свой код на те два запроса, что я написал.

[Аннушка ❤️]

Rsa97, не сработало :(
В базе никаких новых записей не появилось

[Rsa97]

miss_allegory, А ошибок в логе никаких нет?
Или выполните эти запросы в PHPMyAdmin

[Аннушка ❤️]

Rsa97, нет, я поменяла код с цикла на ваш, и ничего это не дало :(

[Аннушка ❤️]

Rsa97, была ошибка, я переменную $time удалила случайно, и всё заработало

[Аннушка ❤️]

Rsa97, а можете помочь одинаковые записи удалить в contacts по тем же данным user = 2 и ho = 2? А то я первые несколько раз неправильно была сделала и теперь у меня 2-3 раза повторяются записи

[Rsa97]

miss_allegory, Одинаковые записи не могут появиться при работающем уникальном ключе. А ключ невозможно установить, пока есть дубликаты.

DELETE `t1`.*
  FROM `contacts` AS `t1`
  JOIN `contacts` AS `t2`
    ON `t2`.`user` = `t1`.`user`
      AND `t2`.`ho` = `t1`.`ho`
      AND `t2`.`time` > `t1`.`time`

[Аннушка ❤️]

Rsa97, я это уже поняла, просто я на другой таблице пробовала сначала, чтобы не испорить, но всё таки испортила

[Аннушка ❤️]

Rsa97, после того как этот код ввожу, у меня сайт долго грузится очень, а сам sql запрос весит в загрузке :(

[Аннушка ❤️]

Rsa97, попыталась еще раз 5, на 5-й раз получилось, 210 секунд заняло, но дублированные строки не удалило :(

[Rsa97]

miss_allegory, А дубли с разным временем? Если с одним и тем же, то сравнивать надо по id, а не по time.

DELETE `t1`.*
  FROM `contacts` AS `t1`
  JOIN `contacts` AS `t2`
    ON `t2`.`user` = `t1`.`user`
      AND `t2`.`ho` = `t1`.`ho`
      AND `t2`.`id` = `t1`.`id`

[Аннушка ❤️]

Rsa97, этот запрос вообще все строки удалил

[Rsa97]

miss_allegory, Прошу прощения, ошибся. В последнем условии должно быть меньше или больше, смотря какие записи надо оставить - первые или последние.
AND `t2`.`id` > `t1`.`id` или AND `t2`.`id` < `t1`.`id`

[Аннушка ❤️]

Rsa97, а код который рассылку делает, он может создать еще один диалог когда уже есть существующий? Если нет, то у меня создало

[Rsa97]

miss_allegory, что такое "диалог"? Запись в contacts? Если создан уникальный ключ, то не может.
А ключ не может быть создан, пока есть дубликаты.

[Аннушка ❤️]

Rsa97, блинн точно! Забыла добавить, всё работает! Спасибочки!! Без вас я бы еще долго с этим мучалась, буду данную тему изучать тоже

Answer 2

[Sanes]

Запустить в фоновом режиме. Например через Crontab. Сто сообщений хостинг даже не почувствует.

Comments

[Аннушка ❤️]

Ну, а если будет больше? Хостинг то у меня не совсем то и по ресурсам хороший.

[Sanes]

miss_allegory, только циклом или с помощью стороннего сервиса.

[jaygen]

Crontab, тут не поможет, нужно думать как уменьшить число запросов к бд

[Sanes]

jaygen, поможет. Любая рассылка должна делаться в фоновом режиме.

[Аннушка ❤️]

Sanes, ну у меня через форму рассылка делается. Как его сделать чтобы ввел текст, а crontab его выполнил?

[Sanes]

miss_allegory, крутите Cron хоть каждую минуту. Это уже вам решать.