Правильно ли пишу класс для бота Telegram?

Question

[driverx18]

Знаю что есть дофига готовых SDK для ботов Telegram, также знаю что ООП дается мне тяжко, и захотелось написать свой класс для работы с Telegram Bot API, очень хотел бы, чтобы знающие подсказали, в верном векторе я двигаюсь, нужно ли создавать свойства такие как token, api, или это абсурд и надо по другому как то порешать. Прошу подкорректировать по возможности. Код то сам по себе работает, но хочу научиться писать правильно. Спасибо

<?php
class Bot {
   private $token = ""; // тут ставлю токен сразу 

   private $api; // в конструкторе выставлю базовый адрес для вызова методов

   public $chatId; // ID чата

   public $message; // полученное сообщение от юзера

   public $output;

   public function __construct() { // сразу заинициализирую в конструкторе адрес апи, чат айди, сообщение..
      $this->api = 'https://api.telegram.org/bot' . $this->token;
      $this->output = json_decode(file_get_contents('php://input'), TRUE);
      $this->chatId = $this->output['message']['chat']['id'];
      $this->message = $this->output['message']['text'];
   }

   public function sendMessage($msg) { // метод отправки сообщений
      file_get_contents($this->api . '/sendMessage?chat_id=' . $this->chatId . '&text=' . $msg);
   }

   public function sendContact($phone, $name) { // метод отправки контакта
      file_get_contents($this->api . '/sendContact?chat_id=' . $this->chatId . '&phone_number=' . $phone . '&first_name=' . $name);
   }

}

$bot = new Bot; 
switch($bot->message) { // проверяем, какое сообщение получено
   case 'hi':
   $bot->sendMessage('halo');
   break;
   case 'contact':
   $bot->sendContact('380662312031', 'Anna');
   break;
   default:
   $bot->sendMessage($bot->message);
   break;
}

Answer 1

[Антон]

Много не скажу, на PHP давно не писал
1. Токен и остальные найстройки надо задавать аргументом конструктора, и брать его из окружения/конфиг файла
2. Можно использовать PHP7 тайпинги и ООП:

...
send($message: Message){}
...
$bot->send(new Message('text'))
...

3. Использовать http клиент, например, docs.guzzlephp.org/en/stable
4. Добавить парсинг комманд и чейнинг

$bot
  ->command('start', function() {})
  ->command('hello', function() {});

(не знаю, как коллбек-стайл будет смотреться в php)

Comments

[driverx18]

по первому пункту:
делаем файл config.php, там есть константа с токеном, в классе бота мы в конструкторе задаем свойству $token этого класса значение из того файла config?
по 3 пункту, зачем http клиент, если можно Curl, file_get_contents? (Я не спорю, просто интересно, я с ними (http клиентами) не работал никогда)

[Антон]

driverx18: да, $bot = new Bot(BOT_TOKEN)

1) это стандарт, в дальнейших проектах будет проще, особенно когда сложность требуемых запросов возрастет 2) так удобней 3) что, если потребуется послать кастомный запрос, например, POST? file_get_contents отпадает, встроенные функции для работы для работы с curl морально устарели 4) что, если тут $this->chatId . '&text=' . $msg); в $msg попадет &param=value? Это уже надо строку специально обрабатывать. И таких примеров много.

[driverx18]

Антон: да, учту, спасибо большое
дык а сам код конструктора моего имеет место быть? свойство $api, инициализация других свойств, или всё плохо? (ну помимо токена, там я понял)

[Антон]

driverx18: да, остальное можно оставить в конструкторе. Повторяющийся код, например, билдинг строки запроса почти одинаковый в sendMessage и sendContact, можно вынести в

protected function request($method, $payload)
...
sendMessage($message) { 
  $this->request('sendMessage', $message)
...

[driverx18]

Антон: спасибо!

[entermix]

Антон: По поводу file_get_contents + POST ;)

https://stackoverflow.com/questions/2445276/how-to...
fr2.php.net/manual/en/context.http.php

[Антон]

entermix: я, в принципе, и не сомневался, что так можно сделать, но зачем?

[entermix]

Антон: например, чтобы не тянуть лишние зависимости, но, это уже совсем другая история.

[Fortop]

Антон:

что, если потребуется послать кастомный запрос, например, POST? file_get_contents отпадает

Это еще почему?

Answer 2

[Stalker_RED]

Вместо сборки строки запроса вручную можно использовать http_build_query(), заодно отпадут проблемы типа "что, если тут $this->chatId . '&text=' . $msg); в $msg попадет &param=value?"

В switch/case обычно отступами размечают так, чтобы хорошо были видны границы блоков.

switch($bot->message) { // проверяем, какое сообщение получено
   case 'hi':
       $bot->sendMessage('halo');
       break;
   case 'contact':
       $bot->sendContact('380662312031', 'Anna');
       break;
   default:
       $bot->sendMessage($bot->message);
}

Comments

[driverx18]

Та про процедурный стиль то понятно, я просто про само содержимое класса спрашивал, корректно ли