Kalkulator napiwków. Co dalej?

Question

Napisałem prostą aplikację na androida, która może okazać się pomocna w restauracji kiedy nie chce nam się liczyć ile musimy zapłacić ze znajomymi za biesiadę tak żebvy nikt nie był pokrzywdzony. Fajnie by było usłyszeć zdanie innych ludzi o tym projekcie. Mi się podoba bardzo ale wiadomo jako twórca nie jestem obiektywny :D

Dodatkowo po zakończeniu pracy nad kalkulatorem pojawił się problem braku pomysłu na kolejne projekty i też byłoby fajnie gdybyście sypnęli jakimiś ideami na nową aplikację. Wiem że jest pełno aplikacji i po prostu mogę napisać coś co już jest ale jak widać jeszcze tego nie zrobiłem i chyba lepiej mi się pracuję keidy ktoś mi powie weź zrób to i to bo może być fajne.

Link do projektu na githubie: https://github.com/ShrekForEver/MyFirstApp
Link do apk (można sobie samemu skomplikować projekt do apk ale jako że ide android studio jest ciężkie to wrzucam link): http://hostuje.net/file.php?id=c6b53eed9166f4881e3e65a77500ff6a
Obrazek z aplikacji:

Answer 1

Jeżeli chodzi o gita, to istnieje coś takiego jak ".gitignore" i tam wrzuca się pliki i ścieżki które nie będą wrzucane do gita. Np vendory, pliki/foldery edytora z którego korzystamy.

https://github.com/ShrekForEver/MyFirstApp/blob/master/app/src/main/java/com/example/michal/myfirstapp/MyActivity.java

Ja Cię kręcę!

wysokoscNapiwkuTV, razemDoZaplaty, doZaplatyNaOsobe,wysokoscRachunkuET,liczbaOsobET,innyWyborET

oraz

onCreateOptionsMenu, onCreate, onResume 

Przy tym:

wysokoscNapiwkuTV.setText(String.valueOf(BillData.wysokoscNapiwku)); 

Zdecyduj się, czy piszesz po Polsku czy Angielsku? 

public void ustalWysokoscNapiwku(View view) {
		EditText innyProcentNapiwku = (EditText) findViewById(R.id.inny_wybor_et);
		int left  = (int) this.getResources().getDimension(R.dimen.padding_left_right_text);
		int right = (int) this.getResources().getDimension(R.dimen.padding_left_right_text);
		int top = (int) this.getResources().getDimension(R.dimen.padding_top_bottom_text);
		int bottom = (int) this.getResources().getDimension(R.dimen.padding_top_bottom_text);
		if (view == findViewById(R.id.radio1)) {
			innyProcentNapiwku.setVisibility(View.INVISIBLE);
			BillData.procentNapiwku = 10;
			(( RadioButton ) findViewById(R.id.radio2)).setChecked(false);
			((RadioButton ) findViewById(R.id.radio3)).setChecked(false);
			tip.setBackground(this.getResources().getDrawable(R.drawable.text_view_background_default_rounded));
			tip.setPadding(left, top, right, bottom);
		} else if (view == findViewById(R.id.radio2)) {
			innyProcentNapiwku.setVisibility(View.INVISIBLE);
			BillData.procentNapiwku = 15;
			((RadioButton ) findViewById(R.id.radio1)).setChecked(false);
			((RadioButton ) findViewById(R.id.radio3)).setChecked(false);
			tip.setBackground(this.getResources().getDrawable(R.drawable.text_view_background_default_rounded));
			tip.setPadding(left, top, right, bottom);

		} else if (view == findViewById(R.id.radio3)) {
			innyProcentNapiwku.setVisibility(View.VISIBLE);
			((RadioButton ) findViewById(R.id.radio2)).setChecked(false);
			((RadioButton ) findViewById(R.id.radio1)).setChecked(false);
			tip.setBackground(this.getResources().getDrawable(R.drawable.text_view_background_default));
			tip.setPadding(left, top, right, bottom);

		}
	}

Zdajesz sobię sprawę, że ten kod wykonuje:

• trzy weryfikacje warunków
• zmienia status checked dwóch buttonów
• zmienia wygląd  

Jest to zbyt wiele odpowiedzialności jak na jedną funcję. 

Pozostawiony zakomentowany kod, który nie wygląda profesjonalnie.

Tak samo masa innych funkcji, która posiada masę elsifów.  

 

Jeżeli chodzi o wizualną stronę to jest przeciętnie - można się postarać o dobranie dużo ładniejszych kolorów bardziej współgrających ze sobą. 

 

Comments

No i gdzie są testy?