Skip to content
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

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Conversation

guliash
Copy link

@guliash guliash commented Aug 9, 2016

Приложение разбито на 3 модуля - клиент (app), контент провайдер(provider), контракты для провайдера(singerscontracts).

Шаги установки:

  • Вначале ставим provider
  • Потом ставим app

Для планшетов стоит квалификатор sw600dp.

DescFragment.java - Диалог фрагмент с подробной информацией
PreviewFragment.java - Превью фрагмент с фото и кнопкой перехода на подробную
ListFragment.java - Фрагмент со списком для планшетов (sw600dp)
TabsFragment.java - Фрагмент с вкладками для телефонов
Provider - классы для работы с ContentProvider

Демонстрация:

Genres.TABLE_NAME,
SingersGenres.TABLE_NAME + "." + SingersGenres.GENRE_ID,
Genres.TABLE_NAME + "." + Genres.ID,
Singers.TABLE_NAME + "." + Singers.ID);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Почему String.format ? Чем просто плюсик не подошел?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Запрос большой. Хотелось его хотя бы как-то привести в читаемый вид.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Если не делать вложенные запросы, то этот огромный запрос тебе не понадобится.
Или можешь посмотреть в сторону CREATE VIEW - по-моему, он делает как раз то, что ты хочешь.

@zagayevskiy
Copy link
Contributor

Виден лаг, когда возвращаешься из деталей в пейджер.

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);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bind удобнее в onViewCreated делать.
Про unbind в onDestroyView помнишь?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не вспомнил, но нагуглил:)

@guliash
Copy link
Author

guliash commented Aug 9, 2016

Лаг с возвратом пофикшен. Вроде бы нашлось применение hide/show:)

}
}

private List<Singer> mSingers;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Почему в списке? ведь их может быть очень много.

Copy link
Contributor

@zagayevskiy zagayevskiy Aug 11, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А нам мапкит результаты поиска (СЕРПа) списками отдаёт, может они чего-то не знают?..

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;");

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

для foreign_keys тоже есть фция, если что

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Да, но там min_sdk требуется 16. А у меня как назло стоит 15. Точнее, студия сама по дефолту выставляет при создании модуля.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ок, понял. Я и забыл уже про 15 АПИ.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants