[PHP] OO beginnersvraagje

[Verwijderd]

Hallo iedereen,

Sinds kort ben ik beginnen Object-georiënteerd programmeren in PHP (Wat ik normaal in Java doe)

Maar ik zit met volgend bizar probleem dat ik maar niet opgelost krijg. (Weet echt niet wat er fout aan is ondanks wat opzoekingswerk)

Ik heb volgend eenvoudig stukje code

PHP:

        <?php
            include 'domein/Driver.php';
            include 'domein/Equipe.php';

            $pilot = new Driver("Lavaert", "Mathias");
            $co_pilot = new Driver("Selie", "Peter");

            $equipe = new Equipe($pilot, $co_pilot);

            echo $equipe->__toString();
        ?>

De klasse Driver heeft gewoon twee attributen :: Naam en voornaam + een toString methode (die werkt...)
De klasse Equipe heeft twee attributen $pilot en $co_pilot + een toString methode die dezelfde methode aanroept bij de klasse Driver.

Nu krijg ik volgende foutmelding:

Fatal error: Call to a member function getName() on a non-object in C:\xampp\htdocs\Inschrijvingen\domein\Equipe.php on line 19

Klasse Equipe ::

PHP:

    class Equipe {
        private $pilot;
        private $co_pilot;
        private $car;

        public function __construct($pilot, $co_pilot) {
            $this->pilot = $pilot;
            $this->co_pilot = $co_pilot;
        }

        public function __toString() {
            return $pilot->getName();
        }

    }

Wat moet ik hiermee?

Mathias Lavaert

[ Voor 15% gewijzigd door Verwijderd op 17-08-2010 22:59 ]

[RobIII]

Verwijderd schreef op dinsdag 17 augustus 2010 @ 22:55:
Fatal error: Call to a member function getName() on a non-object in C:\xampp\htdocs\Inschrijvingen\domein\Equipe.php on line 19


Wat moet ik hiermee?

Beginnen met equipe.php te posten en dan vooral de code rondom regel 19

[orf]

Ik denk dat je de classes even moet posten, of in ieder geval de relevante code.

Te laat

[ Voor 11% gewijzigd door orf op 17-08-2010 22:58 ]

[InfoTracer]

Ik gok dat een var niet wordt geset met een object en dat je op die var een methode probeert uit te voeren, wat natuurlijk dan helemaal niet kan. En het kan idd rond regel 19 liggen

[ Voor 12% gewijzigd door InfoTracer op 17-08-2010 23:01 ]

[RobIII]

RobIII schreef op dinsdag 17 augustus 2010 @ 22:56:
[...]

Beginnen met equipe.php te posten en dan vooral de code rondom regel 19

Daar kom ik even op terug. Begin maar eens eerst met Debuggen: Hoe doe ik dat?

[Gwaihir]

$this->pilot->getName();

(Al heb je niet afgedwongen dat er ook echt 'n Driver in zit.)

[ Voor 57% gewijzigd door Gwaihir op 17-08-2010 23:04 ]

[Verwijderd]

Regel 19 is regel 12 in het stukje dat ik gepast heb

[Verwijderd]

Gwaihir schreef op dinsdag 17 augustus 2010 @ 23:02:
$this->pilot->getName();

Ok ja,

Nog even wennen aan de syntaxis van PHP dus

[Gwaihir]

PHP kent wel type safety voor objecten. Daar zou ik zeker gebruik van maken als je specifieke methodes van zo'n Driver aanroept in de Equipe. Overigens zou ik het netter vinden de __toString() van die pilot aan te roepen dan de getName().

[Verwijderd]

Gwaihir schreef op dinsdag 17 augustus 2010 @ 23:06:
PHP kent wel type safety voor objecten. Daar zou ik zeker gebruik van maken als je specifieke methodes van zo'n Driver aanroept in de Equipe. Overigens zou ik het netter vinden de __toString() van die pilot aan te roepen dan de getName().

Normaal roep ik __toString() aan maar omdat het niet werkte heb ik vanalles geprobeert

[jip_86]

Dat (__toString()) zal dan nu wel weer werken als je $this hebt

[ Voor 12% gewijzigd door jip_86 op 17-08-2010 23:18 ]

[Matis]

Je moet ook $this->pilot->getName(); doen, de variabele pilot valt namelijk niet binnen de scope van je functie, maar is een globaal gedefinieerde variabele.
In Java hoef je vaak niet met this te werken in dit soort gevallen, maar in PHP wel

Edit; Jip

[ Voor 3% gewijzigd door Matis op 17-08-2010 23:16 ]

[HuHu]

Als je PHP 5 gebruikt zou ik ook type hinting gaan gebruiken. Verder zou ik een bepaalde coding standard gaan hanteren, want het is vrij gebruikelijk om protected/private variabelen met een underscore (_) te prefixen. Dat alles voor het overzicht.

Dan krijg je dus dit:

PHP:

class Equipe
{ 
  private $_pilot; 
  private $_coPilot; 
  private $_car; 

  public function __construct(Driver $pilot, Driver $coPilot) 
  { 
    $this->_pilot = $pilot; 
    $this->_coPilot = $coPilot; 
  } 

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

[YopY]

HuHu schreef op woensdag 18 augustus 2010 @ 08:36:
Verder zou ik een bepaalde coding standard gaan hanteren, want het is vrij gebruikelijk om protected/private variabelen met een underscore (_) te prefixen. Dat alles voor het overzicht.

De verplichte $this-> 'prefix' geeft imho al genoeg aan dat het een private of protected variabele (class variabele) betreft, . Eerst de basis, dan pas beginnen met coding standaarden - ik zie weinig mis met de gebruikte in dezen.

@topic, $this-> idd. Nee het is niet logisch ($pilot wordt niet gedefinieerd in de function scope, logischerwijs zou de interpreter dan een niveau hoger (class scope) moeten kijken en als dat ook niet werkt nog eentje hoger (global scope).

[ZpAz]

YopY schreef op woensdag 18 augustus 2010 @ 08:57:
[...]


De verplichte $this-> 'prefix' geeft imho al genoeg aan dat het een private of protected variabele (class variabele) betreft, . Eerst de basis, dan pas beginnen met coding standaarden - ik zie weinig mis met de gebruikte in dezen.

PHP:

class Foo {
    private $_bar;
    public $bar;

    function doFoo() {
        echo $this->_bar; //Werkt
        echo $this->bar; //Werkt
    }
}

$fooBar = new Foo();
echo $fooBar->bar; //Public
echo $fooBar->_bar; //Private werkt dus niet

Enkel $this geeft nog niet aan dat het binnen de private of protected scope valt. Dan kan het nog net prima een public var zijn.

[Hipska]

En nog even melden dat je ook gewoon echo $equipe; kunt doen. __toString is een magic method.

[Gwaihir]

ZpAz in "[PHP] OO beginersvraagje"
Enkel $this geeft nog niet aan dat het binnen de private of protected scope valt. Dan kan het nog net prima een public var zijn.

Die _ is een erfenis van PHP4 waarin private en protected niet bestond. Op die manier gaf je aan dat het als niet public beschouwd moest worden. In PHP5 is dat niet meer nodig daar het in de taal zit.

Publieke class variables hoor je eigenlijk nooit te hebben. Dus, wil je die toch onderscheiden, dan zou ik juiste die enkele publieke een prefix geven. (Iets als "todo" wellicht? )

[ Voor 4% gewijzigd door Gwaihir op 18-08-2010 10:10 ]

[ZpAz]

Gwaihir schreef op woensdag 18 augustus 2010 @ 10:09:
[...]

Die _ is een erfenis van PHP4 waarin private en protected niet bestond. Op die manier gaf je aan dat het als niet public beschouwd moest worden. In PHP5 is dat niet meer nodig daar het in de taal zit.

Publieke class variables hoor je eigenlijk nooit te hebben. Dus, wil je die toch onderscheiden, dan zou ik juiste die enkele publieke een prefix geven. (Iets als "todo" wellicht? )

True, in andere talen moet je vaak expliciet getters en setters aanmaken. Bij PHP heb je wel 'magic' methods __get() en __set() waarmee je wel bepaalde functionaliteit kan uitvoeren als je dat wil als een variabele 'geset' of 'geget' wordt.

[Voutloos]

Je hebt gelijk, maar toch kan die conventie geen kwaad.

[NMe]

HuHu schreef op woensdag 18 augustus 2010 @ 08:36:
Als je PHP 5 gebruikt zou ik ook type hinting gaan gebruiken. Verder zou ik een bepaalde coding standard gaan hanteren, want het is vrij gebruikelijk om protected/private variabelen met een underscore (_) te prefixen. Dat alles voor het overzicht.

Zo gebruikelijk is het ook weer niet hoor. Ik kan zelf dat soort prefixes bijvoorbeeld niet lijden. Ze slaan nergens op, omdat een goeie editor ze toch niet als optie ziet en je waarschuwt als je ze wil gebruiken. Bovendien gebruik je als het goed is inderdaad meer private en protected members dan public members, dus zou ik áls je dan al dingen wil prefixen, dat inderdaad vooral bij public variabelen doen omdat die de uitzondering zijn, niet de afgeschermde variabelen.

[HuHu]

NMe schreef op woensdag 18 augustus 2010 @ 12:42:
[...]

Zo gebruikelijk is het ook weer niet hoor. Ik kan zelf dat soort prefixes bijvoorbeeld niet lijden. Ze slaan nergens op, omdat een goeie editor ze toch niet als optie ziet en je waarschuwt als je ze wil gebruiken. Bovendien gebruik je als het goed is inderdaad meer private en protected members dan public members, dus zou ik áls je dan al dingen wil prefixen, dat inderdaad vooral bij public variabelen doen omdat die de uitzondering zijn, niet de afgeschermde variabelen.

In PHP is het niet direct noodzakelijk, maar in andere talen is het wel handig. Het kan volgens mij geen kwaad om het je aan te leren.

PHP:

class Iets {
  private $x;

  public function getX() {
    return $this->x;
  }

  public function setX($x) {
    $this->x = $x;
  }
}

Java:

public class Iets {
  private int x;

  public int getX() {
    return x;
  }

  public void setX(int x) {
    this.x = x;
  }
}

In PHP moet je de $this-> gebruiken, maar in Java is dat niet per se noodzakelijk. Het prefixen met een _ maakt het dan wel duidelijker of het om een variabele binnen de class-scope of function-scope gaat (en je kunt niet ook niet vergissen). In C++ wordt het dan nog leuker:

C++:

class Iets {
private:
  int x;

public:
  int getX() const { return x; }

  void setX(int xx) { x = xx; }
}

Opeens moet je de parameter van setX een andere naam geven, want je kunt hem geen x meer noemen. Dan vind ik het duidelijker als er dit staat:

C++:

void setX(int x) { _x = x; }

Als je dan jezelf aanleert altijd met een _ te prefixen (of nog beter: te suffixen, want dan heb je geen botsingen met bestaande functies als _main en _exit), dan kan dat zeker geen kwaad.

[NMe]

Zowel in C++ als in Java heb je daar "this" voor en die gebruik ik 1000 keer liever expliciet dan dat ik met underscores moet gaan werken. Daarom: leg niet aan anderen jouw persoonlijke voorkeuren als absolute waarheid op.

[.oisyn]

HuHu schreef op woensdag 18 augustus 2010 @ 15:25:
(of nog beter: te suffixen, want dan heb je geen botsingen met bestaande functies als _main en _exit)

Sinds wanneer kan een lokale variabele of member botsen met een globale functie?

[Gwaihir]

HuHu schreef op woensdag 18 augustus 2010 @ 15:25:
[...]

In PHP is het niet direct noodzakelijk, maar in andere talen is het wel handig. Het kan volgens mij geen kwaad om het je aan te leren.

In PHP moet je de $this-> gebruiken, maar in Java is dat niet per se noodzakelijk. Het prefixen met een _ maakt het dan wel duidelijker of het om een variabele binnen de class-scope of function-scope gaat (en je kunt niet ook niet vergissen).

Volgens die logica zou je variabelen in java dan wellicht ook met een $ moeten beginnen; dat maakt het wel duidelijker dat het om een variabele gaat. Ondertussen ben je erg ver van de standaard java code convention van Sun, welke camelCasing voorschrijft, zonder underscores. Die conventie vind ik een stuk belangrijker.

Bovendien gebruik ik liever (in zowel PHP als Java) mijn editor (eclipse), die het allemaal netjes weergeeft d.m.v. syntax highlighting met aparte kleuren voor lokale en klasse variabelen, dan een conventie waarvan ik als ik code lees nooit zeker weet of hij geheel gevolgd is of niet.

[HuHu]

.oisyn schreef op woensdag 18 augustus 2010 @ 17:12:
[...]

Sinds wanneer kan een lokale variabele of member botsen met een globale functie?

Ik heb dat uit de JSF Air Vehicle C++ Coding Standards:

AV Rule 47:
Identifiers will not begin with the underscore character ‘_’.

Rationale: ‘_’ is often used as the first character in the name of library functions (e.g. _main, _exit, etc.) In order to avoid name collisions, identifiers should not begin with ‘_’.

Ben het zelf nog niet in de praktijk tegengekomen eerlijk gezegd, dus heb er geen ervaring mee.