[PHP][MySQL] Formularz edycji danych - nie zapisuje w bazie. Opcje

[bartekm1]

Witam,

mam taki błąd, mam połączenie z bazą, zrobiony formularz (niżej podany kod), i po wypełnieniu danych, nie zapisuje ich w mysql.

Co może być nie tak?

[PHP] pobierz, plaintext 
<?php
if(!empty($_POST["imienazwisko"]) && !empty($_POST["adres"]) && !empty($_POST["kodpocztowy"]) && !empty($_POST["miasto"]) )
{
    $_GET['id'] = (int)$_GET['id'];
    $sql = "select * from gracze where gracz = ".$_GET['id']."";
    $res = mysql_query($sql);
    $num = mysql_num_rows($res);
    if($num > 0)
    {
              //tutaj możesz dać walidację danych z formularza
              //następnie zapytanie sql
              $sql="UPDATE gracze SET imienazwisko='".$_POST["imienazwisko"]."',adres='".$_POST["adres"]."',kodpocztowy='".$_POST["kodpocztowy"]."',miasto='".$_POST["miasto"]."' WHERE id=$id";
              $result = mysql_query($sql)
        or die("blad");
            {
                      echo'update ok';
                  }
              //oczywiście zmienne $adres, $kod,$miasto to przefiltrowane zmienne pochodzące z formularza
    }
}
else
{
      $body = '<center>
      <h3>Edytuj Swoje Dane Osobowe</h3>
      <table>
      <tr>
      <td>
      <form action="vprofil" method="post">
      <h4>Imię i nazwisko</h4>
      <input type="text" name="imienazwisko" id="input" maxlength="30" value="'.$_POST['imienazwisko'].'" />
      <h4>Adres</h4>
      <input type="text" name="adres" id="input" maxlength="30" value="'.$_POST['adres'].'" />
      <h4>Kod pocztowy</h4>
      <input type="text" name="kodpocztowy" id="input" maxlength="30" value="'.$_POST['kodpocztowy'].'" />
      <h4>Miasto</h4>
      <input type="text" name="miasto" id="input" maxlength="30" value="'.$_POST['miasto'].'" /><br /></td></tr>
      <tr><td align="right">
      <input type="submit" id="button" value="Zapisz" />
      </form>
      </td></tr>
      </table>
      </center>' ;
      echo $body;
}
?>
[PHP] pobierz, plaintext 

A struktura tej bazy wygląda tak:

[SQL] pobierz, plaintext 
      CREATE TABLE `gracze` (
      `gracz` int(6) NOT NULL AUTO_INCREMENT,
      `haslo` varchar(64) COLLATE utf8_unicode_ci NOT NULL,
      `email` varchar(64) COLLATE utf8_unicode_ci NOT NULL,
      `data` datetime NOT NULL,
      `telefon` int(15) NOT NULL,
      `imienazwisko` varchar(99) COLLATE utf8_unicode_ci NOT NULL,
      `adres` varchar(99) COLLATE utf8_unicode_ci NOT NULL,
      `kodpocztowy` varchar(99) COLLATE utf8_unicode_ci NOT NULL,
      `miasto` varchar(99) COLLATE utf8_unicode_ci NOT NULL,
      PRIMARY KEY (`gracz`)
      ) ENGINE=MyISAM DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci ROW_FORMAT=DYNAMIC AUTO_INCREMENT=735 ;
[SQL] pobierz, plaintext 

[lobopol]

Zapewne dlatego, że nie masz ustawionego $_GET['id'] formularz idzie na vprofil. I skrypt zatrzymuję ci się na tym ifie:

[PHP] pobierz, plaintext 
if($num > 0)
[PHP] pobierz, plaintext 

[bartekm1]

Za bardzo nie znam sie na PHP A co zrobić żeby nie przenosiło tego na vprofil, tylko zeby zostało na tym vedycja, ale jak kliknie Zapisz, to żeby mu się wyświetliłą informacja "Poprawnie Zmieniłeś Swoje Dane" i tam pod spodem, hiperłącze do zakłądki "vprofil" z napisem (wróć do profilu).

Jak tak sie zrobi to będzie działało że zapisywało będzie?

[jackraymund]

po co ty to zmieniałeś lol, jakiś czas temu zrobiłem ci ten kodzik to na mus musiałeś znowu to wstawić lol
http://forum.php.pl/index.php?showtopic=19...mp;#entry949043
tam wystarczyło by tylko dodać jeden nawias do ifa

Ten post edytował jackraymund 19.03.2012, 19:38:09

[bartekm1]

No tak, ale był błąd w bazie. Również nie zapisywało tego w bazie

Co tu może być problemem?

[jackraymund]

[PHP] pobierz, plaintext 
$sql="UPDATE gracze SET imienazwisko='".$_POST["imienazwisko"]."',adres='".$_POST["adres"]."',kodpocztowy='".$_POST["kodpocztowy"]."',miasto='".$_POST["miasto"]."' WHERE id=$id";
              $result = mysql_query($sql)
[PHP] pobierz, plaintext 

niema pola id

[PHP] pobierz, plaintext 
  CREATE TABLE `gracze` (
      `gracz` int(6) NOT NULL AUTO_INCREMENT,
      `haslo` varchar(64) COLLATE utf8_unicode_ci NOT NULL,
      `email` varchar(64) COLLATE utf8_unicode_ci NOT NULL,
      `data` datetime NOT NULL,
      `telefon` int(15) NOT NULL,
      `imienazwisko` varchar(99) COLLATE utf8_unicode_ci NOT NULL,
      `adres` varchar(99) COLLATE utf8_unicode_ci NOT NULL,
      `kodpocztowy` varchar(99) COLLATE utf8_unicode_ci NOT NULL,
      `miasto` varchar(99) COLLATE utf8_unicode_ci NOT NULL,
      PRIMARY KEY (`gracz`)
      ) ENGINE=MyISAM DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci ROW_FORMAT=DYNAMIC AUTO_INCREMENT=735 ;
[PHP] pobierz, plaintext 

[Niktoś]

A powiedz jak wygląda link do strony chodzi mi o parametr id.

[bartekm1]

ID to jest kolumna "gracz"

[Niktoś]

A masz go w URL?Ja widzę metodę post,a ty getem coś skrobiesz.A co to właściwie za tworek?

$_GET['id'] = (int)$_GET['id'];

Ten post edytował Niktoś 19.03.2012, 19:48:55

[bartekm1]

To może innaczej

Będzie łatwiej jak udostępnie Wam plik rejestracji, ponieważ ja tego nie robiłem i sie za bardzo nie orientuje.

[PHP] pobierz, plaintext 
<h3>Rejestracja</h3>
<?
if($oGracz[gracz] > 0){
header("Location: ?a=konto");
}
$akcja = trim($_REQUEST["akcja"]);
if ($akcja=="rejestracja") {								
  $login = htmlspecialchars(stripslashes(trim($_POST["login"])), ENT_QUOTES);
  $haslo = htmlspecialchars(stripslashes(trim($_POST["haslo"])), ENT_QUOTES);
  $email = htmlspecialchars(stripslashes(trim($_POST["email"])), ENT_QUOTES);
  $in = htmlspecialchars(stripslashes(trim($_POST["in"])), ENT_QUOTES);
  $adres = htmlspecialchars(stripslashes(trim($_POST["adres"])), ENT_QUOTES);
  $kodpocztowy = htmlspecialchars(stripslashes(trim($_POST["kodpocztowy"])), ENT_QUOTES);
  $miasto = htmlspecialchars(stripslashes(trim($_POST["miasto"])), ENT_QUOTES);
  $plec = htmlspecialchars(stripslashes(trim($_POST["plec"])), ENT_QUOTES);
  $telefon = htmlspecialchars(stripslashes(trim($_POST["telefon"])), ENT_QUOTES);
  $data_urodzenia = date($_POST['data_urodzenia']['rok'].'-'.$_POST['data_urodzenia']['miesiac'].'-'.$_POST['data_urodzenia']['dzien']);
 
  if (strlen($login)<3 or strlen($login)>24
      or !eregi("^[a-zA-Z0-9_.]+$",$login)) { $blad++;
    echo "<div class='alert'><div class='inner'>Login musi mieć od 3 do 24 znaków bez polskich znaków i spacji!</div></div>";
  } else {
    if ($baza = mysql_connect($host_bazy_danych, $uzytkownik_bazy_danych, $haslo_bazy_danych)) {
      if (mysql_select_db($nazwa_bazy_danych)) {
       $wynik=mysql_query("SELECT * FROM gracze WHERE login='$login'");
	   $wynik2=mysql_query("SELECT email FROM gracze WHERE email='$email'");
      } else echo "Nie można połączyć się z bazą.";
      mysql_close($baza);
    } else echo "Nie można połączyć się z bazą.";
    if (mysql_num_rows($wynik)<>0) { $blad++;
      echo "<div class='alert'><div class='inner'>Taki login już jest w naszej bazie!</div></div>";
    }
	if (mysql_num_rows($wynik2)<>0) { $blad++;
      echo "<div class='alert'><div class='inner'>Taki e-mail już jest w naszej bazie danych!</div></div>";
    }
  }
 
  if (strlen($haslo)<6 or strlen($haslo)>50
     or !eregi("^[a-zA-Z0-9]+$",$haslo)) { $blad++;
    echo "<div class='alert'><div class='inner'>Hasło musi mieć od 6 do 50 znaków bez polskich znaków i spacji!</div></div>";
  }
 
  if (!eregi("^[0-9a-z_.-]+@([0-9a-z-]+\.)+[a-z]{2,4}$",$email)) { $blad++;
    echo "<div class='alert'><div class='inner'>E-mail nie został podany prawidłowo!</div></div>";
  }
	if (strlen($in)<3) {$blad++;
    echo "<div class='alert'><div class='inner'>Podaj prawidłowo imię i nazwisko!</div></div>";
    } 
    if (strlen($adres)<3) {$blad++;
    echo "<div class='alert'><div class='inner'>Podaj prawidłowo adres!</div></div>";
    }
    if (strlen($kodpocztowy)<3) {$blad++;
    echo "<div class='alert'><div class='inner'>Podaj prawidłowo kod pocztowy!</div></div>";
    }
    if (strlen($miasto)<3) {$blad++;
    echo "<div class='alert'><div class='inner'>Podaj prawidłowo miasto!</div></div>";
    } 
	if(isset($_POST['regulamin']) && 
	$_POST['regulamin'] == '0') {
	}
	else
	{ $blad++;
	    echo "<div class='alert'><div class='inner'>Musisz zaakceptować regulamin!</div></div>";
	}
	 require_once('var/recaptchalib.php');
  $privatekey = "6LcTWM4SAAAAAASgWIbHpdar89ZQDZPShyE8pT_B";
  $resp = recaptcha_check_answer ($privatekey,
                                $_SERVER["REMOTE_ADDR"],
                                $_POST["recaptcha_challenge_field"],
                                $_POST["recaptcha_response_field"]);
	if (!$resp->is_valid) {
    echo "";
  }
  if ($blad==0) {
    $kod = uniqid(rand());
    $haslo = md5($haslo);
    if ($baza = mysql_connect($host_bazy_danych, $uzytkownik_bazy_danych, $haslo_bazy_danych)) {
      if (mysql_select_db($nazwa_bazy_danych)) {
		$polecony = $_COOKIE['strona'];
        $wynik = mysql_query("INSERT INTO gracze SET login='$login', haslo='$haslo', kod='$kod', status='1', imienazwisko='$in', adres='$adres', kodpocztowy='$kodpocztowy', miasto='$miasto', telefon='$telefon', data_ur='$data_urodzenia', plec='$plec', polecony='$polecony', data=NOW(), email='$email'");
      } else echo "Nie można połączyć się z bazą.";
      mysql_close($baza);
    } else echo "Nie można połączyć się z bazą.";
 
    if ($wynik) {
$list="
Aby potwierdzić Twoją rejestrację, kliknij w poniższy link lub skopiuj go do paska adresu Twojej przeglądarki i potwierdź klawiszem Enter.
 
<a href="http://strona.pl/potwierdz:$kod" target="_blank">http://strona.pl/potwierdz:$kod</a>
 
Jeżeli nie rejestrowałeś się w serwisie, zignoruj ten list.
";
      mail($email, "Rejestracja", $list, "From: strona.pl <admin@strona.pl>");
      echo "<div class='ok'><div class='inner'>Aby dokończyć proces rejestracji, kliknij w link wysłany na Twój e-mail.</div></div>";
    }
  } else $akcja="";
}
 
if ($akcja=="") {
?>
<script type="text/javascript">
 var RecaptchaOptions = {
    theme : 'custom',
    custom_theme_widget: 'recaptcha_widget'
 };
[PHP] pobierz, plaintext 

Część druga pliku rejestracji:

[PHP] pobierz, plaintext 
</script>
<table width="100%"><tr><td><form action="?a=rejestracja" method="post">
<input type="hidden" name="akcja" id="input" value="rejestracja" />
<h4>Login</h4>
<input type="text" name="login" id="input" maxlength='24' value="<?php $login ?>" />
<h4>Hasło</h4>
<input type="password" name="haslo" id="input" maxlength='50' value="<?php $haslo ?>" />
<h4>E-mail</h4>
<input type="text" name="email" id="input" maxlength='50' value="<?php $email ?>" />
<h4>Imię i nazwisko</h4>
<input type="text" name="in" id="input" maxlength='30' value="<?php $in ?>" />
<h4>Adres</h4>
<input type="text" name="adres" id="input" maxlength='30' value="<?php $adres ?>" />
<h4>Kod pocztowy</h4>
<input type="text" name="kodpocztowy" id="input" maxlength='30' value="<?php $kodpocztowy ?>" />
<h4>Miasto</h4>
<input type="text" name="miasto" id="input" maxlength='30' value="<?php $miasto ?>" />
<h4>Płeć</h4>
<select name="plec" id="select"><option value="1">Mężczyzna</option><option value="2">Kobieta</option></select>
<h4>Data urodzenia</h4>
<select name="data_urodzenia[dzien]" id="styled"><?php for ($i=1; $i<=31; $i++){echo "<option value='$i'>$i</option>";}?></select> - 
<select name="data_urodzenia[miesiac]" id="select"><?php for ($i=1; $i<=12; $i++){echo "<option value='$i'>$i</option>";}?></select> - 
<select name="data_urodzenia[rok]" id="select"><?php for ($i=1998; $i>=1900; $i=$i-1){echo "<option value='$i'>$i</option>";} ?></select>
<h4>Numer telefonu</h4>
<input type="text" name="telefon" id="input" maxlength='15' value="<?php $telefon ?>" />
</td><td align="right"><script type="text/javascript"><!--
google_ad_client = "ca-pub-1157982378110244";
/* Duży kwadrat */
google_ad_slot = "8564127234";
google_ad_width = 300;
google_ad_height = 250;
//-->
</script>
<script type="text/javascript"
src="http://pagead2.googlesyndication.com/pagead/show_ads.js">
</script></td></tr></table>
<table cellspacing="20" width="600"><tr><td><h4>Przepisz kod potwierdzający</h4>
 
<div id="recaptcha_widget" style="display:none">
   <div class="recaptcha_only_if_incorrect_sol" style="color:red">Kod potwierdzający jest nieprawidłowy!</div>
   <input type="text" id="recaptcha_response_field" class="input" name="recaptcha_response_field" /></td><td>
      <br /><div id="recaptcha_image" style="border: 3px solid #043a4b"></div>
   <div><a href="java script:Recaptcha.reload()">Przeładuj obrazek</a></div>
 </div>
 </td></tr></table>
 
 <script type="text/javascript"
    src="http://www.google.com/recaptcha/api/challenge?k=6LcTWM4SAAAAAPQJ1kkwj8pYwa9avtqhQK658mLw ">
 </script>
 <noscript>
   <iframe src="http://www.google.com/recaptcha/api/noscript?k=6LcTWM4SAAAAAPQJ1kkwj8pYwa9avtqhQK658mLw "
        height="300" width="500" frameborder="0"></iframe><br>
   <textarea name="recaptcha_challenge_field" rows="3" cols="40">
   </textarea>
   <input type="hidden" name="recaptcha_response_field"
        value="manual_challenge">
 </noscript>
 <input name="regulamin" type="checkbox" name="regulamin" value="0">Zapoznałem/am się z <a href="?a=regulamin" target="_blank">regulaminem</a> serwisu<br />
<input type="submit" id="button" value="Dalej" />
</form>
<?php
KONIEC;
}
 
 
if ($akcja=="potwierdz") {
$kod = htmlspecialchars(stripslashes(trim($_GET["kod"])), ENT_QUOTES);
  if ($kod<>"") {
    if ($baza = mysql_connect($host_bazy_danych, $uzytkownik_bazy_danych, $haslo_bazy_danych)) {
      if (mysql_select_db($nazwa_bazy_danych)) {
        $wynik = mysql_query("DELETE FROM gracze
          WHERE data<=DATE_SUB(NOW(),INTERVAL 2 DAY) and status=1");
        $wynik = mysql_query("UPDATE gracze
          SET status='2', data=NOW() WHERE kod='$kod' and status=1");
        $wynik = mysql_query("SELECT * FROM gracze
          WHERE kod='$kod' and status=2");
      } else echo "Nie można połączyć się z bazą";
      mysql_close($baza);
    } else echo "Nie można połączyć się z bazą";
    if (mysql_num_rows($wynik)==1) {
      $dane = mysql_fetch_array($wynik);
      echo "<div class='ok'><div class='inner'>Dziękujemy. Rejestracja została zakończona poprawnie.</div></div>";
    }
  }
  if ($kod=="" or mysql_num_rows($wynik)<>1) {
    echo "<div class='error'><div class='inner'>Rejestracja nie może zostać dokończona - sprawdź czy link jest poprawny!</div></div>";
  }
}
 
 
 
if ($akcja=="przypomnij") {
echo <<<KONIEC
Aby skorzystać z funkcji przypominania hasła, należy wpisać swój adres email podany przy rejestracji.<br /><br />
<table width="100%"><tr><td><br /><br /><br /><form action="vrejestracja&akcja=wyslijhaslo" method="post">
<input type="hidden" name="akcja" value="wyslijhaslo" />
<h4>E-mail</h4>
<input type="text" name="email" id="input" value="$email" /><br />
<input type="submit" id="button" value="Dalej" />
</form></td>
<td align="right"><script type="text/javascript"><!--
google_ad_client = "ca-pub-1157982378110244";
/* Duży kwadrat */
google_ad_slot = "8564127234";
google_ad_width = 300;
google_ad_height = 250;
//-->
</script>
<script type="text/javascript"
src="http://pagead2.googlesyndication.com/pagead/show_ads.js">
</script></td></tr></table>
KONIEC;
}
 
 
function haslo() {
  $min = 6; $max = 12;
  for($i=0;$i<rand($min,$max);$i++) {
    $znak=chr(rand(48,122));
    if (eregi("[0-9a-zA-Z]",$znak)) $haslo .= $znak;
    else $i--;
  }
  return $haslo;
}
 
 
if ($akcja=="wyslijhaslo") {
 
 
  $email = htmlspecialchars(stripslashes(trim($_POST["email"])), ENT_QUOTES);
  $hasloczytelne = haslo();
  $haslo = md5($hasloczytelne);
  if ($email<>"") {
    if ($baza = mysql_connect($host_bazy_danych, $uzytkownik_bazy_danych, $haslo_bazy_danych)) {
      if (mysql_select_db($nazwa_bazy_danych)) {
        $wynik = mysql_query("UPDATE gracze SET haslo='$haslo' WHERE email='$email'");
        $wynik = mysql_query("SELECT * FROM gracze WHERE email='$email'");
      } else echo "Nie można połączyć się z bazą";
      mysql_close($baza);
    }
    if (mysql_num_rows($wynik)==1) {
      $dane = mysql_fetch_array($wynik);
      $email = $dane["email"];
      $list="Twoje nowe hasło to: $hasloczytelne";
      mail($email, "Przypomnienie hasła", $list,"From: strona.pl <admin@strona.pl>");
      echo "<div class='ok'><div class='inner'>Hasło zostało wyslane na Twój adres e-mail.</div></div>";
    } else {
      echo "<div class='alert'><div class='inner'>Podałeś nieprawidłowy adres e-mail!</div></div>";
    }
  }
}
[PHP] pobierz, plaintext 

Byłby ktoś w stanie pomóc??

Jakoś się odwdzięcze

[Niktoś]

A możesz to na live puścić?Chciałbym się dowiedzieć jak budowane są linki,bo w tym skrypcie nie widze innych parametrów jak paramter akcja a ty się odwołujesz do tablicy GET np:
$kod = htmlspecialchars(stripslashes(trim($_GET["kod"])).
$_GET['id'] = (int)$_GET['id'];

PS.Dla mnie ten formularz jest strasznie nie czytelny,istne spagetti,trudno rozeznać o co w nim chodzi.Zastanów się dobrze nad napisaniem nowego ,albo daj to komuś do poprawy w ramach jakiejś kary.

[bartekm1]

Wiem że ten formularz jest nie bardzo czytelnny.

Potrzebuje kogoś kto, pomoże stworzyć nowy który będzie działał. Ja nie ogarniam zbytnio PHP.

Dla kogoś kto umie to kilka minut. Prosze o pomoc

[Niktoś]

Trening czyni mistrza ,a jak Jesteś zbyt leniwy to do działu oferty.Ja też wszystkiego nie wiem ale uczę się.Zacznij od czegoś prostszego.Od prostego formularza rejestracji bez recaptchy, bez walidacji.
Potem stwórz stronę logowania też w jak najprostszej formie.Jak zbaczysz ,że jest dobra i zaczaisz o co chodzi to dodawaj kolejne element np.walidację czy system odzyskiwania hasła.Nie od razu Rzym zbudowano.A jak nie chcesz się uczyć ,a chcesz Mieć tą stronkę to musisz Komuś zapłacić , bo czas to pieniądz.Może Cie to zmotywuje.

Ten post edytował Niktoś 19.03.2012, 20:42:28

[bartekm1]

uczę sie małymi kroczkami

Ale ten formularz potrzebuje na dzisiaj

Od rana siedze na internecie, ucząc sie z tutków itp, i nie wiem jak to zrobić zeby zapisywało poprawnie. Już mnie oczy bolą od monitora.

[Niktoś]

Co Ci po tym formularzu jak nie będziesz wiedział o co chodzi.Weź poszukaj jakiś formularz w najprostszej formie na google , puść go u siebie na kompie i zobacz jak to działa.

Ten post edytował Niktoś 19.03.2012, 20:46:29

[nospor]

@bartekm1 masz już konto na forum to z niego korzystaj!
Na nowym koncie robisz dokładnie te same przewinienia co na starym. Nie dość że zakładasz nowe konto to nadodatek nie wyciągasz żadnych wniosków z błędów pierwszego.
Myślisz, że to przejdzie?