dvede/kvaksha

KvaKsha messenger

kvaksha's Issues

Accuracy & logic

0. В текущей реализации не вполне очевидна логика определения областей видимости полей классов активности;

1. Пакеты в java принято именовать с маленькой буквы;

2. Пока не обрабатывается большое количество ошибок (пустые обработчики);

3. Если вы приняли (хорошее) решение использовать текстовые ресурсы, то при обработке ошибок/ выводе сообщений тоже стоит применять именно их;

4. Пока не очень понятно, что произойдёт при частично удачной попытке завести, например, пользователя в FireBase - судя по всему, данные частично там и останутся;

5. Объекты используемых view стоит инициализировать однократно (лучше - в onCreate; для части активностей так и сделано, но, например, для Register - нет).

Implementation improvement

1. В текущей реализации часть логики, относящейся к слою моделей - а именно, сохранение и выборка данных из удалённой БД - попала в слой контроллера. Хотелось бы, чтобы chatroom-ы, user-ы и т.д. умели себя выбирать и сохранять сами - помочь в реализации может, например, паттерн Observable;

2. Дублирующуюся логику хелперов (например, чекер входных данных, одинаковые определения которого встречаются в нескольких активностях) лучше вынести в отдельный вспомогательный класс (кроме того, эти методы - вполне статические), туда же можно инкапсулировать повторяющиеся (static) константы - например, обеспечивающие логику работы с Firebase.