PHP - Zend dire evitare i metodi magici?

Question

Stavo leggendo questa pagina - http://deaduseful.com/blog/posts/50-php-optimisation-tips-revisited

E una delle raccomandazioni era quello di evitare l'uso di metodi magici, citato da uno Zend prestazioni PDF che non fornisce alcuna ragione per la sua raccomandazione per evitarli.

Dopo qualche ricerca su Google (e finire qui per una domanda non correlata) mi sono chiesto se qualcuno avesse qualche raccomandazione su quel fronte?

Io uso __get() molto nel mio codice, in genere per salvare variabili che non sempre utilizzo, ad es.

io possa avere una tabella con nome, disc, category_id, time_added

mio get sarebbe simile a questa:

 
public function __get($name) { 
    switch($name) { 
     case 'name': 
     case 'desc': 
     case 'category': 
     case 'time_added': 
      $result = do_mysql_query(); 
      $this->name = $result['name']; 
      $this->desc = $result['desc']; 
      $this->category = $result['category']; 
      $this->time_added = $result['time_added']; 
      return $this->{$name}; 
     break; 
     default: 
      throw Exception("Attempted to access non existant or private property - ".$name); 
    } 
} 

Questo mi sembra un ottimo modo per fare le cose come ho sempre e solo ottenere qualcosa dal database se è necessario e posso riconfermare cose come $ article-> time_added piuttosto che armeggiare con array.

Questo potrebbe essere considerato una cattiva pratica e un carico aggiuntivo sul server?

Spesso estenderò le classi con metodi magici e farò qualcosa del genere se la classe figlio non corrisponde a qualcosa in un get.

 
public function __get($name) { 
    switch($name) { 
     case 'name': 
     case 'desc': 
     case 'category': 
     case 'time_added': 
      $result = do_mysql_query(); 
      $this->name = $result['name']; 
      $this->desc = $result['desc']; 
      $this->category = $result['category']; 
      $this->time_added = $result['time_added']; 
      return $this->{$name}; 
     break; 
     default: 
      return parent::__get($name); 
    } 
} 

Sarebbe una cattiva pratica e un male per le prestazioni? Il numero massimo di livelli che ho quando si estendono i metodi magici è tre.

Answer 1

È vero, sono più lenti ... ma la differenza è così piccola che la velocità rispetto al codice è un fattore. Vale la pena preoccuparsi della differenza per uno sviluppo e una manutenzione più rapidi?

Vedi magic benchmarks per le statistiche

Answer 2

considerare l'utilizzo di array accessors.

class Record implements ArrayAccess { 

    /** 
    * @see ArrayAccess::offsetExists() 
    * 
    * @param offset $offset 
    */ 
    public function offsetExists($offset) { 

    } 

    /** 
    * @see ArrayAccess::offsetGet() 
    * 
    * @param offset $offset 
    */ 
    public function offsetGet($offset) { 
     //fetch and cache $result 

     return $result[$offset]; 
    } 

    /** 
    * @see ArrayAccess::offsetSet() 
    * 
    * @param offset $offset 
    * @param value $value 
    */ 
    public function offsetSet($offset, $value) { 

    } 

    /** 
    * @see ArrayAccess::offsetUnset() 
    * 
    * @param offset $offset 
    */ 
    public function offsetUnset($offset) { 

    } 

Answer 3

Ho fatto alcuni test con metodi magici PHP e operazioni nativo get/set (su una proprietà pubblica)

I risultati:

metodi magici sono molto più lento di accesso nativo. MA il tempo di accesso è ancora così piccolo, che non farà la differenza nel 99,9% di tutti i casi.

Anche se si fa 1 milione di metodo magico accede all'interno di una richiesta, ancora vogliono solo circa 0,1 secondi ...

---

"Sola lettura" significa accesso tramite metodi magici. L'immagine mostra i risultati di PHP 5.5.9 e PHP 7.0.

Ecco lo script di riferimento: https://github.com/stracker-phil/php-benchmark/blob/master/benchmark.php