Wątpliwości co do jakości kodu [java]

pytanie zadane 15 lipca 2016 w Java przez michalxi1410 Początkujący (480 p.)

Witam zaczynam dopiero się uczyć javy, więc postanowiłem poszukać sobie jakichś zadań do zrobienie. Zatrzymałem się na zadaniu 5 - http://www.jkozak.pl/przedmioty/java/zadania1.pdf ponieważ mam wątpliwości co do jakości mojego kodu... Czy można by było np ustalić jakiś stały zbiór wartości dla danej zmiennej i dać to do pętli żeby się wykonywała dotąd, aż użytkownik poda poprawne dane?

import java.util.Scanner;

class Sklep {

	public void raty(int cena, int raty){
		
		if(cena <= 100000 && cena >= 10000 && raty >= 6 && raty <= 48){	
			
			double cenaKoncowa;
			
			if(raty >= 6 && raty <=12){		
				cenaKoncowa = cena + cena * 0.025; 
			}else if(raty > 12 && raty <=24){
				cenaKoncowa = cena + cena * 0.05;			
			}else{
				cenaKoncowa = cena + cena * 0.1;
			}
			
			System.out.println("Rata: " + cenaKoncowa/raty + " Odsetki: " + (cenaKoncowa - cena));	
			
		}else{
			
			Scanner in = new Scanner(System.in);
				while(!(cena < 100000 && cena > 10000 && raty >= 6 && raty <= 48)){
					System.out.println("Podaj cene z odpowiedniego zakresu: ");
					cena = in.nextInt();
					System.out.println("Podaj liczbe rat: ");
					raty = in.nextInt();
				}	
			in.close();
		}	
	}	
}

1 odpowiedź

odpowiedź 15 lipca 2016 przez Mr. Szanowny Bywalec (2,180 p.)
edycja 15 lipca 2016 przez Mr. Szanowny

Jak rozumiem chodzi Ci o to, że jak ktoś poda dobrą cenę a zły zakres rat to i tak od nowa musi wpisać cenę? Jeśli tak to może po prostu pętla dla każdej osobnej zmiennej, dopóki nie wpisze prawidłowej, będzie się wykonywać. Ja sprawdzanie czy jest czasem błędne dał przed prawidłowymi obliczeniami. Czyli najpierw dwa if co sprawdzają każdą zmienną czy jest zła - jeśli tak to pętla. A potem prawidłowe obliczenia.

while(cena<100 || cena>10000) {...}
while(raty<6 || raty>48) {...}

prawidłowe obliczenia..

komentarz 15 lipca 2016 przez michalxi1410 Początkujący (480 p.)

Dzięki za poradę o to mi chodziło, ale chciałem też zapytać czy da się określić jakoś zakres zmiennej, żeby nie musieć pisać tych długich warunków.

komentarz 15 lipca 2016 przez Mr. Szanowny Bywalec (2,180 p.)

Nie spotkałem się z czymś takim. Właśnie po to są if'y, żeby samemu decydować co się dzieje. Gdyby były takie ograniczone int'y czy float'y to powodowałoby by raczej więcej błędów niż pożytku. Ustaliłbyś zakres od 1 do 10 i ktoś przypisałby 12 to by wtedy error wyskoczył - wtedy trzeba by było dać try i catch.

Edit. co do mojego głównego komentarza, zamiast if daj tam while.

komentarz 15 lipca 2016 przez event15 Szeryf (93,790 p.)

Zawsze długie warunki można przenieść do metody, która nazywa się na przykład isValid czy coś. wtedy bedzie po prostu if(cena.isValid) jeśli tylko Cenę rozbijesz na Encję i Value Object

W końcu pieniądz to Waluta + Wartość -> 100PLN