Zapytanie mysql nie dodaje nic do bazy

pytanie zadane 29 listopada 2015 w PHP przez Widemo Użytkownik (920 p.)
zmienione kategorie 29 listopada 2015 przez Arkadiusz Waluk

Mam taki problem. Nie dziala mi zapytanie do bazy mysql a dokladnie $db -> Exec(INSERT INTO.....)

Ponizej wklejam kod

<?php

  require "head.php";
  require "db.php";

  if (empty($_POST['name']) || empty($_POST['lastname']) || empty($_POST['login']) || empty($_POST['pass']) || empty($_POST['passrep']) || empty($_POST['email'])) {
    header("Location: register_form.php?error=0");
    exit;
  }
  $name = $_POST["name"];
  $lastname = $_POST["lastname"];
  $login = $_POST["login"];
  $pass = md5($_POST["pass"]);
  $passrep = md5($_POST["passrep"]);
  $email = $_POST["email"];
  $sex = $_POST["sex"];

  if ($pass != $passrep) {
    header("Location: register_form.php?error=1");
    exit;
  }


  $answer = $db -> Query('SELECT `login` FROM `users` WHERE `login`="' . $login . '"');
  $answer = $answer -> fetchAll();
  
  if(!empty($answer)) {
    header("Location: register_form.php?error=2");
    exit;
   }
    
  $answer = $db -> Query('SELECT `email` FROM `users` WHERE `email`="' . $email . '"');
  $answer = $answer -> fetchAll();
  
  if(!empty($answer)) {
    header("Location: register_form.php?error=3");
    exit;
   }

    $db -> Exec('INSERT INTO `users` VALUES ("", "' . $name . '", "' . $lastname . '", "' . $login . '", "' . $pass . '", "' . $email . '", "' . $sex . '"');
    exit;
  
  echo "$name<br/> $lastname<br/> $login<br/> $pass<br/> $email<br/> $sex<br/>";
  

?>

Połączenie z bazą

<?php 
session_start();
require("head.php");
$db = new PDO("mysql:host=mysql.hostinger.pl;dbname=dbname", "user", "password");

komentarz 14 sierpnia 2016 przez xandros Nałogowiec (29,420 p.)

Użyj bindowania parametrów, jak to napisał Abel. `md5` to słabe hashowanie haseł. Tego powinieneś używać:

http://php.net/manual/pl/function.password-hash.php

http://php.net/manual/pl/function.password-verify.php

2 odpowiedzi

odpowiedź 29 listopada 2015 przez R[a]=d(ek); Mądrala (6,370 p.)
edycja 29 listopada 2015 przez R[a]=d(ek);

Hmm a sprobuj moze uzyc zwyklego $db->Query("INSERT INTO..... "); ?

edit::

Tylko nastepnym razem postaraj sie wybrac odpowiedni dzial zadajac jakies pytanie okej? ;)

komentarz 29 listopada 2015 przez criss Mędrzec (172,570 p.)

Dokladnie, wrzucanie wszystkiego pod C i C++ to ostatnio jakaś plaga. Tylko wczoraj przycisk "zgłoś" wcisnąłem kilka razy; dzisiaj pierwszy raz, ale tylko dlatego, że na forum jestem od kilku minut ;_; Ludzie, ogarnijcie sie.

odpowiedź 14 sierpnia 2016 przez Abel Mądrala (5,140 p.)
edycja 14 sierpnia 2016 przez Abel

Nie jestem jakiś wybitny specjalista , dopiero się uczę

Niemniej wydaje mi się że masz błąd

$answer = $db -> Query('SELECT `login` FROM `users` WHERE `login`="' . $login . '"');
  $answer = $answer -> fetchAll();
   
  if(!empty($answer)) {
    header("Location: register_form.php?error=2");
    exit;
   }

powinno moim zdaniem wyglądać tak

$answer = $db -> Query('SELECT `login` FROM `users` WHERE `login`="' . $login . '"');
  $row = $answer -> fetchAll();
   
  if(!empty($row)) {
    header("Location: register_form.php?error=2");
    exit;
   }

INSERT ja robie w ten sposób

$zapytanie = $polaczenie->prepare("INSERT INTO maile (mail_id, tytul, tresc, nadawca, odbiorca, stan, data_nadania)
		VALUES (0, :tytul, :tresc, :nadawca, :odbiorca, :stan, Now())");
		
		
		$zapytanie->bindParam(':tytul' , $temat, PDO::PARAM_STR);
		$zapytanie->bindParam(':tresc' , $tresc, PDO::PARAM_STR);
		$zapytanie->bindParam(':nadawca' , $nadawca, PDO::PARAM_STR);
		$zapytanie->bindParam(':odbiorca' , $odbiorca, PDO::PARAM_STR);
		$zapytanie->bindParam(':stan' , $nowa, PDO::PARAM_STR);
               if($zapytanie->execute())
		{
		echo "treść komunikatu  ";
		}
}
	catch(PDOException $e)
		{
		echo $zapytanie . "<br>" . $e->getMessage();
		}
	$polaczenie = null;

I wszystko działa jak należy