0

Помогите пожалуйста с вопросом. Изучаю java и решил после разного рода упражнений, сделать модное сейчас - телеграмм бота. На данный момент бот умеет отвечать на вопрос "hi" и универсальный ответ на то что он не "знает".

Пример кода ниже. У меня есть два вопроса

  1. Есть такое ощущение что код можно сократить и упростить без потери функционала. Но как?
  2. Хочу выводить лог в консоль бота для просмотра статистики но даты и не смотря на то что на сервере (Intelj Idea) время стоит правильное и у отправителя тоже, но в консоли вывод такой: "Mon Jan 19 20:42:03 MSK 1970"
public void onUpdateReceived(Update update) {
        if (update.hasMessage() && update.getMessage().hasText()) {
               // Берем данные из API Telegram
            String message = update.getMessage().getText().trim();
            String chatId = update.getMessage().getChatId().toString();
            Integer time = update.getMessage().getDate();
            Date mytime = new Date(time);
            String user_name = update.getMessage().getFrom().getFirstName();
            String last_name = update.getMessage().getFrom().getLastName();
            String nick = update.getMessage().getFrom().getUserName();
             // Пишем лог в консоль.
            System.out.println("Пользователь: " + user_name + " " + last_name + " aka " + nick + " написал: "
                    + message + " " + mytime + " "  );

        SendMessage msg = new SendMessage();
            msg.setChatId(chatId);
            msg.setText(cmd.inputmsg(update.getMessage().getText()));
            try {
                execute(msg);
            } catch (TelegramApiException e) {
                e.printStackTrace();
            }
        }

    }

1 ответ 1

0

"Упростить"

не меняя сути код можно, убрав лишние переменные.
К примеру, Integer time существует только для того, чтобы быть переданной в конструктор Date(), от неё вполне можно избавиться:

Date mytime = new Date(update.getMessage().getDate());

Вместо заведения кучи переменных под данные пользователя - получите User user и в дальнейшем используйте его:

User user = update.getMessage().getFrom();
System.out.println("Пользователь: "+user.getFirstName();
...

для SendMessage можно использовать builder.

Ошибка вывода даты:

в соответствии с документацией TelegramBot API параметр date имеет тип integer и обозначает Date the message was sent in Unix time. Обратите внимание, что unix time это количество секунд В то же время конструктор Date в качестве параметра хочет видеть миллисекунды, т.е. полученное время нужно умножить на 1000:

Date mytime = new Date(update.getMessage().getDate() * 1000);
// возможно, нужно будет привести к long

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.