Esto representa la POO ?

[gfuentes]

Estimados:

Gracias por su atencion, queria preguntar si para ustedes este ejemplo es representativo para la programcion orientada a objeto o como se podria mejorar.

Gracias.

Código PHP:

   
class Casa { 
         
        var $exterior = "Jardin"; 
        var $estructura = "Concreto"; 
         
        function armaCasa($color){ 
            $this->validaArmado($color); 
            $tuCasa="Tu casa tiene un lindo {$this->exterior} 
            y su estructura es de {$this->estructura} de color {$color}"; 
            return $tuCasa; 
        } 
         
        function validaArmado ($cosaValido){ 
            if(empty($cosaValido)){ 
                echo 'Error # 1 : El dato esta vacio'; 
                exit(); 
                } 
        } 
    } 
     
    $laCasa = new Casa; 
    echo $laCasa->armaCasa('Rojo');  
   

[eulloa]

No se lo q quieras decir con "representativo", pero de q se puede mejorar si q se puede.
Empieza por quitar var y comenzar a utilizar modificadores de acceso. Darse una vuelta por el manual tampoco estaría mal.
salu2

[gfuentes]

Gracias, por la respuesta representativa me refiero algo caracteristico o tipico, bueno le dare una vuelta, ya que necesito un ejemplo para comenzar la implementacion de la POO en la empresa, una consulta los modificadores por acceso depende de que version de PHP tenga montada desde que version van desde la 5?.

[GatorV]

Por otro lado es mejor que uses excepciones para manejar los mensajes de error en lugar de usar echo/exit.

Saludos.

[dggluz]

Más representativo de la POO sería que le pusieses accesors y matators (también conocidos como getters y setters) y utilizar modificadores de acceso, como bien dice Eulloa. ¡Suerte!

[gfuentes]

dggluz con respecto a los accesor y matator tomare en cuenta tu recomendacion, lo que me sugerio Eulloa lo aplique, lo de las excepciones lo dejare para mas adelante, ya que no quiero marearlos con tanta informacion, es mejor regurgitar el conocimiento de a poco jeje, Gracias nuevamente.

[gfuentes]

Buenos muchachos entonces segun lo sugerido sin contar lo de las excepciones el codigo estaria asi

Código PHP:

class Casa {
        
        private $exterior = "Jardin";
        private $estructura = "Concreto";
        public $tuCasa;
        
        function getCasa(){
            return $this->tuCasa;
        }
        function setCasa($color){
            $this->validaArmado($color);
            $this->tuCasa="Tu casa tiene un lindo $this->exterior
            y su estructura es de $this->estructura de color $color";
        }
            
        function validaArmado ($cosaValido){
            if(empty($cosaValido)){
            echo 'Error # 1 : El dato esta vacio';
            exit();
            }
        }
    }
    
    $laCasa = new Casa;
    
    $laCasa->setCasa('Amarilla');
    
    echo $laCasa->getCasa();
 

Hay me avisan si notan algun detalle a se me olvido poner acceso a las funciones :P . Gracias nuevamente

[gfuentes]

Gracias por su ayuda muchachos uno aprende demaciado en esta comunidad