[HTML][PHP]Własna funkcja w php przestała działać, Własna funkcja w php działająca w jednym pliku w drugim już nie działa Opcje

[rulfy]

Witam.
Napisałem własną funkcję do wyboru nazw drużyn z bazy danych za pomocą: <select name=""> o nazwie drużyna.
Kiedy użyłem ją we wcześniejszym pliku działała bez zarzutu, ale po próbie wykorzystania jej w następnym już nie. Jak ktoś jest wstanie mi powiedzieć dlaczego to by było super (IMG:style_emoticons/default/smile.gif)

Ciąg Dalszy Postu (IMG:style_emoticons/default/smile.gif)

Kod Funkcji druzyna($a) z pliku o nazwie druzyna.php:

[PHP] pobierz, plaintext 
<?php
	function druzyna($a)
	{
 
 
		$zapytanie = mysql_query("SELECT nazwa FROM druzyny");
		if($a == 2)
		{
			echo '<select name="goscie">';
				echo '<option> --- </option>';	
				while($wynik = mysql_fetch_array($zapytanie))
				{
					echo '<option value="'.$wynik['nazwa'].'">';
					echo $wynik['nazwa'];
					echo '</option>';
				}
 
				if(isset($_POST['goscie']))
				{
					$dr = $_POST['goscie'];
				}
 
			echo '</select>';
		}
 
		if($a == 1)
		{
			echo '<select name="gospodarze">';
				echo '<option> --- </option>';	
				while($wynik = mysql_fetch_array($zapytanie))
				{
					echo '<option value="'.$wynik['nazwa'].'">';
					echo $wynik['nazwa'];
					echo '</option>';
				}
 
				if(isset($_POST['gospodarze']))
				{
					$dr = $_POST['gospodarze'];
				}
 
			echo '</select>';
		}
 
		return $dr;
	}
?>
[PHP] pobierz, plaintext 

[mortus]

W funkcji druzyna() masz przypisania:

[PHP] pobierz, plaintext 
// ...
$dr = $_POST['gospodarze'];
// ...
$dr = $_POST['goscie'];
[PHP] pobierz, plaintext 

Jeśli w tablicy $_POST nie przekażesz choćby pustej wartości zmiennej $_POST['gospodarze'] lub/i $_POST['goscie'], to do przypisania nigdy nie dojdzie. Dodatkowo zmienna $dr nie jest nigdzie zainicjowana domyślną wartością. Tak na dobrą sprawę funkcja zwraca to, o czym pisałem w poprzednim poście. Jest to błąd logiczny, ponieważ w tablicy $_POST nie zawsze będą znajdować się odpowiednie dane. Poza tym sama funkcja ze zmiennych $_POST['gospodarze'] i $_POST['goscie'] nie korzysta, a jedynie je zwraca - bez sensu, bo tutaj funkcji nie potrzeba.

Przeanalizuj cały skrypt jeszcze raz, bo ewidentnie coś jest nie tak z logiką biznesową aplikacji.

Poza tym niby tworzysz funkcję, ale nadal nie przestrzgasz reguły DRY. Przecież ta funkcja może wyglądać lepiej:

[PHP] pobierz, plaintext 
<?php
function druzyna($a) {
  $zapytanie = mysql_query("SELECT nazwa FROM druzyny");
  $name = ($a == 2 ? 'goscie' : 'gospodarze');
  echo '<select name="'.$name.'">';
  echo '<option> --- </option>';
  while($wynik = mysql_fetch_array($zapytanie)) {
    echo '<option value="'.$wynik['nazwa'].'">';
    echo $wynik['nazwa'];
    echo '</option>';
  }
  echo '</select>';
  // Uważam, że poniższy kod jest bez sensu i nie ma prawa bytu, a przepisałem go tylko po to, żeby pokazać, jak może wyglądać Twoja funkcja
  if(isset($_POST[$name])) {
    $dr = $_POST[$name];
  }
  return $dr;
}
?>
[PHP] pobierz, plaintext 

Co do uwagi aachi: Twoja funkcja nazywa się druzyna() a wywołujesz funkcję druzyny().