nie odczytuje mi komentarzy o id newsa, mysql Opcje

[kemsi]

Witam!! Stworzyłem swój taki prosty system newsów na bazie mysql. Gdy próbuje odczytać komentarze do danego newsa to nie wiem czemu, ale nie mogę ich odczytać. Może coś się nie zapisuje, ale już sam nie wiem. oto kody trzech plików(jest jeszcze więcej, ale tylko te będą prawdopodobnie potrzebne):

Odczytywanie newsów:

[PHP] pobierz, plaintext 
<? 
   $polancz = mysql_connect(&#092;"localhost\",\"root\",\"krasnal\") or die(); 
   mysql_select_db('news'); 
 
   $odczyt = mysql_query(&#092;"SELECT `id`, `temat`, `tresc`, `nick`, `mail` FROM `news` ORDER BY `id` DES
C\"); 
   while($wiersz = mysql_fetch_array($odczyt)) { 
   echo &#092;"<table border=1 width=300><tr><td>Temat: \".$wiersz['temat'].\"</td></tr> 
		 <tr><td>&#092;".$wiersz['tresc'].\"</td></tr> 
		 <tr><td><a href=&#092;"mailto:\".$wiersz['mail'].\"\">\".$wiersz['nick'].\"</a></td></tr> 
		 <tr><td><a href=&#092;"komentuj.php?id=\".$wiersz['id'].\"\">Komentarze</a></table><br><br>\"; 
   } 
?>
[PHP] pobierz, plaintext 

Odczytywanie i dodawanie komenatrzy:

[PHP] pobierz, plaintext 
<? 
  //łączenie z bazą 
  $polancz=mysql_connect(&#092;"localhost\",\"root\",\"krasnal\") or die(); 
  mysql_select_db('news'); 
 
  $odczytaj_kom=mysql_query(&#092;"SELECT `id`,`komentarz`,`nick`,`mail` FROM `komentarze` ORDER BY `id` DESC\"); 
  while($a=mysql_fetch_array($odczytaj_kom)){ 
  if($id==$a['id']){ 
  //odczytywanie komentarzy 
  echo &#092;"<table border=1><tr><td>\".$a['komentarz'].\"</td></tr> 
  <tr><td>Nick: &#092;".$a['nick'].\"   Mail:  \".$a['mail'].\"<br><br>\"; 
  } 
  } 
 
  //formularz dodawania komentarzy 
  echo &#092;"<form action=\"kom.php\" method=\"POST\"><table border=\"1\"> 
  <tr><td valign=&#092;"top\">Tresc:<textarea cols=\"36\" rows=\"6\" name=\"komentarz\"></textarea></td></tr> 
  <tr><td>Nick:<input type=&#092;"text\" name=\"nick\"></td></tr> 
  <tr><td>Mail:<input type=&#092;"text\" name=\"mail\"></td></tr> 
  <tr><td><input type=&#092;"submit\" value=\"WYŚLIJ\">	<input type=\"reset\" value=\"WYCZYSC\"></table></form>\"; 
?>
[PHP] pobierz, plaintext 

Zapisywanie komentarzy:

[PHP] pobierz, plaintext 
<? 
  $poloncz=mysql_connect(&#092;"localhost\",\"root\",\"krasnal\") or die(); 
  mysql_select_db('news'); 
 
  $dodaj=&#092;"INSERT INTO komentarze (id, komentarze, nick, mail) VALUES ('$id','$komentarze','$nick','$mail')\"; 
  $query=mysql_query($dodaj); 
?>
[PHP] pobierz, plaintext 

Z góry dzięki za pomoc.

[Darti]

a może tak? :

[PHP] pobierz, plaintext 
<?php
 
$odczytaj_kom=mysql_query(&#092;"SELECT `id`,`komentarz`,`nick`,`mail` FROM `komentarze` WHERE id=\".$_GET['id'].\"\"); 
  while($a=mysql_fetch_array($odczytaj_kom)){ 
 
  echo &#092;"<table border=1><tr><td>\".$a['komentarz'].\"</td></tr> 
  <tr><td>Nick: &#092;".$a['nick'].\"   Mail:  \".$a['mail'].\"<br><br>\"; 
  
  } 
 
?>
[PHP] pobierz, plaintext 

a z dodawaniem spróbuj tak:

[PHP] pobierz, plaintext 
<?php
$poloncz=mysql_connect(&#092;"localhost\",\"root\",\"krasnal\") or die(); 
  mysql_select_db('news'); 
 
  $dodaj=&#092;"INSERT INTO komentarze (id, komentarze, nick, mail) VALUES ('\".$_POST['id'].\"','\".$_POST['komentarze'].\"','\".$_POST['nick'].\"','\".$_POST['mail'].\"')\"; 
  $query=mysql_query($dodaj);
?>
[PHP] pobierz, plaintext 

Ten post edytował Darti 10.03.2005, 20:17:45

[kemsi]

Niestety to nic nie dało. A na dodatek pokazywały się błędy z nie poprawnym kodem. Ma ktoś jeszcze jakiśpomysł?

[tiraeth]

Proponuję napisać od nowa... i najlepiej z taką strukturą bazy:

[SQL] pobierz, plaintext 
CREATE TABLE `news` (
`newsid` INT NOT NULL AUTO_INCREMENT ,
`news_title` VARCHAR( 100 ) DEFAULT 'Brak tematu' NOT NULL ,
`news_body` TEXT NOT NULL ,
`news_user` VARCHAR( 20 ) DEFAULT 'Anonymous' NOT NULL ,
`news_date` INT( 10 ) NOT NULL ,
PRIMARY KEY ( `newsid` ) 
);
 
CREATE TABLE `news_comments` (
`commentid` INT NOT NULL AUTO_INCREMENT ,
`comment_newsid` INT( 4 ) NOT NULL ,
`comment_body` VARCHAR( 255 ) NOT NULL ,
`comment_nick` VARCHAR( 20 ) DEFAULT 'Anonymous' NOT NULL ,
`comment_date` INT( 10 ) NOT NULL ,
`comment_ip` VARCHAR( 50 ) DEFAULT '127.0.0.1' NOT NULL ,
`comment_host` VARCHAR( 255 ) DEFAULT 'localhost' NOT NULL ,
PRIMARY KEY ( `commentid` ) 
);
[SQL] pobierz, plaintext 

Tak będzie łatwiej... pamiętaj, że *_date to stempel czasu (funkcja time();)

[rogrog]

@Tiraeth: no oczywiście projekt bazy na pewno bardziej elegancki od tego co ma kemsi, tylko dlaczego używasz INTów do reprezentowania daty zamiast wbudowanych typów SQLa?? To wprowadza wg mnie tylko zamieszanie w bazie, utrudnia korzystanie z bazy i uniemożliwia używanie SQLowych uławień w korzystaniu z dat - nie widze zalet.

[Darti]

Niestety to nic nie dało. A na dodatek pokazywały się błędy z nie poprawnym kodem. Ma ktoś jeszcze jakiśpomysł?

ok poprawilem - brakowało "

[kemsi]

Dzięki Tiraeth za nową strukture, ale mam pytanie jak w systemie newsów stworzonym na twojej sktrukturze odczytam komentarze do danego newsa? Bo w bazach danych już jakośtego nie rozumiem.

[tiraeth]

[SQL] pobierz, plaintext 
SELECT * 
FROM news_comments WHERE comment_newsid=$_GET['nid']
[SQL] pobierz, plaintext 

gdzie $_GET['nid'] to numer newsa...

[kemsi]

Oto moje pytania co do twojej struktury:
1. comment_newsid=$_GET['nid'] - skąd wziąłeś to $_GET['nid'], bo z tego co kojarze to takie coś powinno być w tabeli w news_comments.
2.Czy jak dodaje numer komentarza do tabeli news_comments to zawsze taki sam numer newsa do tej tabeli muszę dodawać i potem używać tego co podałeś w poprzednim poście?

[Ociu]

1. comments.php?newsid=1
Jak chcesz zobaczyć komentarze to robisz coś takiego (a właściwie link do ich zobaczenia)

[PHP] pobierz, plaintext 
<?php
# tu wczesniejszy kod
print &#092;"<a href=comments.php?newsid=\".$a['id'].\">\";
?>
[PHP] pobierz, plaintext 

2. Tak

edit:
zapomniałem dodać: jeśli zrobisz tak jak ja napisałem wtedy dajesz $_GET['newsid']; (tak dla informacji...)
pozdrawiam

Ten post edytował Ociu 11.03.2005, 22:05:09

[SongoQ]

Panowie jak mozna dopuscic do takiej rzeczy:

[PHP] pobierz, plaintext 
<?php
 
$odczytaj_kom=mysql_query(&#092;"SELECT `id`,`komentarz`,`nick`,`mail` FROM `komentarze` ORDER BY `id` DESC\"); 
  while($a=mysql_fetch_array($odczytaj_kom)){ 
  if($id==$a['id']){ 
  //odczytywanie komentarzy 
  echo &#092;"<table border=1><tr><td>\".$a['komentarz'].\"</td></tr> 
  <tr><td>Nick: &#092;".$a['nick'].\"   Mail:  \".$a['mail'].\"<br><br>\"; 
  } 
  }
 
?>
[PHP] pobierz, plaintext 

Kto Cie uczyl baz danych. Nie zrozum mnie zle, jest tabelka ktora ma 50 milionow komentarzy i zwracasz je posortowane i spawdzasz w kodzie i wyswietlasz ten komentarz i tak czas wykonania takiego zapytania bedzie bardzo dlugi, jesli nie wierzysz to sprawdz i sie przekonaj, nastepnie jesli serwer php i bazy danych jest na osobnych maszynach, a nawet jesli sa to inne lacza to ja dziekuje za uzywanie takiego oprogramowania. Jesli był bym uzytkownikiem to by mnie "szlak" trafił ze sie tak długo ładuje, a jesli był bym adminem to bym Ci odrazu wyciachał baze i konto ftp. Takie rzczy nie mogą być dopuszczone.

Polecam najpierw poczytanie o bazach danych a nastepnie pisanie aplikacji.

[Ociu]

SongoQ: Szczerze mówiąc, nie zauważyłem tego.
IHMO: Zrobienie dobrego kodu SQL to 3/5 mniej roboty...

[PHP] pobierz, plaintext 
<?php
$id = $_GET['news_id'];
$odczytaj_kom = mysql_query(&#092;"SELECT `id`,`komentarz`,`nick`,`mail` FROM `komentarze` WHERE `news_id` = '\".$id.\"' ORDER BY `id` DESC\"); 
?>
[PHP] pobierz, plaintext 

pozdrawiam

[SongoQ]

Przegladajac to forum widac ze czesc ludzi prezentuje wysoki poziom, maja dobre pomysly, doswiadczenie ale jest tez inna czesc uzytkownikow co tak naprawde zanim zaczna cos pisac w php i sql powinni poczytac o bazach danych.

Przy duzych projektach klient np zazada przetestowania dla duzej ilosci danych i co wtedy: "przepraszam ale to dziala dla 10 rekordów" (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) ?

Na pewno ponownie nie skorzysta z waszych usług.

[kemsi]

SongoQ-trochę nie rozumiem o co ci chodzi z tym, że "przepraszam ale to dziala dla 10 rekordów". Wogóle jakośnie nie zrorumiałem tych twoich postów.

" jest tabelka ktora ma 50 milionow komentarzy i zwracasz je posortowane i spawdzasz w kodzie i wyswietlasz ten komentarz i tak czas wykonania takiego zapytania bedzie bardzo dlugi" - to jak mam to inaczej zrobić? Jak mozesz to wytłumacz to w jakiś prostszy sposób.
Z góry dzięki za odp.

[Ociu]

Jemu chodzi o to, że twój kod będzie działał wolno... gdyż pętla będzie 'pętlić się' tyle razy ile masz rekordów w bazie, a w niej dopiero sprawdzi czy "1 = 1". Lepiej zrobić to za pomocą SQL, ponieważ baza odrzuci te rekordy, które się nie równają czemuś tam. Różnica pomiędzy tym jest niewielka dla małej ilości wierszy... np. 10, ale gdy masz 5 000 rekordów i 15 jest z nich poprawnych ? wtedy kicha.. Twój skrypt będzie sprawdział 5 000 rekordów, a gdy dasz SQL, to on sam wykluczy i pętla będzie wyświetlać tylko 15 razy... różnica niewielka ?
Skorzystaj z mojego kodu.. ( (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) )

pozdrawiam

[SongoQ]

@Ociu dzieki za wytlumaczenie @kemsi.

Jak juz @Ociu pisal rozwiazanie masz powyzej.
Bazy danych sa stworzone po to zeby wyciagac takie rokordy ktore nas interesuja, w Twoim przypadku komentarze do newsa i nie ma potrzeby wyciagac reszty. Tak sie stosuje jesli masz dane w plikach.

[kemsi]

Dobra juzprawie wszystko rozumiem tylko przy wyświetlaniu tego idnewsa i id komentarza do newsa pojawił mi siętaki błąd:

Kod

Parse error: syntax error, unexpected T_VARIABLE in e:\usr\krasnal\www\php\skrypty\mysql\news\komentuj.php on line 7

A oto kod tego mojego pliku:

[PHP] pobierz, plaintext 
<?
  //łączenie z bazą
  $polancz=mysql_connect(&#092;"localhost\",\"root\",\"krasnal\") or die();
  mysql_select_db('news_comments');
 
  $id=$_GET['comment_news_id']
  $odczytaj_kom=mysql_query(&#092;"SELECT `commentid`,`comment_newsid`,`comment_body`,`comment_nick` FROM `new
s
comments` WHERE commentid='\".$_GET.\"' ORDER BY `commentid` DESC\");
  while($a=mysql_fetch_array($odczytaj_kom)){
  //odczytywanie komentarzy
  echo \"<table border=1><tr><td>\".$a['comment_body'].\"</td></tr>
  <tr><td>Nick: \".$a['comment_nick'].\"<br><br>\";
  }
 
  //formularz dodawania komentarzy
  echo \"<form action=\"kom.php\" method=\"POST\"><table border=\"1\">
  <tr><td valign=\"top\">Tresc:<textarea cols=\"36\" rows=\"6\" name=\"comment_body\"></textarea></td></tr>
  <tr><td>Nick:<input type=\"text\" name=\"comment_nick\"></td></tr>
  <tr><td><input type=\"submit\" value=\"WYŚLIJ\">	<input type=\"reset\" value=\"WYCZYSC\"></table></form>\";
?>
[PHP] pobierz, plaintext 

Z góry dzięki za odp.

[Darti]

bo nie masz średnika w linii 6 na końcu (lub na początku 7)
(IMG:http://forum.php.pl/style_emoticons/default/rolleyes.gif)

[SongoQ]

[PHP] pobierz, plaintext 
<?php
$odczytaj_kom=mysql_query(&#092;"SELECT `commentid`,`comment_newsid`,`comment_body`,`comment_nick` FROM `new
s
comments` WHERE commentid='\".$_GET.\"' ORDER BY `commentid` DESC\");
?>
[PHP] pobierz, plaintext 

$_GET - cos Ci tam brakuje => $_GET['comment_news_id'] albo $id

[kemsi]

Dzięki wielkie. To pomogło w tym problemie, ale teraz jest błąd taki:

Kod

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in e:\usr\krasnal\www\php\skrypty\mysql\news\komentuj.php on line 8

linia 8 to jest umnie z pętlą while.