Witam
W tej chwili konstruktor wymusza 3 połączenie do bazy, czy taki zapis konstruktora jest poprawny, czy można to zoptymalizować?
<?php
$id = 1;
$Player = new Player($id);
$USER['nick'] = $Player->nick;
$USER['level'] = $Player->level;
$USER['praca'] = $Player->praca;
$USER['kasa'] = $Player->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(){
$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(){
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();
}
}
}
?>