Tymczasowy obiekt Opcje

[michalos13]

Ostatnio zmagam się z problemem którego nie mogę rozwiązać. Tworzę klasę która doda do tablicy wartość wygenerowaną z innej klasy. Robię w funkcji "tymczasowy obiekt" (nie wiem jak to nazwać) klasy, wywołuję z niej funkcję która zwraca wartość i zostaje zapisana ona do tablicy. Potem pobieram z tablicy interesującą mnie wartość. Za pierwszym razem zwracana wartość jest dobra ale potem wartość == null.

[PHP] pobierz, plaintext 
<?php
class Klasa
{
    private $controls = array();
    
    public function addElement( $id , $name , $config )
    {
        $reflect = new ReflectionClass( $name );
        $element = $reflect->newInstance();
        $this->controls[$id] = $element->createElement( $config  );
    }
    
    public function getElement( $id )
    {
        return $this->controls[$id] ; 
    }
}
 
$a = new Klasa();
$a->addElement( 'id' , 'text' , array( 'cos' => 'costam' ) );
 
echo $a->getElement( 'id' );
?>
[PHP] pobierz, plaintext 

[wookieb]

Pokaż proszę kod klasy text u mnie coś takiego działa

[PHP] pobierz, plaintext 
<?php
class text
{
    public function createElement($conf)
    {
        return 'hehe';
    }
}
 
class Klasa
{
   private $controls = array();
   
   public function addElement( $id , $name , $config )
   {
       $reflect = new ReflectionClass( $name );
       $element = $reflect->newInstance();
       $this->controls[$id] = $element->createElement( $config  );
   }
   
   public function getElement( $id )
   {
       return $this->controls[$id] ;
   }
}
 
$a = new Klasa();
$a->addElement( 'id' , 'text' , array( 'cos' => 'costam' ) );
 
echo $a->getElement( 'id' );
echo $a->getElement( 'id' );
?>
[PHP] pobierz, plaintext 

Ale nie wiem czy do końca dobrze zrozumiałem.

[michalos13]

Dobrze mnie zrozumiałeś (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Ale klasa (text) u mnie wyglądało mniej-więcej tak:

[PHP] pobierz, plaintext 
<?php
class text
{
    private $txt;
    public function createElement($conf)
    {
        $this->txt = $this->txt . '123';
        $this->txt = $this->txt . 'abc';
        return $this->txt;
    }
}
?>
[PHP] pobierz, plaintext 

Ale kiedy zrobię tak jak w twoim przykładzie (zwracam od razu tekst) to działa.

Ten post edytował michalos13 19.06.2009, 17:24:15

[wookieb]

Więc już chyba wszystko ok?
A jeżeli nie ok to rozumiem, że to nie jest dokładny kod i wtedy musisz raczej pokazać ten, który powoduje problemy (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[michalos13]

No więc:

[PHP] pobierz, plaintext 
<?php
class checkbox implements phpfElement
{
    private $fullElement;
 
    function createElement( $element )
    {
        require_once( 'namespace.php' );
        if( isset( $element[ $namespace[ 'name' ] ] ) )
        {
            $this->elementAdd( '<input type="checkbox" name="' . $element[ $namespace[ 'name' ] ] . '" ' );
            if( isset( $element[ $namespace[ 'value' ] ] ) )
            {
                $this->elementAdd( 'value="' . $element[ $namespace[ 'value' ] ] . '" ' );
            }
            if( isset( $element[ $namespace[ 'size' ] ] ) )
            {
                $this->elementAdd( 'size="' . $element[ $namespace[ 'size' ] ] . '" ' );
            }
            if( isset( $element[ $namespace[ 'maxlength' ] ] ) )
            {
                $this->elementAdd( 'maxlength="' . $element[ $namespace[ 'maxlength' ] ] . '" ' );
            }
            if( $element[ $namespace[ 'readonly' ] ] ) 
            {
                $this->elementAdd( 'readonly="readonly" ' );
            }
            if( $element[ $namespace[ 'disabled' ] ] ) 
            {
                $this->elementAdd( 'disabled="disabled" ' );
            }
            if( isset( $element[ $namespace[ 'tabindex' ] ] ) )
            {
                $this->elementAdd( 'tabindex="' . $element[ $namespace[ 'tabindex' ] ] . ' ' );
            }
            if( isset( $element[ $namespace[ 'class' ] ] ) )
            {
                $this->elementAdd( 'class="' . $element[ $namespace[ 'class' ] ] . '" ' );
            }
            if( isset( $element[ $namespace[ 'id' ] ] ) )
            {
                $this->elementAdd( 'id="' . $element[ $namespace[ 'id' ] ] . '" ' );
            }
            if( $element[ $namespace[ 'checked' ] ] ) 
            {
                $this->elementAdd( 'checked="checked" ' );
            }
            $this->elementAdd( ' />');
            
            
            if( isset( $element[ $namespace[ 'label' ] ] ) )
            {
                $this->elementAdd( '<label for="' . $element[ $namespace[ 'id' ] ] . '">'
                                   . $element[ $namespace[ 'label' ] ] . '</label>'
                                 );
            }
        }
        return $this->fullElement;
    }
    
    private function elementAdd( $string )
    {
        $this->fullElement = $this->fullElement . $string;
    }
}
?>
[PHP] pobierz, plaintext 

i namespace.php

[PHP] pobierz, plaintext 
<?php
$namespace[ 'name' ] = 'name';
$namespace[ 'text' ] = 'text';
$namespace[ 'value' ] = 'value';
$namespace[ 'size' ] = 'size';
$namespace[ 'maxlength' ] = 'maxlength';
$namespace[ 'readonly' ] = 'readonly';
$namespace[ 'disabled' ] = 'disabled';
$namespace[ 'input' ] = 'input';
$namespace[ 'class' ] = 'class';
$namespace[ 'label' ] = 'label';
$namespace[ 'id' ] = 'id';
$namespace[ 'multiple' ] = 'multiple';
$namespace[ 'optgroup' ] = 'optgroup';
?>
[PHP] pobierz, plaintext 

[wookieb]

Problem polega na tym ze użyłeś require_once, który działa tak, że jeżeli raz wcześniej dołączono plik do skryptu to drugi raz tego nie zrobi. Użyj samego require.

[michalos13]

Wiedziałem że to moja głupota ;-p Dzięki.