Mvp screens #3

eldarovich99 · 2018-10-27T18:06:20Z

No description provided.

Yundin · 2018-10-28T10:40:59Z

app/src/main/java/com/styleru/styleruapp/navigation/ScreenKeys.java

-    public static final String PROFILE_ACTIVITY = "profile";
-    public static final String EVENTS_ACTIVITY = "events";
-    public static final String PEOPLE_ACTIVITY = "people";
+    public static final String DIRECTIONS_FRAGMENT = "category";


Тут ошибка

переименовал все Category в Directions, кроме этого места

Yundin · 2018-10-28T10:41:50Z

app/src/main/java/com/styleru/styleruapp/presentation/login_screen/LoginPresenter.java

@@ -12,7 +12,7 @@
 @InjectViewState
 public final class LoginPresenter extends MvpPresenter<LoginView> {
    private final StyleruRouter mRouter;
-    public AuthorizationInteractor mInteractor;
+    private AuthorizationInteractor mInteractor;


Лучше еще и final

Yundin · 2018-10-28T10:43:04Z

app/src/main/java/com/styleru/styleruapp/presentation/main_screen/MainActivityPresenter.java

+    MainActivityPresenter(StyleruRouter router){this.mRouter = router;}
+
+    void changeScreen(String key){
+        switch (key) {


А зачем здесь свич, если метод принимает ключ экрана?

Yundin · 2018-10-28T10:44:48Z

...va/com/styleru/styleruapp/presentation/main_screen/directions_screen/DirectionsFragment.java

+            View item = inflater.inflate(R.layout.item_category, mLinearLayout, false);
+            TextView directionName = item.findViewById(R.id.category_name);
+            directionName.setText(element);
+            mLinearLayout.addView(item);


Можешь объяснить, что делает этот цикл?

заполняет направления (добавляет в layout каждое направление)

А как должен выглядеть этот экран с направлениями?
Если нужно динамически заполнить что-то однотипное, надо использовать ресайклер

зачем? там всего 4 направления...их все равно не будет больше, чем поместится на экране

Чтобы избавится от говнокода

и ухудшить производительность

Эм, сделать производительность хуже, чем 4 х (inflate + findViewById) сложно

Увидел дизайн.
Нужен ресайклер, тк направления в теории могут добавляться/изчезать/фильтроваться итд.
Ну и эта реализация максимально плохая по производительности

Yundin · 2018-10-28T10:45:22Z

...va/com/styleru/styleruapp/presentation/main_screen/directions_screen/DirectionsFragment.java

+    @Override
+    public void onDestroyView() {
+        super.onDestroyView();
+        mUnbinder.unbind();


Зачем unbind?

я объяснял уже, это используется во фрагментах, если не ошибаюсь, чтобы не было утечки памяти

Фрагменты сами справятся со своим уничтожением, там все ок

Yundin · 2018-10-28T10:47:35Z

...va/com/styleru/styleruapp/presentation/main_screen/directions_screen/DirectionsFragment.java

+            mPresenter.changeScreen(menuItem);
+            return true;}
+        );
+        return view;


Все листенеры и другие несрочные действия лучше сделать в onViewCreated

Это для всех фрагментов

Yundin · 2018-10-28T10:47:46Z

...a/com/styleru/styleruapp/presentation/main_screen/directions_screen/DirectionsPresenter.java

+
+@InjectViewState
+public final class DirectionsPresenter extends MvpPresenter<DirectionsView> {
+    private StyleruRouter mRouter;


Yundin · 2018-10-28T10:51:03Z

.../main/java/com/styleru/styleruapp/presentation/main_screen/events_screen/EventsFragment.java

+        EventItem sampleProfile = new EventItem("https://upload.wikimedia.org/wikipedia/commons/thumb/1/17/Bufotes_oblongus.jpg/275px-Bufotes_oblongus.jpg", "java meetup", "31.12.2018", "red square");
+        List<EventItem> mProfiles = new ArrayList<>();
+        for (int i = 0; i < 100; i++){
+            mProfiles.add(sampleProfile);


Используй fill

нет такого метода (для mProfiles)

Collections.fill

Yundin · 2018-10-28T10:53:48Z

...main/java/com/styleru/styleruapp/presentation/main_screen/events_screen/EventsPresenter.java

+                break;
+            case R.id.profile_menu:
+                mRouter.replaceScreen(ScreenKeys.PROFILE_FRAGMENT);
+                break;


Я тут подумал, что презентер теперь слишком сильно привязан к конкретной реализации, это плохо.
Принимать ключ тоже не очень хорошо, вдруг надо сделать что-то перед уходом на конкретный экран.
Сделай в таких кейсах отдельный метод презентера для каждого следующего экрана и дергай его

Yundin · 2018-10-28T10:56:28Z

...ain/java/com/styleru/styleruapp/presentation/main_screen/profile_screen/ProfileFragment.java

@@ -100,14 +111,6 @@ private void setLinks(HashMap<String, String> links, View view){
                    linkTextViewID = R.id.textView2;
                    linkEditTextID = R.id.editText2;
                    break;


Господь, это здесь зачем?

пустые строчки для дополнительных ссылок

Можно дизайн? Это выглядит максимально крипово и ненужно

Все равно не понял, что это за линки и что они делают

дополнительные ссылки (опциональные)

По какому дизайну ты верстаешь?
Или это ты придумал эти ссылки?

так Глеб сказал, по дизайну еще ничего не менял

А что именно он сказал? Что это за ссылки? Обязательно ли их создавать таким говнокодом или сюда тоже можно ресайклер?

Yundin · 2018-10-28T11:00:00Z

app/src/main/res/layout/fragment_profile.xml

-        android:layout_width="0dp"
+    <android.support.design.widget.BottomNavigationView xmlns:android="http://schemas.android.com/apk/res/android"
+        xmlns:app="http://schemas.android.com/apk/res-auto"
+        android:id="@+id/bottom_navigation"


А зачем во фрагменте свой навбар? Он один в MainActivity

там меняется название, а еще ConstraintLayout не поддерживает нормально Include...

Еще как поддерживает.
Название чего меняется? Можно дизайн?

хз, я bottomnavigation пытался заинклюдить, всегда вверх улетает...

Тебе не нужен собственный бар в каждом фрагменте от слова совсем.
Так все, что не имеет констрейнтов улетает вверх, заинклюженных лэйаутов это тоже касается

в любом случае улетает и констрейнты не помогут

Размеры ему задай

Yundin · 2018-10-28T11:00:24Z

gradle/wrapper/gradle-wrapper.properties

 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-4.4-all.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-4.6-all.zip


Это надо в gitignore

eldarovich99 added 5 commits October 24, 2018 23:34

added mvp templates (directions, events and profile screens)

5019208

added profile screen

c7d722a

fragments stack fixed

15edb06

bottom navigation selection partially fixed

74c5904

move some logic to presenter

264c459

eldarovich99 requested a review from Yundin October 27, 2018 18:06

Yundin reviewed Oct 28, 2018

View reviewed changes

eldarovich99 added 12 commits October 30, 2018 23:22

make all Router fields in presenters final

9f8a4f1

removed switch and key from MainActivityPresenter

9bdfce8

moved non-urgent logic in fragments to onViewCreated method

c947dd5

editted filling the list of profiles

7321e4c

replaced a listview and a dog-nail by recyclers

a2d64c6

removed another bottom navigation from fragment_profile

b76fd23

methods of replacing screens moved from presenter to fragments

35e4d41

added listener in PeopleFragment

d9c7eaa

fixed recyclerview (overScrollMode) in PeopleFragment

46f764c

added edit button in profileFragment

c4cf1c9

now the whole page in ProfileFragment scrolls

3985e3b

added margin

bbed032

eldarovich99 added 6 commits November 16, 2018 14:17

added edit page

b794bfc

added buttons for edit page

70079cd

added 3 categories of events (ViewPager+TabLayout)

35b3bc3

added ParticularEventFragment

84eaca4

added ParticularEventInteractor

ef5b174

added toolbar style instead of boilerplate

4a2e8d2

Mvp screens #3

Are you sure you want to change the base?

Mvp screens #3

Conversation

eldarovich99 commented Oct 27, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Yundin Oct 28, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Yundin Oct 28, 2018 •

edited

Loading