Forum PHP.pl

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

Taki przykład.

PSEUDOKOD:


zamienia sie w:


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

Ten post edytował PrinceOfPersia 1.05.2015, 13:27:07

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 

[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:

|| 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)


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

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.

Ten post edytował peter13135 3.05.2015, 13:58:03