Migracion hacia __set() y __get()

[bigwhite]

Hola!

Tengo un problema... una clase que usaba setters y getters "tradicionales" y ahora la estoy pasando a los metodos magicos correspondientes.

¿ el problema ? me ha cambiado la sintaxis... y ahora debo editar muchisimos archivos. Estoy bien ? o bien confundido?

Antes:
$video->setColor1('888888');

Ahora:
$video->color1 = '888888';

Es asi?
No es que la idea subyacente en POO es abstraerse de la implementacion ?

En esta clase "hija" (extendida) no logro poder ver los atributos de la clase padre!

Código PHP:

  <?
class youtube extends video {  #clase hija: youtube  
 // url = source
    
  function __construct($source=""){       
    parent::__set('source',$source);
    parent::setDefault();
    parent::__set('brand','youtube');    
  }
  
  public function __set($var,$val){
    if ($var=='id'){ // sobre-escribo el metodo "setId"
      parent::__set('id',$id); 
      parent::__set('source'," www  .youtube . com/v/$val");
    }
  }
  
  public function __get($var){
    return parent::__get($var);
  }
  
  function __toString (){ 
    return (parent::__toString());
  }   
 
 
}  # fin de clase 
?>   

Cuando intento acceder a un atributo definido en un vector en la clase padre.... (ej: keywords)

Fatal error: Call to undefined method youtube::keywords()

Ah! la clase padre ("video") comienza asi:

Código PHP:

  <?
class video{
  protected $vector; 
  
  public function __set($var, $val){
    $this->vector[$var]=$val;
  }
  public function __get($var){    
    if(isset($this->vector[$var])){
      return $this->vector[$var];
    }else{
      return NULL;
    }      
  }
...
?>   

[Seppo]

Si te pone undefined method es porque estás haciendo $obj->keywords(), deberías hacer $obj->keywords para acceder al getter... Si querés que sea un método, deberías implementar el método mágico __call

[bigwhite]

Seppo: que tal? podrias darme un ejemplo de como puedo implementar esto ? quisiera que fuera una solucion general para todos los atributos que quiero leer

gracias desde ya por la pista

[GatorV]

Un ejemplo:

Código PHP:

class Foo {
      private $_attributes = array();
 
      public function __construct() {}
 
      public function __set($sVariable, $sData) {
              $this->_attributes[$sVariable] = $sData;
      }
 
      public function __get($sVariable) {
              return $this->_attributes[$sVariable];
      }
 
      public function dumpAttributes() {
              var_dump($this->_attributes);
      }
}
 
$bar = new Foo();
$bar->foo = "baz";
$bar->baz = "foo";
$bar->anumber = 1;
 
$bar->dumpAttributes();
 

Saludos.

[bigwhite]

mil gracias GatorV pero en mi caso tengo diriamos una clase hija 'Bar' y el problema lo tengo al intentar acceder la los atributos implementados en $_attributes en la clase padre.

Ahi es que me falla y no me devuelve nada:

Código PHP:

  class Bar extends Foo {
 
  public function __get($var){ 
      return parent::__get($var); 
    } 
 
} 
   

Es lo mismo ? eso parece funcionar......aunque me quedan dudas.... porque por ejemplo no encuentro la forma de "settear" en la clase hija:

Código ejemplo:

<?
class Foo {
      protected $_attributes = array();
 
      public function __construct() {}
 
      public function __set($sVariable, $sData) {
              $this->_attributes[$sVariable] = $sData;
      }
 
      public function __get($sVariable) {
              return $this->_attributes[$sVariable];
      }
 
      public function dumpAttributes() {
          echo '<pre>'; 
              var_dump($this->_attributes);
          echo '</pre>';              
      }
}
 
class Bar extends Foo { 
 
  function __construct(){
    $this->_attributes['foo']='otra cosa';
  } 
  
  public function __get($var){  
      return parent::__get($var);  
    }  
 
} 
 
$bar = new Bar();
$bar->foo = "baz";
$bar->baz = "foo";
$bar->anumber = 1;
 
$bar->dumpAttributes();
?>
 

Esto no cambia 'foo' en el array

[GatorV]

Define los atributos como protected, si usas private solo son visibles para esa clase, si lo quieres leer en la clase heredada debes de usar protected.

Te recomiendo que leas sobre la visibiladad en la POO.

Saludos.

[bigwhite]

Iniciado por GatorV

Define los atributos como protected, si usas private solo son visibles para esa clase, si lo quieres leer en la clase heredada debes de usar protected.

Te recomiendo que leas sobre la visibiladad en la POO.

Saludos.

Te agradezco PERO... no fue justo lo que hice ?

[GatorV]

Que versión de PHP manejas, el código que editaste (después de mi respuesta) me funciona tal cual, la salida es la esperada:

Código:

array(3) {
  ["foo"]=>
  string(3) "baz"
  ["baz"]=>
  string(3) "foo"
  ["anumber"]=>
  int(1)
}

Saludos.

[bigwhite]

Hola GatorV!, yo esperaba otra cosa porque en el constructor de la clase hija Bar hice:

$this->_attributes['foo']='otra cosa';

uso PHP5

el código que editaste (después de mi respuesta)

No recuerdo haber hecho eso... pero estoy sin dormir.....todo puede ser (al menos no el 'protected')

[masterpuppet]

el problema es que estas redefiniendo la propiedad foo,
despues del new tenes $bar->foo = "baz";

[bigwhite]

Iniciado por masterpuppet

el problema es que estas redefiniendo la propiedad foo,
despues del new tenes $bar->foo = "baz";

MUY cierto! gracias! ahora me queda ver porque no me funciona la clase de videos
gracias a todos!

[enriqueplace]

Que tal Bigwhite,

Amén que ya tienes el problema resuelto, te sugiero que si no estás haciendo algo raro como un generador de código o similar, evites todo lo que puedas el uso de los get y set mágicos de PHP, que lo único que hacen en la mayoría de los casos es crear un efecto de "atributos públicos" y estos están prohibidos conceptualmente desde los inicios de la POO.

Tu clase no debería exponer sus atributos, a lo sumo hacer un get cuando no te quede otra alternativa y en muy contados casos un set, y siempre que puedas, evita hacer a la vez set / get.

Saludos

[bigwhite]

Que tal Bigwhite,

Amén que ya tienes el problema resuelto, te sugiero que si no estás haciendo algo raro como un generador de código o similar, evites todo lo que puedas el uso de los get y set mágicos de PHP, que lo único que hacen en la mayoría de los casos es crear un efecto de "atributos públicos" y estos están prohibidos conceptualmente desde los inicios de la POO.

Que tal.... Enrique? si.... me parecia eso......y me llamaba mucho la atencion

Gracias por tus conceptos
abrazo

[Webstudio]

Igualmente, si bien es sano dejar de lado __set y __get, siempre podés hacer uso del método __call, en el cual podés establecer una implementación de setters y getters que incluso tenga control de scope. No debería ser muy dificil hacer algo asi :

Código PHP:

<?php
class A {
    protected $_attributes = array('nombre', 'apellido', 'edad');
    protected $_data = array();
    
    function __call($name, $argument)
    {
        $type = strtolower(substr($name, 0, 3));
        $att_name = strtolower(substr($name, 3));
        
        switch($type)
        {
            case 'get':
                if(in_array($att_name, $this->_attributes))
                {
                    return $this->_data[$att_name];
                }
                throw new Exception('Atributo no existe, DANGER !!');
            break;
            
            case 'set':
                if(in_array($att_name, $this->_attributes))
                {
                    $this->_data[$att_name] = $argument[0];
                    return $argument[0];
                }
                throw new Exception('Atributo no existe, DANGER !!');
            break;
            
            default:
                throw new Exception('llamaste algo que no existe macho, media pila.');
            break;
        }
    }
}
try {
    $a = new A;
    $a->setApellido('pablo');
} catch(Exception $e) {
    echo $e->getMessage();
}
var_dump($a);
?>
 

[bigwhite]

EXCELENTE webstudio! yo despues de lo que dijo Enrique... habia pensado en algo parecido, tambien con arrays de metodos pero usando esto dentro de los metodos __set y __get

Esa clase A podria ser la clase padre.... de todas las que utilize, verdad ? asi no tendria que andar copiando todo ese codigo........

[Webstudio]

Hay varios caminos que podrías tomar, pero bueno, heredar a A es uno de ellos. Aunque una clase hija (B) tendría que resetear en su constructor el array $_attributes y con eso ya estarías.

[enriqueplace]

Claramente eso es "sobreingeniería", un mal endémico en el desarrollo de software.

La herencia no debe aplicarse SOLO en base al "reuso de código", la herencia es una jerarquía donde debe haber una relación de parentesco.

Cuidado con eso, es muy común ver ese error de conceptos.