Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP][MySQL] Księga gości
Forum PHP.pl > Forum > Przedszkole
kamil_lk
Witam. Napisałem sobie księgę gości wraz ze stronnicowaniem i chciałbym się dowiedzieć, czy jest zrobiona w miarę dobrze, czy czegoś brakuje, ewentualnie jest do poprawki.
Od razu mam pytanie, co mam zrobić aby wpisy na stronie były wyświetlane od razu, a nie po kliknięciu dopiero na którąś podstronę (czyli na index.php a nie index.php?strona=1)?

index.php
  1. <?php
  2. $imie = $_POST['imie'];
  3. $email = $_POST['email'];
  4. $www = $_POST['www'];
  5. $tresc = $_POST['tresc'];
  6.  
  7. if($email == "") $email = "brak";
  8. if($www == "") $www = "brak";
  9.  
  10. $polacz = mysql_connect('localhost', 'root', 'krasnal') or die ("Nie połączono z bazą");
  11.  
  12. mysql_create_db('ksiega1');
  13. mysql_select_db('ksiega1');
  14.  
  15. //dodawanie wpisów do bazy
  16. if(!empty($imie) && !empty($tresc)){
  17. $zapytanie = 'CREATE TABLE dane (id int NOT NULL AUTO_INCREMENT, imie char(50), email char(50), www char(50), tresc text, PRIMARY KEY(id))';
  18. $idzapytania = mysql_query($zapytanie);
  19.  
  20. $zapytanie = "INSERT INTO `dane` (`id`, `imie`, `email`, `www`, `tresc`) VALUES ('', '$imie', '$email', '$www', '$tresc')";
  21. $idzapytania = mysql_query($zapytanie);
  22. }
  23.  
  24. // stronnicowanie i wyświetlanie wpisów
  25. $nastronie = 25;
  26. $start = ($_GET['strona']-1) * $nastronie;
  27.  
  28. $zapytanie = "SELECT * FROM `dane` ORDER BY 'id' DESC LIMIT $start, $nastronie";
  29. $idzapytania = mysql_query($zapytanie);
  30.  
  31. echo "<table>";
  32. while($wynik = mysql_fetch_row($idzapytania)){
  33. echo "<tr><td>Imie:</td><td>".$wynik[1]."</td></tr>";
  34. echo "<tr><td>E-mail:</td><td>".$wynik[2]."</td></tr>";
  35. echo "<tr><td>Www:</td><td>".$wynik[3]."</td></tr>";
  36. echo "<tr><td>Treść:</td><td>".$wynik[4]."</td></tr>";
  37. }
  38. echo "</table>";
  39.  
  40. // linki do stron
  41. $ile = 0;
  42. $zapytanie = "SELECT `id` FROM `dane`";
  43. $idzapytania = mysql_query($zapytanie);
  44. while($id = mysql_fetch_row($idzapytania)){
  45. $ile++;
  46. }
  47.  
  48. for($i = 1; $i<$ile/$nastronie+1; $i++){
  49. echo "<a href=index.php?strona=".$i." style=\"text-decoration: none\">[".$i."]</a> ";
  50. }
  51. mysql_close($polacz);
  52. ?>
nospor
Czemu za kazdym razem, gdy dodajesz nowy wpis do bazy, tworzysz ponownie tabelę?
ADeM
  1. $start = ($_GET['strona']-1) * $nastronie;
Zamień na:
  1. if( isset( $_GET[ 'strona' ] ) ) {
  2. $start = ( $_GET[ 'strona' ] - 1 ) * $nastronie;
  3. }
  4. else {
  5. $start = 0;
  6. }

Po co za każdym razem, przy dodawaniu wpisu, próbujesz też tworzyć bazę i tabelę w niej?

EDIT:
Spóźniłem się...
kamil_lk
hym.. myślałem że gdy mysql zobaczy że jest już taka tabela to opuści to zapytanie i przejdzie dalej winksmiley.jpg
nospor
Nie, mysql wywali blad, ze probujesz utworzyc tabele ktora juz istnieje. To raz. A dwa to to, ze generujesz niepotrzebnie jedno zapytanie. Ucz sie pisac optymalnie.
kamil_lk
Cytat
mysql wywali blad

nie wywala mi błędu

Cytat
Kod
      if( isset( $_GET[ 'strona' ] ) ) {
      $start = ( $_GET[ 'strona' ] - 1 ) * $nastronie;
      }
      else {
      $start = 0;
      }

dzięki, działa smile.gif
nospor
Cytat
nie wywala mi błędu
wywala, tylko nie widzisz.
kamil_lk
A jak mogę sprawdzić czy dana tabela już istnieje?

Możesz podać kod wyświetlający wszystkie błędy? smile.gif
CuteOne
Nie bądź taki leniwy... i zacznij sam szukać odpowiedzi.. google naprawdę nie boli
kamil_lk
  1. $tabela = mysql_list_tables('ksiega1');
  2. if(!empty($imie) && !empty($tresc)){
  3. if(!$tabela){
  4. $zapytanie = 'CREATE TABLE dane (id int NOT NULL AUTO_INCREMENT, imie char(50), email char(50), www char(50), tresc text, PRIMARY KEY(id))';
  5. $idzapytania = mysql_query($zapytanie);
  6. }
  7.  
  8. $zapytanie = "INSERT INTO `dane` (`id`, `imie`, `email`, `www`, `tresc`) VALUES ('', '$imie', '$email', '$www', '$tresc')";
  9. $idzapytania = mysql_query($zapytanie);
  10. }


czy tak dobrze?
ADeM
Potrzebne Ci to w ogóle jest?
Jeśli raz stworzyłeś tabelę w bazie danych, to ona tam jest cały czas, dopóki jej ktoś nie usunie.
kamil_lk
Potrzebne, ponieważ na razie uczę się, testuje i usuwam czasem tą bazę, albo może ktoś będzie chciał sobie to wykorzystać na stronie to wystarczy tylko wkleić i nie trzeba nic modyfikować oprócz połączenia z MySql.
Dodałem jeszcze linijkę, a właściwie to zmieniłem
  1. $db_list = mysql_list_dbs($polacz);
  2. if(!$db_list) mysql_create_db('ksiega1');


Dodałem także wyświetlanie wszystkich błędów i tu mam problem ponieważ wyrzuca mi
"Notice: Undefined index: imie in c:\usr\krasnal\www\1\index.php on line 24" gdzie ta linia to $imie = $_POST['imie']; podobnie jest z resztą przypisać z formularza, powie ktoś coś na temat tego błedu?

------------

Problem z błędami rozwiązałem smile.gif

Chodziło o to że pod każdą zmienną podstawiam wartość, która może nie istnieć i wywala mi błędy, ale już naprawiłem smile.gif
ADeM
To może przenieś sobię tworzenie bazy i tabel do osobnego pliku. Dla porządku.
Jeśli chodzi o te zmienne. To odwołujesz się do nie istniejących po prostu. Sprawdz wcześniej czy takie istnieją. Np.:
  1. if( isset( $_POST[ 'imie' ] ) ) {
  2. $imie = $_POST[ 'imie' ];
  3. }
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.