Skip to content

Commit 93e036a

Browse files
committed
Some cleanup.
1 parent 286c52a commit 93e036a

File tree

9 files changed

+30
-32
lines changed

9 files changed

+30
-32
lines changed

app/src/main/java/com/fernandocejas/android10/rx/sample/activity/ActivityBackpressureSamples.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,11 @@ protected void onCreate(Bundle savedInstanceState) {
4242

4343
@Override
4444
protected void onDestroy() {
45-
this.subscription.unsubscribe();
45+
subscription.unsubscribe();
4646
super.onDestroy();
4747
}
4848

4949
private void initialize() {
50-
this.subscription = Subscriptions.empty();
50+
subscription = Subscriptions.empty();
5151
}
5252
}

app/src/main/java/com/fernandocejas/android10/rx/sample/activity/ActivityObservableCompositionSamples.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,14 +34,14 @@ protected void onCreate(Bundle savedInstanceState) {
3434
super.onCreate(savedInstanceState);
3535
setContentView(R.layout.activity_samples_observable_composition);
3636
ButterKnife.bind(this);
37-
this.initialize();
37+
initialize();
3838
}
3939

4040
private void initialize() {
41-
this.navigator = new Navigator();
41+
navigator = new Navigator();
4242
}
4343

4444
@OnClick(R.id.btn_sampleConcatVsFlatMap) void navigateToObservableConcatVsFlatMap() {
45-
this.navigator.navigateToObservableConcatVsFlatMap(this);
45+
navigator.navigateToObservableConcatVsFlatMap(this);
4646
}
4747
}

app/src/main/java/com/fernandocejas/android10/rx/sample/activity/ActivityObservableConcatVsFlatMapSample.java

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@
3838
public class ActivityObservableConcatVsFlatMapSample extends BaseActivity {
3939

4040
private static final String LOG_TAG = "ConcatVsFlatMap";
41-
4241
private static final String SEPARATOR = " ";
4342

4443
@Bind(R.id.tv_streamOriginalOrder) TextView tv_streamOriginalOrder;
@@ -64,14 +63,14 @@ protected void onCreate(Bundle savedInstanceState) {
6463

6564
@Override
6665
protected void onDestroy() {
67-
this.subscription.unsubscribe();
66+
subscription.unsubscribe();
6867
super.onDestroy();
6968
}
7069

7170
private void initialize() {
72-
this.dataManager = new LocalDataManager(new RandomStringGenerator(), new NumberGenerator(),
71+
dataManager = new LocalDataManager(new RandomStringGenerator(), new NumberGenerator(),
7372
JobExecutor.getInstance());
74-
this.subscription = Subscriptions.empty();
73+
subscription = Subscriptions.empty();
7574
}
7675

7776
@OnClick(R.id.btn_flatMap) void onFlatMapClick() {
@@ -95,7 +94,7 @@ private void initialize() {
9594
}
9695
};
9796

98-
this.buildNumbersObservable()
97+
buildNumbersObservable()
9998
.flatMap(dataManager::squareOfAsync)
10099
.observeOn(AndroidSchedulers.mainThread())
101100
.subscribe(observer);
@@ -122,14 +121,14 @@ private void initialize() {
122121
}
123122
};
124123

125-
this.buildNumbersObservable()
124+
buildNumbersObservable()
126125
.concatMap(dataManager::squareOfAsync)
127126
.observeOn(AndroidSchedulers.mainThread())
128127
.subscribe(observer);
129128
}
130129

131130
private Observable<Integer> buildNumbersObservable() {
132-
return this.dataManager.numbers();
131+
return dataManager.numbers();
133132
}
134133

135134
private void showToast(String message) {
@@ -141,11 +140,11 @@ private void debugLog(String message) {
141140
}
142141

143142
private void printFlatMapResult(String result) {
144-
this.tv_flatMapResult.setText(result);
143+
tv_flatMapResult.setText(result);
145144
}
146145

147146
private void printConcatMapResult(String result) {
148-
this.tv_concatMapResult.setText(result);
147+
tv_concatMapResult.setText(result);
149148
}
150149

151150
private void populateData() {

app/src/main/java/com/fernandocejas/android10/rx/sample/activity/BaseActivity.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,5 +24,10 @@ public abstract class BaseActivity extends Activity {
2424

2525
@Override protected void onCreate(Bundle savedInstanceState) {
2626
super.onCreate(savedInstanceState);
27+
initialize();
28+
}
29+
30+
private void initialize() {
31+
this.navigator = new Navigator();
2732
}
2833
}

app/src/main/java/com/fernandocejas/android10/rx/sample/activity/MainActivity.java

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
import butterknife.ButterKnife;
2020
import butterknife.OnClick;
2121
import com.fernandocejas.android10.rx.sample.R;
22-
import com.fernandocejas.android10.rx.sample.navigation.Navigator;
2322

2423
public class MainActivity extends BaseActivity {
2524

@@ -28,22 +27,17 @@ protected void onCreate(Bundle savedInstanceState) {
2827
super.onCreate(savedInstanceState);
2928
setContentView(R.layout.activity_main);
3029
ButterKnife.bind(this);
31-
this.initialize();
32-
}
33-
34-
private void initialize() {
35-
this.navigator = new Navigator();
3630
}
3731

3832
@OnClick(R.id.btn_sampleObserver) void navigateToObserverSample() {
39-
this.navigator.navigateToSubscriberSample(this);
33+
navigator.navigateToSubscriberSample(this);
4034
}
4135

4236
@OnClick(R.id.btn_sampleBackpressure) void navigateToBackpressureSample() {
43-
this.navigator.navigateToBackpressureSamples(this);
37+
navigator.navigateToBackpressureSamples(this);
4438
}
4539

4640
@OnClick(R.id.btn_sampleObservableComposition) void navigateToObservableCompositionSamples() {
47-
this.navigator.navigateToObservableCompositionSamples(this);
41+
navigator.navigateToObservableCompositionSamples(this);
4842
}
4943
}

app/src/main/java/com/fernandocejas/android10/rx/sample/adapter/ElementsAdapter.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,17 +53,17 @@ public ElementsAdapter(Context context, ElementAddedListener listener) {
5353
}
5454

5555
@Override public ElementViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
56-
final View view = this.layoutInflater.inflate(R.layout.row_element, parent, false);
56+
final View view = layoutInflater.inflate(R.layout.row_element, parent, false);
5757
return new ElementViewHolder(view);
5858
}
5959

6060
@Override public void onBindViewHolder(ElementViewHolder holder, int position) {
61-
final String element = this.elements.get(position);
61+
final String element = elements.get(position);
6262
holder.textViewText.setText(element);
6363
}
6464

6565
@Override public int getItemCount() {
66-
return (this.elements != null) ? this.elements.size() : 0;
66+
return elements.size();
6767
}
6868

6969
@Override public void onNext(String stringElement) {

app/src/main/java/com/fernandocejas/android10/rx/sample/data/LocalDataManager.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ public Observable<Integer> numbers() {
4747
}
4848

4949
public Observable<Integer> squareOfAsync(int number) {
50-
return Observable.just(number * number).subscribeOn(Schedulers.from(this.jobExecutor));
50+
return Observable.just(number * number).subscribeOn(Schedulers.from(jobExecutor));
5151
}
5252

5353
public Observable<String> elements() {
@@ -61,12 +61,12 @@ public Observable<String> newElement() {
6161
}
6262

6363
public List<Integer> getNumbersSynchronously() {
64-
return this.numbers;
64+
return numbers;
6565
}
6666

6767
private void populateElementsList() {
6868
for (int i = 0; i < 10; i++) {
69-
this.elements.add(randomStringGenerator.nextString());
69+
elements.add(randomStringGenerator.nextString());
7070
}
7171
}
7272
}

app/src/main/java/com/fernandocejas/android10/rx/sample/data/RandomStringGenerator.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ public RandomStringGenerator() {
4343

4444
public RandomStringGenerator(int stringLength) {
4545
this.stringLenght = stringLength;
46-
this.charArray = new char[this.stringLenght];
46+
this.charArray = new char[stringLenght];
4747
this.random = new Random();
4848
}
4949

app/src/main/java/com/fernandocejas/android10/rx/sample/executor/JobExecutor.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,13 +47,13 @@ public static JobExecutor getInstance() {
4747
private JobExecutor() {
4848
this.workQueue = new LinkedBlockingQueue<>();
4949
this.threadPoolExecutor = new ThreadPoolExecutor(INITIAL_POOL_SIZE, MAX_POOL_SIZE,
50-
KEEP_ALIVE_TIME, KEEP_ALIVE_TIME_UNIT, this.workQueue);
50+
KEEP_ALIVE_TIME, KEEP_ALIVE_TIME_UNIT, workQueue);
5151
}
5252

5353
@Override public void execute(Runnable runnable) {
5454
if (runnable == null) {
5555
throw new IllegalArgumentException("Runnable to execute cannot be null");
5656
}
57-
this.threadPoolExecutor.execute(runnable);
57+
threadPoolExecutor.execute(runnable);
5858
}
5959
}

0 commit comments

Comments
 (0)