Forum PHP.pl

Napisałem sobie skrypt. Łączy się z bazą danych, odczytuje i wypisuje dane z 4 tabel (dosłownie 10 rekordów), wypisuje 4 formularze - po jednym do modyfikacji danych w każdej tabeli. Na lokalu <wamp 2.1 PHP 5.3.3 MySQL 5.5.8 i Apache 2.2.17> wszystko pięknie śmiga.

Po krótkich testach chciałem zobaczyć jak będzie to wyglądało w internecie i wrzuciłem na swój serwer. Wypełniłem trochę danymi i się bawiłem i przez około 20 minut wszystko działało ładnie, po czym po którymś przesłaniu danych w formularzu pusta strona (biała). I tyle mi się wyświetla. Jak już łaskawie ukaże mi się formularz logowania to po zalogowaniu się dalej pusto.

Treść dziennika logów:
[Tue Jan 10 16:05:22 2012] [warn] [client 31.175.91.238] mod_fcgid: stderr: PHP Fatal error: Allowed memory size of 67108864 bytes exhausted (tried to allocate 66846721 bytes) in /ścieżka_do_folderu/plik_z_funkcjami_dołączonymi_do_skryptu on line 69
Fatal error: Allowed memory size of 67108864 bytes exhausted (tried to allocate 66846721 bytes) in /ścieżka_do_folderu/plik_z_funkcjami_dołączonymi_do_skryptu on line 69

Skrypt wygląda tak:
index.php

[PHP] pobierz, plaintext 
<?php
session_start();
 
if(isset($_POST['check']))
{
 if(($_POST['username']==$form_user) && ($_POST['pass']==$form_pass))
 {
  $_SESSION['logged'] = true;
  $_SESSION['login'] = $_POST['username'];
 }
 else
 {
	  echo('Błędne dane');
 }
}
 
if(isset($_SESSION['logged']) && (!empty($_SESSION['login'])))
{
	 include('db_select.inc'); // połączenie się z bazą
	include('funckje.php');
 
	dodaj('typ1');
	dodaj('typ2');
 
 mysql_close($connection);
}
else
{
 
  echo('         <form method="POST" action="" class="login">
      <fieldset>
                <input class="dane" type="text" name="username" value="Login"  onfocus="if (this.value == \'Login\') {this.value = \'\';}" onblur="if (this.value == \'\') {this.value = \'Login\';}" />
                <input class="dane" type="password" name="pass" value="Hasło" onfocus="if (this.value == \'Hasło\') {this.value = \'\';}" onblur="if (this.value == \'\') {this.value =\'Hasło\';}" />
                <input class="login-button" type="submit" value="Zaloguj">
      </fieldset>
 
      <input class="hidden" type="hidden" value="1" name="check">
</form>
');
}
 
?>
[PHP] pobierz, plaintext 

funkcje.php

[PHP] pobierz, plaintext 
<?php	
 
	function cennik($rodzaj)
	{
 
		if($rodzaj == 'tpy1'){
			$query_typ = 'SELECT * FROM typ;';
 			$query_kategoria = 'SELECT * FROM kategoria;';
		}
		if($rodzaj == 'typ2')
		{
				$query_typ = 'SELECT * FROM typind;';
 				$query_kategoria = 'SELECT * FROM kategoria2;';
		}
 
	$result_typ = mysql_query($query_typ);
	$result_kategoria = mysql_query($query_kategoria);
 
	if(!$result_typ or !$result_kategoria){die("Nie można wykonać zapytania! <br />". mysql_error());}
	$i = 0;
	while($result_row = mysql_fetch_row($result_typ)){
  	$result_array[$i] = $result_row[1];
  	$result_array[$i+1] = $result_row[2];
	$result_array[$i+2] = $result_row[3];
	$i=$i+3;
	}
 
	$j = 0;
	while($result_row = mysql_fetch_row($result_kategoria))
	{
		$result_array_tytul[$j+1] = $result_row[0];
		$result_array_tytul[$j] = $result_row[1];	
		$j=$j+2;
	}
 
	    echo('  <h4>Tytuł</h4>
			 <div id="tabela_html">
			 <div id="tytul_gora">Tabela<p>'.date("d.m.Y").'r.</p></div>');
 
		for($i; $i>=0;)
		{		
			if($j>=2){
				echo('<div class="kategoria"><h6 class="tytul_kat">'.$result_array_tytul[$j-2].'</h6>');
			}
 
			while($result_array[$i] == $result_array_tytul[$j-1]){
 
				echo('<div class="pozycja"><span class="tytul">'.$result_array[$i-2].'</span><span class="cena">'.$result_array[$i-1].'</span></div>');
				$i=$i-3;
				if($i <=0) {break;}
			}
			echo('</div>'); // wg. logów tutaj jest błąd
			$j=$j-2;
		}
 
	echo('</div>');
 
	}
 
 
 
 
	function lista_kategorii($rodzaj)
	{
		if($rodzaj == 'typ1'){
 			$query_kategoria = 'SELECT * FROM kategoria;';
		}
		if($rodzaj == 'typ2')
		{
 				$query_kategoria = 'SELECT * FROM kategoria2;';
		}
 
	$result_kategoria = mysql_query($query_kategoria);
 
	if(!$result_kategoria){die("Nie można wykonać zapytania! <br />". mysql_error());}	
	$j = 0;
	while($result_row = mysql_fetch_row($result_kategoria))
	{
		$result_array_tytul[$j+1] = $result_row[0];
		$result_array_tytul[$j] = $result_row[1];	
		$j=$j+2;
	}
 
		$select = '<select name="kategoria">		
		<option></option>';
 
for($j; $j>=2; $j=$j-2)
	{		
		$select.= '<option value="'.$result_array_tytul[$j-1].'">'.$result_array_tytul[$j-2].'</option>';
	}
 
		$select.= '</select>';
 
		return $select;	
	}
 
	function dodaj($rodzaj)
	{
		$potwierdzenie = '';
 
		if($rodzaj == 'typ1'){
			$query_typ = 'SELECT * FROM typ;';
 			$query_kategoria = 'SELECT * FROM kategoria;';
			$rodzaj2 = 'kategoria';
			$option = lista_kategorii('typ1');
		}
		if($rodzaj == 'typ2')
		{
			$query_typ = 'SELECT * FROM typind;';
 			$query_kategoria = 'SELECT * FROM kategoria2;';
			$rodzaj2 = 'kategoria2';
			$option = lista_kategorii('typ2');
		}
 
 if (isset($_POST['nazwa']))
 {
	if($_POST['ktoredodaj'] == 'kategoria'){
	 	if($_POST['nazwa'] != '')
	 	{
   			$insert = 'INSERT INTO '.$_POST['rodzaj'].' VALUES (';
   			$id = 'NULL, ';
   			$nazwa = $_POST['nazwa'];
   			$query = $insert.$id.'"'.$nazwa.'");';
 
   			mysql_query($query);
 
   			$potwierdzenie ='Nowa kategoria została zapisana!';
   			unset($_POST['nazwa']);
   			unset($_POST['rodzaj']);
	 	}
	 	else
	 	{
		echo('<p style="color:red; font-weight: bold; padding: 5px;">Wpisz nazwę kategorii!</p>');	 
	 	}
	}
	else
	{
		if($_POST['ktoredodaj'] == 'pozycja')
		{
			if($_POST['nazwa'] != '')
	 	{
			if($_POST['rodzaj'] == 'kategoria') $tabela = 'typ';
			if($_POST['rodzaj'] == 'kategoria2') $tabela = 'typ2';
   			$insert = 'INSERT INTO '.$tabela.' VALUES (';
   			$id = 'NULL, ';
   			$nazwa = $_POST['nazwa'];
   			$cena = $_POST['cena'];
   			$id_kat = $_POST['kategoria'];
   			$query = $insert.$id.'"'.$nazwa.'", '.$cena.', '.$id_kat.');';   
   			mysql_query($query);
 
   			$potwierdzenie ='Nowa pozycja została zapisana!';
   			unset($_POST['nazwa']);
   			unset($_POST['rodzaj']);
	 	}
	 	else
	 	{
		echo('<p style="color:red; font-weight: bold; padding: 5px;">Wszystkie pola musza być uzupełnione!</p>');	 
	 	}		
		}	
	}  
  }
else
{
	$_POST['rodzaj'] = '';
	$_POST['nazwa'] = '';
}
 
		echo('
		<h4>Nowa kategoria</h4>
		<p style="color:green; font-weight: bold; padding: 5px;">'.$potwierdzenie.'</p>
		<form action="#nowe" method="POST" class="dodaj_kategorie">
		Nazwa kategorii:</td><td><input size="30" type="text" name="nazwa" value="" />
		<input type="hidden" name="rodzaj" value="'.$rodzaj2.'" />
		<input type="hidden" name="ktoredodaj" value="kategoria" />
		<input type="submit" value="Dodaj nową kategorię" />
		</form>
 
		<h4>Nowa pozycja</h4>
		form action="#nowe" method="POST" class="dodaj_kategorie">
		Nazwa pozycji:<input type="text" name="nazwa" value="" />
		Cena: <input type="text" name="cena" value="" />
		<input type="hidden" name="rodzaj" value="'.$rodzaj2.'" />
		<input type="hidden" name="ktoredodaj" value="pozycja" />
		Wybierz kategorię: '.$option.'
		<input type="submit" value="Dodaj nową pozycję" />
		</form>
 
');
 
	}
 
?>
[PHP] pobierz, plaintext 

Macie jakieś pomysły/sugestie jak rozwiązać ten problem?


za każdym razem gdy prześlę formularz to strona się przeładowuje i łączy się ponownie z bazą danych tzn tak jak na schemacie:
-łączy z bazą
-wyświetla formularz
-zamyka polaczenie
i tak w kółko za każdym przesłaniem formularza.
Czy to może szkodzić mojemu skryptowi? Tzn czy jest on przez to duży, zajmuje dużo zasobów i czy można by to zrobić lepiej? (raz łączyć z bazą danych i przeładowywać stronę bez tracenia tego połączenia)? Czy może to w niczym nie przeszkadza jak sobie dwa razy dziennie połączę się z bazą danych po 5 razy w ramach tego skryptu?


//EDIT: poprawione, wcześniej nie miałem dostępu do źródeł skryptu dlatego tak to napisałem.

Ten post edytował Kulfon 10.01.2012, 18:40:07

Pokaż lepiej dokładnie jak wygląda twój skrypt a nie walisz pseudokod, z którego nic nie wynika.

1) Nie edytuj posta jako odpowiedź na mojego posta, bo niby skad mam wiedzieć, że edytowałes?
Pisz normalnie jak człowiek kolejnego posta, wówczas wiadomo, że jest nowa rzecz w temacie

2) Wcześniej pokazałeś kod, na którym wywałał się skrypt. Teraz tego kodu nie widzę.

Już poprawiłem
Plik funkcje.php - komentarzem zaznaczyłem gdzie jest błąd
Linia 52

Ten post edytował Kulfon 10.01.2012, 17:16:53

[PHP] pobierz, plaintext 
for($i; $i>=0;){
[PHP] pobierz, plaintext 

I jeszcze się dziwisz, że pożera pamięć w nieskończoność?

Ale później jest kolejna pętla i przy spełnionym warunku (jeżeli już mi wypisze wszystkie pozycje z kategorii) to:

[PHP] pobierz, plaintext 
$i=$i-3;
[PHP] pobierz, plaintext 

Nie wydziwiaj, tylko zacznij od zapisania tego po ludzku, bo na 90% jest któraś z pętli nieskończona.

Zaraz to wszystko sprawdzę jeszcze raz (IMG:style_emoticons/default/smile.gif)

Swoją drogą to do pewnego momentu działało a ja nic nie zmieniałem w kodzie wtedy więc czemu nagle zaczęło mu coś nie pasować? Gdyby pętla była nieskończona to w ogóle by nie ruszyło tylko od razu wypluło ten błąd (przynajmniej tak mi się wydaje).

Ten post edytował Kulfon 10.01.2012, 20:46:53

Użyj debuggera i uruchom krok po kroku cały skrypt.

Po to są narzędzia, żeby z nich korzystać.

Hmm póki co xDebug'a nie udało mi się odpalić u siebie...
Mam wamp'a 2.1
phpinfo(); wyrzuca że xDebug jest zainstalowany <http://wklej.org/id/666164/>
html_errors = On
display_errors = 1
tak mam ustawione w php.ini
coś jeszcze powinienem skonfigurować <zrobić inaczej?>


Zauważyłem jeszcze jedną rzecz:
w pliku index.php wywołuję 2 razy funkcję

[PHP] pobierz, plaintext 
cennik('typ1');
cennik('typ2');
[PHP] pobierz, plaintext 

lokalnie działa to w porządku
na serwerze natomiast wywala błąd


na serwerze wszystko działa jeżeli zakomentuję drugie wywołanie funkcji

[PHP] pobierz, plaintext 
cennik('typ1');
//cennik('typ2');
[PHP] pobierz, plaintext 

wtedy także nie ma błędu o przepełnionej pamięci


EDIT://
Wyczyściłem bazę danych na serwerze. Wysłałem pliki jeszcze raz i wszystko działa jak należy (jak przed błędem)
Mimo tego, czy możecie mi powiedzieć co mogło być domniemana przyczyną takich błędów?

Ten post edytował Kulfon 11.01.2012, 18:01:42

O to:

[PHP] pobierz, plaintext 
 
		for($i; $i>=0;)
		{		
			if($j>=2){
				echo('<div class="kategoria"><h6 class="tytul_kat">'.$result_array_tytul[$j-2].'</h6>');
			}
 
			while($result_array[$i] == $result_array_tytul[$j-1]){
 
				echo('<div class="pozycja"><span class="tytul">'.$result_array[$i-2].'</span><span class="cena">'.$result_array[$i-1].'</span></div>');
				$i=$i-3;
				if($i <=0) {break;}
			}
			echo('</div>'); // wg. logów tutaj jest błąd
			$j=$j-2;
		}
[PHP] pobierz, plaintext 

Zrozum wkoncu ze tu ci się zapetla. A zapetla się, bo $i zmniejszasz warunkowo w petli while, do której albo może nie wejsc w ogole, albo wejdzie za malo razy by zmniejszyc dostatecznie to $i
Teraz ci działa, bo baze wyczyściłeś i pewnie wszystkie zależności warunkowe masz ok. A przestało ci działać, bo w pewnym momencie tak zmieniłeś dane w bazie, że zależności warunkowe zrobiły się złe i $i nie zostało dostatecznie zmniejszone by wyjść z pętli.

Ok, dziękuję, wreszcie do mnie dotarła głupota jaką zrobiłem....
Pytanie tylko czy da się to zrealizować w inny sposób? Moglibyście mnie naprowadzić na jakieś lepsze rozwiązanie?
2 tabele, jedna przechowuje nazwy nagłówków druga trzyma nazwy pozycji
wypisuję nagłówek i potem pozycje które odpowiadają temu nagłówkowi - to miała w założeniu realizować ta konstrukcja.
Da się w takim razie zrobić to inaczej?

A jeszcze troszkę bardziej na php z pseudo kodu? ;p Chyba już rozumiem jak to ma iść i zaraz spróbuję to przepisać, ale mimo wszystko jakbyś mógł jeszcze mi to przybliżyć to byłbym wdzięczny. Napiszę za jakiś czas co mi z tego wyszło (IMG:style_emoticons/default/smile.gif)

I tak praktycznie gotowca dostałeś.

Dobra, zobaczymy co mi z tego wyjdzie, a jakby co to napiszę jeszcze (IMG:style_emoticons/default/smile.gif)

Dzięki wielkie za pomoc, skrypt działa (IMG:style_emoticons/default/smile.gif)