[PHP][MySQL]Rejestracja Opcje

[jaca121212]

Czy mógłby mi to ktoś poprawić tak aby skrypt działał poprawnie

[HTML] pobierz, plaintext 
Rejestracja
<form action="rejestracja.php" method="POST">
Nazwa Użytkownika/login: <input type="text" name="login" /><br />
Hasło: <input type="password" name="haslo" /><br />
Powtórz hasło: <input type="password" name="haslo1" /><br />
Adres E-mail: <input type="text" name="email" /> <br />
<input type="submit" value="Zarejestruj się!!" /><br />
<?php
if(isset($_SESSION['blad1'])) echo $_SESSION['blad'];
?>
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
session_start();
 
if ((!isset($_POST['login'])) || (!isset($_POST['haslo'])) || (!isset($_POST['haslo1'])) || (!isset($_POST['email'])))
{
	header('Location: index.php');
	exit();
}
 
require_once "polaczenie.php";
$polaczenie = @new mysqli($host, $db_user, $db_password, $db_name);
if ($polaczenie->connect_erron!=0)
{
	echo "Error: ".$polaczenie->connect_errno . "Opis: ". $polaczenie->connect_error;
}
else
{
$login = $_POST['login'];
$haslo = $_POST['haslo'];
$haslo1 = $_POST['haslo1']; 
$email = $_POST['email'];
 
$login = htmlentities($login, ENT_QUOTES, "UTF-8");
$haslo = htmlentities($haslo, ENT_QUOTES, "UTF-8");
$haslo = htmlentities($haslo1, ENT_QUOTES, "UTF-8");
$haslo = htmlentities($email, ENT_QUOTES, "UTF-8");
 
 if($resultat = @$polaczenie->query(
 sprintf("INSERT INTO `uzytkownicy` (`id`, `user`, `pass`, `email`) VALUES ('', '%s', '%s', '%s', '%s')",
 mysqli_real_escape_string($polaczenie, $login),
 mysqli_real_escape_string($polaczenie, $haslo),
  mysqli_real_escape_string($polaczenie, $haslo1),
   mysqli_real_escape_string($polaczenie, $email))))
 {
	 $ilu_userow = $resultat->num_rows;
	 if($ilu_userow>0)
	 {
		 $_SESSION['zarejestrowany'] = true;
 
 
 
		unset($_SESSION['blad']);
		$resultat->free_result();
	header('Location: index.php');
	 }else
	 {
		 $_SESSION['blad'] = '<span style="color:red" >Nieprawidłowy Login lub Hasło! </span>';
		 header('Location: index.php');
	 }
 }
$polaczenie->close();
}
 
?>
[PHP] pobierz, plaintext 

do tej pory jest taki błąd

[SQL] pobierz, plaintext 
Notice: Undefined property: mysqli::$connect_erron IN C:\xampp\htdocs\rejestracja.php ON line 12
[SQL] pobierz, plaintext 

Ten post edytował jaca121212 6.04.2015, 08:04:49

[Raito]

Kod

if ($polaczenie->connect_erron!=0)

nie powinno być errno?

Ten post edytował Raito 6.04.2015, 08:22:33

[jaca121212]

ale nawet nie zapisuje rekordów do bazy danych może źle jest napisana skladnia INSERT INTO

Ten post edytował jaca121212 6.04.2015, 10:44:51

[Raito]

if($resultat = @$polaczenie->query( - usuń @

[Turson]

Zacznij od wywalenia wszystkich @ z kodu i Temat: Jak poprawnie zadac pytanie

[jaca121212]

Kod

if ($polaczenie->connect_erron!=0)

nie powinno być errno?

jeśli usunę linie 12 to wyskakuje kolejny błąd z else w lini 16

[Raito]

To nie usuwaj całej linii, tylko zamień if ($polaczenie->connect_erron!=0) na if ($polaczenie->connect_errno!=0).

[jaca121212]

To nie usuwaj całej linii, tylko zamień if ($polaczenie->connect_erron!=0) na if ($polaczenie->connect_errno!=0).

jak zmieniłem na if ($polaczenie->connect_errno!=0) to zaden błąd się nie pojawił ale jest problem bo nie zamisuje się do bazy żaden rekord możliwe jest to że źle jest napisany skrypt dodawania usera do bazy teraz kod cały wygląda tak:

[PHP] pobierz, plaintext 
<?php
session_start();
 
if ((!isset($_POST['login'])) || (!isset($_POST['haslo'])) || (!isset($_POST['haslo1'])) || (!isset($_POST['email'])))
{
	header('Location: index.php');
	exit();
}
 
require_once "polaczenie.php";
$polaczenie = new mysqli($host, $db_user, $db_password, $db_name);
if ($polaczenie->connect_errno!=0)
{
	echo "Error: ".$polaczenie->connect_errno . "Opis: ". $polaczenie->connect_error;
}
else
{
$login = $_POST['login'];
$haslo = $_POST['haslo'];
$haslo1 = $_POST['haslo1']; 
$email = $_POST['email'];
 
$login = htmlentities($login, ENT_QUOTES, "UTF-8");
$haslo = htmlentities($haslo, ENT_QUOTES, "UTF-8");
$haslo = htmlentities($haslo1, ENT_QUOTES, "UTF-8");
$haslo = htmlentities($email, ENT_QUOTES, "UTF-8");
 
 if($resultat = $polaczenie->query(
 sprintf("INSERT INTO `uzytkownicy` (`id`, `user`, `pass`, `email`) VALUES ('', '%s', '%s', '%s', '%s')",
 mysqli_real_escape_string($polaczenie, $login),
 mysqli_real_escape_string($polaczenie, $haslo),
  mysqli_real_escape_string($polaczenie, $haslo1),
   mysqli_real_escape_string($polaczenie, $email))))
 {
	 $ilu_userow = $resultat->num_rows;
	 if($ilu_userow>0)
	 {
		 $_SESSION['zarejestrowany'] = true;
 
 
 
		unset($_SESSION['blad']);
		$resultat->free_result();
	header('Location: index.php');
	 }else
	 {
		 $_SESSION['blad'] = '<span style="color:red" >Nieprawidłowy Login lub Hasło! </span>';
		 header('Location: index.php');
	 }
 }
$polaczenie->close();
}
 
?>
[PHP] pobierz, plaintext 

[Raito]

[PHP] pobierz, plaintext 
 sprintf("INSERT INTO `uzytkownicy` (`id`, `user`, `pass`, `email`) VALUES ('', '%s', '%s', '%s', '%s')",
 mysqli_real_escape_string($polaczenie, $login),
 mysqli_real_escape_string($polaczenie, $haslo),
  mysqli_real_escape_string($polaczenie, $haslo1),
   mysqli_real_escape_string($polaczenie, $email))))
[PHP] pobierz, plaintext 

Próbujesz dodać do 4 wartości (`id`, `user`, `pass`, `email`) 5 wartości ('', '%s', '%s', '%s', '%s').

[jaca121212]

[PHP] pobierz, plaintext 
 sprintf("INSERT INTO `uzytkownicy` (`id`, `user`, `pass`, `email`) VALUES ('', '%s', '%s', '%s', '%s')",
 mysqli_real_escape_string($polaczenie, $login),
 mysqli_real_escape_string($polaczenie, $haslo),
  mysqli_real_escape_string($polaczenie, $haslo1),
   mysqli_real_escape_string($polaczenie, $email))))
[PHP] pobierz, plaintext 

Próbujesz dodać do 4 wartości (`id`, `user`, `pass`, `email`) 5 wartości ('', '%s', '%s', '%s', '%s').

ok już działa teraz pytanie jak sprawdzić czy hasła są takie same wpisane i jak kodować hasło md5 lub sh1 które kodowanie będzie najlepszę

[Raito]

Pobierasz z bazy zakodowane hasło i porównujesz z zakodowanym wpisanym hasłem.
Lepsze jest md5.
md5(cos);

[jaca121212]

mam jeszcze problem bo jak wpisałem haslo to zamiast w bazie zapisac haslo zapisuje adres e mail w miejsce wpisania hasla tak wyglada to w bazie (IMG:http://www.iv.pl/images/33107898534028340754.png)

[Turson]

Pobierasz z bazy zakodowane hasło i porównujesz z zakodowanym wpisanym hasłem.
Lepsze jest md5.
md5(cos);

Czemu lepsze md5? Już dawno zostało złamane. Lepsza opcja to hash+sól

[jaca121212]

Czemu lepsze md5? Już dawno zostało złamane. Lepsza opcja to hash+sól

a mógłby mi ktoś pokzać przykład jak ma być to poprawinie zrobione lub jak to powinno wyglądać

[Raito]

@jaca121212:
http://www.dobreprogramy.pl/Ardziej/Gdy-zw...malo,29553.html
@Turson:
Dzięki za info, nie wiedziałem o tym. Sam przy okazji sobie o tym poczytam (IMG:style_emoticons/default/smile.gif)

[jaca121212]

mam problem z sprawdzaniem zmiennych czy istnieja w polach przy rejestracji np. czy haslo jest takie same jak haslo powtórzone w formularzu tak samo czy są wszystkie pola uzupełnione poprawilibyście ten skrypt

[PHP] pobierz, plaintext 
<?php
session_start();
 
if ((!isset($_POST['login'])) || (!isset($_POST['haslo'])) || (!isset($_POST['haslo1'])) || (!isset($_POST['email'])))
{
	header('Location: index.php');
	exit();
}
 
require_once "polaczenie.php";
$polaczenie = new mysqli($host, $db_user, $db_password, $db_name);
if ($polaczenie->connect_errno!=0)
{
	echo "Error: ".$polaczenie->connect_errno . "Opis: ". $polaczenie->connect_error;
}
else
{
 
$login = $_POST['login'];
$haslo = md5($_POST['haslo']);
$haslo1 = md5($_POST['haslo1']); 
$email = $_POST['email'];
 
$login = htmlentities($login, ENT_QUOTES, "UTF-8");
$haslo = htmlentities($haslo, ENT_QUOTES, "UTF-8");
$haslo1 = htmlentities($haslo1, ENT_QUOTES, "UTF-8");
$email = htmlentities($email, ENT_QUOTES, "UTF-8");
 
if(empty($_POST['login'])){
	echo "nie podałeś loginu";
 
}elseif(empty($_POST['haslo']==$_POST['haslo1'])){
	echo "hasła się różnią";
}elseif(empty($_POST['email'])){
	echo "proszę podać adres e-mail";
}else
 
 if($resultat = $polaczenie->query(
 sprintf("INSERT INTO `uzytkownicy` (`id`, `user`, `pass`, `email`) VALUES ('', '%s', '%s', '%s')",
 mysqli_real_escape_string($polaczenie, $login),
 mysqli_real_escape_string($polaczenie, $haslo),
   mysqli_real_escape_string($polaczenie, $email))))
 {
	 $ilu_userow = $resultat->num_rows;
	 if($ilu_userow>0)
	 {
		 $_SESSION['zarejestrowany'] = true;
 
 
 
		unset($_SESSION['blad']);
		$resultat->free_result();
	header('Location: index.php');
	 }else
	 {
		 $_SESSION['komunikat'] = '<span style="color:red" >Zostałeś/aś  poprawnie zarejestrowany/na. Zaloguj się!</span>';
		 header('Location: index.php');
	 }
 }
$polaczenie->close();
}
 
?>
[PHP] pobierz, plaintext 

[HTML] pobierz, plaintext 
Rejestracja
<form action="rejestracja.php" method="POST">
Nazwa Użytkownika/login: <input type="text" name="login" /><br />
Hasło: <input type="password" name="haslo" /><br />
Powtórz hasło: <input type="password" name="haslo1" /><br />
Adres E-mail: <input type="text" name="email" /> <br />
<input type="submit" value="Zarejestruj się!!" name="wyślij" /><br />
<?php
if(isset($_SESSION['komunikat'])) echo $_SESSION['komunikat'];
?>
[HTML] pobierz, plaintext 

[Raito]

[PHP] pobierz, plaintext 
}elseif(empty($_POST['haslo']==$_POST['haslo1'])){
[PHP] pobierz, plaintext 

A coż to?

[PHP] pobierz, plaintext 
elseif($_POST['haslo']!=$_POST['haslo1']){
echo "Wpisane hasła różnią się.";}
elseif(empty($_POST['haslo'])){
...
}elseif(empty($_POST['haslo1'])){
...
}
[PHP] pobierz, plaintext 

Ten post edytował Raito 6.04.2015, 19:40:04

[Boshi]

Dla sprawdzenia czy wszystkie pola zostały uzupelnione wystarczy pętla foreach po tablicy POST

Ten post edytował Boshi 6.04.2015, 19:43:01

[jaca121212]

W jaki sposób sprawdzić czy login taki jest w bazie jeśli tak to żeby pokazał komunikat

Ten post edytował jaca121212 7.04.2015, 06:49:33

[Turson]

Zapytaniem liczącym ilośc wierszy gdzie login = wpisany_login