-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
@guliash & Artem Gilmudinov #5
base: master
Are you sure you want to change the base?
Conversation
Genres.TABLE_NAME, | ||
SingersGenres.TABLE_NAME + "." + SingersGenres.GENRE_ID, | ||
Genres.TABLE_NAME + "." + Genres.ID, | ||
Singers.TABLE_NAME + "." + Singers.ID); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Почему String.format ? Чем просто плюсик не подошел?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Запрос большой. Хотелось его хотя бы как-то привести в читаемый вид.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Если не делать вложенные запросы, то этот огромный запрос тебе не понадобится.
Или можешь посмотреть в сторону CREATE VIEW - по-моему, он делает как раз то, что ты хочешь.
Виден лаг, когда возвращаешься из деталей в пейджер. |
public View onCreateView(LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) { | ||
View view = inflater.inflate(R.layout.desc_fragment, container, false); | ||
|
||
ButterKnife.bind(this, view); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bind удобнее в onViewCreated делать.
Про unbind в onDestroyView помнишь?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Не вспомнил, но нагуглил:)
Лаг с возвратом пофикшен. Вроде бы нашлось применение hide/show:) |
} | ||
} | ||
|
||
private List<Singer> mSingers; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Почему в списке? ведь их может быть очень много.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А нам мапкит результаты поиска (СЕРПа) списками отдаёт, может они чего-то не знают?..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
АПИ с сервером всегда на списках.
Хотя есть решение с курсорами - у гугла вроде видел в каком-то апи.
@Override | ||
public void onOpen(SQLiteDatabase db) { | ||
super.onOpen(db); | ||
db.execSQL("PRAGMA foreign_keys=ON;"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
для foreign_keys тоже есть фция, если что
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Да, но там min_sdk требуется 16. А у меня как назло стоит 15. Точнее, студия сама по дефолту выставляет при создании модуля.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ок, понял. Я и забыл уже про 15 АПИ.
Приложение разбито на 3 модуля - клиент (app), контент провайдер(provider), контракты для провайдера(singerscontracts).
Шаги установки:
Для планшетов стоит квалификатор sw600dp.
DescFragment.java - Диалог фрагмент с подробной информацией
PreviewFragment.java - Превью фрагмент с фото и кнопкой перехода на подробную
ListFragment.java - Фрагмент со списком для планшетов (sw600dp)
TabsFragment.java - Фрагмент с вкладками для телефонов
Provider - классы для работы с ContentProvider
Демонстрация: