[php] overerving van functie werkt niet

[wboard]

waarom zou ik deze functie niet kunnen overerven in een onderliggende klasse?

PHP:

    public function toArray()
    {
        $arr = array();

        foreach($this as $key => $value)
        {
            $arr[$key] = $value;
        }

        return $arr;
    }

als ik de functie binnen de onderliggende klasse plaatst werkt hij perfect ,
van de magic functions weet ik dat je die niet kunt overerven (logisch), zoals __call e.d.

[RobIII]

Is dat alle (relevante) code? Laat je inherited class eens zien? En zit je functie wel in een class?

[ Voor 70% gewijzigd door RobIII op 20-07-2009 20:55 ]

[Voutloos]

Geef aub een glazen bol, dan kunnen we helpen. Als dat wat lastig is, is het geven van de definities van beide classes, en de foutmelding wellicht ook al een eind in de richting.

[HuHu]

Ik gok op het foutief gebruik van $this, maar dat is maar een gokje.

[wboard]

excuses, ik dacht dat ik alles gepost had, wanneer ik hem in deze class zet werkt hij zonder problemen:

PHP:

<?
require_once(dirname(__FILE__) . "/../globalConfig.php");

class product extends webshop
{
    //Empty constructor for Class
    public function __construct()
    {}

    //Return a named array with all data from object
    public function toArray()
    {
        $arr = array();

        foreach($this as $key => $value)
        {
            $arr[$key] = $value;
        }

        return $arr;
    }   
}
?>

maar wanneer hij in de parent class staat pakt hij hem niet (hij returned een LEGE ARRAY)

PHP:

<?
require_once(dirname(__FILE__) . "/../globalConfig.php");

class webshop
{
    //Attributes for class Type
    //Methods for class Type

    //Empty constructor for Class
    public function __construct()
    {
        $this->dbConnect();
    }

    //Return a named array with all data from object
    public function toArray()
    {
        $arr = array();

        foreach($this as $key => $value)
        {
            $arr[$key] = $value;
        }

        return $arr;
    }       
}
?>

[wboard]

HuHu schreef op maandag 20 juli 2009 @ 21:17:
Ik gok op het foutief gebruik van $this, maar dat is maar een gokje.

goeie gok denk ik, want daar denk ik ook al een tijd aan, maar ik begrijp dan niet waarom hij perfect werkt in de onderliggende classe

[Voutloos]

En waar maak je de instance aan, waar hoe ziet de call naar toArray eruit? Is die niet stiekem static en negeer je een E_STRICT error, etc. etc.

[masq]

Visibility

The visibility of a property or method can be defined by prefixing the declaration with the keywords: public, protected or private. Public declared items can be accessed everywhere. Protected limits access to inherited and parent classes (and to the class that defines the item). Private limits visibility only to the class that defines the item.

http://www.php.net/manual/en/language.oop5.visibility.php

Ligt het hier niet aan? Aangezien de methode in de supertype gedefinieerd is.

[ Voor 7% gewijzigd door masq op 20-07-2009 21:51 ]

[wboard]

Voutloos schreef op maandag 20 juli 2009 @ 21:32:
En waar maak je de instance aan, waar hoe ziet de call naar toArray eruit? Is die niet stiekem static en negeer je een E_STRICT error, etc. etc.

error_reporting(E_ALL);


de call is niet static, de instance maak ik gewoon in een functie
$blaat = new product;
$boe = $blaat->open(); //hier vult hij de properties in
$meuh = $blaat->toArray();

let er op dat het bij open() gewoon goed gaat (ik heb nl. gecheckt of ze ingevuld zijn)

[Verwijderd]

^{product extends webshop... Een product is dus een webshop?}

[wboard]

Verwijderd schreef op maandag 20 juli 2009 @ 21:57:
^{product extends webshop... Een product is dus een webshop?}

lol, nee dat klopt niet

[Rian]

PHP:

 foreach($this as $key => $value)
        {
            $arr[$key] = $value;
        }

Vziw kun je niet zomaar een compleet object in een foreach statement stoppen. Als je dynamisch alle waarden van je object in een array wilt teruggeven, moet je een method gebruiken die alle variabelen ophaalt. Een voorbeeld daarvan is get_class_vars, ik zou daar eens naar kijken in de php manual.

Wat ik persoonlijk zou doen is een functie schrijven die dmv get_class_vars alle variabelen returned. Wanneer je de variabelen weet, kun je met het grootste gemak de bijbehorende waardes ophalen.

[masq]

Rian schreef op maandag 20 juli 2009 @ 22:13:
Vziw kun je niet zomaar een compleet object in een foreach statement stoppen.

Jawel hoor, zie http://php.net/manual/en/language.oop5.iterations.php

TS: Wat is de visibility van de class members? Anders valt er nog steeds niets nuttigs over te zeggen.

[HuHu]

Doe eens:

PHP:

error_reporting(E_ALL | E_STRICT);

[Verwijderd]

wboard schreef op maandag 20 juli 2009 @ 22:04:
[...]

lol, nee dat klopt niet

^{Just wondering... Het kwam me al enigszins vreemd over}

[wboard]

-De visibility van de class members is PRIVATE, maar dat zou toch niet uit moeten maken als je een methode overerft??

-Die error reporting zal ik eens aanzetten, post het zo ff

-@rian: ja dat kan wel, vandaar ook dat het binnen de class zelf wel werkt

[Voutloos]

wboard schreef op dinsdag 21 juli 2009 @ 09:21:
-De visibility van de class members is PRIVATE, maar dat zou toch niet uit moeten maken als je een methode overerft??

Probeer het eens en leer er van.

[Verwijderd]

wboard schreef op dinsdag 21 juli 2009 @ 09:21:
-De visibility van de class members is PRIVATE, maar dat zou toch niet uit moeten maken als je een methode overerft??

-Die error reporting zal ik eens aanzetten, post het zo ff

-@rian: ja dat kan wel, vandaar ook dat het binnen de class zelf wel werkt

Als je in je base class een member als private declareert, dan is dat member alleen visible voor die base class. Wil je echter dat de bewuste member visible blijft in de overervingsstructuur, dus in de derived classes van die base zul je ze als protected moeten declareren.
Voorbeeld (C#, ik verwacht dat PHP hetzelfde zal doen ):

C#:

namespace Inheritance
{
    class Base
    {
        private string _myPrivateMember = string.Empty;
        protected string _myProtectedMember = string.Empty;
        public string _myPublicMember = string.Empty;

        public Base()
        {
            // alle 3 de members kunnen in deze constructor gezet worden
            _myPrivateMember = "private";
            _myProtectedMember  = "protected";
            _myPublicMember = "public";
        }
    }

    class Derived : Base
    {
        public Derived()
        {
            // alleen de protected en public kunnen in deze constructor gezet worden
            // de private member is dus niet zichtbaar voor deze class
            _myProtectedMember = "protected";
            _myPublicMember = "public";
        }
    }
}

[Zyppora]

$this verwijst naar de instance van de class waarin het aangeroepen wordt. Als jij dat in een inherited method aanroept, en de variables staan in de inheriting class (of vice versa), zul je die niet zien.

[HuHu]

wboard schreef op dinsdag 21 juli 2009 @ 09:21:
-De visibility van de class members is PRIVATE, maar dat zou toch niet uit moeten maken als je een methode overerft??

-Die error reporting zal ik eens aanzetten, post het zo ff

-@rian: ja dat kan wel, vandaar ook dat het binnen de class zelf wel werkt

Gozer... post dan een werkend voorbeeld waarmee het fout gaat. Alsof we kunnen ruiken of je class members public/protected/private zijn :{w.

[RobIII]

HuHu schreef op dinsdag 21 juli 2009 @ 10:36:
[...]

Gozer... post dan een werkend voorbeeld waarmee het fout gaat. Alsof we kunnen ruiken of je class members public/protected/private zijn :{w.

Hoewel HuHu het hiet wat lomp brengt heeft HuHu wel een punt: als je voorbeeld in de TS niet overeen komt met de werkelijke situatie dan kun je niet van ons verwachten dat we zinnige uitspraken kunnen doen. Zorg dus (voortaan) voor code die overeenkomt (of zelfs een 1:1 copy/paste is) met het daadwerkelijke probleem.

[wboard]

oeps, dit is wel erg lomp , bedankt iig