[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 

[Hpsi]

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 smile.gif

jakis blad? albo cos?

[rulfy]

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

Kod z którym ta funkcja nie działa o nazwie pan_podsumowanie.php:

[PHP] pobierz, plaintext 
<?php
	include "include/druzyna.php";
 
	$sezon = $_GET['sezon'];
	$runda = $_GET['runda'];
 
	$flaga = $_POST['flaga'];
	$nazwa = $_POST['nazwa'];
	$id = $_POST['id'];
	$punkty = $_POST['punkty'];
 
 
	if(isset($_POST['ok']))
	{
		if($flaga == p)
		{
                  ...
                }
 
		if($flaga == s)
		{
			...
		}
 
		if($flaga == k)
		{
			if(is_numeric($punkty))
			{
				zap_upd = mysql_query("UPDATE krol SET nazwisko = '$nazwisko', imie = '$imie', zespol = '$nazwa', pkt = '$punkty' WHERE id = '$id'");
				if($zap_upd = true)
				{
					echo "<script>setTimeout(function(){document.location.href='index.php?go=$go&sezon=$sezon&runda=$runda';alert('Operacja Zakończona Sukcesem');}, 0);</script>";
				}
				else
				{
					echo "<script>setTimeout(function(){document.location.href='index.php?go=$go&sezon=$sezon&runda=$runda';alert('Operacja Nieudana');}, 0);</script>";
				}
			}
			else
			{
				echo "<script>setTimeout(function(){document.location.href='index.php?go=$go&sezon=$sezon&runda=$runda';alert('Błędne Dane');}, 0);</script>";
			}
		}
	}
 
	if(isset($_POST['edit']))
	{
		if($flaga == p)
		{
			...
		}
 
		if($flaga == s)
		{
			...
		}
 
		if($flaga == k)
		{
			echo '<h1>Król Strzelców - Edycja</h1>';
			echo '<br/>';
 
			echo '<div>';
			echo '<form action="" method="post">';
 
			echo '<p>Nazwisko</p>';
				echo '<input type="text" name="nazwisko" size="50" />';
 
			echo '<p>Imię</p>';
				echo '<input type="text" name="imie" size="20" />';
 
			echo '<p>Zespół</p>';
				$nazwa = druzyny(1);
 
			echo '<p>Kosze</p>';
				echo '<input type="text" name="punkty" size="4" />';
 
				echo '<input type="hidden" name="flaga" value="'.$flaga.'" />';
				echo '<input type="hidden" name="id" value="'.$id.'" />';
				echo '<p><input class="przycisk" type="submit" name="ok" value="Zapisz" /></p>';
 
				echo '<br />';
			echo '</form>';
			echo '</div>';
		}
 
		echo '<br />';
		echo "<a href='index.php?go=$go&sezon=$sezon&runda=$runda'>Anuluj</a>";
		echo '<br />';
	}
	else
	{
		...
 
		echo '<h1>Król Strzelców - Edycja'.$sezon.'</h1>';
		echo '<br/>';
 
		$zapytanie_k = mysql_query("SELECT * FROM krol WHERE sezon='$sezon'");
		echo '<table style="table-layout: auto; width: 100%; border: 3px solid #000000;">';
			echo '<tr>';
				echo '<th style="border: 3px solid #000000;">Nazwisko</th>';
				echo '<th style="border: 3px solid #000000;">Imie</th>';
				echo '<th style="border: 3px solid #000000;">Zespół</th>';
				echo '<th style="border: 3px solid #000000;">Kosze</th>';
			echo '</tr>';
			while($wynik = mysql_fetch_array($zapytanie_k))
			{
				echo '<tr>';
					echo '<td style="border: 3px solid #000000;">'.$wynik['nazwisko'].'</td>';
					echo '<td style="border: 3px solid #000000;">'.$wynik['imie'].'</td>';
					echo '<td style="border: 3px solid #000000;">'.$wynik['zespol'].'</td>';
					echo '<td style="border: 3px solid #000000;">'.$wynik['pkt'].'</td>';
					echo '<td style="border: 3px solid #000000;">';
						echo '<form action="" method="post">';
							echo '<input type="hidden" name="flaga" value="k" />';
							echo '<input type="hidden" name="id" value="'.$wynik['id'].'" />';
							echo '<input class="przycisk" type="submit" name="edit" value="Edytuj" />';
						echo '</form>';
					echo '</td>';
				echo '</tr>';
			}
			echo '</table>';
 
		echo '<br />';
		echo "<a href='index.php?go=$go&sezon=$sezon'>Powrót</a>";
		echo '<br />';
	}	
?>
[PHP] pobierz, plaintext 

[sarxos]

ale po próbie wykorzystania jej w następnym już nie

W jakim sensie, że już nie? Znaczy się dokładnie "co jest objawem niedziałania tej funkcji".

Może to off-topic, ale powiem tylko, że uzywanie tak notorycznie echo nie jest zbyt eleganckim rozwiązaniem.

[rulfy]

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

Kod z którym ta funkcja działa o nazwie pan_wyniki.php:

[PHP] pobierz, plaintext 
<?php
 
	include "include/druzyna.php";
 
 
	$sezon = $_GET['sezon'];
	$runda = $_GET['runda'];
 
	$gosp_pkt_m1 = $_POST['gosp_pkt_m1'];
	$gosc_pkt_m1 = $_POST['gosc_pkt_m1'];
	$gosp_pkt_m2 = $_POST['gosp_pkt_m2'];
	$gosc_pkt_m2 = $_POST['gosc_pkt_m2'];
	$gospodarze = $_POST['gospodarze'];
	$goscie = $_POST['goscie'];
	$id = $_POST['id'];
 
	if($runda == 'J')
	{
		echo '<h1>Runda Jesienna</h1>';
	}
	else
	{
		echo '<h1>Runda Wiosenna</h1>';
	}
 
	echo '<br />';
 
	if(isset($_POST['edit']))
	{
		echo '<div>';
			echo "<p>$gospodarze - $goscie</p>";
		echo '</div>';
 
		echo '<form action="" method="post">';
			if($go == 6)
			{
                          ...
			}
 
			if($go == 5)
			{
				...
			}
 
 
			echo '<div>';
				echo '<input type="hidden" name="id" value="'.$id.'" />';
				echo '<p><input class="przycisk" type="submit" name="ok" value="Zapisz" /></p>';
			echo '</div>';
		echo '</form>';
 
		echo "<a href='index.php?go=$go&sezon=$sezon&runda=$runda'>Anuluj</a>";
	}
	else
	{
		...
 
	if($go == 5)
		{
			echo '<h2 style="text-align: left;">Dodaj Mecz</h2>';
 
			echo '<div>';
				echo '<form action="" method="post">';
					$kolejka = kolejka();
					$data = data();
 
					echo '<p>Gospodarze</p>';
						$gospodarze_add = druzyna(1);
 
					echo '<p>Goście</p>';
						$goscie_add = druzyna(2);
 
					echo '<div>';
						echo '<p><input class="przycisk" type="submit" name="add" value="Dodaj" /></p>';
					echo '</div>';
				echo '</form>';
			echo '</div>';
		}	
 
		echo "<a href='index.php?go=$go&sezon=$sezon'>Powrót</a>";
 
	}
 
	if(isset($_POST['ok']))
	{
		if($go == 6)
		{
			...
		}
 
		if($go == 5)
		{
			...
		}
 
 
	}
 
	if(isset($_POST['add']))
	{	
		$zap_add = mysql_query("INSERT INTO `wyniki` (`id`, `sezon`, `kolejka`, `data`, `gospodarze`, `goscie`, `runda`) VALUES ('', '$sezon', '$kolejka', '$data', '$gospodarze_add', '$goscie_add', '$runda')")or die(mysql_error());
		if($zap_add == true)
		{
			echo "<script>setTimeout(function(){document.location.href='index.php?go=$go&sezon=$sezon&runda=$runda';alert('Operacja Zakończona Sukcesem');}, 0);</script>";
		}
		else
		{
			echo "<script>setTimeout(function(){document.location.href='index.php?go=$go&sezon=$sezon&runda=$runda';alert('Operacja Nieudana');}, 0);</script>";
		}
	 }
 
 
 
 
 
?>
[PHP] pobierz, plaintext 

Dodam jeszcze że w pliku pan_podsumowanie działa to po napisaniu kodu funkcji w miejscu jej wywołania :

[PHP] pobierz, plaintext 
$zapytanie = mysql_query("SELECT nazwa FROM druzyny");
	echo '<select name="nazwa">';
		echo '<option> --- </option>';	
			while($wynik = mysql_fetch_array($zapytanie))
			{
				echo '<option value="'.$wynik['nazwa'].'">';
				echo $wynik['nazwa'];
				echo '</option>';
			}
 
			if(isset($_POST['nazwa']))
			{
				$nazwa = $_POST['nazwa'];
			}
	echo '</select>';
[PHP] pobierz, plaintext 

Bardzo proszę o pomoc. (IMG:style_emoticons/default/smile.gif)

P.S.: Ale się rozpisałem (IMG:style_emoticons/default/tongue.gif)

[Hpsi]

OMFG. sorry 99% kodu jest niepotrzebna tego co dałeś.
Co do tego, że nie jest elegancko dawać tak notorycznie echo zgodze się...
Jakiś błąd wywala Ci? jeśli nie to czy masz włączone error_raporting ?

Dodam jeszcze że w pliku pan_podsumowanie działa to po napisaniu kodu funkcji w miejscu jej wywołania :

nic dziwnego w koncu kazaleś mu zrobić bezpośrednio to co masz w funkcji (IMG:style_emoticons/default/smile.gif)

Ten post edytował Hpsi+ 15.06.2011, 15:33:10

[rulfy]

Nie wyskakują żadne błędy, ale wygląda na to jak by funkcja przestała zwracać wybraną drużynę. Zmienna $nazwa jest cały czas pusta.
Sam formularz się pojawia tylko nie zapisuje tego co powinna ta funkcja zwrócić do zmiennej do której jest przypisana.

Jakiś błąd wywala Ci? jeśli nie to czy masz włączone error_raporting ?

Jak sprawdzić czy to jest włączone ? (IMG:style_emoticons/default/tongue.gif)

[Hpsi]

A przyczepionych tematów się nie czyta?
Temat: Jak poprawnie zadac pytanie nospor bardzo dobrze Tam wytłumaczył (IMG:style_emoticons/default/smile.gif)

[rulfy]

Po włączeniu będów wyskoczyło takie coś.

Notice: Undefined variable: dr in c:\.\.\.\.\.\.\druzyna.php on line 45

tylko ze wydaje mi sie ze to blad związany z tym ze zmienna $dr jest pusta. Trudno żeby była wypełniona jak błąd wyskakuje przed dokonaniem wyboru.

[modern-web]

To nie błąd tylko informacja, która nie ma prawa spowodować jakiegokolwiek błędu.
Dodaj znacznik @ przed definiowaniem zmiennej dr -> @$dr = ...
Jest to operator tłumienia błędów.

Co do Twojego problemu.. nie wiem. To błąd logiczny, a nie problem ze składnią.
Przeanalizuj kod od początku do końca.
Ja w tym momencie nie mam na to czasu dlatego nie wyręczę Cię z tego obowiązku.

Ten post edytował modern-web 15.06.2011, 16:28:50

[Hpsi]

Dodaj znacznik @ przed definiowaniem zmiennej dr -> @dr = ...

zły nawyk wg mnie ....
Co do reszty to sie zgodze, czytalem kod i tez nie zauwazlem bledu.

[modern-web]

Owszem, zły nawyk ale gdy włącza się wyświetlanie wszystkich komunikatów i błędów warto nie zaśmiecać sobie ekranu (w przypadku rozbudowanych skryptów) i tymczasowo dodać @ przed definiowaniem zmiennej, na którą wskazuje informacja (Notice). Błędy (Warning) należy od razu usuwać - oczywiście w miarę możliwości.
Tak czy inaczej @ nie robi wielkiej różnicy gdy E_NOTICE jest wyłączone (IMG:style_emoticons/default/wink.gif)

[rulfy]

No cóż po patrze jeszcze trochę w kod może coś znajdę, ale jak byście coś jednak do juta znaleźli to piszcie. Dzięki z pomoc (IMG:style_emoticons/default/smile.gif)

[Fifi209]

A połączenie z bazą masz w momencie wywołania funkcji?

[rulfy]

Połączenie z bazą mam na pewno bo inne aktualizowane dane z tego formularza się dodają tylko jest problem z tą funkcja.

[mortus]

Ale Panie. Przecież Twoja funkcja zwraca zawartość $_POST['gospodarze'] lub $_POST['goscie'], a w pliku pan_podsumowanie.php nigdzie z tablicy $_POST takich danych nie odbierasz, bo zapewne ich nie ma, w przeciwieństwie do pliku pan_wyniki.php.

Wniosek: funkcja zwraca nie to, co trzeba!

[aachi]

Ja patrzę w ten plik w którym funkcja nie działa i nie widzę żadnego jej wywołania.... Możesz podać numer linii w której ją wywołujesz?

Jest tylko jakaś druzyny(1) w linii 74.... więc to nie to (IMG:style_emoticons/default/wink.gif) Ty przecież zdefiniowałeś druzyna()

Ten post edytował aachi 16.06.2011, 09:36:58

[rulfy]

Ale Panie. Przecież Twoja funkcja zwraca zawartość $_POST['gospodarze'] lub $_POST['goscie'], a w pliku pan_podsumowanie.php nigdzie z tablicy $_POST takich danych nie odbierasz, bo zapewne ich nie ma, w przeciwieństwie do pliku pan_wyniki.php.

Wniosek: funkcja zwraca nie to, co trzeba!

To chyba nie to bo próbowałem dodać do funkcji ten sam kod tylko z $_POST['nazwa']; i przypisać go parametrowi 3 ,tak jak to jest zrobione w pan_wyniki.php. (parametr 1 - gospodarze, 2 - goscie, 3 - nazwa) Niestety w ten sposób tez nie chciało działać :/

Ja patrzę w ten plik w którym funkcja nie działa i nie widzę żadnego jej wywołania.... Możesz podać numer linii w której ją wywołujesz?

Jest tylko jakaś druzyny(1) w linii 74.... więc to nie to (IMG:style_emoticons/default/wink.gif) Ty przecież zdefiniowałeś druzyna()

Wywołanie funkcji jest w linii 73:

[PHP] pobierz, plaintext 
<?php		
		if($flaga == k)
		{
			...
			echo '<p>Zespół</p>';
				$nazwa = druzyny(1);
 
			...
		}
?>
[PHP] pobierz, plaintext 

Jeśli chodzi o definicje funkcji to:

[PHP] pobierz, plaintext 
<?php
	function druzyna($a)
	{
	...
        }
?>
[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().

[rulfy]

Zmieniłem trochę funkcję druzyna($a) i zaczęło działać.
Błędem było złe przekazywanie tablicy $_POST tak jak to opisał 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.

Teraz funkcja jako parametr przyjmuję odpowiednie odwołanie do tablicy $_POST. A oto kod:

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

I przykładowe wywołanie (z pliku pan_wynik.php):

[PHP] pobierz, plaintext 
$gospodarze_add = druzyna('gospodarze');
[PHP] pobierz, plaintext 

Napiszcie co myślicie o tym rozwiązaniu.

Jeśli chodzi o pozbycie się tego fragmentu kodu o którym pisze mortus:

[PHP] pobierz, plaintext 
<?php
...
  // 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 

To bez tego funkcja nie działa. Pozdrawiam (IMG:style_emoticons/default/smile.gif)

P.S: mortus, a oco chodzi w tej regule DRY (IMG:style_emoticons/default/questionmark.gif)

Ten post edytował rulfy 20.06.2011, 19:39:24