Poniżej jest moja pierwsza klasa. Chciałbym żebyście zaproponowali rozwiązania dążące to zoptymalizowania tego kawałka kodu aby był wydajniejszy. Jak widać są 3 połączenia z bazą, może macie pomysł na zrobienie z dwóch ostatnich jedną, próbowałem ale miałem pewien problem. Jeśli macie jakiekolwiek inne odczucia, propozycje, pomysły czy epitety skierowane w moją stronę, to również ich wysłucham.

[PHP] pobierz, plaintext 
<?php
class register {       
      public $nickname;
      public $password; 
      public $password2;
      public $email;
      public $answer=array(); // przechowuje odpowiedzi na otrzymane dane
      public $sended=array(); // przechowuje poprawne dane, uzupelnia formularz     
      function check($nickname,$email,$password,$password2){ // sprawdza czy wszystko ok
            $this->sended['nickname']=$this->nickname=$nickname; 
            $this->sended['email']=$this->email=$email; 
            $this->sended['password']=$this->password=$password; 
            $this->sended['password2']=$this->password2=$password2;  
            try
            {
                  $pdo = new PDO('mysql:host=xxx;dbname=xxx_stmn;port=3306', 'xxx_stmn', 'xxx');
                   $query = $pdo->prepare('SELECT COUNT(id) FROM forum_users WHERE  email=:email || nickname=:nickname');       
                  $query -> bindValue(':nickname', $this->nickname, PDO::PARAM_STR);
                  $query -> bindValue(':email', $this->email, PDO::PARAM_STR);        
                  $query -> closeCursor();
                  $query -> execute();        
                  $counted = $query -> fetch();  
                   ($counted[0]>0 || $this->password!=$this->password2 ||  $this->nickname=="" || $this->password=="" ||  $this->email=="") ? $this->whatswrong($counted) :  $this->createaccount();
            }
            catch(PDOException $e)
            {
                   array_push($this->answer, "Error: ".$e->getMessage());
            }
      }         
      function createaccount(){ // tworzy konto jesli wszystko ok
            array_push($this->answer,"zrobilbym Ci konto ale jeszcze nie dokonczylem funkcji");     
      }            
      function whatswrong($counted){ // informuje co jest zle
            if($this->nickname==""){
                  array_push($this->answer, "Uzupelnij nick.");
            }
            if($this->password==""){
                  array_push($this->answer, "Uzupelnij haslo.");
            }
            if($this->email==""){
                  array_push($this->answer, "Uzupelnij e-mail.");
            }
            if($this->password!=$this->password2){
                  array_push($this->answer, "Twoje hasla nie sa identyczne.");
                  unset($this->sended['password']);
                  unset($this->sended['password2']);
            } 
            if($counted[0]>0) {   
                  $pdo = new PDO('mysql:host=xxx;dbname=xxx_stmn;port=3306', 'xxx_stmn', 'xxx');
                  $query = $pdo->prepare('SELECT nickname FROM forum_users WHERE nickname=:nickname');       
                  $query -> bindValue(':nickname', $this->nickname, PDO::PARAM_STR);     
                  $query -> closeCursor();
                  $query -> execute();       
                  $data = $query -> fetch();  
                  if($data['nickname']==$this->nickname && $this->nickname!=""){
                        array_push($this->answer, "Nick jest zajety.");
                        unset($this->sended['nickname']);
                  }
                  $query = $pdo->prepare('SELECT email FROM forum_users WHERE email=:email');       
                  $query -> bindValue(':email', $this->email, PDO::PARAM_STR);        
                  $query -> closeCursor();
                  $query -> execute();  
                  $data = $query -> fetch(); 
                  if($data['email']==$this->email && $this->email!=""){
                        array_push($this->answer, "Email jest zajety.");
                        unset($this->sended['email']);
                  }
            } 
      }
    
    }
    
    if($_SERVER['REQUEST_METHOD'] == 'POST')
    {    
     $register = new register();
     $register->check($_POST['nickname'],$_POST['email'],$_POST['password'],$_POST['password2']);
     $smarty->assign("answer",$register->answer); // wysyla odpowiedzi
     $smarty->assign("fillform",$register->sended); // wysylam do formularza dane ktore byly poprawne
    }
    $smarty->display("index.tpl");
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
class A extends PDO
{
    public function __construct()
    {
         try{
      
            parent::__construct( 'polaczenie z baza przez pdo' );
      
        }catch( PDOException $e ){
          
            throw new MysqlException( 'Błąd połączenia z bazą danych' );
             
        }
    }
    
    public function doSmth()
    {
        $this->query();
    }
}
?>
[PHP] pobierz, plaintext 

lub przekazywanie uchwytu do klasy w konstruktorze.

Nie twórz połączenia w każdej metodzie potrzebującej dostępu do bazy danych. Utwórz je w innej części kodu, a potem przekaż wszystkim klasom, które go wymagają.

[PHP] pobierz, plaintext 
<?php
class register {
  protected $pdo;
 
  public function setConnection(PDO $pdo)
  {
    // Od teraz masz jedno połączenie w całym obiekcie.
    $this->pdo = $pdo;
  }
 
  // rest of the code
 
}
?>
[PHP] pobierz, plaintext 

ja, na twoim miejscu, od samego poczatku uczyl bym sie rozdzielac logike od widoku...

owszem wiecej klepania w klawisze, ale o roznicach miedzy programowaniem strukturalnym a oop mozna by ksiazki pisac (by byc scislym - to i napisano... )

tutaj masz projekt MVC (nie czepiajcie sie, wiem ze prosty, by nie zec prymitywny :] )
dzieki takiemu podejsciu zyskujesz wiele mozliwosci, np jesli chcesz zmieniac wersje jezykowe, inne komunikaty, inny wyglad, inna baza, etc..

ucz sie od poczatku oddzielac to co jest widokiem, co jest logika aplikacji, oraz jej model...

to jest jedynie szkic
przyznaje ze w zasadzie nie do konca to przemyslalem, pewnie przy kodowaniu zmienilbym pare spraw, tym niemniej jednak glowny pomysl jest moim zdaniem sluszny

[PHP] pobierz, plaintext 
<?php
//tutaj wyswietlasz wszystkie widoki (komunikaty, etc)
class vievRegister{
    private $_myController
    public function __construct(){
        $this->$_myController = new Controller;
    }
    public function loguj(){
        if($this->$_myController->login($login,$pass) )
            echo "jest ok";
        else
            echo"nie jest dobrze";
    }
    
    public function wyloguj(){
        if($this->$_myController->logout($login,$pass) )
            echo "jest ok";
        else
            echo"nie jest dobrze";
    }
    public function etc(){}
}#class vievRegister
 
//tutaj pracujesz nad logika aplikacji
class Controller{
    
    protected $_MyModel;
 
    public function __construct(){
    //zainicjuj odpowiedni model
    }
 
    //logujemy, @return bool
    public function login($login=null, $pass=null) {}
    // wylogujemy @return bool
    public function logout() {}
    
 
 
    //rejestrujemy @return mixed (bool gdy udany, w przeciwnym razie kod bledu)
    public function register($login=null, $pass=null, $etc=array()) {}
    
    //wywalamy @return bool
    public function remove() {
}#class Controller
 
// tutaj wykonujesz wszystkie zapytanie do bazy danych
class Model{
    protected static $_uchwytDoDB;
 
    public function __construct() {}
    protected function login() {}
    protected function searchLogin() {}
    protected function searchEmail() {}
    protected function coTamJeszczeTza(){}
    public function __destruct(){}
}#class Model
 
// wyjatki
class MyException {
//tutaj obslugujesz bledy
}
?>
[PHP] pobierz, plaintext 

Jak poprzednicy + co mi się rzuciło w oczy to $sended zamiast $sent

Racja, muszę przypomnieć sobie czasowniki nieregularne.
Oczywiście dzięki za wszystkie odpowiedzi, na pewno będą miały znaczenie dla mojej przyszłości.

Może miast

[PHP] pobierz, plaintext 
<?php
array_push($this->answer, "Uzupelnij nick.");
?>
[PHP] pobierz, plaintext 

daj

[PHP] pobierz, plaintext 
<?php
private function debug($sMessage)
{
   array_push($this->answer, $sMessage);
}
?>
[PHP] pobierz, plaintext 

Chyba będzie wygodniej