cambiar de estructurada a POO

[migueilichenco]

hola a todo, buenos tal ves soy muy preguntos, pero me interesa aprender, y pues soy novato, actualemnte estoy transformando las funciones estructuradas que uso a poo y pues arme esta clase, solo que no la veo que se paresca a la POO, agradeceria si me podrian ayudar

Código PHP:

  <?
    class combobox
        {
            public $sql;
            
            function __construct($table,$option,$value)
                {
                    $this->table=$table;
                    $this->option=$option;
                    $this->value=$value;
                    $sql = "select * from ".$this->table;
                    $this->sql=$sql;
                    $result = mysql_query($this->sql) or die (mysql_error());
                    $this->result=$result;
                    echo "<select name='$this->table' class='botones'>";
                    while($fila = mysql_fetch_assoc($this->result))
                        {
                            echo "<option value='$fila[$value]'>$fila[$option]</option>"; 
                        }    
                    echo "</select>"; 
                    
                    mysql_close();
 
                }
        }
include("bd.php");
Conexion();
$test = new combobox("listas","producto","id_producto");
?>   

Gracias de antemano a todos por sus respuestas, y espero no molestar

[GatorV]

La ventaja de usar POO es que puedes hacer las cosas mas separadas, o utilizar el ya famoso MVC, para separar lo que es la Vista (HTML) de el acceso a datos (PHP)

En si, si estas usando POO, pero no de la forma correcta, deberias de tener todo bien empacado y aislado es decir:
- Una clase ComboBox
- Una clase BD (llamada desde comboclass)
- Una clase Renderer (llamada desde tu comboclass) Para pasar de datos (un array) a un control HTML

Deberias de leer los libros de POO que hay para que te des una idea como debes de aislar todo tu codigo en objetos que se implementan unos a otros.

[migueilichenco]

gracias por tu respuestas GatorV, si tienes razon mi implementacion tiene poco o nada de la estrctura logica de POO, esto es devido a que aunque tengo la logica relativamente clara, no tengo asi, claro algun ejemplo practico, para poder basarme en el, y partir de esa estrctura para estudiarla y empezar a trabajar mas como "deberia ser", agradeceria si alguien se toma la molestia de ayudarme a transformas mi clase en una clase real poo, de paso serviria de ejemplo para otros, gracias a todos por sus posibles respuestas y un saludo

[GatorV]

Pues para empezar puedes realmente romper tu "clase" en una clase con mas propiedades y metodos.

Código PHP:

  class ComboBox {
      private $tabla;
      private $opcion;
      private $valor;
      private $datosCombo;

      public function __construct( $tabla, $opcion, $valor ) {
            $this->tabla = $tabla;
            $this->opcion = $opcion;
            $this->valor = $valor;
            $this->datosCombo = array();
      }

      public function setTabla( $tabla ) {
            $this->tabla = $tabla;
      }

      public function getTabla() {
            return $this->tabla;
      }

      // Aqui faltarian mas "setters" y "getters"

      public function loadData() {          
             $sql = "SELECT * FROM " . $this->table;
             
             $conexion = new Conexion();
             $conexion->Query( $sql );
             $data = array();
             while( $row = $conexion->fetchRow() ) {
                   // Aqui suponemos que tenemos dos valores del query
                   $data[$row[0]] = $row[1];
             }
             
             $this->datosCombo = $data;
      }

      public function dibujaCombo() {
             $html = "<select name=\"%s\" class=\"botones\">%s</select>";
             $opciones = array();
             $opHTML = "<option value=\"%s\"%s>%s</option>";
             foreach( $this->datosCombo as $value => $option ) {
                    $selected = '';
                    if( $value == $this->value ) $selected = ' selected="';
                    
                    $opciones[] = sprintf( $opHTML, $value, $selected, $option );
             }

              return sprintf( $html, $this->table, implode( "\n", $opciones ) );
      }

      public function __toString() {
             return $this->dibujaCombo();
      }
} 
   

y lo puedes usar asi:

Código PHP:

  $combo = new ComboBox( "listas", "producto", "id_producto" );
$combo->loadData();
echo $combo; // Esto llama el metodo de __toString() 
   

[migueilichenco]

gracias GatorV tu como siempre respondiendo y teniendo una pacinecia increible, bueno me sirvio de mucho la clase que construiste y me hace ver lo mucho que me hace falta aprender, trate de hacer la estrcutura de la clase conexion a la que haces referencia pero parece que no funciona

Código PHP:

  class Conexion  {
            private $servidor;
            private $usuario;
            private $password;
            private $sql;
            private $db;
            
            public function __construct($servidor,$usuario,$password,$db){
                    $this->servidor=$servidor;
                    $this->usuario=$usuario;
                    $this->password=$password;                    
                    $this->db=$db;
                    $this->cadena_result=array();
                    $this->cadena=array();
                                        
                    if (!($link =mysql_connect($this->servidor,$this->usuario,$this->password))){
                    
                            echo "Error conectandose a la base de datos.";
                            exit();
                    }
                
                    if (!mysql_select_db($this->db,$link)){
                    
                            echo "Error seleccionando la base de datos.";
                            exit();
                    }
                
                    return $link;        
            }
      
            public function Query($sql){
                    $this->cadena = mysql_query($sql);
                    return $this->cadena;
                    
                
            }
            public function fetchRow() {
                    $this->cadena_result = mysql_fetch_assoc($this->cadena);
                    return $this->cadena_result;
        }
    
} 
   

jejeje eso fue lo que logre, bueno no es nada bonito me deja lo mucho que me falta aprender, gracias gatorV por tus respuestas Saludos

[GatorV]

Creo es bueno que quieras romper con tus paradigmas de hacer las cosas a la antiguita por hacerlas ya de la forma moderna.

Esto te va a ayudar mucho cuando por decir quieras cambiarte a Java o a lenguajes de ese tipo, saludos!

[migueilichenco]

Gracias GatorV por tu ayuda bueno despues de estudiar y tratar de meter las ideas que me has dado y estructurarlas, e creado una clase creo que lo mas cercana posible a una clase "normal", aunque aun le falta, me gustaria una opinion para saber por donde toy que le fallo y por donde voy bien, gracias a todos por las posibles respuestas que coloquen y gracias a GatorV por su infinita paciencia

Este es el Codigo:

Código PHP:

  class content
{
        
        public $direccion;
        public  $expresion;
        protected $salida;
        protected $cadena;
        protected $url;
        protected $valor;
        protected $result;
        protected $out;
 
        function __construct($direccion)
            {
                
                $this->direccion=$direccion;
                $this->expresion=$expresion;
                $this->valor=$valor;
                $this->url=$url;
                $this->result=$result;
                $this->out=$out;
                
                
                $this->expresion="#content=\"(.*?)\"#i";
                $this->valor="iso";
            }        
        function verify()
        {                    
                $this->url=fopen($this->direccion, "r");
                if (!$this->url) 
                {
            echo "<p>Erro pagina inexistente.\n";
            exit;
                }
                $this->display();
        }    
        function display()
        {        
                while (!feof ($this->url)) 
                    {
                        $this->result = fgets ($this->url, 8000);                    
                        if (preg_match_all($this->expresion, $this->result, $this->out))
                                {    
                                        if (!strpos($this->out[1][0],$this->valor))
                                                {
                                                        echo $this->out[1][0];
                                                        echo "\n";
                                                        echo "<br>";
                                                }    
                                }
                    }
                fclose($this->url);    
 
        }
    }
 
    
 
        $contenido = new content("http://www.peru.com");
        $contenido->verify(); 
   

[GatorV]

El unico problema que yo le veo es en el metodo verify, porque? las clases no deberian nunca terminar la ejecucion de un script, a menos que le sea pedido (o mejor lanzar una excepcion:

Código PHP:

  public function verify() {
         $this->url=fopen($this->direccion, "r");
         if (!$this->url) 
         {
               throw new Exception( "Pagina no existente, o no contactable" );
         }
         $this->display(); 
} 
   

[migueilichenco]

gracias algo nuevo se aprende siempre, la clase funciona bien pero en mi editor me corre bien, pero en la parte inferior me avisa esto:

Notificación: D:\servidor\clases1\href2.php línea 99 - Undefined variable: expresion
Notificación: D:\servidor\clases1\href2.php línea 100 - Undefined variable: valor
Notificación: D:\servidor\clases1\href2.php línea 101 - Undefined variable: url
Notificación: D:\servidor\clases1\href2.php línea 102 - Undefined variable: result
Notificación: D:\servidor\clases1\href2.php línea 103 - Undefined variable: out

Aunque corre, normal, pero dime eso ha que se deve, alguna parte que no esta donde deveria, disculpa la persistencia, pero me gusta aprender, y entender para poder mejorar y despues yo tambien poder ayudar

[GatorV]

Se debe a que esas variables no las definiste antes de utilizarlas, como el ejemplo de $this->expresion = $expresion; $expresion no tiene ningun valor anterior por lo cual te tira ese error.