Conversation
IvanShafran
left a comment
IvanShafran
left a comment
There was a problem hiding this comment.
Сложно проверять всё вместе, если честно.
По сервису бекапа: 7 баллов. +3, когда поправишь закрытие стрима правильно.
По touch: 5 баллов. +5, когда используешь CompatGestureDetector
По релизной сборке: 6 баллов + 4 балла, когда сделаешь, чтобы релизная сборка собиралась хитро. Один ключ использовался при отсутствии второго: https://github.com/IvanShafran/action-tracker/blob/master/app/action-tracker-signing.gradle
Проблема твоего варианта, что если скачать репозиторий, то нельзя релизную сборку запустить. Приватный ключ отдавать плохо, поэтому и нужен второй тестовый.
| try { | ||
| fileOutputStream = openFileOutput(fileName, Context.MODE_PRIVATE); | ||
| fileOutputStream.write(databaseBackup.getBytes()); | ||
| fileOutputStream.close(); |
There was a problem hiding this comment.
Так неправильно закрывать stream. Используй try with resources лучше
| } | ||
|
|
||
| @Override | ||
| public boolean onTouchEvent(MotionEvent event) { |
There was a problem hiding this comment.
Вместо рукописной обработки используй CompatGestureDetector
|
|
||
| personId = -1; | ||
|
|
||
| if (findViewById(R.id.fragmentContainer2).getVisibility() == View.VISIBLE) { |
There was a problem hiding this comment.
Добавь детектор только на экране(фрагменте) профиля человека, тогда достаточно вызвать onBackPressed у активити вместо этого кода
IvanShafran
left a comment
IvanShafran
left a comment
There was a problem hiding this comment.
По сервису и release -- 10. По свайпу - 8. +2 за SimpleGestureListener
| editText.setText(savedInstanceState.getString(ARG_DESCRIPTION)); | ||
| } | ||
|
|
||
| gestureDetectorCompat = new GestureDetectorCompat(getContext(),this); |
There was a problem hiding this comment.
вместо того, чтобы имплеметировать все методы, можно подать в аргументы SimpleGestureListener и имплементировать только нужный метод onFling
| personId, | ||
| updatedValues); | ||
| */ | ||
| /* ContentValues updatedValues = new ContentValues(); |
There was a problem hiding this comment.
А почему код закоментирован? Если он не нужен, то надо удалить
2 participants
No description provided.