Klasa do ładowania plików, duże obciążenie... Opcje

[cojack]

No więc tak, mam sobie klasę do ładowania plików, i gdy odpaliłem profilera php to zobaczyłem że dość duże obciążenie mam na tej klasie, bo z wszystkich wywołań metod wszystkie wywołania a jest ich 33 z tego co dobrze widzę, klasyfikują na 2 miejscu w stopniu najbardziej obciążającym system, no i szczerzę powiedziawszy to nie mam bladego pojęcia co jest tak mega super złego w tej klasie że takie obciążenie powoduje, oto kod:

[PHP] pobierz, plaintext 
<?php
/**
 * BDT_Loader klasa odpowiedzialna za ładowanie plików
 *
 * @author     Przemysław Czekaj <przemyslaw.czekaj@aichra.pl>
 * @link       http://aichra.pl
 * @version    0.1
 * @since      03.29.2010
 * @package    BDT
 * @subpackage lib
 * @charset    utf8
 **/
 
/**
 * @author  cojack
 * @TODO    Błędy w postaci GETTEXT
 */
class BDT_Loader {
 
   private static $_path;
 
   /**
    * Konstruktor klasy
    *
    * @return  void
    */
   public static function initialize() {
      chdir( './..' );
      self::$_path = getcwd() . '/';
   }
 
   /**
    * Metoda przyjmuje argument jako tablice iteracyjną
    * Adres do pliku powinien zaczynać się od ./ i uwzględnić
    * zmianę ścieżki podczas wczytywania danych.
    * Zachowywujemy się tak jakbyśmy byli w / tego fw.
    *
    * @access  public
    * @param   array    $files   Tablica w postaci array ('plik', 'plik', 'plik');
    * @return  void
    */
   public static function loadFile( $files = array() ) {
      foreach( $files as $file ) {
         if( self::_checkFile( $file ) ) {
               require_once( self::$_path.$file );
         }
      }
   }
 
   /**
    * Metoda wczytuje pojedyncze pliki XML
    *
    * @access  public
    * @param   string    $files   Względna ścieżka do pliku
    * @return  object    Xml object
    */
   public function loadFileXML( $file ) {
      if( self::_checkFile( $file ) ) {
         libxml_use_internal_errors( TRUE );
         $sxe = new SimpleXMLElement( self::$_path.$file, NULL, TRUE );
         if( $sxe === FALSE ) {
            $errorMsg  = 'Błąd w pliku XML: ' . $file . "<br />";
            $errorMsg .= 'Ścieżka: ' . self::$_path.$file . "<br />";
            foreach( libxml_get_errors() as $error ) {
               $errorMsg .=  "<p>" . $error->message . "</p><br />";
            }
            trigger_error( $errorMsg , E_USER_WARNING );
         }
      }
      return $sxe;
   }
 
   /**
    * Metoda sprawdza czy dany plik jest możliwy do wczytania
    *
    * @param   string   $file    Względna ścieżka do pliku
    * @return  boolean  true, false
    */
   private static function _checkFile( $file ) {
      if( is_file( self::$_path.$file ) ) {
         if( is_readable( self::$_path.$file ) ) {
            return TRUE;
         } else {
            trigger_error( 'Plik: '. $file . ' jest nie dla odczytu, sprawdź uprawnienia.' , E_USER_WARNING );
         }
      } else {
         trigger_error( 'Zła ścieżka i/lub nazwa dla pliku: '. $file . "\n" . 'Ścieżka: ' . self::$_path . $file , E_USER_WARNING );
      }
      return FALSE;
   }
}
[PHP] pobierz, plaintext 

Gdyby ktoś mógł mi wskazać jakieś wskazówki by poprawić wydajność tej klasy, będę wdzięczny, pomysły z autoload zachować dla siebie, od razu mówię nie, nie i jeszcze raz nie, także bez nich.

Ten post edytował cojack 23.08.2010, 20:18:19

[wookieb]

• w funkcji ładującej możesz zrobić cache w zmiennej statycznej i sprawdzać via in_array. IMO będzie szybsze niż odwoływanie się do funkcji I/O

Wiem, że w php-ie można spodziewać się różnych krzaków ale no sorry erix :/

[PHP] pobierz, plaintext 
 
class Loader
{
    private static $loaded = array();
 
    public static function load($file)
    {
        if (!in_array($file, self::$loaded))
        {
            require $file;
            self::$loaded[] = $file;
        }
    }
}
 
$start = microtime(true);
for ($i=0; $i<1000; $i++)
{
    require_once 'test2.php';
}
echo microtime(true) - $start;
echo '<br/>';
 
$start = microtime(true);
for ($i=0; $i<1000; $i++)
{
    Loader::load('test2.php');
}
echo microtime(true) - $start;
 
[PHP] pobierz, plaintext 

Wynik

Kod

test0.0037469863891602
test0.050887823104858

Gdzie test jest zawartością pliku test2.php

Oczywiście, że require_once jest wolniejsze od require ale tylko przy pierwszym wywołaniu. I teraz pytanie. Czy warto się męczyć o zysk 1 setnej sekundy? I to w dodatku z ryzykiem wielu błędów (jeżeli nie korzystamy z autoloadera)? Nie sądzę.

Napomnę jeszczę taką kwestię iż nie zawsze korzystanie z funkcji autoloadera będzie ok.
Czasem w nagłówkach klas z łapy podaje się require_once i ścieżkę do pliku, żeby nie szukał jej autoloader.

Prosiłbym jeszcze kogoś o przetestowanie powyższego kodu z jakimś akceleratorem. Ponieważ APC mi nie działa na windowsie :/

Ten post edytował wookieb 24.08.2010, 10:56:04