-
Notifications
You must be signed in to change notification settings - Fork 17
Воронина Веда #2
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
base: master
Are you sure you want to change the base?
Conversation
| this.name = name; | ||
| } | ||
|
|
||
| public void setCover_small(String cover_small) { |
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.
Немного не джава-вэй. camalCaseEverywhere:)
|
Подправила, чуть позже добавляю AutoValue + Moshi + AutoParcel |
| Log.w("ListFragment", "OnCreateView"); | ||
| View rootView = inflater.inflate(R.layout.fragment_list, container, false); | ||
| ButterKnife.bind(this, rootView); | ||
| if (getActivity() instanceof ListProvider && getActivity() instanceof PerformerSelectedListener) { |
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.
Исключение всё же кидать надо, если активити эти интерфейсы не реализует.
| selectionArgs, null, null, sortOrder); | ||
|
|
||
| cursor.setNotificationUri(getContext().getContentResolver(), | ||
| CONTENT_URI); |
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.
setNotificationUri для чего, если не секрет?
| +");"); | ||
|
|
||
| db.execSQL("CREATE INDEX idx_genres_" + Genre.ID + | ||
| " ON " + GENRES + "(" + Genre.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.
Нужен?
ВСЕ, ЖДУ ОБНОВЛЕНИЙ |
|
Встроенную функцию для журнала не нашла, остальное все вроде поправила |
| mDbOpenHelper = new DBHelper(context); | ||
| } | ||
|
|
||
| public void insertSinger(Singer singer) { |
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.
Транзакцию не хочешь оформить?
| public void insertSinger(Singer singer) { | ||
| SQLiteDatabase db = mDbOpenHelper.getWritableDatabase(); | ||
| ContentValues values = createCV(singer); | ||
| db.beginTransaction(); |
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.
У меня вопрос - почему при транзакциях использут try-finally блоки?
# Conflicts: # server/src/main/java/com/example/vorona/server/db/DbBackend.java
Воронина Веда