[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]

Może i szybciej, ale wyobraź sobie, że te zapytanie, może być jeszcze w międzyczasie modyfikowane. Nie w tym konkretnym przykładzie, ale jednak.

Załóżmy, że chcesz skasować pokaźną liczbę userów z BD. Ich id(ientyfikatory) są przekazane w tablicy $_GET['ids']. Tworzysz funkcję (albo lepiej - metodę), gdzie w prametrze przekazujesz tą tablicę.

[PHP] pobierz, plaintext 
<?php
 
public function deleteUsers($ids) {
	//podstawowe zapytanie znajduję się w zmiennej
	//$this->sql = 'DELETE FROM users WHERE';
	foreach ($ids as $id) {
		$this->sql .= ' id = '.$id.' OR';
	}
	//wiem, że po zakończeniu iteracji przez tablice
	//zapytanie będzie zawierać nadmiarowe OR, ale
	//jest to tylko przykład. sam się możesz pokusić o
	//poprawną implementację.
 
	$this->executeQuery();
	//wysyłasz zapytanie do BD, pobierasz wynik, przetwarzasz etc.
}
 
?>
[PHP] pobierz, plaintext 

Mam nadzieję, że zrozumiałeś.

P.S. Moga byc błędy, bo się spieszę, ale nie mogłem tego zostawić bez odpowiedzi.
P.P.S. Nie trzeba od razu robić, tak jak napisałem, ale jest to chyba przydatny nawyk dla programisty. W jakimś stopniu zwiększa póżniejszą zdolność programu do adaptacji/przeróbek przy mniejszej, włożonej w to pracy.