PHP OOP - Jak skonstruować weryfikacje przy rejestracji Opcje

[Utilaft]

Witam. Mam taki kod:

[PHP] pobierz, plaintext 
<?php
include_once 'class/user_class.php';
session_start();
 
?>
 
<?php
 
$user = new User();
 
if($user->get_session())
{
header("Location: login.php");
exit;
}
 
if($_SERVER["REQUEST_METHOD"] == "POST")
{
 
$register = $user->register_user($_POST['login'], $_POST['password'], $_POST['email']);
 
if($register) 
{
 
echo 'Rejestracja przebiegła pomyślnie.';
} else {
echo 'Login jest zajęty.';
}
 
 
}
 
?>
[PHP] pobierz, plaintext 

i klasa:

[PHP] pobierz, plaintext 
<?php
include_once 'db/config.php';
 
class User
{
 
public function __construct()
{
$db = new DB_Class();
}
 
//Register user
public function register_user($login, $password, $email)
{
 
$password = md5($password);
$sql = mysql_query("SELECT login from users WHERE login = '$login' LIMIT 1");
$no_rows = mysql_num_rows($sql);
 
if($no_rows == 0)
{
$result = mysql_query("INSERT INTO users SET login='$login', password='$password', email='$email'");
return $result;
} else {
return FALSE;
}
 
}
 
//Getting session
public function get_session()
{
return isset($_SESSION['login']);
}
 
 
}
 
?>
[PHP] pobierz, plaintext 

Chcę jakoś dobrze skonstruować weryfikację. Np. czy pole nie jest puste, czy ma dobre znaki. Możecie jakoś podpowiedzieć (IMG:style_emoticons/default/tongue.gif) ? // Jestem początkujący

I w ogóle chciałbym się dowiedzieć, jak dobrze jakoś zbudować rejestracje/logowanie użytkownika, zarządzanie użytkownikami

Ten post edytował Utilaft 3.03.2013, 18:10:43

[!*!]

Walidacja to osobna klasa. Wysyłasz do niej dane np. POST + to jakie one mają być, sprawdzasz czy nie ma błędów i gotowe.

[Utilaft]

No to podobnie myślałem.

np. public function($var) {
if(blabla)
{
return costam
}
}

I takie coś?


I jeszcze tak na marginesie, nie rozumiem stosowania return. O co w tym chodzi? Po co mi zwracana wartość, jak mogę to wykorzystać.?

[nospor]

I jeszcze tak na marginesie, nie rozumiem stosowania return. O co w tym chodzi? Po co mi zwracana wartość, jak mogę to wykorzystać.?

Skoro nie potrzebujesz, by Twoja funkcja coś zwracała, to nie zwracaj nic. Kiedyś może jednak będziesz potrzebował informacji od funkcji - po to właśnie jest RETURN

[mstraczkowski]

Dobrą praktyką jest trzymanie się tego, aby każda metoda coś zwracała, chociażby obiekt klasy $this, przez co możliwy jest method chaining, oczywiście nie zawsze jest sens.