Naszła mnie ochota, żeby zrobić poprawki do modułu aplikującego kolory ze zrobionych przeze mnie stylów do podanych elementów View. Jedna mała klasa która steruje całym procesem. Dzięki niej mam możliwość posiadania 3210941241 stylów w aplikacji i zmiany ich jednym kliknięciem ( ale dopiero po otworzeniu nowej aktywności, nie chce bawić mi się w dynamiczną implementację), oraz aplikacja kolorów ze stylów do obiektów które "nie są obsługiwane" automatycznie przez parametry stylów systemu Androida.
Czyli poprawianie wyglądu kodu tego gniota którego nie ruszałem od wielu miesięcy:).
Założenia które musiałem tu uwzględnić:
- Ma działać ze starymi wersjami Android API ( xxx < API version 21), czyt. ustawienie w plikach xml. koloru jako "?android:nazwa_atrybutu_ze_stylu" działa tylko dla kilku atrybutów; ma to działać np. na API version 16 oraz dla wszelkiego rodzaju dodanych własnoręcznie atrybutów;
- Kolory odczytywane są na podstawie podanego atrybutu (R.attr.nazwa_atrybutu) z bieżącego aktywnego stylu ( wszystkie style mają taki sam zestaw atrybutów, nie ma możliwości, że kolor odpowiadający atrybutowi nie zostanie odnaleziony);
- Kolor może być ustawiony dla typowego obiektu View ( kolor tła - background) lub dla View z wektorem jako tło (atrybut wektora "fillPath" - nie można ustawić koloru tego atrybutu bezpośrednio w java., ale można ustawić w xml. wektora DOMYŚLNY kolor biały i w java. DODAĆ na ten biały kolor filtr który da wygląd kolorowego obiektu);
- Kolory ustawione będą w obiekcie View jako tło ( nazwaView.setBackgroundColor(nazwaKoloru)) a dla View z vectorem jako colorFilter (nazwaView.getBackground.setColorFilter(nazwaKoloru));
- Obiekty będące polem klasy implementującej layout można przekazać bezpośrednio ( np. przekazać button mButton do metod wstawionych poniżej). Jeśli obiekt jest zagnieżdżony w innym View nie należy tworzyć nowych zmiennych za pomocą findViewById(...). Tak samo jeśli nie ma swojej reprezentacji jako pole w tamtej klasie ( np. ImageView ze zdefiniowanym obrazkiem, nie robimy pola w klasie MojaKlasa i findViewById(...)/ButterKnife bo nie ma po co). Ogólnie chodzi o to, że jeśli jest kilka obiektów tego samego typu w widoku, należy je wyciągnąć po typie ( oszczędzanie miejsca - przykład widok z 20 obiektami Button- dałby 20 linijek z wywołaniem metody, ale można iterować po wszystkich obiektach wchodzących w skład widoku, wyciągnąć obiekty danego typu (Button) i w ten sposób dodać referencję do nich do jakiejś listy).
- Istnieją następujące przypadki:
a) mamy bezpośredni dostęp do obiektu dla którego chcemy ustawić kolor, jeden obiekt ( np. ustaw kolor dla tego obiektu ImageButton który jest polem w klasie implementującej layout).
b) nie mamy bezpośredniego dostępu do obiektu dla którego chcemy ustawić kolor, jeden obiekt ( np. obiekt jest wstawiony w inny obiekt ( np. ListLayout z TextView i ImageView w środku i chcemy ustawić kolor dla ImageView)
c) nie mamy bezpośredniego dostępu do obiektu dla którego chcemy ustawić kolor, więcej niż jeden obiekt ( np. Fragment mający w sobie 10 ImageView i dla nich ma być ustawiony kolor)
d) wszystkie powyższe jeszcze raz, ale nie wiemy, czy mamy do czynienia z ustawieniem koloru tła czy dodaniem filtru koloru dla obiektu z wektorem;
Powyższe to wszystko co pamiętam na ten moment.
Podejmie się ktoś dyskusji co tu poprawić/połączyć/wywalić? :D
Klasa której kod chcę poprawić:
public class StyleController {
private Context mContext;
public StyleController(Context context) {
mContext = context;
}
public void applyColorAsBackground(View view, int attrId) {
int colorId = getAttrColor(attrId);
view.setBackgroundColor(colorId);
}
public void applyColorToSingleKnownView(View view, int attrId) {
int colorId = getAttrColor(attrId);
setColor(view, colorId);
}
public void applyColorToSingleUnknownNestedView(ViewGroup rootView, int attrId, Class<?> lookedType) {
View view = pickSingleNestedView(rootView, lookedType);
if (view != null) {
applyColorToSingleKnownView(view, attrId);
}
}
public void applyColorToMultipleViews(ViewGroup rootView, int attrId, Class<?> lookedType) {
List<View> viewList = new ArrayList<>();
pickMultipleNestedViews(rootView, viewList, lookedType);
applyColorToView(viewList, attrId);
}
private void pickMultipleNestedViews(ViewGroup viewGroup, List<View> viewList, Class<?> lookedType) {
for (int i=0; i< viewGroup.getChildCount(); i++) {
if (viewGroup.getChildAt(i) instanceof ViewGroup) {
pickMultipleNestedViews((ViewGroup) viewGroup.getChildAt(i), viewList, lookedType);
} else if (lookedType.isInstance(viewGroup.getChildAt(i))){
viewList.add(viewGroup.getChildAt(i));
}
}
}
private View pickSingleNestedView(ViewGroup viewGroup, Class<?> lookedType) {
for (int i=0; i< viewGroup.getChildCount(); i++) {
if (viewGroup.getChildAt(i) instanceof ViewGroup) {
pickSingleNestedView((ViewGroup) viewGroup.getChildAt(i), lookedType);
} else if (lookedType.isInstance(viewGroup.getChildAt(i))){
return viewGroup.getChildAt(i);
}
}
return null;
}
private void applyColorToView(List<View> viewList, int attrId) {
int colorId = getAttrColor(attrId);
for (View view : viewList) {
setColor(view, colorId);
}
}
public int getAttrColor(int attrId) {
TypedValue typedValue = new TypedValue();
// !!! set boolean resolveRes as true to make it work!!!
mContext.getTheme().resolveAttribute(attrId, typedValue, true);
return typedValue.data;
}
private void setColor(View view, int colorId) {
if (isBackgroundInvisible(view)) {
setBackgroundColor(view, colorId);
} else {
setColorFilter(view, colorId);
}
}
private boolean isBackgroundInvisible(View view) {
return view.getBackground() == null || !view.getBackground().isVisible();
}
private void setBackgroundColor(View view, int colorId) {
view.setBackgroundColor(colorId);
}
private void setColorFilter(View view, int colorId) {
view.getBackground().setColorFilter(colorId, PorterDuff.Mode.MULTIPLY);
}
public int getStyleAsInteger(String themeName) {
int themeId;
switch (themeName) {
case "Light":
themeId = R.style.Light;
break;
case "Dark":
themeId = R.style.Dark;
break;
case "Candy":
themeId = R.style.Candy;
break;
case "Dracula":
themeId = R.style.Dracula;
break;
case "Ocean":
themeId = R.style.Ocean;
break;
case "Forest":
themeId = R.style.Forest;
break;
default:
themeId = R.style.Light;
}
return themeId;
}
}
I przykładowe użycie wygląda tak ( przyciski mają w sobie obrazek wektora, więc wymagane dla nich jest ustawienie filtru koloru a nie zmiana koloru tła:
public class ExampleFragment extends BasicFragment {
@BindView(R.id.reset_button) ImageButton mRefreshButton;
@BindView(R.id.pause_button) ImageButton mPlayPauseButton;
@BindView(R.id.lock_button) ImageButton mLockButton;
@BindView(R.id.map_button) ImageButton mMapButton;
@BindView(R.id.gps_button) ImageButton mGpsButton;
@BindView(R.id.network_button) ImageButton mNetworkButton;
@BindView(R.id.barometer_button) ImageButton mBarometerButton;
private StyleController styleController;
private void setButtonColorsByStyle() {
ViewGroup viewContainer = (ViewGroup) getView();
Class<?> lookedType = ImageButton.class;
applyColorToMultipleViews(viewContainer, R.attr.colorButtonPrimary, lookedType);
}
}
Mi w oczy od razu rzuciły się:
- możliwy null zwrócony (!!!) w metodzie "pickSingleNestedView"
- nazewnictwo "applyColorToSingleKnownView" i "applyColorToSingleUnknownNestedView" - robiłem to, więc wiem =, że Known i Unknown miało oznaczać czy mamy dostęp do obiektu ( bezpośredni) i czy trzeba ustawić tam tło, cz filter, ale nie sądzę, żeby obca osoba łatwo to odczytała
- możliwość połączenia metod "applyColorAsBackground", "applyColorToSingleKnownView" i "applyColorToSingleUnknownNestedView" - wszystkie ustawiają kolor dla jednego widoku, więc należałoby tutaj pokombinować...
- metoda "getStyleAsInteger" długi switch
Jakiekolwiek uwagi mile widziane. Istotą wstawienia tego jest refactoring tego kodu do lepszej postaci, nie czy jest zasadny ( chociaż też uwagi przyjmę) :)