Skip to content

Commit 082e515

Browse files
committed
Refactor codes
1 parent 6fa6669 commit 082e515

File tree

1 file changed

+17
-26
lines changed

1 file changed

+17
-26
lines changed

colorpickerview/src/main/java/com/skydoves/colorpickerview/ColorPickerView.java

Lines changed: 17 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -236,14 +236,11 @@ private void onFinishInflated() {
236236
final int persisted = preferenceManager.getColor(getPreferenceName(), -1);
237237
if (palette.getDrawable() instanceof ColorHsvPalette && persisted != -1) {
238238
post(
239-
new Runnable() {
240-
@Override
241-
public void run() {
242-
try {
243-
selectByHsvColor(persisted);
244-
} catch (IllegalAccessException e) {
245-
e.printStackTrace();
246-
}
239+
() -> {
240+
try {
241+
selectByHsvColor(persisted);
242+
} catch (IllegalAccessException e) {
243+
e.printStackTrace();
247244
}
248245
});
249246
}
@@ -339,12 +336,9 @@ public boolean isHuePalette() {
339336
private void notifyColorChanged() {
340337
this.debounceHandler.removeCallbacksAndMessages(null);
341338
Runnable debounceRunnable =
342-
new Runnable() {
343-
@Override
344-
public void run() {
345-
fireColorListener(getColor(), true);
346-
notifyToFlagView(selectedPoint);
347-
}
339+
() -> {
340+
fireColorListener(getColor(), true);
341+
notifyToFlagView(selectedPoint);
348342
};
349343
this.debounceHandler.postDelayed(debounceRunnable, this.debounceDuration);
350344
}
@@ -471,19 +465,17 @@ private void notifyToFlagView(Point point) {
471465
flagView.setRotation(0);
472466
flagView.setX(posX);
473467
flagView.setY(centerPoint.y - flagView.getHeight());
474-
flagView.onRefresh(getColorEnvelope());
475468
} else {
476469
flagView.setRotation(180);
477470
flagView.setX(posX);
478471
flagView.setY(centerPoint.y + flagView.getHeight() - selector.getHeight() * 0.5f);
479-
flagView.onRefresh(getColorEnvelope());
480472
}
481473
} else {
482474
flagView.setRotation(0);
483475
flagView.setX(posX);
484476
flagView.setY(centerPoint.y - flagView.getHeight());
485-
flagView.onRefresh(getColorEnvelope());
486477
}
478+
flagView.onRefresh(getColorEnvelope());
487479
if (posX < 0) flagView.setX(0);
488480
if (posX + flagView.getMeasuredWidth() > getMeasuredWidth()) {
489481
flagView.setX(getMeasuredWidth() - flagView.getMeasuredWidth());
@@ -592,6 +584,7 @@ public void setDebounceDuration(long debounceDuration) {
592584
private Point getCenterPoint(int x, int y) {
593585
return new Point(x - (selector.getMeasuredWidth() / 2), y - (selector.getMeasuredHeight() / 2));
594586
}
587+
595588
/**
596589
* gets a selector.
597590
*
@@ -600,6 +593,7 @@ private Point getCenterPoint(int x, int y) {
600593
public ImageView getSelector() {
601594
return this.selector;
602595
}
596+
603597
/**
604598
* gets a selector's selected coordinate x.
605599
*
@@ -681,14 +675,11 @@ public void setInitialColor(@ColorInt final int color) {
681675
|| (getPreferenceName() != null
682676
&& preferenceManager.getColor(getPreferenceName(), -1) == -1)) {
683677
post(
684-
new Runnable() {
685-
@Override
686-
public void run() {
687-
try {
688-
selectByHsvColor(color);
689-
} catch (IllegalAccessException e) {
690-
e.printStackTrace();
691-
}
678+
() -> {
679+
try {
680+
selectByHsvColor(color);
681+
} catch (IllegalAccessException e) {
682+
e.printStackTrace();
692683
}
693684
});
694685
}
@@ -961,7 +952,7 @@ public void onDestroy() {
961952

962953
/** Builder class for create {@link ColorPickerView}. */
963954
public static class Builder {
964-
private Context context;
955+
private final Context context;
965956
private ColorPickerViewListener colorPickerViewListener;
966957
private int debounceDuration = 0;
967958
private FlagView flagView;

0 commit comments

Comments
 (0)