Forum PHP.pl

Mam kod który pobiera mi dane do galerii zawiera on jednak sporo zapytań.
Mogę go trochę zoptymalizować?

[PHP] pobierz, plaintext 
$id = (int)$_GET['id'];
 		$liczba = 2;
 	    $id = (int)$_GET['id'];
 		$liczba = 2;
		$sql = $db->query('select * from galeria_foto where kategoria = "'.$kat.'" and id = "'.$foto.'"');
 		$row = $sql->fetch_array();
 		$numer = $row[numer];
 
		$sql = $db->query('select * from galeria_foto where kategoria = "'.$kat.'" and id = "'.$foto.'"');
    	$num = $sql->num_rows;
      	$ile = ceil($num/$liczba);
 
		if(!isset($_GET['id'])) $_GET['id'] = 0;
    	$strona = (int)$_GET['id'];
 
 
		$q = $db->query('SELECT `id` FROM `galeria_foto` WHERE `id` > "'.$foto.'" ORDER BY `id` ASC');
		$wiersz = $q->fetch_array();
 
		$quer = $db->query('SELECT `id` FROM `galeria_foto` WHERE `id` < "'.$foto.'" ORDER BY `id` DESC');
		$wiersz_2 = $quer->fetch_array();
 
 		$sql = $db->query('select * from galeria_foto where  id = "'.$foto.'"');
    	$num = $sql->num_rows;
      	$ile = ceil($num/$liczba);
 
		if(!isset($_GET['id'])) $_GET['id'] = 0;
    	$strona = (int)$_GET['id'];
 
 
		$q = $db->query('SELECT `id` FROM `galeria_foto` WHERE `id` > "'.$foto.'" ORDER BY `id` ASC');
		$wiersz = $q->fetch_array();
 
		$quer = $db->query('SELECT `id` FROM `galeria_foto` WHERE `id` < "'.$foto.'" ORDER BY `id` DESC');
		$wiersz_2 = $quer->fetch_array();
 
  		$zap = $db->query('select name,id from galeria_kategorie where id = "'.$kat.'"');
		$r = $zap->fetch_array();
[PHP] pobierz, plaintext 

Pozdrawiam

Najpierw nasuwa się pytanie jakiej biblioteki do połączenia z bazą używasz.

i drugie:

[PHP] pobierz, plaintext 
$sql = $db->query('select * from galeria_foto where kategoria = "'.$kat.'" and id = "'.$foto.'"');
[PHP] pobierz, plaintext 

czemu dwa razy?

No jasne że możesz. Zauważ, że nawet nie używasz czegoś takiego jak limit, choć wszędzie pobierasz tylko jeden wiersz. Nawet nie mówię, że pewne zapytania się dublują. Ja bym powiedział, że ów kod jest naprawdę bardzo nieoptymalny i zupełnie inaczej bym go napisał nie dlatego, że zapytania są źle napisane, ale już sam kod php jest w wielu miejscach bezsensowny. Całe linijki kodu produkują te same dane, choć są zbędne i wystarczy je wykonać raz. Popatrz ile razy sprawdza Ci zmienną $_GET[id] i liczy kompletnie zbędnie $strona. Co Ci daje w 10 linii liczenie ilości wierszy wyniku skoro w zapytaniu szukasz rekordu z konkretnym id, czyli będzie tam zawsze 0 albo 1. Takich nielogiczności masz więcej w raptem nie 40 linijkowym kodzie... Musisz go mocno pozmieniać by był optymalny.

EDIT: W zasadzie jakoś połowa tego kodu jest zbędna (IMG:style_emoticons/default/winksmiley.jpg)

Ten post edytował thek 26.11.2009, 22:55:12

To mi przypomina kod pewnego serwisu, który ostatnio zamknęła policja.
Już same 4 pierwsze linijki świadczą o tym, że kod pisał jakiś pomyleniec

[PHP] pobierz, plaintext 
$id = (int)$_GET['id'];
$liczba = 2;
$id = (int)$_GET['id'];
$liczba = 2;
[PHP] pobierz, plaintext 

W tak prostym kodzie powtórzenia?

Zamiast

SELECT * ...

(zamiast gwiazdki) proponuje też wstawić nazwy kolumn.