[PYTANIE] JAK ZROBIC STRONE REJESTRACYJNĄ POMOCY!

[westernstart]

SZUKAM KOGOŚ KTO POWIE MI JAK ZROBIĆ STRONĘ REJESTRACYJNĄ NA WLASNEJ STRONIE WWW

[goukan]

Zwykły formularz>Po wysłaniu formularza wpisuję hasło login i ew. email do bazy danych... KONIEC

Logowanie: Formularz>Wpisz login, hasło>Sprawdzanie czy haslo jest prawidlowe do loginu>KONIEC...

Rejestracja.. Oczywiście musisz mieć połączenie z bazą danych

i taką tabelę:

CREATE TABLE IF NOT EXISTS `uzytkownik` (
  `id` int(10) unsigned NOT NULL,
  `username` varchar(30) COLLATE latin1_general_cs NOT NULL,
  `password` varchar(50) COLLATE latin1_general_cs NOT NULL,
  `email` varchar(100) COLLATE latin1_general_cs NOT NULL,
  `data` datetime NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 COLLATE=latin1_general_cs;

 

<?php
	$erno = $_POST['send'];
	if($_GET['reg'] == 1 &&  $erno == 1) //jesli wyslano formularz
	{
		if(empty($_POST['login']) or empty($_POST['haslo']) or empty($_POST['haslo2']) or empty($_POST['email']))
		{
			$wiadomosc = "Wypełnij wszystkie pola!";
		}
		elseif($_POST['haslo'] != $_POST['haslo2'])
		{
			$wiadomosc = "Wprowadzone hasła są różne!";
		}
		elseif(preg_match('/[^0-9A-Za-z@.]/', $_POST['email']))
		{
			$wiadomosc = "Wprowadzony adres email jest niepoprawny";
		}
		else
		{
			$data = date("Y-m-d H:i");
			$login = $_POST['login'];
			$haslo = sha1($_POST['haslo']);
			$email = $_POST['email'];
			$sql = mysql_query("SELECT * FROM uzytkownik where username='$login'");
			$result = mysql_num_rows($sql);
			if($result != 0) echo'Taki login już istnieje.';
			else
			{
			$dn2 = mysql_num_rows(mysql_query('select id from uzytkownik'));
			$id = $dn2+1;
			mysql_query("insert into uzytkownik values('$id', '$login', '$haslo', '$email', '$data')");
			$wiadomosc = "Rejestracja zakończona pomyślnie.";
			}
		}
		echo $wiadomosc;
		echo '<br><a href="javascript: history.back()">Wróć do poprzedniej strony</a>';
	}

if($erno==0)
{
?>
<section>
	<header>
		<h2>Zarejestruj się</h2>
		<h3>Na naszej stronie </h3>
	</header>
	<form method="post" action="?reg=1">
		<table width="50%">
			<tr>
				<td>Podaj login:</td>
				<td><input name="login" type="text"></td>
			</tr>
			<tr>
				<td>Podaj hasło:</td>
				<td><input name="haslo" type="password"></td>
			</tr>
			<tr>
				<td>Powtórz hasło:</td>
				<td><input name="haslo2" type="password"></td>
			</tr>
			<tr>
				<td>Podaj e-mail:</td>
				<td><input name="email" type="text"></td>
			</tr>		
			<tr>
				<td colspan="2"><input type="hidden" name="send" value="1"><input class="classname" type="submit" name="submit" value="Wyślij!"></td>
			</tr>
		</table>
	</form>
</section>
<?php
}
?>

[3nichilat0]

zmienne :

$login
$email

nie zostały zabezpieczone filtrem znaków niebezpiecznych dla db.

[Wilaft]

A żeby zabezpieczyć..

 

htmlspecialchars ew. mysl_real_escape_string

[3nichilat0]

@Wilaft

 

Przed wysłaniem do bazy:

mysl_real_escape_string 

Po pliku:

htmlspecialchars 

[goukan]

Albo preg_match()

 

@edit poprawka

<?php
	$erno = $_POST['send'];
	if($_GET['reg'] == 1 &&  $erno == 1) //jesli wyslano formularz
	{
		if(empty($_POST['login']) or empty($_POST['haslo']) or empty($_POST['haslo2']) or empty($_POST['email']))
		{
			$wiadomosc = "Wypełnij wszystkie pola!";
		}
		elseif($_POST['haslo'] != $_POST['haslo2'])
		{
			$wiadomosc = "Wprowadzone hasła są różne!";
		}
		elseif(preg_match('/[^0-9A-Za-z@.]/', $_POST['email']))
		{
			$wiadomosc = "Wprowadzony adres email jest niepoprawny";
		}
		elseif(preg_match('/[^0-9A-Za-z]/', $_POST['login']))
		{
			$wiadomosc = "Wprowadzony login jest niepoprawny";
		}
		elseif(preg_match('/[^0-9A-Za-z]/', $_POST['haslo']))
		{
			$wiadomosc = "Wprowadzone hasło jest niepoprawne.";
		}		
		else
		{
			$data = date("Y-m-d H:i");
			$login = $_POST['login'];
			$haslo = sha1($_POST['haslo']);
			$email = $_POST['email'];
			$sql = mysql_query("SELECT * FROM uzytkownik where username='$login'");
			$result = mysql_num_rows($sql);
			if($result != 0) echo'Taki login już istnieje.';
			else
			{
			$dn2 = mysql_num_rows(mysql_query('select id from uzytkownik'));
			$id = $dn2+1;
			mysql_query("insert into uzytkownik values('$id', '$login', '$haslo', '$email', '$data')");
			$wiadomosc = "Rejestracja zakończona pomyślnie.";
			}
		}
		echo $wiadomosc;
		echo '<br><a href="javascript: history.back()">Wróć do poprzedniej strony</a>';
	}

if($erno==0)
{
?>
<section>
	<header>
		<h2>Zarejestruj się</h2>
		<h3>Na naszej stronie </h3>
	</header>
	<form method="post" action="?reg=1">
		<table width="50%">
			<tr>
				<td>Podaj login:</td>
				<td><input name="login" type="text"></td>
			</tr>
			<tr>
				<td>Podaj hasło:</td>
				<td><input name="haslo" type="password"></td>
			</tr>
			<tr>
				<td>Powtórz hasło:</td>
				<td><input name="haslo2" type="password"></td>
			</tr>
			<tr>
				<td>Podaj e-mail:</td>
				<td><input name="email" type="text"></td>
			</tr>		
			<tr>
				<td colspan="2"><input type="hidden" name="send" value="1"><input class="classname" type="submit" name="submit" value="Wyślij!"></td>
			</tr>
		</table>
	</form>
</section>
<?php
}
?>

[3nichilat0]

A co jak w if-e wywołasz $_POST o wartości "mysql_guery(DROP DATABASE)" ???

[goukan]

preg_match zezwala tylko jak widać na cyfry a-zA-Z i koniec.. nie przepuści nawiasów i _

[3nichilat0]

ale wcześniej masz

elseif($_POST['haslo'] != $_POST['haslo2'])

kasacja tabeli nie jest równa 1 więc straci bazę.

[goukan]

tu też można dodać tego preg_matcha ;p

[3nichilat0]

A nie lepiej dać na samym początku i mieć spokój z późniejszym zabezpieczaniem?