Skip to content

@aleien, Манюхина - #1

Open
aleien wants to merge 19 commits into
yamblz-native:masterfrom
aleien:master
Open

@aleien, Манюхина#1
aleien wants to merge 19 commits into
yamblz-native:masterfrom
aleien:master

Conversation

@aleien

@aleien aleien commented Jul 13, 2016

Copy link
Copy Markdown

Тестим как работают ПР-ы)


@NonNull private final Set<String> dataResults = new LinkedHashSet<>();
@NonNull
private final Set<String> dataResults = new LinkedHashSet<>();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

А как же потокобезопасность коллекций?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Про потокобезопасность не подумала, сегодня еще посмотрю, спасибо

@Archinamon

Archinamon commented Jul 14, 2016

Copy link
Copy Markdown
Collaborator

assert smth != null
Никогда не сработает, т.к. JVM на телефонах запускается с флагом игнора ассертов ;) Будет NPE обычный строчкой ниже.

Плюс, никак не хендлится поворот :)

@Tagakov

Tagakov commented Jul 14, 2016

Copy link
Copy Markdown
Member

Повороты не обязательные. Лекция была не об этом.

@aleien

aleien commented Jul 14, 2016

Copy link
Copy Markdown
Author

О, про assert не знала. Я их обычно не использую вообще никогда (:
Повороты - ок, лучше сосредоточусь на потоках тогда.

aleien added 9 commits July 16, 2016 12:46
# Conflicts:
#	app/src/main/java/ru/yandex/yamblz/concurrency/LoadProducer.java
#	app/src/main/java/ru/yandex/yamblz/concurrency/PostConsumer.java
#	app/src/main/java/ru/yandex/yamblz/ui/fragments/ContentFragment.java
@aleien aleien changed the title Стандартное решение aleien, Манюхина Jul 16, 2016
@aleien aleien changed the title aleien, Манюхина @aleien, Манюхина Jul 16, 2016
@aleien

aleien commented Jul 16, 2016

Copy link
Copy Markdown
Author

Провожу эксперименты, пробую разные варианты
https://github.com/aleien/concurrency/wiki

@BindView(R.id.hello)
TextView helloView;

public static Exchanger<Set> EXCHANGER = new Exchanger<>();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

final?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants