Forum PHP.pl

Witam, mam problem, gdyż rejestrując się mogę zrobić dużo kont o tym samym nicku... nie da się potwierdzać ( chyba przez to że się źle rejestrują userzy:D ) i logować... Plik db.php jest prawidłowo napisany więc nie przez to...
Daje wam kody kilku plików rejestracji (pliki z formularzami pomijam [np. logowanie, rejestracja, itd.] ) :
to kod pliku rejestr2.php:

[PHP] pobierz, plaintext 
<?php
if ($_POST['koniec']) {
include 'db.php'; // połączenie się z bazą danych
/*
CREATE TABLE `user` (
      `id` INTEGER UNSIGNED NOT NULL AUTO_INCREMENT,
      `login` VARCHAR(30) NOT NULL,
      `pass` VARCHAR(30) NOT NULL,
      `email` VARCHAR(100) NOT NULL,
      `data` VARCHAR(20) NOT NULL,
      `aktywny` TINYINT(1) DEFAULT 0,
      PRIMARY KEY (`id`)
);
*/
$login = $_POST['login'];
$pass = $_POST['pass'];
$pass2 = $_POST['pass2'];
$email = $_POST['email'];
$email2 = $_POST['email2'];
$data = date('d-m-Y');
 
	if (strlen($login) < 3 or strlen($login) > 30) { $komunikat = 'Błędny login lub puste pole';}
	if (!eregi("^[0-9a-z_.-]+@([0-9a-z-]+\.)+[a-z]{2,4}$", $email)) { $komunikat = 'Błędny adres email';}
	if (strlen($pass) < 6 or strlen($pass) > 30 ) { $komunikat = 'Błędne hasło lub puste pole';}
	if ($email == $email2 ) { $komunikat = 'Emaile nie zgadzają się';}
	if ($pass == $pass2) { $komunikat = 'Hasła nie zgadzają się';}
 
		$pass = md5($pass);
 
$wynik = mysql_query("SELECT * FROM user WHERE login='$login'");
	if (mysql_num_rows($wynik) <> 0) { $komunikat = "Ten login jest zajety";}
 
	$wynik = mysql_query("INSERT INTO $tabela VALUES('', '$login', '$pass', '$email', '$data', '0')");
 
	if ($wynik) {	
	$list="Witaj $login!
	Dane do zalogowania:
	Login: $login
	Haslo: takie jakie podałeś przy rejestracji
	Aby aktywować swoje konto kliknij w link: #STRONKA#/logow/potwierdz.php?user=$login
	Dziękuje za rejestracje, administrator strony";
	mail($email, "Link Aktywacyjny", $list,"From: <Admin@STRONKA>");
 
$komunikat = "<p>Dziękuję za rejestracje ".$login."! W ciagu nabliższych 5 minut dostaniesz wiadomość e-mail z linkiem aktywacyjnym konta.</p> <br> Aktywuj konto i <a href='formularz.php'>Zaloguj się</a>";
} else { echo 'Nie zarejestrowałeś się, spróbuj ponownie!'; }
mysql_close($polaczenie);
}
?>
[PHP] pobierz, plaintext 

to kod potwierdz.php:

[PHP] pobierz, plaintext 
<? include('head.php'); ?>
<?php
include('db.php');
$login = $_GET['user'];
/*
   mysql_query("UPDATE user SET aktywny=1 WHERE login=$login ");
   if(mysql_affected_rows()==0)
        {
                 print'Aktywacja ukończona pomyślnie. Możesz już korzystać z naszego serwisu. <a href="formularz.php">Zaloguj się!</a>';
         }
         else
         {
                  print'Podano nieistniejącego użytkownika!';
          }
*/
 
    $wynik = mysql_query("SELECT * FROM $tabela
          WHERE login=$login and aktywny=1");
 
    if (mysql_num_rows($wynik) == 1) {
        echo '<span class="powodzenie">Aktywowałeś już swoje konto.</span>';
        exit; 
    } else {
//        $wynik = mysql_query("UPDATE $tabela
//          SET aktywny='1', data=NOW() WHERE login=$login and aktywny=0");
        $wynik = mysql_query("SELECT * FROM $tabela
          WHERE login=$login and aktywny=1");
        if (mysql_num_rows($wynik) == 1) {
            echo '<span class="powodzenie">Dziękujemy. Rejestracja została zakończona poprawnie. Możesz się teraz zalogować.</span>';
        }
    }
 
    // jeżeli został wprowadzony zły link, wyświetla się błąd
    if (!$kod or mysql_num_rows($wynik)<>1) {
        echo '<p>Aktywowanie konta nie powiodło się.</p>';
    }
    mysql_close($polaczenie);
 
?>
<? include('foot.php'); ?>
[PHP] pobierz, plaintext 

to kod login.php:

[PHP] pobierz, plaintext 
<?php
if ($_POST['koniec']) { // jeżeli formularz został wysłany, to wykonuje się poniższy skrypt
 
include('db.php');
session_start();
 
 // sprawdzanie czy to nie jest admin
include("admin.php");
$login =  $_POST['login'];
$pass = $_POST['pass'];
if ($login == $adl) {
if ($pass == $adp) {
$_SESSION['loged'] = $login;
$_SESSION['rang'] = "admin";
echo '<meta http-equiv="Refresh" content="0 url=admp.php" >';
exit;
}
}
        $pass = md5($pass);
 
        $wynik=mysql_query("SELECT * FROM $tabela WHERE
        login='$login' and pass='$pass' and aktywny=0");
        // jeżeli nie aktywował konta
        if (mysql_num_rows($wynik) == 1) {
            $informacja = mysql_fetch_array($wynik);
            $komunikat = 'To konto nie jest aktywne!<br>Musisz je aktywowac by sie zalogowac!<br>Wejdz na E-mail, odszukaj wiadomość z linkiem aktywacyjnym i aktywuj konto.<br>';
            exit;
        }
        // użytkownika loguje
        $wynik=mysql_query("SELECT * FROM $tabela WHERE
        login='$login' and pass='$pass' and aktywny=1");
        if (mysql_num_rows($wynik) == 1) {
            $informacja = mysql_fetch_array($wynik);
            $_SESSION["login"] = $informacja["login"];
            header('Location: index.php ');
        } else {
            echo 'Zostały wprowadzone nieprawidłowe dane!';
        }
        mysql_close($polaczenie);
 
			$_SESSION['loged'] = $login;
			$_SESSION['rang'] = "user";
}
?>
[PHP] pobierz, plaintext 

Możesz zrobić dużo kont o tym samym nicku bo sprawdzasz czy nick istnieje i jeśli tak, to do zmiennej $komunikat przypisujesz "Ten login jest zajety". Nic więcej z tym nie robiąc, albo ja niedowidzę.

te linijki:

[PHP] pobierz, plaintext 
$wynik = mysql_query("SELECT * FROM user WHERE login='$login'");
if (mysql_num_rows($wynik) <> 0) { $komunikat = "Ten login jest zajety";}
[PHP] pobierz, plaintext 

Ten post edytował Damonsson 2.05.2012, 08:57:57

Jak wyżej, do tego projektując bazę danych, polecam dodawać indexy UNIQUE na unikalne wartości, taka rada z własnego doświadczenia (IMG:style_emoticons/default/wink.gif)

UNIQUE czyli że nie może być ta sama wartość w tabeli, tak? i zaraz zmienie na UNIQUE, w php tak samo - dodam exit; (IMG:style_emoticons/default/smile.gif)