Problem z rejestracją

Question

Witaj, mam problem z rejestracją. 

Parse error: syntax error, unexpected 'mysql_select_db' (T_STRING) in C:\xampp\htdocs\osadnicy\connect.php on line 9

Index.php

<form method="post" action="rejestracja.php" >
<table width="300" border="0">
	<tr>
		<td>Login:</td>
		<td><input type="text" name="user"</td>
	</tr>
		<tr>
		<td>Hasło:</td>
		<td><input type="password" name="pass"</td>
	</tr>
		<tr>
		<td>Email:</td>
		<td><input type="text" name="email"</td>
	</tr>
	</tr>
		<tr>
		<td></td>
		<td><input type="submit" name="Zarejestruj się"</td>
	</tr>
</table>
</form>

 

Rejestracja.php 

 

<?php

	include"connect.php";
	
$user = $_POST['user'];
$pass = $_POST['pass'];
$email = $_POST['email'];
	

$insert = 'INSERT into uzytkownicy(user, pass, email) VALUES(`$user`, `$pass`,`$email`)';

mysql_query($insert);


?>

 

Answer 1

Po pierwsze niepozamykane znaczniki <input> w formularzu. Powinno być np. tak:

<td><input type="text" name="user"></td>

I jeszcze masz jeden </tr> za dużo za emailem.

Po drugie brak zabezpieczeń przed sql injection.

Po trzecie takie zapytanie nie przejdzie, wartości zapisujemy w apostrofy ' a nazwy kolumn można w ` ale nie trzeba. Tak powinno być okej:

$insert = "INSERT INTO `uzytkownicy` (`user`, `pass`, `email`) VALUES ('$user', '$pass','$email')";

Po czwarte nie używaj mysql_ a chociaż mysqli_ a najlepiej PDO.

Po piąte to przechowywanie czystego hasła w bazie to raczej bardzo kiepski pomysł.

I wreszcie po szóste to błąd jest gdzieś w pliku connect.php, przy wyborze bazy danych. Pokaż co masz w pliku connect, dane bazy możesz sobie zakryć.

Comments

Nie wiem jak dać te okienko do kodu źródłowego, więc daje tak http://wklej.to/OMkyF . Tutaj zdjęcie bazy: http://i.imgur.com/o7ZucyL.png

---

Ale to okienko nie było potrzebne. Zawartość pliku connect.php pokaż ;)

Co prawda widzę kolejny błąd tutaj bo masz kolumnę data z NOT NULL w bazie, a w zapytaniu wartości do tej kolumny nie dodajesz - a skoro ma NOT NULL czyli że nie może być pusta to jest wymagana.

---

http://wklej.to/OMkyF tutaj jest connect.php Jak bylbys tak dobry i moglbys to wszystko popoprawiac.

---

Co do connect.php to po pierwsze linijka 8 czyli z mysql_connect_db nie ma średnika na końcu przez to ten błąd. Po drugie w linijce 9 czyli z mysql_select_db powinny być dwa parametry, 1 to nazwa bazy tak jak podajesz a drugi to identyfikator połączenia czyli $connect. Tak to powinno wyglądać:

<?php

	$host = "localhost";
	$db_user = "root";
	$db_password = "";
	$db_name = "osadnicy";
	
	$connect = mysql_connect("localhost","root","");
	mysql_select_db('osadnicy', $connect);
?>

Do tego nie wiem jeszcze po co definiujesz wszystkie dane do połączenia z bazą jako zmienne a w połączeniu i tak wpisujesz je ręcznie - ale to myślę że tylko chwilowo do jakiś testów zrobiłeś czy coś.

Byłbym dobry ale tego wszystkiego za Ciebie popoprawiać nie mogę z dwóch powodów. Dlatego, że jednak jest to trochę pracy i dlatego że jak dostaniesz gotowy kod to się niczego nie nauczysz ;) Próbuj, pisz, szukaj, jak coś nie będzie Ci szło albo będziesz potrzebował wyjaśnienia to chętnie z czymś konkretnym pomogę ale całego kodu za Ciebie nie napiszę.

---

Okej, dzieki za cenne rady :) Jutro wszystko sproboje poprawic.

---

Nie ma sprawy, mam nadzieję, że sobie poradzisz ;)

Answer 2

Zapomnij o wszystkim co ma przedrostek mysql_ w php