[MySQL i PHP] Proste stronicowanie Opcje

[JaKo]

Otóż napisałem w oparciu o małe, ale jednak doświadczenie skrypt stronicujący zapytania z MySQL. Z tego co testowałem działa świetnie. Chciałbym jednak się dowiedzieć, czy kod nie ma wad, które przeoczyłem. Prosiłbym jakiegoś "fachowca" o rzucenie okiem oto skrypt:

[PHP] pobierz, plaintext 
<?php
 
ini_set ('display_errors' , 1);
error_reporting (E_ALL & ~E_NOTICE);
 
//Pod??czenie i wyb?r bazy
 
 
if($dbc=@mysql_connect ('localhost' , 'user' , 'pass')) {
 
	if (!@mysql_select_db ('news')) {
		die('Nie mo?na wybra? bazy danych');
	}
 
 
}else {
	die('Nie mozna po??czy? sie z MySQl');
}
 
$limit =  5; //Liczba newsów na jednej stronie
 
$strona = $_GET['strona'];
 
 
if (!isset($strona)) {
	$limit1 = 0;
	$limit2 = $limit;
} else {
	$limit1 = $limit * $strona - $limit;
	$limit2 = $limit * $strona;
}
 
//Definicja zapytania
$zapytanie = "SELECT * FROM news_pozycje ORDER BY id DESC LIMIT $limit1,$limit2";
 
 
if ($r = mysql_query ($zapytanie)) {
 
	//Odczytywanie w wy?wietlenie kolejnych rekord?w.
	while ($wiersz = mysql_fetch_array($r)) {
		print "<p align="left"><b>{$wiersz['tytul']}</b> :: {$wiersz['data_utworzenia']}<br /><br />
			{$wiersz['tresc']}<br /><br /><b>Autor: </b>{$wiersz['autor']}</p>
			<hr width="70%" size="1" />n";
	}
 
} else {
	die ('<p>Zapytanie nie zosta?o wykonane, bo <b>' . mysql_error() . "</b>. Zapytanie: $zapytanie.</p>");
 
}
 
echo "<center>wybierz stronę:";
list($ile)=mysql_fetch_row(mysql_db_query("news","SELECT count(*) FROM news_pozycje"));
 
$liczba_stron = $ile / $limit;
$liczba_stron = ceil($liczba_stron);
 
for ($v = 1; $v <= $liczba_stron; $v++) {
 
	if ($liczba_stron == 1) {
	echo " $v";
	} else {
	echo " <a href="?strona=$v">$v</a>";
	}
}
 
 
mysql_close(); //Zamkni?cie po??czenia.
 
 
?>
[PHP] pobierz, plaintext 

Pozdrawiam,
Konrad Janisz

Ten post edytował JaKo 28.03.2007, 15:39:14

[misiek172]

niechce mi sie czytać całego kodu ale na poczaku wydaje mi się że to :

if (!@mysql_select_db ('news')) {
die('Nie mo?na wybra? bazy danych');
}

można zamienić poprostu na:

mysql_select_db('news') or die('Nie mo?na wybra? bazy danych');

i wtedy nie trzeba używać @ bo to samo powodu nie wyświetlanie podstawowego błędu.

Ten post edytował misiek172 28.03.2007, 21:38:54