crear select con programacion orientado a objetos

[jjhoncv25]

Hola amigos nuevamente, tengo este codigo lo e recontra revisado pero hasta el momento no veo cual es el error weno lo que imprime es frutas:y un combo vacio nada mas, realmente me gustaria que me ayudaran a vieran donde esta este error.

<?php //archivo se llama clases.php

class Select extends Element{
protected $options;

public function _construct(){
parent::_construct();
$this->options=array();
}

public function creaOption($option){
$this->options[]=$option;
}

private function traeOptions(){
return (array)$this->options;
}

public function render(){
echo $this->traeLabel().":<br/>\n";
echo "<select name=\"".$this->traeName()."\">\n";
foreach ($this->traeOptions() as $opt){
echo $opt->render();
//echo $this->traeOptions();
}
echo "</select>";
}
}

class Option extends Element{

public function _construct($value,$label){
parent::_construct();
$this->creaValue($value);
$this->creaLabel($label);
}

public function render(){

echo "<option value=\"".$this->traeValue()."\">".$this->traeLabel()."</option>\n";
}
}

class Element{
private $name;
private $value;
private $label;

public function _construct(){ }

public function creaName($name){
$this->name=$name;
}

public function traeName(){
return $this->name;
}

public function creaValue($value){
$this->value=$value;
}

public function traeValue(){
return $this->value;
}

public function creaLabel($label){
$this->label=$label;
}

public function traeLabel(){
return $this->label;
}

}
?>

<?php

include ('clases.php');

$frutas=new Select();
$frutas->creaLabel('Frutas');
$frutas->creaName('frut_sel');
$frutas->creaOption(new Option('Naranjas','Naranjas'));
$frutas->creaOption(new Option('Fresas','Fresas'));
$frutas->creaOption(new Option('Piñas','Piñas'));
$frutas->creaOption(new Option('Plátanos','Plátanos'));
$frutas->creaOption(new Option('Manzanas','Manzanas'));
$frutas->render();

?>


ojala que me ayuden con esto.

[Triby]

Hay varias cosas a corregir, algunas conceptuales y otras de codigo.

Dentro de las conceptuales: Estas creando una clase que solamente te permitira agregar un elemento por cada instancia. No tiene mucha utilidad, tal vez estas practicando o haciendo una tarea, pero ni como ejercicio lo veo viable.

De codigo (y tambien conceptual):

Código PHP:

// El metodo admite solo un parametro:
public function creaOption($option){
$this->options[]=$option;
}
 
// Sin embargo, al agregar una nueva opcion, creas otro objeto... con que objeto?
$frutas->creaOption(new Option('Naranjas','Naranjas'));
 

Tal vez seria mas conveniente algo como:

Código PHP:

public function creaOption($index, $caption){
    $this->options[$index]=$caption;
}
 
// Y al crear cada opcion:
$frutas->creaOption('naranjas','Naranjas');
 
// Despues, al mostrarlas:
foreach($this->options as $index => $caption) {
    echo "<option value=\"$index\">$caption</option>\n";
}
 

Finalmente, dentro del metodo render, haces:
echo $opt->render();

Creo que esto terminaria en un ciclo infinito.

[jjhoncv25]

gracias por el codigo.. pero no entiendo unas cosas en q parte va el foreach en q clase o q se estaria modificando... y tambien la parte de $opt->render(); en q parte lo pongo o q parte es la q se tiene q modificar .

[masterpuppet]

En realidad lo que esta mal es que te falta un "_" en los constructores, tenes _construct cuando deberia ser __construct.
Y tendrias que revisar varias cosas pero te sugiero que no hagas echo's, simplemente en render concatena y retorna la cadena, para imprimir llama a render en el __toString, algo asi:

Código PHP:

abstract class Element
{
    ...
    abstract public function render();
 
    public function __toString()
    {
        return $this->render();
    }
}
 
class Select extends Element
{
    ...
    public function render()
    {
        $html  = $this->traeLabel();
        $html .= "<select name=" . $this->traeName() . ">";
        foreach ($this->traeOptions() as $opt) {
            $html .= $opt->render();
        }
        $html .= "</select>";
        return $html;
    }
}
 

obviamente el render de option no imprime sino que retorna el string concatenado, luego para utilizar:

Código PHP:

$frutas= new Select();
$frutas->creaLabel('Frutas');
$frutas->creaName('frut_sel');
$frutas->creaOption(new Option('Naranjas','Naranjas'));
$frutas->creaOption(new Option('Fresas','Fresas'));
echo $frutas;
 

Saludos.