[php] Funkcja wyświetlająca menu, pomysły optymalizacji / zabezpieczeń Opcje

[nexis]

Napisałem dla swoich potrzeb funkcję wyświetlającą menu o głębokości dwóch pozycji (kategoria + podkategoria).

Struktura bazy danych wygląda tak:

[SQL] pobierz, plaintext 
id    (INT)
super (INT)
label (VARCHAR)
[SQL] pobierz, plaintext 

przy czym kategorie nadrzędne mają id = 0.

Funkcja generuje listę <ul><li></li></ul>:

[PHP] pobierz, plaintext 
<?php
function menu () {
 
  // POBRANIE IDENTYFIKATORA KATEGORII NADRZEDNEJ
  if ( ( isset( $_GET['cat'] ) ) && ( is_numeric( $_GET['cat'] ) ) ) {
	$query = "SELECT super FROM categories WHERE id = " . $_GET['cat'];
	$query = mysql_query( $query );
	$row   = mysql_fetch_array( $query );
	$super = $row['super'];
  }
  // POBRANIE WSZYSTKICH NADRZEDNYCH KATEGORII
  $query  = 'SELECT id, super, label FROM categories WHERE super = 0 ORDER BY label ASC';
  $query  = mysql_query( $query );
  // UTWORZENIE LISTY KATEGORII NADRZEDNYCH
  $output = '<ul>';
  while ( $row = mysql_fetch_array( $query ) ) {
	// WYSWIETLENIE PODKATEGORII
	if ( ( isset( $_GET['cat'] ) ) && ( is_numeric( $_GET['cat'] ) ) && ( ( $row['id'] == $_GET['cat'] ) || ( $row['id'] == $super ) ) ) {
	  $output .= '<li><a href="?cat=' . $row['id'] . '" class="redb">' . $row['label'] . '</a>';
 
	  $query2 = "SELECT COUNT(id) FROM categories WHERE super = " . $row['id'];
	  $query2 = mysql_query( $query2 );
	  $count  = mysql_fetch_array( $query2 );
 
	  if ( $count[0] > 0 ) {
		$query2  = 'SELECT id, super, label FROM categories WHERE super = ' . $row['id'] . ' ORDER BY label ASC';
		$query2  = mysql_query( $query2 );
		$output .= '<ul>';
		while ( $row2 = mysql_fetch_array( $query2 ) )
		  $output .= '<li><a href="?cat=' . $row2['id'] . '" class="red">' . $row2['label'] . '</a></li>';
		$output .= '</ul>';
	  }
 
	  $output .= '</li>';
	} else
	  $output .= '<li><a href="?cat=' . $row['id'] . '">' . $row['label'] . '</a></li>';
  }
  $output .= '</ul>';
 
  return $output;
 
}
?>
[PHP] pobierz, plaintext 

Aktualnie wybrana kategoria jest przekazywana poprzez

[PHP] pobierz, plaintext 
<?php
$_GET['cat']
?>
[PHP] pobierz, plaintext 

.

Macie jakieś pomysły na optymalizację lub zabezpieczenie tej funkcji?

W przypadku wyświetlanie podkategorii myślałem o rekurencji, ale odstąpiłem od tego pomysłu z tego względu, że dla potrzeb dwupoziomowego menu tak jest chyba prościej.

Ten post edytował nexis 9.07.2007, 12:38:51