[php] system newsów Opcje

[Hectic]

Napisałem skrypt newsów, który dzieli je na strony według idów. Wszystko jest ok dopóki nie skasuje któregoś ida np. 12, 11, 9, 8… Kiedy tak zrobię na końcu 1 i początku 2 wyświetla się ten sam news. I nie wiem dlaczego (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

[PHP] pobierz, plaintext 
<?php
/****************************/
$page = $_GET['page'];
$page2 = $page - 1;
$select_id = 'Select id FROM news';
$id_news = mysql_query($select_id);
$wiersze = mysql_num_rows($id_news);
$idy = $wiersze + 1;
$idy2 = $page2 * $news_page;
/****************************/
if(!isset($_GET['page']))
{
	$stop = $idy;
}
else
{ 
	$stop = $idy - $idy2;	
}
/****************************/
 
$select_news = 'Select title, text, nick, date  FROM news WHERE id < '.$stop.' ORDER BY id DESC LIMIT '.$news_page.'';
$zapytanie = mysql_query($select_news);
while($wiersz = mysql_fetch_row($zapytanie)) 
{
	if (empty($wiersz[0]) and empty($wiersz[1]) and empty($wiersz[2]) and empty($wiersz[3]))
		{ } // nie rób nic
	else 
	{
		echo '<BR><table class="news" align="center" width="95%" border="0">'; 
		echo '<tr><td><b>Tytuł:</b> ';
		echo $wiersz[0]; 
		echo '</td><td align="right">';
		echo '<b>Data:</b> ';
		printf ("%s", $wiersz[3]); 
		echo '</td><tr><td colspan="2" align="right"><br>';
		printf ("%s", $wiersz[1]); 
		echo '</td></tr><tr><td><br>';
		echo '<b>Autor:</b> ';
		printf ("%s", $wiersz[2]); 
		echo '</td><td align="right">Komentarze[0]</td></tr>';
		echo '</td></tr></table><hr width="95%">';
	}
}
 
echo '<p class="news" align="center">Strona: ';
$ilosc = ceil($wiersze / $news_page);
 
for($strona = 1; $ilosc >= $strona ; $strona++) 
{
	echo '[<a href="index.php?go=news&page='.$strona.'">'.$strona.'</a>]';
}
?>
[PHP] pobierz, plaintext 

Oraz ten fragment kodu:

[PHP] pobierz, plaintext 
<?php
$select_id = 'Select id FROM news';
$id_news = mysql_query($select_id);
$wiersze = mysql_num_rows($id_news);
?>
[PHP] pobierz, plaintext 

Może bardzo obciążyć serwer mysql (w przypadku dużej ilości newsów i odwiedzających). Jak inaczej to rozwiązać?

Ten post edytował Hectic 1.05.2006, 21:41:09

[LBO]

[PHP] pobierz, plaintext 
<?php
$select_id = 'Select id FROM news';
$id_news = mysql_query($select_id);
$wiersze = mysql_num_rows($id_news);
?>
[PHP] pobierz, plaintext 

Może bardzo obciążyć serwer mysql (w przypadku dużej ilości newsów i odwiedzających). Jak inaczej to rozwiązać?

Nie przesadzaj. Bazy danych Nie są, ąż tak bardzo wrażliwe. Chociaż można twoje zapytanie zoptymalizować.

[SQL] pobierz, plaintext 
SELECT COUNT(*) 
FROM news
[SQL] pobierz, plaintext 

Dalej...

[SQL] pobierz, plaintext 
SELECT title, text, nick, date  
FROM news
 WHERE id < $stop ORDER BY id DESC LIMIT $news_page
[SQL] pobierz, plaintext 

...zamień na...

[SQL] pobierz, plaintext 
SELECT title, text, nick, date  
FROM news
 ORDER BY id DESC LIMIT $news_page OFFSET $idy2
[SQL] pobierz, plaintext 

...zakładając, że $news_page to ilość newsów na stronę. Konstrukcja zapytania z LIMIT (i OFFSET) wyeliminuje Twóje problemy po wywaleniu id z BD.
A teraz czytaj komentarze.

[PHP] pobierz, plaintext 
<?php
//gdzie deklaracja $news_page
/****************************/
//Dzięki konstrukcji poniżej $_GET['page'] posiada domyslną wartość (int) 1
//$page = isset($_GET['page']) ? $_GET['page'] : 1;
$page = $_GET['page'];
//a nie lepiej zwyczajnie $page--? Pozbedziesz się dodatkowej zmiennej. Za dużo ich produkujesz.
$page2 = $page - 1;
//tak jak radziłem wyżej
$select_id = 'Select id FROM news';
//wtedy to
$id_news = mysql_query($select_id);
//i to niepotrzebne
$wiersze = mysql_num_rows($id_news);
 
$idy = $wiersze + 1;
//gubię się w nazwach zmiennych, strasznie się powtarzasz
//utrudnia to sprawdzanie błędów
$idy2 = $page2 * $news_page;
 
//Jeżeli dostosowałeś się do pierwszej rady, musisz koniecznie zmienic konstrukcję
 tego if'a (najlepiej wywal).
if(!isset($_GET['page']))
{
	$stop = $idy;
}
else
{ 
	$stop = $idy - $idy2;	
}
//Rada wyżej		
$select_news = 'Select title, text, nick, date  FROM news WHERE id < '.$stop.' ORDER BY id DESC LIMIT '.$news_page.'';
 
//Dalej nie sprawdzam. Zakładam, że stronnicowanie i drukowanie danych jest poprawne.
?>
[PHP] pobierz, plaintext 

Tylko nie myśl, że jak wkleisz moje zalecenia to skrypt zacznie cudownie działać. Co to, to nie (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) . Ja staram sie tylko pokazać Tobie właściwy kierunek.

Pozdrowienia.

Ten post edytował LBO 2.05.2006, 00:55:35