[PHP] Stronnicowanie brak stron Opcje

[Sasuke Uchiha]

Witam mam problem gdyż nie wyświetla mi żadnych stron tzn wyświetla mi tylko jedną stronę z 5 rekordami tak jak powinno chodź w bazie jest ponad 10 rekordów spełniających podany warunek to i tak coś kurde nie gra bo jest tylko 1 strona i nie wyświetla ani wstecz ani dalej. Proszę o pomoc oto kod skryptu

[PHP] pobierz, plaintext 
<?php
define('limit_cm', 5);
 
$viev_news = "SELECT id, title, author, icon, date, time, img, vievs, comments, img1, info1, info2
          FROM news WHERE id=".$_GET['id_news']."" or die('Blad');
$result_viev_news = mysql_query($viev_news);
$row_viev_news = mysql_fetch_array($result_viev_news);          
 
$vievs = ++$row_viev_news['vievs'];
mysql_query("UPDATE news SET vievs='$vievs' WHERE id=".$_GET['id_news']."");
 
 if($_POST['add_cm'] == 'yes') {
 if (!empty($_POST['valcm'])){
 if(strlen(trim($_POST['valcm'])) > 15) {
 if(strlen(trim($_POST['valcm'])) < 500) {
 
$date_add = date("Y-m-d");
$time_add =date("G:i:s");
 
$valcm = ahack($_POST['valcm']);
 
mysql_query("INSERT INTO comments (`id`, `category`, `id_news`, `id_user`, `user`, `text`, `date`, `time`) VALUES ('', 'news', '".$_GET['id_news']."', '$id_user', '$login_user', '".mysql_real_escape_string($valcm)."', '$date_add', '$time_add')") or die (mysql_error);
$num_comments_news_add = ++$row_viev_news['comments'];
mysql_query("UPDATE news SET comments='$num_comments_news_add' WHERE id=".$_GET['id_news']."");
  }else {
          $error = 'Twój komentarz jest za dlugi!';
        }
  }else {  
          $error = 'Twój komentarz jest za krótki!';
        }
  }else {
          $error = 'Puste Pole';
        }
 
  }
 
$comments = "SELECT SQL_CALC_FOUND_ROWS * FROM comments WHERE id_news=".$_GET['id_news']." ORDER BY id DESC LIMIT ". mysql_escape_string((int)$_GET['page_cm'] *limit_cm).",".limit_cm;
$comments_query = mysql_query($comments);
$num_rows_comments_news = mysql_num_rows($comments_query);
 
 
 
 
 
echo '
  <table style="margin-bottom: 10px;" valign="top" cellspacing="0" cellpadding="0">
  <tr><td><img src="img/bg_site_news_1.png"></td></tr>
  <tr style="background-image: url(img/bg_site_repeat_2.png);"><td>
 
  <table style="margin-left: 7px;"  valign="top" cellspacing="0" cellpadding="0">
  <tr><td style="background-repeat: no-repeat; padding-left: 10px; color: #151515; font-family: Arial; font-size: 14px; width: 490px; height: 45px; background-image: url(img/bg-title.png);">
  <table cellspacing="0" cellpadding="0"><tr>
  <td><img src="img/icons/'.$row_viev_news['icon'].'"></td>
  <td style="padding-left: 5px;">
  <font style="color: #151515; font-size: 14px;"><b>'.$row_viev_news['title'].'</b></font><br> <font style="color: #505050; font-size: 11px;">Autor: <b>'.$row_viev_news['author'].'</b>  '.$row_viev_news['date'].' '.$row_viev_news['time'].'</font>
  </td></tr></table></td></tr>
  <tr><td><table style="margin-right: 16px; margin-top: 10px; margin-left: 9px;" cellspacing="0" cellpadding="0">
  <tr><td style="text-align: justify; color: #FFFFFF; font-family: Arial; font-size: 11px;"><img style="margin-top: 3px;margin-right: 5px;" align="left" src="img/'.$row_viev_news['img1'].'"><b>'.$row_viev_news['info1'].'</b><br><br></td></tr>
  <tr><td style="text-align: justify; color: #FFFFFF; font-family: Arial; font-size: 11px;">'.$row_viev_news['info2'].'<br><br></td></tr>
  <tr><td style="text-align: justify; color: #FFFFFF; font-family: Arial; font-size: 14px;"><b>KOMENTARZE('.$num_rows_comments_news.'):</b></td></tr>
  </table></td></tr>
  </table>';
$number_id = 1 + $num_rows_comments_news;
 while($comments_row = mysql_fetch_array($comments_query))
{
$avatar = "SELECT avatar
          FROM users WHERE id=".$comments_row['id_user']."" or die('Blad');
$result_avatar = mysql_query($avatar);
$row_avatar = mysql_fetch_array($result_avatar);
 
echo'
   <table style="width: 485px; margin-top: 10px; margin-left: 8px;" align="center" cellspacing="0" cellpadding="0">
   <tr><td><img src="img/bg_comments_1.png"></td></tr>
   <tr><td style="background-image: url(img/bg_comments_2.png);">
   <table cellspacing="0" cellpadding="0"><tr valign="top"><td style="padding-left: 7px;"><img src="img/'.$row_avatar['avatar'].'"></td><td style="padding-left: 5px;">
  <a href="index_test.php?site=profil&&id='.$comments_row['id_user'].'"><font style="font-family: Arial; font-size: 12px;color: #f5b93e;"><b>'.$comments_row['user'].'</b></font></a><font style="padding-left: 5px;font-family: Arial; font-size: 10px;color: #797979;">'.$comments_row['time'].' / '.$comments_row['date'].'</font><font style="margin-left: 235px;font-family: Arial; font-size: 10px;color: #ffffff;"><b>#'.--$number_id.'</b></font><br>
   <font style="font-family: Arial; color: #bbbbbb; font-size: 10px;">'.$comments_row['text'].'</font>
   </td></tr></table>
   </td></tr>                                                                                                                              
   <tr><td><img src="img/bg_comments_3.png"></td></tr>
   </table>';
  }
echo'</td></tr>
<tr><td>'; 
 
    //Pobieramy ilosc danych w bazie
$comments = "SELECT FOUND_ROWS() as Ilosc";
$comments_query = mysql_query($comments);
 
 
 
list($rows_page_cm) = mysql_fetch_row($comments_query);
 
 
echo '<center>'; 
 
if($_GET['page_cm']>0){ 
 
 
   echo '<a href="index_test.php?site=viev_news&&id_news='.$_GET['id_news'].'&&page_cm='.($_GET['page_cm']-1).'">wstecz</a> ';
}
 
for($i = 0;$i<ceil($rows_page_cm/limit_cm);$i++){
   echo '<a href="index_test.php?site=viev_news&&id_news='.$_GET['id_news'].'&&page_cm='.($i).'">[ '.($i+1).' ]</a> ';
}
 
 
if($_GET['page_cm']<ceil($rows_page_cm/limit_cm)-1){  
 
   echo ' <a href="index_test.php?site=viev_news&&id_news='.$_GET['id_news'].'&&page_cm='.($_GET['page_cm']+1).'">dalej</a>';
}
 
echo '</center>';
echo'</td></tr>
  <tr style="background-image: url(img/bg_site_repeat_2.png);">
  <td><form action="index_test.php?site=viev_news&&id_news='.$_GET['id_news'].'" method="post">
  <table align="center" cellspacing="0" cellpadding="0">
  <tr><td style="padding-bottom: 5px;text-align: justify; color: #FFFFFF; font-family: Arial; font-size: 14px;"><b>DODAJ KOMENTARZ:</b></td></tr>
  <tr><td><img src="img/bg_add_cm_1.png"></td></tr>
  <tr align="center"><td align="center">
  <textarea name="valcm" style="margin: 0px;padding-left: 5px; padding-right: 5px; border: 0px;background-image: url(img/bg_add_cm_2.png);resize: none; width: 395px; height:80px;"></textarea>
  </td></tr>
  <tr><td><img src="img/bg_add_cm_3.png"></td></tr>
  <tr><td style="padding-top: 10px;" align="right">';
 
 
 
 if($id_user != 0) {
  echo $error;
echo'
 <input type="hidden" name="add_cm" value="yes" />
 <input type="image" src="img/button_send_cm.png" />
      ';
 }
 else{
 
 echo'Tylko zalogowani użytkownicy mogą dodawać komentarze!';
 
 }
 
echo '</form></td></tr>
  </table>
  </td>
  </tr>
 
  <tr><td><img src="img/bg_site_bottom_3.png"></td></tr>
 
  </table>';   
 
 
 
?>
[PHP] pobierz, plaintext 

[CuteOne]

Dobra rada.. przepisz skrypt na nowo bo tu jest jedno wielkie bagno.

1. Włącz raportowanie błędów
2a. Jeżeli używasz or die(mysql_error()); to staraj się dodać je do każdego zapytania a nie jedno z mysql_error(), drugie z ('blad') a inne bez
2b. Tego or die() nie powinno być przy stringu
$viev_news = "SELECT id, title, author, icon, date, time, img, vievs, comments, img1, info1, info2
FROM news WHERE id=".$_GET['id_news']."" or die('Blad');
3. Po co w URL linków dajesz dwa razy && ?
4. Po co zliczasz rekordy w zapytaniu a później znowu za pomocą mysql_num_rows() - poczytaj o COUNT
5. Używasz zmiennych, których nigdzie nie zadeklarowałeś

i wiele innych...

ps. do paginacji użyj gotowca - będzie o niebo lepszy niż to co masz teraz

Ten post edytował CuteOne 18.06.2012, 04:31:37

[Sasuke Uchiha]

Dziękuej za rady ale to nie rozwiązuje w zadnym stopniu mojego problemu... nie chcę żadnych gotowców bo tak niczgo się nie nauczę.

[peter13135]

Wiem, że jego odpowiedź Cię nie zadowoliła, bo pewnie chciałbyś otrzymać informację "w tej linni powinno być to i tamto zamiast czegoś tam". Niestety, ale Twój skrypt jest taki brudny, że analizowanie go nie sprawi nikomu przyjemności.

Powinieneś się cieszyć, że CuteOne przebrnął przez 10 linii Twojego skryptu i wypisał Ci błędy jakie tam popełniłeś

Ja również nie chcę się zbytnio męczyć. Powiem tyle:
w linii 110 masz ten kod który wyświetla przycisk "dalej".
Jest on wtedy wyświetlany, gdy spełniony jest warunek w linii 108.
Skoro się nie wyświetla, a Twoim zdaniem powinien, to zobacz sobie, co w tych zmiennych siedzi i na podstawie tego będziesz wiedział czemuj warunek nie jest spełniony.

[Sasuke Uchiha]

no właśnie nie wiem co jest źle skoro w poprzednim skrypcie dokładnie tak sam to rozwiązałem i wszystko działa.

[!*!]

Zacznij sprawdzać co jest nie tak przez var_dump(). http://www.phpro.org/tutorials/Pagination-...HP-and-PDO.html tu masz w miarę dobry skrypt paginacji, obiektowy, ale jak choć trochę znasz PHP to załapiesz i przystosujesz.

Ten post edytował !*! 18.06.2012, 09:42:42

[Sasuke Uchiha]

Nie bardzo się rozumiemy gdyż chcę zrozumieć co w tym konkretnym przykładzie jest źle

[!*!]

Postaw się w Naszej sytuacji. Pokazujesz kod który ma ~150 linii, jest źle napisany, nawet to nie jest angielski, już pomijając składnie... Trudno wskazać cokolwiek, ponieważ:

1. nie podałeś żadnego błędu, tylko że Ci się coś nie wyświetla
2. nie mamy tego działającego kodu u siebie, a najzwyczajniej nikt nie kopiuje kodu z forum i nie wymyśla bazy do niego, tylko po to żeby go sprawdzić.

Tak jak pisałem, zacznij wstawiać co kilka linijek var_dump(), co masz sprawdzać? Zmienne jakie odbierasz i porównaj je z tymi co mają być. W ten sposób znajdziesz "błąd".

Obstawiałbym to:

[PHP] pobierz, plaintext 
$comments = "SELECT SQL_CALC_FOUND_ROWS * FROM comments WHERE id_news=".$_GET['id_news']." ORDER BY id DESC LIMIT ". mysql_escape_string((int)$_GET['page_cm'] *limit_cm).",".limit_cm;
[PHP] pobierz, plaintext 

Ten post edytował !*! 18.06.2012, 10:41:29

[peter13135]

Czemu sobie po prostu nie wyświetlisz tych zmiennych sam nie zobaczysz ?
Problem jest oczywisty do rozwiązania.
Nie wyświetla Ci się ten przycisk dalej bo nie spełniony jest warunek
warunek nie jest spełniony bo $_GET['page_cm'] nie jest mniejsze od ceil($rows_page_cm/limit_cm)-1, czyli któraś wartość jest nieprawidłowa
Wyświtl te wartości i zbadaj która wartość jest błędna.
Następnie zastanów się, czemu wartośc jest błędna.

[Sasuke Uchiha]

ale nie wyświetla się żaden błąd w dodatku jest kropka w kropke zrobione tak samo jak poprzedni SKRYPT... a sam kod proszę bardzo podaje linijki w którym jest część tego skryptu:
- od 86 do 112
- 37 oraz 38

Naprawdę reszta nie ma tutaj żadnego znaczenia...

[!*!]

I właśnie dlatego to Ty masz to sprawdzić. Jak, już Ci napisałem, peter13135 zresztą też.

[Sasuke Uchiha]

no ok już chyba wiem co jest nie tak źle zlicza chyba ilość komentarzy bo zamiast 12 w wartości $rows_page_cm wyświetla że jest 1 hmm tylko dlaczego skoro w poprzednim skrypcie dobrze liczy oO

[peter13135]

CuteOne napisał Ci:

4. Po co zliczasz rekordy w zapytaniu a później znowu za pomocą mysql_num_rows() - poczytaj o COUNT

a Ty mówisz, że jego post nie jest pomocny i nie rozwiązuje problemu

[Sasuke Uchiha]

CuteOne napisał Ci:

a Ty mówisz, że jego post nie jest pomocny i nie rozwiązuje problemu

Bo najpierw to zliczam do pętli aby potem móc sobie od tego odejmować id posta żebym nie musiał pisać 2 niepotrzebnych zapytań dodatkowych do bazy przy wprowadzaniu nowego rekordu. W dodatku zmieniłem teraz nawet nazwy zmiennych aby nie były identyczne i co ? i nic dalej jest wartość 1

Ten post edytował Sasuke Uchiha 18.06.2012, 10:55:53

[!*!]

A czytałeś w ogóle co pisaliśmy? Robisz wszytko tak, jak robić się nie powinno. Dopracuj skrypt lepiej. Co ma wspólnego dodawanie rekordów do paginacji? To są 2 różne rzeczy, a Ty chcesz je łączyć w 1? Litości.

[Sasuke Uchiha]

nie nic nie łącze tylko ktoś sie pytał dlaczego robię 2 razy to samo więd odpowiadam jak to rozwiązałem. Pytanie moje brzmi dlaczego ten sam sposób idealnie ten sam... działa w tym skrypcie ?

[PHP] pobierz, plaintext 
<?php
 
define('limit_news', 3);
 
$news = "SELECT SQL_CALC_FOUND_ROWS * FROM news ORDER BY id DESC LIMIT ". mysql_escape_string((int)$_GET['porcja'] *limit_news).",".limit_news;
$news_query = mysql_query($news);
 
echo'<div id="news">';
while($news_row = mysql_fetch_array($news_query))
{
echo'
  <table style="margin-bottom: 10px;" valign="top" cellspacing="0" cellpadding="0">
  <tr><td><img src="img/bg_site_news_1.png"></td></tr>
  <tr style="background-image: url(img/bg_site_repeat_2.png);"><td>
 
  <table style="margin-left: 7px;"  valign="top" cellspacing="0" cellpadding="0">
  <tr><td style="background-repeat: no-repeat; padding-left: 10px; color: #151515; font-family: Arial; font-size: 14px; width: 490px; height: 45px; background-image: url(img/bg-title.png);">
  <table cellspacing="0" cellpadding="0"><tr>
  <td><img src="img/icons/'.$news_row['icon'].'"></td>
  <td style="padding-left: 5px;">
  <font style="color: #151515; font-size: 14px;"><b>'.$news_row['title'].'</b></font><br> <font style="color: #505050; font-size: 11px;">Autor: <b>'.$news_row['author'].'</b>  '.$news_row['date'].' '.$news_row['time'].'</font>
  </td></tr></table></td></tr>
  <tr><td><table style="margin-right: 16px; margin-top: 10px; margin-left: 9px;" cellspacing="0" cellpadding="0">
  <tr><td style="text-align: justify; color: #FFFFFF; font-family: Arial; font-size: 11px;"><img style="margin-top: 3px;margin-right: 5px;" align="left" src="img/'.$news_row['img1'].'">'.$news_row['info1'].'<br><br></td></tr>
  <tr><td style="padding-left: 15px; color: #FFFFFF; font-family: Arial; font-size: 11px; width: 473px; height: 31px; background-image: url(img/bg_info_news_3.png); background-repeat: no-repeat;">Wyświetleń: <font color="#ffc000">'.$news_row['vievs'].'</font>  Komentarzy: <font color="#ffc000">'.$news_row['comments'].'</font> <font style="padding-left: 210px;"><a style="color: #ffc000;" href="?site=viev_news&&id_news='.$news_row['id'].'">czytaj więcej...</a></font></td></tr>
   </table></td></tr>
  </table>
 
  </td></tr>
  <tr><td><img src="img/bg_site_bottom_3.png"></td></tr>
  </table>';
}
//Pobieramy ilosc danych w bazie
$news = "SELECT FOUND_ROWS() as Ilosc";
$news_query = mysql_query($news);
 
 
 
list($iloscWpisow) = mysql_fetch_row($news_query);
 
 
echo '<center>'; 
 
if($_GET['porcja']>0){ 
 
 
   echo '<a href="?porcja='.($_GET['porcja']-1).'">wstecz</a> ';
}
 
for($i = 0;$i<ceil($iloscWpisow/limit_news);$i++){
   echo '<a href="?porcja='.($i).'">[ '.($i+1).' ]</a> ';
}
 
 
if($_GET['porcja']<ceil($iloscWpisow/limit_news)-1){  
 
   echo ' <a href="?porcja='.($_GET['porcja']+1).'">dalej</a>';
}
 
echo '</center></div>';
 
?>
[PHP] pobierz, plaintext 

[peter13135]

Jak już jesteśmy przy niepotrzebnych zapytaniach. To zastanów się po co trzymasz w tabeli newsów liczbę komentarzy

[Sasuke Uchiha]

no i wracamy do punktu wyjścia to nie ma teraz tutaj w tym momencie żadnego znaczenia.

[!*!]

Pytanie moje brzmi dlaczego ten sam sposób idealnie ten sam... działa w tym skrypcie ?

W tym drugim robisz to źle.

[Sasuke Uchiha]

Tak tylko że jest jota w jotę tak samo zrobione schematycznie no ale jest źle no zasadniczo coś musi być źle skoro tutaj napisałem i nie działa