Добавил команду /givepass для продления проходок#4
Добавил команду /givepass для продления проходок#4acrighthere wants to merge 1 commit intoJouTak:masterfrom acrighthere:feature/givepass
Conversation
|
не, лапитана не надо пинговать, @ArtemBalakin всё видит |
There was a problem hiding this comment.
такое грузить не оч хорошо, это должно контролится через gitignore, но следи все равно)
There was a problem hiding this comment.
а как ты видишь, какая цель у этого класса? т.е , какую задачу он решает
There was a problem hiding this comment.
Если на сервере какие-то тех. неполадки и нужно продлить пользователям проходки ибо они не виноваты в технических проблемах.
There was a problem hiding this comment.
Я имел в виду несколько иной подход: класс должен выполнять только действия, связанные с конкретной командой, например, продлением проходок. В текущей реализации он дополнительно занимается чтением и записью данных в файл.
В данном случае, класс небольшой, это не создаёт значительных проблем. Однако в более сложных проектах предпочтительно разделять функциональность, создавая отдельные утилитные классы для работы с файлами. Постоянное создание объектов для чтения и записи (Reader/Writer) вручную, вместо использования готовых экземпляров из одного класса, может привести к сложностям с отладкой ошибок, связанных с их настройками.
There was a problem hiding this comment.
Да,я понимаю, просто посчитал что в данном случае так как проект маленький, то можно реализовать reader/writer не вынося их в отдельный класс. Для масштабируемости конечно лучше вынести, если надо, то могу реализовать
There was a problem hiding this comment.
Кстати, этого в задаче не было, но чисто на "подумать". Часто бывает, что либо в БД, либо с фронта( если он есть), приходит Json, с полями, названия которых отличаются от бековских.
Типа: в коде у нас lastPaymentDate, а приходит last_payment_date, есть представление, как такое разруливать?
There was a problem hiding this comment.
В Gson если я правильно понимаю можно использовать аннотацию @SerializedName в котором указывается имя переменной с которой к нам приходит Json.
3 participants
PlayerJoinListner-фиксирует заход каждого игрока и добавляет информацию по нему в json файл нужного формата
PlayerPass-содержит структуру Json
GivePassCommand-Реализация команды раздачи проходок