Poprawność konstruktora? Opcje

[Nortonek]

Witam

W tej chwili konstruktor wymusza 3 połączenie do bazy, czy taki zapis konstruktora jest poprawny, czy można to zoptymalizować?

[PHP] pobierz, plaintext 
<?php
 
  $id = 1;
 
  $Player = new Player($id);
 
  $USER['nick']   = $Player->nick;
  $USER['level']  = $Player->level;
  $USER['praca'] = $Player->praca;
  $USER['kasa']  = $Player->kasa;
 
  echo $USER['kasa'];
 
 
 
 
class Players {
  public $nick;
  public $level;
  public $id_players;
  public $kasa;
  public $praca;
 
 
  function __construct($id)   {  
      $this -> id_players= $id;
      self::get_dane();
      self::update_kasa();
      self::get_dane();
  }
 
  public function get_dane(){
    global $db, $table_players;
 
        $stats = $db -> Execute("SELECT *  FROM  $table_players 	WHERE 	`id_players`='".$this -> id_players."'  AND `aktywny`='1'	");
        $this -> nick = $stats -> fields['nick'];
        $this -> level= $stats -> fields['level'];
        $this -> kasa= $stats -> fields['kasa'];
				$this -> praca= $stats -> fields['praca'];
        $stats -> Close();
  }
 
  public function update_kasa(){
		global $db, $table_players;
 
    if ( $this -> praca <= time() ) {
        $kasa = 100;
        $result = $db -> Execute("UPDATE $table_players SET `kasa`= `kasa` + '$kasa', `praca`='0' WHERE `id_players`='".$this -> id_players."' ");
        $result -> Close();
    }
  }
 
 
}
 
?>
[PHP] pobierz, plaintext 

[Turson]

Napisałeś klasę, która wymusza jakaś akcję już w konstruktorze. To nie najlepszy pomysł, bo za to powinny odpowiadać poszczególne metody. Powiedzmy, że będziesz chciał pobrać same dane użytkownika metodą get_dane(), ale klasa przy okazji zrobi update w bazie. Kiepsko. Metoda get_dane() powinna przyjmować jakiś parametr np. id usera w przypadku, gdy będziesz chciał wykonać jedynie pobranie danych.