Witam,

Mam taki kawalek kodu :

[PHP] pobierz, plaintext 
function trainFilter($spam) {
            $table = $this->tokenizeFile();
            foreach($table as $value) {
                $totalsrow = new TotalsRow(new MysqlDatabase('localhost','root', 'wmateusz', 'spam'));
                $totalsrow->checkTotals($spam);
                $tokenrow = new TokenRow(new MysqlDatabase('localhost','root', 'wmateusz', 'spam'));
                $token = $tokenrow->setToken($value);
                $tokenrow->checkToken($token,$spam);
                //$row->insert();
            }
        }
 
        function getTokensValues() {
            $table = $this->tokenizeFile();
            foreach($table as $value) {
                $tab = new TokenTable(new MysqlDatabase('localhost','root', 'wmateusz', 'spam'));
                $gettoken[] = $tab->find($value);
            }
            return $gettoken;
        }
 
        function getTotalsValues() {
            $totals = new TotalsTable(new MysqlDatabase('localhost','root', 'wmateusz', 'spam'));
            $gettotals = $totals->findAll();
            return $gettotals;
        }
[PHP] pobierz, plaintext 

Jak widac mamy tutaj 3 funkcje, ale co od razu rzuca sie zapewne w oczy to to , ze za kazdym razem gdy uruchamiana jest jakas funkcja od nowa nastepuje połączenie z bazą danych. np:

[PHP] pobierz, plaintext 
$totals = new TotalsTable(new MysqlDatabase('localhost','root', 'wmateusz', 'spam'));
[PHP] pobierz, plaintext 

Gdzie new MysqlDatabase to klasa w ktorej konstruktorze nastepuje połączenie z bazą danych, ponadto w powyzszyf funkcjach znajdują sie petle foreach, wiec z kazda iteracja petli jest inicjowane nowe polaczenie z baza.


Prosilbym o wskazówki jak zoptymalizowac kod, zeby tyle razy nie wywolywac połączenia z baza.......

(tak na marginesie, nie wiem czy tego nie powinienem pisac w innym dziale....bo tak z przyzwyczajenia wszystko wrzucam do przdszkola..)

może warto zastosować wzorzec singletona ? Raczej sprawdza się w takich przypadkach. Pozdr

Juz o tym myslalem, ale wlasnie chodzi o inne rozwiazanie, chce zeby kod byl bardziej czytelny, nie podoba mi sie to jak wyglada i zastanawiam sie jakby tu mozna bylo "przeniesc" jakos to połączenie z baza poza te funkcje , zeby bylo nadrzedne ze tak powiem, gotowe cale połączenie w momencie gdy wywoluje funkcje a nie dopiero wywolywane w funkcji, ktos ma jakis wskazowki ?

może przechowywać we właściwości klasy obiekt połaczenia z bazą danych, tworzony w konstruktorze ?

Troche niezrozumiale napisales, mozesz podac przyklad ?

[PHP] pobierz, plaintext 
yourClass 
{
  private $oDatabase;
  public function __construct()
  {
    $this->oDatabase = new MysqlDatabase('localhost','root', 'wmateusz', 'spam'); // tu bym jeszcze sprawdził, czy poprawnie został utworzony objekt itp.
  }
  function trainFilter($spam)
  {
  //...
   $totalsrow = new TotalsRow($this->oDatabase);
  //..
  }
 
  // itd
}
[PHP] pobierz, plaintext 

chodziło mi o coś takiego przykładowo, jednak ja sądzę że lepszy byłby ten singleton.