[PHP] Isset nie reaguje tak jakby, shoutbox Opcje

[Glukozamina]

Witam. Stworzyłem sobie takiego małego mojego shoutboxa i się zastanawiam dlaczego po kliknieciu dodaj nadal sie wykonuje... tzn. dodaje nowe wpisy mimo iż ma polecenie aby tego nie robił gdy pole tresc jest puste. 2. Jak zrobić, żeby podczas odswieżania nie dodawał sie za każdym razem ten wpis ? 3. Jak zrobić ,że gdy klikne na np. pole nick to nazwa tego pola(napis nick) mi zniknie automatycznie z formularza ?

[PHP] pobierz, plaintext 
<form method="GET" action="index.php"> 
<input type="text" name="nick" value="nick" size="20">
<input type="text" name="tresc" size="40">
<input type="submit" value="Dodaj">
</form>
 
 
<?php
 
function dodawanie($id, $nick, $tresc, $time, $ip){
 
  /*sprawdzenie poprawości przekazanych danych*/
 
 
if($tresc == ''){
    echo("Brak danych. Proszę wypełnić pole: Treść.");
    return;
  }
 
  }
  if($id == '') $id = 'NULL';
 
 $time=(date("H:i:s"));
 $ip=($_SERVER['REMOTE_ADDR']); 
 $nick=($_GET['nick']);
 $tresc=($_GET['tresc']);	
 
 
 
  /*Nawiązanie połączenia z bazą*/
 
  if (!$db_lnk = @mysql_connect("x", "x", ":)")){
    echo('Wystąpił błąd podczas próby połączenia z serwerem MySQL...<BR>');
    return;
  }
 
  if(!@mysql_select_db('x')){
    echo('Wystąpił błąd podczas wyboru bazy danych: biblioteka<BR>');
    @mysql_close();
    return;
  }
 
  /*Utworzenie i wykonanie zapytania*/
 
  $query = "INSERT INTO shoutbox VALUES(";
  $query .= "'$id', '$nick', '$tresc', '$time', '$ip'";
  $query .= ")";
 
 
 
  if(!mysql_query($query, $db_lnk)){
    echo 'Wystąpił błąd: zapytanie zostało odrzucone...<br />Zapytanie: <pre>' . $query . '</pre>Odpowiedź MySQL: <pre>' . mysql_error( $db_lnk ) . '</pre>';
    @mysql_close();
    return;
  }
 
 
 
  /*Pobranie informacji o liczbie wstawionych wierszy*/
 
  $rowsNo = mysql_affected_rows($db_lnk);
 
  echo("Dziękuje za pozostawioną wiadomość . <br>");
  echo("Liczba dodanych rekordów: $rowsNo<br>");
 
 
  /*Zamknięcie połączenia z bazą*/
 
  if(!@mysql_close()){
    echo('Wystąpił błąd podczas zamykania połączenia z serwerem MySQL...<br>');
  }
 
if(isSet($_GET['tresc'])){                                            /*<--------------------------- tutaj. to jest dorbze ?*/
  dodawanie($_GET['$id'], $_GET['$nick'], $_GET['tresc'], $_GET['$time'], $_GET['$ip']);
}
else{
  echo("Niepoprawne dane! Wypełnij wszystkie pola.");
}
 
 
?>
[PHP] pobierz, plaintext 

[kapuch]

Nie patrzylem w kod z racji poznej godz. ale moim zdaniem ani warunek !empty, ani isset nie pomoze, gdy odswiezamy strone, dlatego ze w adresie te zmienne nadal sa ustawione.
Mam trzy rozwiazania:
1. Ustawiasz ciasteczko podczas dodawania wpisu, ktore zablokuje na jakis czas danego uzytkownika
2. Sprawdzasz czy dany wpis juz nie istnieje, tzn czy nie jest dokladnie taki sam, jak wczesniejsze (zle rozwiazanie, bo bedzie blokowac krotkie wpisy typu "ok", "tak", ktore moze dodac kilka uzytkownikow).
3. Moim zdaniem najlepsze rozwiazanie, to po dodaniu wpisu albo przekierowujesz klienta do $_SERVER['http_referer'] (chociaz ta zmienna jest 'wadliwa', wiec mozna ja "podrobic" sesjami), albo na strone glowna - wtedy w linku nie bedzie ustawionych zmiennych (IMG:style_emoticons/default/winksmiley.jpg)
BTW. Uzywaj do tego POSTa a nie GETa.

Ten post edytował kapuch 22.07.2010, 00:26:10