Все зависит в рамках какой задачи Вы пишите.
Как бы если этот скрипт написан за минимальный прайс и минимальный тайминг, на какой нибудь халтурке (фриланс), то ничего страшного другими словами "а на что Вы надеялись".
Если Вы являетесь штатным программистом в конторе или этот проект долгосрочен для Вас и Вы потеряли кучу времени на реализацию этого кода, то Вам нужно пересмотреть свои взгляды на будущее, в современном пхп мире так уже никто не пишит.
1) Сейчас стараются писать в ООП стиле Ваш код имеет вид процедурного кода больше перетекающий в "лапшекод".
Как минимум необходимо находить независимые атомарные операции и обворачивать в классы
например голый sql запрос не может быть на одном уровне бизнес логикой и проверкой входных данных (см. паттерн Репозиторий), так же нельзя тут же обращаться к апи телеграмма может быть стоить выделить в отдельный универсальный класс отправки сообщений (вдруг вам надо будет отправлять вместо телеграмма, в вайбер) (можно глянуть паттерн Декоратор) и т. д.
2)
include "bitrix.php";
include "function.php";
require_once '../src/db.php';
Но при этом
use Aura\Database;
3) избегайте if else особенно когда они занимают пол экрана это очень ухудшает читаемость и дальнейшее расширение приложения
4) используйте Exception правильное использования сократит количество if else
можно дальше продолжать но начните с этого