class all{ private $pdo; // Połączenie mysql! public $error = 0; public function _mysql(){ try{ $this->pdo = new PDO('mysql:host=localhost;dbname=webserv;', 'root', '', }catch(PDOException $e){ } } public function register($user, $pass, $passt, $email){ $err['user'] = 'Błędny nick! Nick może zawierać litery, liczby i spacje.'; $this->error++; }else{ if($this->reg('user', $user) == true){ $err['user'] = 'Podany nick jest zajęty! Proszę wybrać inny.'; $this->error++; } } }else{ $err['user'] = 'Nie wypełniono pola NICK!'; $this->error++; } if($pass == $passt){ $err['pass'] = 'Błędne hasło! Hasło może zawierać litery, liczby i spacje.'; $this->error++; } }else{ $err['pass'] = 'Nie wypełniono pola HASŁO!'; $this->error++; } }else{ $err['pass'] = 'Hasła nie są takie same!'; $this->error++; } $err['email'] = 'Błędny email!'; $this->error++; }else{ if($this->reg('email', $email) == true){ $err['email'] = 'Na podany email już zostało zarejestrowane konto!'; $this->error++; } } }else{ $err['email'] = 'Nie wpisałeś emaila!'; $this->error++; } if($this->error == 0){ }else{ foreach($err as $value){ } } } /* Funkcja sprawdzająca czy user oraz email są użyte w bazie! */ public function reg($which, $value){ switch ($which){ case 'email' : $con = $this->pdo->prepare('SELECT `email` FROM `users` WHERE `email` = :m'); break; case 'user' : $con = $this->pdo->prepare('SELECT `login` FROM `users` WHERE `login` = :m'); break; } $con->bindValue(':m', $value, PDO::PARAM_STR); $con->execute(); $count = $con->rowCount(); if($count > 0){ return true; }else{ return false; } } }
Czy da się zoptymalizować kod ? Bo tutaj dużo ifów jest ;P