class.panel.php
class panel
{
public $haslo;
public $haslo2;
public $login;
public $email;
public $email2;
public $blad = 0;
public $tabela = 'uzytkownik';
public $wyslane;
function Rejestracja($wyslane, $loginn, $hasloo, $hasloo2, $emaill, $emaill2)
{
$this->wyslane = $wyslane;
if(!isset($_SESSION['login'])) {
if ($this->wyslane)
{
if (strlen($this->login) < 3 or
strlen($this->login) > 30 or
!eregi("^[a-zA-Z0-9_.]+$", $this->login)) {
$this->blad++;
echo '<p>Proszę wprowadzić poprawny login (od 3 do 30 znaków).</p>'; } else {
$wynik = mysql_query("SELECT * FROM $this->tabela WHERE login='$this->login'"); {
$this->blad++;
echo '<p>Podana nazwa użytkownika została już zajęta.</p>'; }
}
if ($this->haslo !== $this->haslo2)
{
$this->blad++;
echo '<p> Podane hasła nie są ze sobą zgodne. </p>'; }
if (!eregi("^[0-9a-z_.-]+@([0-9a-z-]+\.)+[a-z]{2,4}$", $this->email)) {
$this->blad++;
echo '<p> Proszę wprowadzić poprawnie adres e-mail.</p>'; } else {
$wynik = mysql_query("SELECT * FROM $this->tabela WHERE email='$this->email'"); {
$this->blad++;
echo '<p>Podany adres e-mail jest już zajęty.</p>'; }
}
if ($this->email !== $this->email2)
{
$this->blad++;
echo '<p>Podane adresy e-mail nie są ze sobą zgodne.</p>'; }
if ($this->blad == 0)
{
$this->haslo = md5($this->haslo);
$wynik = mysql_query("INSERT INTO $this->tabela VALUES('', '$this->login', '$this->haslo', '$this->email', '$kod', NOW(), '')"); if ($wynik)
{
$list = "Witaj $this->login ! Kliknij w poniższy link, aby aktywować swoje konto. $kod";
mail($email, "Rejestracja użytkownika", $list, "FROM: <>"); echo '<p>Dziękujemy za rejestrację! W ciągu najbliższych 5 minut dostaniesz wiadomość e-mail z dalszymi wskazówkami rejestracji.</p>'; }
}
}
// Tworzenie formularza
<form action="index.php" method="post">
<input type="hidden" name="wyslane" value="TRUE" />
<p>Login <font color="black">*</font>: <input type="text" name="login" /></p>
<p>Hasło <font color="red">*</font>: <input type="password" name="haslo" /></p>
<p>Powtórz hasło <font color="red">*</font>: <input type="password" name="haslo2" /></p>
<p>Adres e-mail <font color="red">*</font>: <input type="text" name="email" /></p>
<p>Powtórz adres e-mail <font color="red">*</font>: <input type="text" name="email2" /></p>
<p><input type="submit" value="wyślij" /></p></form>
KONIEC;
} else
header('Location: index.php'); }
}
index.php
require_once('class.db.php');
require_once('class.panel.php');
$po = new db('plej', 'jakies', 'cos', 'localhost');
$panel = new panel();
$panel->Rejestracja($_POST["wyslane"], $_POST['login'], $_POST['haslo'], $_POST['haslo2'], $_POST['email'], $_POST['email2']);
Skrypt działa i dodaje użytkownika ale mi chodzi czy dobrze pisze np. czy dobrze zmienne dodaje itd. czy lepiej jakoś inaczej je przekazywać, czy styl pisania w OOP jest dobry dlatego pisze do was żeby od razu się uczyć dobrze : )) z góry mówię że w oop nie siedzę długo dopiero w nim raczkuje.
Mam nadzieje że rozumiecie o co chodzi.
Pozdrawiam.