Комментарии (18)

1. 

   Skycaptain

   01.04.2019 05:54

   #19966806

   Из какого вы года, путешественник во времени?

1. 

   evgwed

   01.04.2019 07:00

   #19966846

   Задумка статьи интересная, спасибо!
   
   За такое нужно руки отрывать в 2к19
   

   global $tg_bot_token;

   
   

   1. 

      KillerAngel Автор

      01.04.2019 08:24

      #19967038

      Всегда пожалуйста.

   
   

   1. 

      iDm1

      01.04.2019 13:13

      #19968636

      -1

      Если своим сообщением вы хотите сделать мир лучше, то правильно было бы сразу привести примет верного подхода в данном случае.
      А так выглядит как хвастовство: «Смотрите какой-то говнокодер, не то что я! :)»

      
      

      1. 

         evgwed

         01.04.2019 13:24

         #19968700

         +1

         Не думал, что нужно пояснять такие очевидные вещи.
         
         

         function Send_Out($user_id, $text, $is_end = true)
         {
           global $tg_bot_token;
           if (strlen($user_id) < 1 || strlen($text) < 1) {return;}
           $json = file_get_contents('https://api.telegram.org/bot' . $tg_bot_token . '/sendMessage?chat_id=' . $user_id . '&text=' . $text);
         }

         
         
         Этот код можно решить в несколько путей:
         
         1. В функции добавить еще один входной параметр в функцию:
         

         function Send_Out($tg_bot_token, $user_id, $text, $is_end = true)
         {
           if (strlen($user_id) < 1 || strlen($text) < 1) {return;}
           $json = file_get_contents('https://api.telegram.org/bot' . $tg_bot_token . '/sendMessage?chat_id=' . $user_id . '&text=' . $text);
         }

         
         
         2. Использовать классы и создать класс похожего типа:
         

         class TelegramClient 
         {
             private $botToken;
             
             function __construct(string $botToken) 
             {
                 $this->botToken = $botToken;
             }
             
             public function sendMessage(int $userId, string $message, bool $isEnd = true): void
             {
                 if ($userId < 1 || mb_strlen($message) < 1) {
                     throw new TelegramClientException(...);
                 }
                 $json = file_get_contents('https://api.telegram.org/bot' . $tg_bot_token . '/sendMessage?chat_id=' . $user_id . '&text=' . $text);
             }
         }

         
         Запросы на сервер не нужно отправлять через file_get_contents, лучше использовать Guzzle.
         

         
         

         1. 

            KillerAngel Автор

            01.04.2019 16:11

            #19969758

            -1

            А можно все завернуть в классы и сделать так:
            
            

              include_once 'webhook_class.php';
            
              $tg_bot_token = "_____YOUR_BOT_TOKEN_____";
            
              $Telegram_Cli = new Telegram_Cli_Class($tg_bot_token);
            
              $Webhook = new Webhook_Class();
            
              $Webhook->Set_Type('telegram');
              $Webhook->Get_Data();
              $Webhook->Parse_Data();
              $Webhook->Parse_Tokens();
              if ($Webhook->is_Out())
              {
                $Telegram_Cli->Send($Webhook->user_id, $Webhook->out_msg);
              }
            }
            

            
            

1. 

   2PAE

   01.04.2019 08:16

   #19966996

   Готовые примеры для чего это может быть полезно, где-то есть посмотреть?

   
   

   1. 

      KillerAngel Автор

      01.04.2019 08:20

      #19967020

      В открытом виде нет, используется для управление домашней автоматикой, к аккаунту пользователя привязывается id от Алисы и от Telegram. Команды управление получаются одинаковые.

1. 

   zhulan0v

   01.04.2019 08:16

   #19967000

   Код конечно печальный, но плюсанул за актуальную тематику. Если б не она, то вас бы слили (может еще и сольют).

   
   

   1. 

      KillerAngel Автор

      01.04.2019 08:23

      #19967032

      Код примитивный, кто умеет красиво и сам разберется за час.

      
      

      1. 

         zhulan0v

         01.04.2019 08:26

         #19967046

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

1. 

   0ri0n

   01.04.2019 08:48

   #19967126

   Не знаю что Вы… до парня на счет кода. Как смог так написал. (Я уверен, что если сделаете более читаемый код, он не откажется и опубликует и его.)
   Благодарю за статью, ты заинтересовал меня в возможностях «Алисы».
   

1. 

   win0err

   01.04.2019 10:57

   #19967746

   Да что вы прикопались к коду?
   Нормальный первоапрельский код.

   
   

   Ну а если серьёзно, KillerAngel, не используйте глобальные переменные.
   Почитайте про чистые функции и почему нужно стремиться писать именно чистые функции.
   Почитайте про ООП, с помощью него можно лучше структурировать код.
   И не забывайте про многобайтовые кодировки, вы же с PHP работаете (mb_strlen, например).

   
   

   1. 

      KillerAngel Автор

      01.04.2019 16:23

      #19969830

      -1

      А я и не использую, ну почти.
      Про ООП читал.
      Про кодировки — да, когда срезал всю шелуху, упустил strlen
      
      Все переделал, теперь все вот так, что один что другой скрипт:
      

        include_once 'webhook_class.php';
      
        $Yandex_Alice_Cli = new Yandex_Alice_Cli_Class();
      
        $Webhook = new Webhook_Class();
      
        $Webhook->Set_Type('yandex_alice');
        $Webhook->Get_Data();
        $Webhook->Parse_Data();
        $Webhook->Parse_Tokens();
        if ($Webhook->is_Out())
        {
          $Yandex_Alice_Cli->Set_Sess_Id($Webhook->data_msg_sess_id);
          $Yandex_Alice_Cli->Send($Webhook->user_id, $Webhook->out_msg);
        }
      

      
      

      
      

      1. 

         Roman_S1

         02.04.2019 15:24

         #19974584

         Это, конечно, лучше чем было, но ведь есть PSR и код оформлять желательно в соответствии с ним

         
         

         1. 

            KillerAngel Автор

            02.04.2019 15:38

            #19974658

            Согласен

1. 

   Disinterpreter

   02.04.2019 15:24

   #19974586

   Идея хорошая, кому не нравится, сделайте лучше.
   
   А вообще, у меня была подобная проблема, когда я писал единого бота для VK&Discord&LINE.
   
   Однако, там я решил эту проблему, написав несколько «драйверов», которые формировали единый JSON-массив и отправляли его в RabbitMQ, а на стороне считывания уже работал с этими стандартизированными данными. Так что, вам есть к чему стремиться.

   
   

   1. 

      KillerAngel Автор

      02.04.2019 15:36

      #19974650

      Так это был только обрезанный костяк. Суть статьи — идея, ну и ее простейшая реализация, чтобы можно было описать каждый ход и он был понятен всем. А там где это используется есть и проверки и обработчик который работает с базой через свои классы-адаптеры.