vldf/fedclient

Simple Federation Messenger Client

fedclient's Issues

Implementation

1. Я не до конца согласна с тем,  что предложенная реализация клиента — MVC в чистом виде (посмотрите, пожалуйста, пример, который  я Вам прислала на почту). Детали работы с конфигом при инициализации API стоит вынести в Api, так же, как и цикл поллинга сообщений. Для того, чтобы перерисовывать графику при этом, можно реализовать паттерн Publish-Subscribe и подписываться в main на обновления;
2. Относительно работы с конфигом — см. комментарий про архитектуру на исключениях к проекту сервера. Аналогично про ошибки — текстовые константы, про логирование, про возможность корректно завершить работу клиента не только в качестве исключения;
3. "/n" - не единственный вариант завершения строки. Для того, чтобы код был устойчив к возможным переездам между разными версиями ОС, имеет смысл пользоваться, например, методом System.lineSeparator(). Аналогичное верно для разделителя путей ФС;
4. if (lastAddedChatTextLen > 10) — магические константы — все еще — не очень хорошая идея;
5. Циклы перерисовки чата по полученным сообщениям/ по внутренним изменениям с логической точки зрения должны были бы быть разнесены (или условия должны быть модифицированы; пока читается с трудом) — но это, видимо, после publish/ subscribe уйдет;
6. Вспомогательный метод для тестов лучше держать в тестах;

Tests

1. generateToken, раз уж он у Вас так часто встречается, вообще говоря, можно попробовать вынести в   отдельную библиотеку и подключать ее через тот же Gradle;

2. См. комментарии относительно тестирования к серверу.