satesalovskiy / eventme Goto Github PK

1. Кастомные AlertDialog можно переделать в DialogFragment, чтобы работать с ними как с отдельными фрагментами и вынести эту логику из основного фрагмента
2. Логику для работы с SharedPredferences лучше вынести в отдельный класс
3. Много дублирующего кода (например работа с Picasso), вынести в отдельный класс
4. Code style: имена переменных, лишние строки, блоки кода
5. Подумать над возможностью редактирования событий
6. Подумать над хранением данных в локальной базе данных, тогда вместо передачи большого числа данных в Bundle можно будет просто получать элемент по его id в БД

1. Необходимо разбить классы по пакетам. Есть два способа:
   по назначению
   -database
   -network
   -reader
   -ui

по классам
-activities
-adapters
-fragments
-model
-utils
-widgets
Выбери наиболее удобный
2. Adapter-ы и ViewHolder-ы нужно выносить в отдельные классы
3. Многовато активити, лучше всего делать не более 5
4. Активити должно называться со словом Activity в конце, например ExampleActivity
5. Нужно создать модель, которая объединяет все эти свойства, и сделать ее Parcalable:
detailsIntent.putExtra("event_photo", eventImage);
detailsIntent.putExtra("event_topic", eventTopic);
detailsIntent.putExtra("event_day", eventDay);
detailsIntent.putExtra("event_month", eventMonth);
detailsIntent.putExtra("event_year", eventYear);
detailsIntent.putExtra("event_description", description);
detailsIntent.putExtra("event_ref", eventID);
detailsIntent.putExtra("event_creator_email", eventCreatorEmail1);
Тогда можно будет положить ее в интент одной строчкой.