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

[erix]

require_once jest wolniejsze od require, z tego co pamiętam. (IMG:style_emoticons/default/winksmiley.jpg)

Poza tym, dlaczego dokonujesz tylu sprawdzań na poziomie systemu plików? Samo is_readable wystarczy, po co is_file?

ad 1) nie używam stałych (DEFINE) w całym FW, taka filozofia

Hmm, filozofia w stylu kobiecej bo tak!, czy jakaś konkretna argumentacja? Nie wyobrażam sobie większego systemu bez choćby jednej stałej określającej katalog roboczy aplikacji.

ad 2) bo to jest klasa do ładowania plików?

No niech Ci będzie, ale też tego nie rozumiem.

ad 6) Bez sensu byłoby stosowanie tutaj wyjątków, ponieważ kod byłby tak naszpikowany try {} catch {} że to się mija z celem, a błędy i tak gromadzę i pakuje w jedno miejsce, i tak czy siak używam trigger_error. A poza tym używam tej klasy też poza ciałem klas, także wyjątki w tym miejscu nie mają prawa bytu, gdyż ładuje w ten sposób klasy z których dana kasa dziedziczy, no i pomyśl sobie jakby to wyglądało.

Takie myślenie jest bez sensu. Są wyjątki, zwiększają czytelność kodu i usprawniają wiele rzeczy, np:

[PHP] pobierz, plaintext 
 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 

Na co tu tyle? Zrób sobie error_handler, który pluje wyjątkiem przy E_WARNING. PHP nie jest wielowątkowe, więc przy samym require_once wypluje wyjątek w dokładnie tym miejscu, co trzeba, a będziesz miał centralne zarządzanie odpowiednimi zdarzeniami.

Poza tym, żeby require_once wypluło błąd, to też musi sprawdzić, jaki on jest, czyli też sprawdza, czy plik da się odczytać, czy są do niego uprawnienia. Więc wywołanie is_readable jest - de facto - ponownym wykonaniem czegoś, co miało miejsce. Więc po co? (IMG:style_emoticons/default/winksmiley.jpg)