Ramm

https://github.com/EnterpriseQualityCoding/FizzBuz...
Это была элементарная задачка FizzBuzz

Ну что же, начнём? Только не плакать потом, сами попросили.

Итак, начнем с архитектуры. Начнем с того, что код чем только не занимается, он и из консоли запускает и как веб-хук и т.д. Сам по себе он является куском битрикса в том или ином виде.
Далее - код использует ООП, но написан в процедурном стиле.

Уже за это надо проявлять насилие над личностью.

<?php
include "bitrix.php";
include "function.php";
require_once  '../src/db.php';

Просто используйте require_once.

use должен быть вверху файла.

$id = $_GET['id'];
Будет генерировать ошибки, если скрипт был вызван без id.

$select = $db->query("SELECT * FROM tg_bot WHERE  tg_bot.id='$id'");

Добро пожаловать в мир SQL-инъекций.

$bot_token = $select[0]['api_key'];
Что произойдет, если бот не существует? Ошибки.

define('BOT_TOKEN', "$bot_token");
Бить железной линейкой по пальцам. Сильно. Больно. Долго.

Нельзя использовать переменные внутри define. Эта инструкция придумана для объявления констант. Объявлять константы используя переменные недопустимо.

if (php_sapi_name() == 'cli') {
    apiRequest('setWebhook', array('url' => isset($argv[1]) && $argv[1] == 'delete' ? '' : WEBHOOK_URL));
    exit;
}

Жуть какая-то.

$content = file_get_contents("php://input");
$update = json_decode($content, true);

if (!$update) {
    exit;
}

Я так понимаю, что здесь происходит вызов самого веб-хука.

Ну а дальше просто идет месиво. Смешались в кучу кони, люди...

Что с этим можно сделать?

Вызубрить ООП и getjump.github.io/ru-php-the-right-way

Вспомните принцип "Разделяй и властвуй".
Разделяйте код по функционалу и уровням абстрации.
Например разделите код на работу с базой, на работу с API, на бизнес-логику.

Например ваш код мог бы выглядеть так:

// добавляем автозагрузку классов
require_once 'vendor/autoload.php';
// импортируем конфигурацию
require_once 'config.php';

use Chaly/Bot;

// проверяем, что наш скрипт вызван с индентификтором бота
if (!isset($_GET['id'])) {
  exit(1); // ненулевой статус говорит об ошибке
}

// загрузить робота из базы данных
$bot = Bot::loadById($_GET['id']);
if (!$bot) {
  exit(1);
}

// запустить робота
$bot->run();

Как может выглядеть тело run()

public function run() {
  if (Cli::isValid()) {
    switch (Cli::getCommand()) {
        case 'delete':
            $this->delete();
            break;
        case 'register':
            $this->register();
            break;
    }
    return;
  }

  $payload = $this->getWebHookPayload();
  if (!$payload) {
    return;
  }

  if (isset($payload['message'])) {
    $this->interpretMessage($payload['message']);
  }

  // и так далее, думаю смысл понятен
}

array_combine(array_keys($arr), array_column($arr, 'VALUE'))

или

array_map(function($n) {
  return $n['VALUE'];
}, $arr)

эмм.... кагбе никакой фреймворк не мешает создать объект на основании внешних данных.

Можно. Но PHP тут (почти) не при чём.

Надо создать трансляцию, скопировать её секретный URL и запустить на сервере, например, ffmpeg так, чтобы он стримил на этот URL нужный файл в нужном формате, зацикленно.

Ссылки для ознакомления:

• Прямые трансляции в ВК
  
• Как создать трансляцию при помощи ПО
  
• Как настроить Open Broadcast Software (со своего компа)
  
• ffmpeg streaming guide (на англ.) – как стримить с сервера, запустив там ffmpeg
  

download_images_from_csv.sh (допилите под себя если нужно)

Статические ссылки на текущий класс, такие как self:: или __CLASS__, вычисляются используя класс, к которому эта функция принадлежит, как и в том месте, где она была определена
...
Позднее статическое связывание пытается устранить это ограничение, предоставляя ключевое слово, которое ссылается на класс, вызванный непосредственно в ходе выполнения.

Позднее статическое связывание

Решение:

public static function getMessage(): string
{
  return 'message - ' . static::getString();
}

Поделюсь с вами вот такой ссылкой:
https://www.youtube.com/playlist?list=PLswdBLT9llb...

Это самое верное и правильное решение. В нем нет ни грамма мудрёности. Если вам так сказали то вероятно уровень собеседующего был ниже (на уровне плинтуса).

Смотрите. Есть такая простая иерархия классов:

class A
{
    public static $text = 'class A';

    public function selfTest()
    {
        echo self::$text;
    }

    public function staticTest()
    {
        echo static::$text;
    }
}

class B extends A
{
    public static $text = 'class B';
}

Мы создаем экземпляк субкласа и вызываем методы, определенные в предке.

$obj = new B();
$obj->selfTest(); // выведет "class A"
$obj->staticTest(); // выведет "class B"

self всегда указывает на тот класс, в котором он написал. Здесь метод описан в классе A, и self указывает на класс A, хоть и вызывается из класса B.
Значение static вычисляется при вызове. И static указывает на класс объекта в котором произошел вызов. В нашем случае он указывает на B, хотя сам код описан в классе A.

Зачем это надо? Для того, что бы можно было переопределять статичные члены вашего класса в наследниках и вы могли обращаться к новым значениям из методов, описанных в предке.

С обычными не статичными членами это и так работает, потому что они собираются, когда вы создаете объект класса. Что бы это работало для статичных методов, надо использовать static

Проблема в том, что при выполнении каждого запроса, происходит автокомит, что в свою очередь нагружает диск.
Установи autocomit=0 и после выполняй COMMIT, после порции отправленных данных.

Транзакция помогает именно по тому, что старт транзакции устанавливает autocomit в 0, а по завершении возвращает его в исходное состояние.

Ну, и как говорили, объединяй данные в группы. Так же если это не критично и единичная операция, можно отключать проверку внешних связей (foreign_key_checks=0) и уникальных ключей (unique_checks=0)

Быстрый вариант

class Sms
{
    protected static $instances = [];

    public static function getInstance($id = 'default')
    {
        $map = [
            'first' => 'FirstSmsHandler',
            'second' => 'SecondSmsHandler',

            'default' => 'FirstSmsHandler',
        ];

        if (!isset(self::$instances[$id])) {
            if (!isset($map[$id])) {
                throw new \Exception(sprintf('Unknown hadler `%s`', $id));
            }

            $handler = new $map[$id];

            self::$instances[$id] = new self($handler);
        }

        return self::$instances[$id];
    }

    protected $handler;

    public function __construct(SmsHandler $handler)
    {
        $this->setHandler($handler)
    }

    public function setHandler(SmsHandler $handler)
    {
        return $this->handler = $handler;
    }

    public function changeHandlerByName($handlerName)
    {
        $map = [
            'first' => 'FirstSmsHandler',
            'second' => 'SecondSmsHandler',

            'default' => 'FirstSmsHandler',
        ];

        if (!isset($map[$handlerName])) {
            throw new \Exception(sprintf('Unknown handler `%s`', $handlerName));
        }

        $handler = new $map[$handlerName];

        $this->setHandler($handler);

        return $this;
    }

    public function __call($name, $arguments)
    {
        return call_user_func_array([$this->handler, $name], $arguments);
    }
}

$sms = new Sms(new FirstSmsHandler());
$sms->changeHandlerByName($_POST['provider']);

$sms = Sms::getInstance($_POST['provider']);

$sms->getBalance();

Можно еще почитать как ларавел реализовывает контракты, сервис контайнеры и сервис провайдеры. Там через конфиг просто меняем контейнер и уже работаем с другим классом. Почитайте.