Forum PHP.pl > [Java]Jak zastąpić IF polimorfizmem

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [Java]Jak zastąpić IF polimorfizmem - czysty kod

Forum PHP.pl > Forum > Przedszkole

gregi

1.05.2015, 13:15:38

czy może ktoś podrzucić przykład jak zastąpić if, case polimorfizmem ?
Chciałbym to zrozumieć : )

PrinceOfPersia

1.05.2015, 13:25:51

Taki przykład.

PSEUDOKOD:

Kod

function  drawRectangle() {
}

function  drawCircle() {
}

switch (figure.type) {
   case 'rectangle':
       drawRectangle();
       break;
   case 'circle':
       drawRectangle();
       break;
}

zamienia sie w:

Kod

class Rect implements Figure {
   function draw() {
      // rysuje prostokat
   }
}

class Circle implements Figure {
   function  draw() {
      // rysuje okrag
   }
}

figure.draw(); // wywolujemy polimorficzna metode draw zamiast switch/case.

i to był pseudokod, a jak to będzie w konkretnym języku programowania to sobie doczytasz.

gregi

2.05.2015, 07:18:44

Mam standardową i pewnie nieoptymalną konstrukcję z if. Czy da się to zrobić optymalnej zgodnie z zasadami clean code?
JAVA

[PHP] pobierz, plaintext 
if (0 == punktyUser)
		{
			connector.updateScheduledTimeSlowko_TODO(slowko_ang, 3); // update czasu
			connector.updatePointsUser(slowko_ang, 1); //update punktów
			System.out.println("Matryca =0p");
		}
 
		else if (-1 == punktyUser)
		{
 
			connector.updateScheduledTimeSlowko_TODO(slowko_ang, 3); // update
			connector.updatePointsUser(slowko_ang, 1);															// czasu
			System.out.println("Matryca =-1p");
		} else if (-2 == punktyUser)
		{
			connector.updateScheduledTimeSlowko_TODO(slowko_ang, 3);
			connector.updatePointsUser(slowko_ang, 1);	
			System.out.println("Matryca =-2p");
		}
 
		else if (-2 > punktyUser)
		{
			connector.updateScheduledTimeSlowko_TODO(slowko_ang, 3);
			connector.updatePointsUser(slowko_ang, 1);	
			System.out.println("Matryca =-<<<-2p");
 
		}
 
		else if (1 == punktyUser)
		{
			connector.updateScheduledTimeSlowko_TODO(slowko_ang, 10);
			connector.updatePointsUser(slowko_ang, 2++punktyUser);	
			System.out.println("Matryca =1");
		} else if (2 == punktyUser)
		{
			connector.updateScheduledTimeSlowko_TODO(slowko_ang, 10);
			connector.updatePointsUser(slowko_ang, ++punktyUser);	
			System.out.println("Matryca =2");
		}
 
		else if (2 < punktyUser)
		{
			connector.updateScheduledTimeSlowko_TODO(slowko_ang, 15);
			connector.updatePointsUser(slowko_ang, ++punktyUser);
			System.out.println("Matryca <<<<<<<2");
		} else
		{
			System.out.println("matryca........!!!!!!!!!......./n/n");
			System.err.println("matryca........!!!!!!!!!......./n/n punkty usera: "+punktyUser);
		}
	}
[PHP] pobierz, plaintext

PrinceOfPersia

2.05.2015, 11:44:24

[JAVA] pobierz, plaintext 
System.out.println("Matryca =-1p");
[JAVA] pobierz, plaintext

wnioskuję, że to nic nie znaczy tylko do debugowania? Więc to olewam.

Co do reszty kodu to przede wszystkim, jedyne co tak naprawdę zmieniasz to liczby w parametrach, a czasem nawet nie to:

[JAVA] pobierz, plaintext 
connector.updatePointsUser(slowko_ang, 2++punktyUser); // swoja droga tu nie ma literowki? czy ta 2 nie jest tu przez przypadek?
....
.....
connector.updatePointsUser(slowko_ang, ++punktyUser);
...
...
connector.updatePointsUser(slowko_ang, ++punktyUser);
[JAVA] pobierz, plaintext

albo:

[JAVA] pobierz, plaintext 
connector.updateScheduledTimeSlowko_TODO(slowko_ang, 3);
...
...
connector.updateScheduledTimeSlowko_TODO(slowko_ang, 10);
[JAVA] pobierz, plaintext

Ja bym zanim przechodził na polimorfizm, to zaczął od doprowadzenia tego, żeby to w formie if wyglądało normalnie. Czyli jak robisz to samo:

[JAVA] pobierz, plaintext 
connector.updatePointsUser(slowko_ang, ++punktyUser)
[JAVA] pobierz, plaintext

nie ma sensu tego pisać ponownie.

Jak się parametr, który podajesz zmienia:

[JAVA] pobierz, plaintext 
connector.updateScheduledTimeSlowko_TODO(slowko_ang, 10);
...
...
connector.updateScheduledTimeSlowko_TODO(slowko_ang, 15);
[JAVA] pobierz, plaintext

to nie ma sensu pisać ponownie całej funkcji, skoro możesz sobie wydzielić jakąś zmienną scheduleTime i tam przypisywać 10 albo 15. A potem tę zmienną odpalać:

[JAVA] pobierz, plaintext 
connector.updateScheduledTimeSlowko_TODO(slowko_ang, scheduleTime);
[JAVA] pobierz, plaintext

A co do całokształtu kodu to problem jest z tym, że za cholerę nie wiadomo o co w nim chodzi. Poza tym nie wiem, czy wiesz, ale istnieje coś takiego jak operatory logiczne AND/OR etc. i można ująć kilka warunków w jeden:

Kod

-1 == punktyUser || -2 == punktyUser

|| to OR
&& to AND
bo kod, który tam wykonujesz, niczym się nie różni od siebie (oprócz innego tekstu w System.out.println)

Cytat

A potem tę zmienną odpalać:

*miałem na myśli: a potem tę zmienną podawać do odpalanej funkcji.

peter13135

3.05.2015, 13:01:30

Cytat

Mam standardową i pewnie nieoptymalną konstrukcję z if. Czy da się to zrobić optymalnej zgodnie z zasadami clean code?

Optymanlość, a czystość kodu to dwie różne rzeczy, które niekoniecznie idą w parze.

Tam gdzie wydajność ma priorytetowe znaczenie, tam rezygnuje się z czystości kodu.

Tworzenie dodatkowych warstw abstrakcji zwiększa czytelność kodu, a więc produktywność programistów, ale ma to negatywny wpłw na wydajność.

PS. Nie ma czegoś takiego jak "bardziej optymalne" lub "mniej optymalne" . Albo coś jest optymalne, albo nie.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.