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]

1) Nie latwiej zdefiniować nawet stałej która przechowuje ci getcwd() i dawać ją w potrzebnych miejscach? (że nie wspomnę o używaniu include_path)

[PHP] pobierz, plaintext 
require_once ACTUAL_CWD.'plik.php';
[PHP] pobierz, plaintext 

Bo to co teraz masz nie ma sensu
2) Nie rozumiem, dlaczego pakujesz tutaj ładowanie xml
3) Korzystamy z wyjątków a nie trigger_errorów.
4) Klasa loadera wcale nie jest tak wolna. Czasem narzut pracy wynika po prostu z konieczności sparsowania , wykonania załadowanych plików przez php.
5)

Kod

   * @return  boolean  true, false

Facepalm :/
6)

Kod

* @return  object    Xml object

Typujemy silnie. Czyli nie "object" ale "SimpleXMLElement". Ma to ogromne znaczenie dla wszelkiego rodzaju IDE.
7) Nadal nie widzę sensu korzystania z metody _checkFile. Szczerze mówiąc przyda się tylko w środowisku developerskim gdzie sam na podstawie wywalonego przez php błędu będziesz wiedział co zrobić.

Ten post edytował wookieb 23.08.2010, 20:30:47