[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

[Hectic]

Nie rozumiem tego

//wtedy to
$id_news = mysql_query($select_id);
//i to niepotrzebne
$wiersze = mysql_num_rows($id_news);

W takim razie czym mam to zastąpić?

Ten post edytował Hectic 2.05.2006, 08:58:21

[Hacker]

[PHP] pobierz, plaintext 
<?php
$select_id = 'Select COUNT(*) FROM news';
$id_news = mysql_query($select_id);
$wiersze = mysql_fetch_array($id_news);
$wiersze = $wiersze[0];
?>
[PHP] pobierz, plaintext 

[Hectic]

Ale co oznacza offset? Znalazłem dokumentacje tylko w języku angielskim (IMG:http://forum.php.pl/style_emoticons/default/worriedsmiley.gif)

[nospor]

Ale co oznacza offset? Znalazłem dokumentacje tylko w języku angielskim

troche samodzielnosci:
http://www.google.com/search?hs=3Tp&hl=pl&...ukaj&lr=lang_pl
juz pierwszy wynik zawiera to czego szukasz

[Kayne]

Witam

Denerwuje mnie fakt, że piszecie w ten sposób:

Kod

$select_id = 'Select COUNT(*) FROM news';
$id_news = mysql_query($select_id);

Czy nie lepiej, szybciej i czytelniej jest tak:

Kod

$id_news = mysql_query('Select COUNT(*) FROM news');

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

[Hacker]

trochę offtop, ale LBO nie lepiej i wydajniej

[SQL] pobierz, plaintext 
DELETE 
FROM users
 WHERE id IN('.imlode(', ', $ids).')
[SQL] pobierz, plaintext 

Ten post edytował Hacker 2.05.2006, 13:33:49

[LBO]

Może i masz rację... zły przykład dałem, bo - akurat w tym przypadku - SQL posiada odpowiednie funkcje/gramatykę do jego obsłużenia (cały czas zapominam o IN, hyhyhy). Ale to nie zmienia faktu, że może zaistnieć wiele innych czynników, gdzie trzymanie podstawowego zapytania i jego modyfikacja, zależna od zaistniałych warunków jest bardzo przydatna. Weź pod uwagę ADOdb - ten layer posiada (musi je posiadać) odpowiednie narzędzia do zmieniania zapytań z powodu np. nieobługiwania przez daną Bazę konkretnej SQL komendy i za pomoca ich robi do na około.

Ten post edytował LBO 2.05.2006, 14:04:40