PHP + MySQL Optymalizacja kodu funkcji

Question

Witam.

Mam małe doświadczenie w PHP i MySQL tym samym proszę o pomoc w optymalizacji kodu. Kod działa jednak wg. mnie wygląda brzydko i może być później problem z jego rozbudową.

Chcę wyciągnąc z tabeli wartość INT od 1 - 5 a następnie zamienić ją na obrazek i przypisać do zmiennej (a_taktyka1, a_taktyka2, a_taktyka3, a_taktyka4), którą w późniejszej części kodu wyświetlam.

Są mi potrzebne cztery zmienne (każda będzie odpowiadać za jedną rundę walki) przez co zrobiłem cztery funkcje. Proszę o pomoc ponieważ wydaje mi się, że mozna to zmniejszyć do jednej funkcji lub rozbudowanego switch-a jednak nie mam pomysłu jak to zrobić.

Próbowałem mieszać z switch(($wiersz['a_taktyka1']) II ($wiersz['a_taktyka2']) II ($wiersz['a_taktyka3']) || ($wiersz['a_taktyka1'])) jednak i tak kod blokuje mi nazwa funkcji.

Z góry dziękuję za każdą pomoc i opinię nawet typu "zostaw jak jest"

 

Działający kod:

	// ...
	
	$rezultat = @$polaczenie->query("SELECT * FROM uzytkownicy WHERE id='$_SESSION[id]'");		
	while (($wiersz = $rezultat->fetch_assoc()) !== null)
	{
		
		$wiersz["a_taktyka1"];
		$wiersz["a_taktyka2"];
		$wiersz["a_taktyka3"];
		$wiersz["a_taktyka4"];
		
		function a_taktyka1($wiersz)
		{				
			switch($wiersz['a_taktyka1'])
			{
				case 1:
					return "<img src='src/a_taktyka1.png' width='150px' height='300px'>";
				break;
				case 2:
					return "<img src='src/a_taktyka2.png' width='150px' height='300px'>";
				break;
				case 3:
					return "<img src='src/a_taktyka3.png' width='150px' height='300px'>";
				break;
				case 4:
					return "<img src='src/a_taktyka4.png' width='150px' height='300px'>";
				break;
				case 5:
					return "<img src='src/a_taktyka5.png' width='150px' height='300px'>";
				break;
			}
		}
		
		function a_taktyka2($wiersz)
		{				
			switch($wiersz['a_taktyka2'])
			{
				case 1:
					return "<img src='src/a_taktyka1.png' width='150px' height='300px'>";
				break;
				case 2:
					return "<img src='src/a_taktyka2.png' width='150px' height='300px'>";
				break;
				case 3:
					return "<img src='src/a_taktyka3.png' width='150px' height='300px'>";
				break;
				case 4:
					return "<img src='src/a_taktyka4.png' width='150px' height='300px'>";
				break;
				case 5:
					return "<img src='src/a_taktyka5.png' width='150px' height='300px'>";
				break;
			}
		}
		
		function a_taktyka3($wiersz)
		{				
			switch($wiersz['a_taktyka3'])
			{
				case 1:
					return "<img src='src/a_taktyka1.png' width='150px' height='300px'>";
				break;
				case 2:
					return "<img src='src/a_taktyka2.png' width='150px' height='300px'>";
				break;
				case 3:
					return "<img src='src/a_taktyka3.png' width='150px' height='300px'>";
				break;
				case 4:
					return "<img src='src/a_taktyka4.png' width='150px' height='300px'>";
				break;
				case 5:
					return "<img src='src/a_taktyka5.png' width='150px' height='300px'>";
				break;
			}
		}
		
		function a_taktyka4($wiersz)
		{				
			switch($wiersz['a_taktyka4'])
			{
				case 1:
					return "<img src='src/a_taktyka1.png' width='150px' height='300px'>";
				break;
				case 2:
					return "<img src='src/a_taktyka2.png' width='150px' height='300px'>";
				break;
				case 3:
					return "<img src='src/a_taktyka3.png' width='150px' height='300px'>";
				break;
				case 4:
					return "<img src='src/a_taktyka4.png' width='150px' height='300px'>";
				break;
				case 5:
					return "<img src='src/a_taktyka5.png' width='150px' height='300px'>";
				break;
			}
		}
			
		$a_taktyka1 = a_taktyka1($wiersz);
		$a_taktyka2 = a_taktyka2($wiersz);
		$a_taktyka3 = a_taktyka3($wiersz);
		$a_taktyka4 = a_taktyka4($wiersz);

	}
	
	// ...
	
	echo $a_taktyka1;
	echo $a_taktyka2;
	echo $a_taktyka3;
	echo $a_taktyka4;
	
	// ...

 

 

Answer 1

nie potrzebujesz do tego żadnej funkcji.

 uwórz tablicę typy string o rozmiarze 5 a w nie wpisz nazwy obrazków np.

 t_obr[1]="a_taktyka1.png"  itd

a do zmiennej przypisz po prostu :

a_taktyka=t_obr[$wiersz['a_taktyka1']

  anajlepiej zrobis zjeśłi nazwy obrazków umieścisz w tabilicy mysqla, i wybierzesz je zapytaniem

 

a potem na podstawie

Comments

 anajlepiej zrobis zjeśłi nazwy obrazków umieścisz w tabilicy mysqla, i wybierzesz je zapytaniem

Raczej nie jest to najlepsze. Co gdy nazwy plików z jakiegoś powodu będzie musiały być zmienione? Po co każdemu użytkownikowi zapisywać nazwę jakiegoś tam obrazka?

---

to jest stała tablica wspólna dla wszystkich użytkowników

lepiej przechowywac takie dane w bazie a nie kodzie programu

jęśli obrazki  mają ulec zmianie to po prostu  podmienisz nazwy w tabeli bazy  a kod pozostaje niezmienny

---

No można zapisać te nazwy plików w tabeli z konfiguracją. O to ci chodziło?

---

tak, to chyba najbardziel eleganckie rowiązanie

Answer 2

Dlaczego definiujesz funkcje wewnątrz pętli? No i po ci ci ta pętla skoro wybierasz tylko jeden rekord (zakładając, że id jest unikalne, bo takie powinno być)?

Po co powielasz te funkcje i switche skoro robią dokładnie to samo?

No i po co ci ta małpa?

Z góry dziękuję za każdą pomoc i opinię nawet typu "zostaw jak jest"

Tylko totalny idiota mógłby powiedzieć, żebyś zostawił tak jak jest. 

Answer 3

O tablicy nawet nie pomyślałem

Dzięki za pomoc

 

PS. Robię sporo błędów ponieważ dopiero się uczę... THX za każdą konstruktywną krytykę