[php] optymalizacja zapytań Opcje

[Lejto]

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

[thek]

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

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