[php] Lekkie przerobienie instrukcji warunkowej, Z góry dziękuję Opcje

[CryWolf]

Witam!
Mam kod:

[PHP] pobierz, plaintext 
<?php
 
session_start();
//wczytanie pliku z funkcjami
  require_once ("funkcje.php");
//rozpoczęcie sesji
 
include "tabelaleft.php";
pol_mysql();
 
include_once "blokada.php";
 
?>
 
<td width="500" height="200">
 
<?php
 
echo "Obecnie Twoje GG to: ";
 
$pokazgg = "SELECT gg FROM users WHERE login = '". $_SESSION['user'] ."'";
$wykonaj = mysql_query($pokazgg);
while($wiersz = mysql_fetch_array($wykonaj))
{
print " ".$wiersz['gg']."<br>";
 
}
 
?>
 
 
<form action="changegg.php" method="POST">
Wpisz nowe GG i potwierdź! <input type="text" name="gg">
<input type="submit" value="Zmień">
</form>
 
<?php
$gg = $_POST['gg']; // zmienna gg z formularza
 
 
mysql_query("UPDATE users SET gg = $gg WHERE login = '". $_SESSION['user'] ."'");
 
 
 
 
?> 
 
</td>
 
<?php
include "tabelaright.php";
?>
[PHP] pobierz, plaintext 

Chodzi o to, że wyświetli mi, że aktualne gg to 2871296, lecz po odświeżeniu strony już nic się nie pokazuje bo chwile dalej część kodu:

[PHP] pobierz, plaintext 
<?php
mysql_query("UPDATE users SET gg = $gg WHERE login = '". $_SESSION['user'] ."'");
?>
[PHP] pobierz, plaintext 

dodaje mi pusty rekord. Najpierw wyświetla się formularz do wypełnienia i potem skrypt nie czeka i dodaje pusty rekord do bazy danych. Wiem, że trzeba to zrobić instrukcją warunkową ale nie wiem jak. Gdy ktos wejdzie w edycję GG ale jednak nie bedzie chcial ostatecznie zmienic gg to po wyjsciu ze strony zostanie pusta niewypełniona kolumna gg. Lecz gdy wejdzie na stronę, wyświetli mu się jego GG i wpisze do formularza nowe GG i potwierdzi to wtedy gg się zachowa. Pewnie wiecie już o co chodzi ale wolę wytłumaczyć, a i czy metoda GET jest tu dobrym pomysłem czy lepiej użyć POST?
Pozdrawiam i z góry dziękuję

Ten post edytował CryWolf 15.07.2007, 14:39:53

[-=Peter=-]

[PHP] pobierz, plaintext 
<?php
if(is_numeric($_POST['gg'])){
  $gg = $_POST['gg']; // zmienna gg z formularza
  mysql_query("UPDATE users SET gg = $gg WHERE login = '". $_SESSION['user'] ."'");
}
?>
[PHP] pobierz, plaintext 

[JaRoPHP]

[PHP] pobierz, plaintext 
<?php
if(!empty($_POST['gg'])) {
  $gg = $_POST['gg']; // zmienna gg z formularza
  mysql_query("UPDATE users SET gg = $gg WHERE login = '". $_SESSION['user'] ."'");
}
?>
[PHP] pobierz, plaintext 

--
małe spóźnienie

Ten post edytował JaRoPHP 15.07.2007, 14:49:19

[CryWolf]

Jeszcze ostatnia sprawa. jestem w tym nowy i nie wiem.
Co tu jest źle? Coś chyba z else ale nie jestem pewien.

[PHP] pobierz, plaintext 
<?php
if(isset($_POST['gg']))
{
$gg = floor($_POST['gg']); // zmienna gg z formularza
mysql_query("UPDATE users SET gg = $gg WHERE login = '". $_SESSION['user'] ."'"); }
 
else { echo "
 
<form action="changegg.php" method="POST">
Wpisz nowe GG i potwierdź! <input type="text" name="gg">
<input type="submit" value="Zmie&#324;">
</form>";
}
?>
[PHP] pobierz, plaintext 

Ten post edytował CryWolf 15.07.2007, 15:53:00

[JaRoPHP]

Dlaczego sprawdzasz, czy pole zostało wypełnione funkcją isset" title="Zobacz w manualu PHP" target="_manual. Nawet jeśli pole to będzie puste, a zostanie przesłane (będzie w formularzu), funkcja zwróci wartość true.

[PHP] pobierz, plaintext 
<?php
if(!empty($_POST['gg'])) {
  $gg = $_POST['gg']; // zmienna gg z formularza
  mysql_query("UPDATE users SET gg = $gg WHERE login = '". $_SESSION['user'] ."'");
} else {
  // po echo powinien być apostrof
  echo ' 
	<form action="changegg.php" method="POST">
	Wpisz nowe GG i potwierdź! <input type="text" name="gg">
	<input type="submit" value="Zmień">
	</form>'; // i znowu apostrof
}
?>
[PHP] pobierz, plaintext 

Ten post edytował JaRoPHP 15.07.2007, 15:59:19