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

Update rx-must-die.md #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
64 changes: 64 additions & 0 deletions panels/rx-must-die.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,3 +19,67 @@ Kevin McDonagh, Novoda
* Hasan Hosgel, Lab1886 - Mercedes-Benz
* Ash Davis, ImmobilienScout24

## Why did you choose to use Rx?
Change of mindset from doing things in the background to defining behaviors.

Solution to threading problems, and handling streams of data.

It was good in good old Java times.

Because it fits nicely with all streams and can be used for all different aspects in the app.

### Why not
Coroutines are there to solve threading problems


Rx is overused . No need to apply to everything .

Do not adapt the latest things, stick to the architecture your team likes

Difficult to get rid of

When not used daily then dangerous to introduce errors and misunderstanding .

If you use only Singleton you probably don't need Rx

It might be less readable . Operators concatenation might be too much.

Stacktrace is not readable .

5000 extra methods

## not to use for
Calls to the backend
Business logic that should be in a function



#### use for
React on change of favorites
Push from the business logic
Rotation change

###alternative tools
Live Data with room
Arrow library? Different league
Coroutines
Rx bindings
Rx kotlin





Think about reactive scenarios.

Do not force an architecture by your library

Rxjava 2 was made to include backpressure

###what is missing ?
More education on real use
Split rxjava
Preserve stacktraces