witam

posiadam w bazie danych tebele o nazwie global w ktorej przechowywane sa informacje na temat ksiazek, znajduje sie tam kilka tysiecy pozycji.

tabela ta ma taka oto strukture:

tabela global:
| id | tytul | id_autor | id_category | .......... i pozostale pola

tabela autor:
| id | autor |

tabela category:
| id | category |

normalne jest ze w tej tabeli moze znajdowac sie wiecej niz jedna pozycja tego samego autora (ten sam autor kilka innych tytulow) oraz to ze znajduja sie wiecej niz jedna kategoria np. horror, przygoda

np:
autor_1 | tytul_10 | category_1
autor_1 | tytul_11 | category_1
autor_1 | tytul_12 | category_1
autor_1 | tytul_13 | category_1
autor_1 | tytul_14 | category_1
autor_2 | tytul_21 | category_2
autor_2 | tytul_22 | category_2
autor_3 | tytul_31 | category_2
autor_3 | tytul_32 | category_2
autor_4 | tytul_41 | category_1
autor_4 | tytul_42 | category_1
autor_5 | tytul_51 | category_3
autor_5 | tytul_52 | category_3


szukam rozwiazania aby wypisac wszystkich autorow pisszacych w tej samej kategorii tematycznej:

tzn klikajac na link: category_1

otrzymalbym cos takiego:

KATEGORIA: category_1
liczba autorow w tej kategorii: 2
liczba ksiazek w tej kategorii: 7

oraz liste autorow:

autor_1 (5)
autor_4 (2)


napisalem skrypt ktory realizuje to zadanie ale wysypuje sie gdy w danej kategorii jest duza liczba autorow: np kategoria science-fiction gdzie jest samych autorow 2tys kazdy autor napisz chodzby dwie ksiazki to juz jest 4tys pozycji:

skrypt nie radzi sobie z czasem wykonania (chyba) dostaje taki oto komunikat:


Fatal error: Maximum execution time of 30 seconds exceeded in search.php on line ...


oto skrypt:

[PHP] pobierz, plaintext 
<?php
$link=mysql_connect (hostTxt,userTxt,passTxt);
	mysql_select_db (bazaTxt);
 
//zmienna $_GET['category'] wchodzi do skryptu informujac o ktora kategorie chodzi
 
	$zapytanie = "SELECT * FROM category WHERE id='".$_GET['category']."'";
	$wykonaj = mysql_query ($zapytanie) or die(mysql_errno() . ": " . mysql_error(). "\n");
	$wiersz=mysql_fetch_array($wykonaj);
	$kategoria=$wiersz['category'];
//w powyzszym zapytaniu pobieram nazwe kategorii
 
 
	$zapytanie = "SELECT * FROM global WHERE id_category='".$_GET['category']."'";
	$wykonaj = mysql_query ($zapytanie) or die(mysql_errno() . ": " . mysql_error(). "\n");
	$i=0;
	while ($wiersz=mysql_fetch_array($wykonaj))
	{
	$wejscie[]=$wiersz['id_autor'];
	$i++;
	}
//w powyzszym zapytaniu laduje wszystkich autorow piszacych w jednej kategorii to 
tablicy $wejscie[]
//moze zdazyc sie tak ze jeden autor napisal wiecej niz jedna ksiazke w tej samej 
kategorii
 
   $wynik = array_unique ($wejscie);
 
//dlatego z tablicy $wejscie trzeba usunac powtarzajacych sie autorow
 
 $ile_akt=0;
 $suma_poz=0;
//powyzsze zmienne beda odpowiedzialne za wyswietlenie szczegolow o kategorii
//$ile_akt - ile autorow jest w danej kategorii
//$suma_poz - ile jest ksiazek nalezaczych do jednej kategorii
 
 
//czesc realizujaca glowne zadanie skryptu
foreach ($wynik as $klucz => $wartosc)
{
  $zapytanie = "SELECT * FROM authors WHERE id='".$wartosc."'";
  $wykonaj = mysql_query ($zapytanie); or die(mysql_errno() . ": " . mysql_error(). "\n");
  $wiersz=mysql_fetch_array($wykonaj);
  $autor=$wiersz['autor'];
  //w powyzszym zapytaniu pobieram nazwe pierwszego autora z zadanej kategorii
 
  //w tym zapytaniu pbieram liczbe tytulow ksiazek nalezacych do zadanej kategorii i
 napisanych przez danego autora
  $zapytanie = "SELECT * FROM global WHERE id_autor='".$wartosc."' and id_category='".$_GET['category']."'";
  $wykonaj = mysql_query ($zapytanie); or die(mysql_errno() . ": " . mysql_error(). "\n");
  $i=0;
  while ($wiersz=mysql_fetch_array($wykonaj))
	{
	$i++;
	}
  $suma_poz=$suma_poz+$i;
	echo"<tr><td><a class=menu_link href=details.php?id=".$wartosc."&cat=".$_GET['category'].">".$autor." (".$i.")</a></td></tr>";
 
 $ile_akt++;
 }
 
// a tutaj wypisuje szczegoly na temat danej zadanej kategorii 
echo"<div>
		   <span>Category: ".$kategoria."</span></br>
		   <span>Total authors in this category: ".$ile_akt."</span></br>
		   <span>Total books in this category: ".$suma_poz."</span>
	  </div>";
 
?>
[PHP] pobierz, plaintext 

wg powyzszego przykladu blad ten mi sie pojawia
w lini 40 i 47
Fatal error: Maximum execution time of 30 seconds exceeded in search.php on line 40
lub
Fatal error: Maximum execution time of 30 seconds exceeded in search.php on line 47

czy moze mi ktos pomoc nie wiem o co chodzi
czy tern skrypt dziala w nieskonczonosc?,
petla nigdy sie nie konczy?
czy jeszcze cos innego
ja nie mam juz pomyslow

a moze ktos mi podsunie inny sposob rozwiazania tego problemu

pozdrawiam

ps. wiem ze ta tabel nie ma jeszcze nic wspolnego z postacia normalna ale teraz nie ma to najmniejzszego znaczenia

o moj ty swiecie....
a o left join lub o zwyklym pobieraniu danych z kilku tabel w jednym zapytaniu slyszal? Przejrzyj se przyklady lef joina, poczytaj o tym, a potem bedziesz mogl polaczyc to zapytanie:

[PHP] pobierz, plaintext 
<?php
 $zapytanie = "SELECT * FROM category WHERE id='".$_GET['category']."'";
?>
[PHP] pobierz, plaintext 

z ta petla generujaca x zapytan:

[PHP] pobierz, plaintext 
<?php
foreach ($wynik as $klucz => $wartosc)
{
  $zapytanie = "SELECT * FROM authors WHERE id='".$wartosc."'";
 ?>
[PHP] pobierz, plaintext 

to wszystko w jednym zapytaniu da sie zrobic

nastepnie:

[PHP] pobierz, plaintext 
<?php
$zapytanie = "SELECT * FROM global WHERE id_autor='".$wartosc."' and id_category='".$_GET['category']."'";
  $wykonaj = mysql_query ($zapytanie); or die(mysql_errno() . ": " . mysql_error(). "\n");
  $i=0;
  while ($wiersz=mysql_fetch_array($wykonaj))
    {
    $i++;
    }
?>
[PHP] pobierz, plaintext 

pobierasz x wierszy tylko po to by zwikeszac licznik o jeden? mysql_num_rows(). ale to i tak zle rozwiazanie. Poczytaj o count w mysql
pozatym tę ostatnia petle tez mozna podpiac do zapytania, o ktorym pisalem wczesniej. poczytaj o group by
Tyle na szybko. zycze milej lektury

ok dzieki,

ale abstrachujac juz od techniki programownia - wiem ze nie jest najlepsza, ciagle sie ucze

dlaczego wystepuje ten blad

tak na chlopski rozum powinno to przeleciec cala tabele i wypluc wyniki

no ale ty przeczytales ten blad? wyraznie ci napisalo, ze skonczyl sie czas na skrypt. ten czas jest ustawiany w php.ini
a skrypt sie wykonuje dlugo, bo masz w nim zero optymalnosci, co juz w poscie wyzej wyjansilem

kapuje,


no coz trzeba bedzie zwiekszyc czas na wykonanie skryptu

Ale najciekawsze jest to, że ani razu nie zapytał: 'a jak to zoptymalizować? Może pomożecie?'
Cenię ludzi samodzielnych, którzy po swojemu rozwiązują problemy.

1- zerknij na temat posta

2 -

Widzisz, chodziło mi o to, że zwiększenie czasu wykonania skryptu nie jest żadnym rozwiązaniem. Problem leży w wielu niepotrzebnych zapytaniach, które niejednokrotnie można zastąpić jednym. Ale o tym wszystkim powiedział Ci już nospor, a Twoim komentarzem było to, że zwiększysz czas.

oczywiscie to byl zart,
zdaje sobie sprawe ze nie jest to zadne rozwiazanie

rozumiem dokladnie o co chodzi

ok poprawilem nieco czas
a to zapytanie ktore uzylem jest chyba jednoznaczne z join'em

[PHP] pobierz, plaintext 
<?php
foreach ($wynik as $klucz => $wartosc)
 {
	$zapytanie = "SELECT * FROM authors, global WHERE authors.id=global.id_autor and authors.
id='".$wartosc."' and global.id_category='".$_GET['category']."'";
	$wykonaj = mysql_query ($zapytanie) or die(mysql_errno() . ": " . mysql_error(). "\n");
	$wiersz=mysql_fetch_array($wykonaj);
	$autor=$wiersz['autor'];
}
 
?>
[PHP] pobierz, plaintext 

(czas sie poprawil znacznie ale dalej 30sek to nie wystarczajaco)

wyzucilem drugie zapytanie generujace liczbe pozycji ale na moim poziomie zaawansowania nie jestem w stanie wyobrazic sobie jak pozbyc sie petli for each generujacej zapytania.

to co pomozecie?

[PHP] pobierz, plaintext 
<?php
$zapytanie = "... WHERE id IN (".implode(',', $wynik).")".
?>
[PHP] pobierz, plaintext 

o w morde!! to working
nigdy bym tego nie przeskoczyl.


dzieki serdeczne

masz u mnie browara