Как можно оптимизировать конструкцию?

Question

[Анастасия]

Здравствуйте. Сразу оговорюсь: мне нужен не код, а помощь в построении верной логики приложения.
Суть такая:
Я храню у себя данные в БД. Через бота я проверяю на стороннем сервисе эти же данные. Если они меняются - я записываю их себе в БД. Вне зависимости от изменений, на каждый запрос я отправляю себе результат запроса в телегу. Данных много, поэтому в телеге у меня они отображаются в виде слайдера (вперёд и назад). В свойствах кнопки хранится ID записи в БД, текущее положение и действие (вперёд или назад)

Порядок операций:
Делаю запрос на чужой сервер.
В результате мне придёт либо массив данных, либо null
Если есть массив данных, то
1. я преобразовываю массив в строку
2. Сообщаю, что данные есть
3. сравниваю данные из БД. В случае их отличия заношу в БД, а в случае совпадения - в отдельное поле ставлю пометку и тут же получаю номер последней записи в БД
4. отправляю оповещение в телеграм, где указываю в параметрах кнопки переключателя слайдера id той записи, которую я только что внесла в БД
Иначе если null, то
- Сообщаю, что данных нет.
- сравниваю данные из БД. В случае их отличия заношу в БД, а в случае совпадения - в отдельное поле ставлю пометку и тут же получаю номер последней записи в БД

spoiler

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

По сути, пока я не внесла данные в БД я не могу вывести оповещение в телегу, потому что мне нужно знать ID записи. Но я не могу знать ID записи, пока я преобразую массив в строку.

Как можно оптимизировать код кроме как выноса этого действия в функцию?

Comments

[FanatPHP]

Я боюсь что никто не понимает этой проблемы, почему нельзя тупо поменять местами пункты 2 и 3

[Анастасия]

FanatPHP, 2 и 3 можно. нельзя 3 и 4 менять

[FanatPHP]

А в чем проблема-то тогда?
Какая проблема сделать в три шага?
1. преобразовываю массив в строку (что бы это не значило)
2. сравниваю данные из БД. В случае их отличия заношу в БД, а в случае совпадения - в отдельное поле ставлю пометку и тут же получаю номер последней записи в БД
3. отправляю оповещение в телеграм, где указываю что данные есть, а в параметрах кнопки переключателя слайдера id той записи, которую я только что внесла в БД

[Анастасия]

FanatPHP, так тут дело не в оповещении,что данные есть, а в том, что я дублирую код вставки в базу. В первом случае - это 3 действо, а во втором случае - 2. Код выполняет одно и тоже и находится в 10 строках друг от друга.

[FanatPHP]

так весь вопрос в расстановке условий что ли?
типа такого?

Если есть массив данных,
    массив в строку
    что сообщить =  есть
иначе
    что сообщить = нету

сравниваю данные из БД. В случае их отличия заношу в БД, а в случае совпадения - в отдельное поле ставлю пометку и тут же получаю номер последней записи в БД

что сообщить .=  в параметрах кнопки переключателя слайдера id той записи, которую я только что внесла в БД

вообще конечно текст настолько путанный, что уверен в том что этот вариант будет отвергнут.
но при этом я реально не могу понять, какая проблема расставить по коду пару условий

[Анастасия]

FanatPHP, вот примерно к такому консенсусу я сама и пришла, только не до конца поняла запись что сообщить .= . Я вообще предполагала сделать вот так:

Если есть массив данных,
    массив в строку
    что сообщить =  есть
иначе
    что сообщить = нету

сравниваю данные из БД. В случае их отличия заношу в БД, а в случае совпадения - в отдельное поле ставлю пометку и тут же получаю номер последней записи в БД

Если есть массив данных,
    отправляю оповещение в телеграм, где указываю в параметрах кнопки...

Просто я дважды спрашиваю: "если есть массив данных". Меня смущало это. Если не делать второй если, то в случае отсутствия данных произойдёт ошибка, так как мне в оповещении нечего отправлять.

[FanatPHP]

что значит "Сообщаю, что данные есть" и "Сообщаю, что данных нет."?
Это отправка сообщения в телеграм?

[Анастасия]

FanatPHP, да. Там тип в два этапа. В случае наличия данных - краткая сводка в телегу, что данные есть (на деле там идут ещё до кучи несколько мат функций, а только потом вывод). Потом должна быть запись в БД, чтобы получить записанный айдишник, а затем вывод в телегу данных в виде слайдера, где в кнопках записан полученный ID

[FanatPHP]

зачем там два этапа?
запись в БД так долго идет что краткой сводки надо полчаса ждать?
ну так надо запись оптимизировать.
и сначала записывать, а потом уже отправлять все сводки, хоть краткие, хоть полные

[Анастасия]

FanatPHP, В два этапа, чтобы в краткой сводке видеть запрос, а ниже как раз листать подробности.
Здесь идиотизм в том, что одно без другого жить не может

В начале формат данных: я меняю порядок записей, в некоторых местах делаю жирный шрифт, массив перевожу в implode для БД (на удивление многомерный массив с двумя implode работает быстрее, чем json utf-8) и складываю все суммы во всех полях, чтобы потом вывести в кратком оповещении.
Потом я могу вывести результат, либо записать в БД. Но в БД я могу записать только после формата данных, потому что иначе записывать нечего.
А потом уже вывожу слайдер, потому что слайдер без ID быть не может

Во втором случае
Вывожу оповещение, что ничего нет
Записываю в БД

[FanatPHP]

омг, опять гоняем наносекунды
ну то есть тот факт что БД льем заведомое дерьмо, ДАЖЕ не джейсон, а совсем уже вонючую жижу, нас не смущает. про то, чтобы данные хранить в нормальном виде разговора даже нет.

но зато целых три наносекунды, примерно одну квинтиллионную от общего времени обработки запроса сэкономили!
ведь знал же что меня тошнить начинает через три комментария, а все равно лезу

[FanatPHP]

и потом вот это вот начинающее программист начнет искать по БД лайками
и гробить базу уже неиллюзорно
потому что сейчас сэкономил наносекунду и три строчки кода на том чтобы записать в данные БД нормально

[Анастасия]

FanatPHP, а есть какие-нибудь ассинхронные возможности это реализовать, чтобы функция отображения слайдера отработала когда в бд запишутся данные?