The bottom line is that if a person who does not have a username in the Telegram settings, that is, he is empty, then when the user logs into the bot, after pressing the / start button, nothing happens to the user, and the bot administrator receives an error

Error working with API. Details: {"ok":false,"error_code":400,"description":"Bad Request: chat_id is empty"}

The bot has a database where information about users is recorded.

Here is the code for checking and recording user data:

 // Функция-обработчик, отвечающая на команды $bot->addHandler(function($update, Telegram &$bot) { if (!isset($update->{"message"}->{"text"})) { $update->{"message"}->{"text"} = " "; } if (empty($update->{"message"}->{"chat"}->{"username"})) { $bot->sendMessage($chatId, "Уважаемый пользователь. У Вас не установлен username. Пожалуйста, установите его в настройках. После этого выполните команду /start еще раз. Спасибо за понимание"); return; } $chatId = $update->{"message"}->{"chat"}->{"id"}; $messageIn = $update->{"message"}->{"text"}; $firstname = ""; $lastname = ""; if (isset($update->{"message"}->{"chat"}->{"first_name"})) { $firstname = $update->{"message"}->{"chat"}->{"first_name"}; } if (isset($update->{"message"}->{"chat"}->{"last_name"})) { $lastname = $update->{"message"}->{"chat"}->{"last_name"}; } //Проверка на пользователя if(DB::uCountColumns("SELECT * FROM `Users` WHERE `chat_id` = '".$chatId."'")){ // обновление DB::uQuery("UPDATE `Users` SET `username`='".$update->{"message"}->{"chat"}->{"username"}."', `lastname`='".$lastname."', `firstname`='".$firstname."' WHERE `chat_id` = '".$chatId."'"); }else{ // запись DB::uQuery("INSERT INTO `Users`(`chat_id`, `username`, `lastname`, `firstname`, `regDate`) VALUES ('".$chatId."','".$update->{"message"}->{"chat"}->{"username"}."','".$lastname."','".$firstname."',NOW())"); } $userInfo = DB::uFetch("SELECT * FROM `Users` WHERE `chat_id` = '$chatId'"); $rMessage = explode(' ', $messageIn); switch ($rMessage[0]) { case '/start': { // Открываем файл для получения существующего содержимого if ($rMessage[1] > 0 AND ($rMessage[1]!=$chatId)){ // проверка есть ли такой пользователь if(DB::uCountColumns("SELECT * FROM `Users` WHERE `chat_id` = '".$rMessage[1]."'") AND !DB::uCountColumns("SELECT * FROM `Reffer` WHERE `chat_id` = '".$rMessage[1]."'")){ DB::uQuery("INSERT INTO `Reffer`(`chat_id`, `reffers_id`, `money`) VALUES ('".$chatId."','".$rMessage[1]."','". REF_MONEY ."')"); DB::uQuery("UPDATE `Users` SET `balanse`= `balanse` + '". REF_MONEY ."' WHERE `chat_id` = '".$rMessage[1]."'"); } } $welcomeMsg = "Текст приветствия"; $buttons = []; $buttons[] = [['text' => "\xF0\x9F\x92\xB5 Получить бонус"],['text' => "\xF0\x9F\x92\xB0 Кошелек"]]; $buttons[] = [['text' => "\xF0\x9F\x91\xAB Друзья"],['text' => "\xF0\x9F\x93\x88 Статистика"]]; $buttons[] = [['text' => "\xF0\x9F\x8C\x8D Язык"],['text' => "\xE2\x84\xB9 Помощь"]]; $bot->sendMessage($chatId, $welcomeMsg, array('keyboard' => $buttons, 'resize_keyboard' => true)); break; } // Помощь case "\xE2\x84\xB9": { help($bot, $chatId); break; }

What is wrong here?

  • structure show - Anatol
  • what is contained in $ rMessage [1]? why don't you use $ chatId variable instead of $ rMessage [1]? - Maistrenko Vitalii
  • and the rest of the code too - Anatol
  • ! Here is the structure - Valery Makinsky
  • Here is all that rMessage besides what I threw off: DB :: uQuery ("INSERT INTO Reffer ( chat_id , reffers_id , money ) VALUES ('". $ ChatId. "', '". $ RMessage [1]. "' , '". REF_MONEY."') "); DB :: uQuery ("UPDATE Users SET balanse = balanse + '". REF_MONEY. "' WHERE chat_id = '". $ RMessage [1]. "'" "); - Valery Makinsky

2 answers 2

Does this code even work out at all? what version of php?

at the very beginning, you need to check username user has a username
from the user we get a string, but we are comparing with an integer

 array (size=2) 0 => string '/start' (length=6) 1 => string '001101001' (length=9)

would also replace magic quotes and concatenation.

corrections:

 //Проверка имени пользователя $username = isset($update->{"message"}->{"chat"}->{"username"}) ? $update->{"message"}->{"chat"}->{"username"} : "no_username" //Проверка на пользователя if(DB::uCountColumns("SELECT * FROM `Users` WHERE `chat_id` = '".$chatId."'")){ // обновление DB::uQuery("UPDATE `Users` SET `username`='".$username."', `lastname`='".$lastname."', `firstname`='".$firstname."' WHERE `chat_id` = '".$chatId."'"); }else{ // запись DB::uQuery("INSERT INTO `Users`(`chat_id`, `username`, `lastname`, `firstname`, `regDate`) VALUES ('".$chatId."','".$username."','".$lastname."','".$firstname."',NOW())"); } $userInfo = DB::uFetch("SELECT * FROM `Users` WHERE `chat_id` = '$chatId'"); $rMessage = explode(' ', $messageIn); switch ($rMessage[0]) { case '/start': { // Открываем файл для получения существующего содержимого // ID рефала как целое число if (intval($rMessage[1]) > 0 AND intval($rMessage[1])!=$chatId){ // проверка есть ли такой пользователь if(DB::uCountColumns("SELECT * FROM Users WHERE chat_id = '".$rMessage[1]."'") AND !DB::uCountColumns("SELECT * FROM Reffer WHERE chat_id = '".$rMessage[1]."'")){ DB::uQuery("INSERT INTO Reffer(chat_id, reffers_id, money) VALUES ('".$chatId."','".$rMessage[1]."','". REF_MONEY ."')"); DB::uQuery("UPDATE Users SET balanse= balanse + '". REF_MONEY ."' WHERE chat_id = '".$rMessage[1]."'")

    The problem is that at the very beginning there is a check on the void of the username and this is a key point in logic. If the verification condition is fulfilled and the user name is empty, the bot must send a message to the user and interrupt the further execution of the function.

      if (empty($update->{"message"}->{"chat"}->{"username"})) { $bot->sendMessage($chatId, "Уважаемый пользователь. У Вас не установлен username. Пожалуйста, установите его в настройках. После этого выполните команду /start еще раз. Спасибо за понимание"); return; }

    However, the problem is that the $chatId variable at the time of the transfer to the sendMessage method sendMessage not contain any value, respectively, the method can not be executed at all and throws an error ( this is a mandatory non-zero parameter ). The $chatId variable is $chatId and its value is filled in further by code.

     $chatId = $update->{"message"}->{"chat"}->{"id"};

    Accordingly, the solution is to transfer this line of code with the definition of the $chatId variable above the check.

    • what are you minus, anonymous? - Anatol