PHP Shorthand & meer..

Over extend:

Stel je hebt 2 classes. Een genaamd 'auto' en de ander genaamd 'wiel'.
Je kunt wel doen wiel extend auto. Een auto heeft ook eigenschappen van een wiel. Het omgekeerde kan dan weer niet, want een wiel heeft niet alle eigenschappen van een auto.

Over namespaces:

Ik heb ze zelf nog nooit gebruikt, maar ze zijn handig als je grote projecten hebt. Je kunt dan de verschillende onderdelen van je project onderverdelen in namespaces.

Het kan wel in één regel maar waarom zou je dit willen? Je code wordt er heel onduidelijk van.

dan zou ik het nog eerder zo doen;

<?php
public function getProduct()
{
 if($select->rowCount() == 0)
 {
 return 'This product does not exist!';
 }


 return $this->product = $select->fetchAll(PDO::FETCH_OBJ);
}

of zo

<?php
class Select
{
 public function rowCount()
 {
 return 0;
 }
}


function getProduct(Select $select)
{
 if($select->rowCount() == 0)
 {
 throw new Exception("The requested product does not exist.");
 }


 return $this->product = $select->fetchAll(PDO::FETCH_OBJ);
}


try
{
 $select = new Select;
 getProduct($select);
}
catch (Exception $e)
{
 echo $e->getMessage();
}

Qua classes zit je wel in de richting ja. Hoe je het zou kunnen zien, je hebt Users en Administratoren.
Maar een administrator is net zo goed een gebruikers, alleen heeft deze extra functionaliteit rechten.

dan krijg je zo iets

<?php


class User
{
 protected $gebruikersNaam;


 function __construct($gebruikersNaam)
 {
 $this->gebruikersNaam = $gebruikersNaam;
 }


 public function getGebruikersNaam()
 {
 return $this->gebruikersNaam;
 }


}


class Admin extends User
{
 public function verwijderAccount()
 {
 // Doe iets om het account te verwijderen.
 }
}


$user = new User('DirkZz');
$admin = new Admin('DirkZz');


// Dit kan
$admin->verwijderAccount(); // Kan
$admin->getGebruikersNaam(); // Kan
$user->getGebruikersNaam(); // Kan
$user->verwijderAccount();  // Kan niet methode verwijderAccount bestaat niet.

(Het forum vernaggelt al mijn inspringingen, sorry daarvoor)

Wat betreft namespaces, deze gebruik je om bijv. Class namen meerdere keren te gebruiken in je code en om deze op een logische wijze te structureren.

Je maakt dan een folderstructuur aan die er zo uit kan zien;

ToonData/Csv/Genereer.php
ToonData/Xml/Genereer.php

met als code voor die 2 php bestanden bijv;

<?php namespace ToonData\Csv;


class Genereer 
{
    public function toon()
    {
        
    }
}

<?php namespace ToonData\Xml;


class Genereer 
{
    public function toon()
    {
        
    }
}

En op de plek waar je het gebruikt;

<?php
use ToonData\Xml\Genereer AS GenereerXML;
use ToonData\Csv\Genereer AS GenereerCSV;


$xmlGenerator = new GenereerXML();
$csvGenerator = new GenereerCSV();


$xmlGenerator->toon();
$csvGenerator->toon();

( Zoals het in dit voorbeeld is gebruikt is het ook handig zijn om het tegen een interface aan te gooien )

Citaat van Victor

Over extend:

Stel je hebt 2 classes. Een genaamd 'auto' en de ander genaamd 'wiel'.
Je kunt wel doen wiel extend auto. Een auto heeft ook eigenschappen van een wiel. Het omgekeerde kan dan weer niet, want een wiel heeft niet alle eigenschappen van een auto.

Over namespaces:

Ik heb ze zelf nog nooit gebruikt, maar ze zijn handig als je grote projecten hebt. Je kunt dan de verschillende onderdelen van je project onderverdelen in namespaces.

Toon Meer

Viktor, hier moet ik je tegenspreken. Een auto inherit(extend) niet van wiel, een auto heeft een wiel (has-a) ipv (is-a). Ipv Inheritance te gebruiken (aka extenden), ga je hiervoor composition gebruiken (Wiel objecten als members van een auto geven). Een beter voorbeeld is 'Voertuig', wat een base class zou zijn, eventueel zelf een 'abstract class' (niet zeker of dit toepasselijk is in PHP, te lang geleden), met als subclassen Auto, vrachtwagen, trein enz. Dit zijn allemaal soorten voertuigen, die voldoen aan wat een auto moet kunnen doen.

Qua namespaces is de uiteg van DirkZz goed!
Namespaces worden inderdaad zo goed als standaard gebruikt bij 'grotere' projecten. Dit is zeker handig in het geval wanneer je libraries gaat includen (dit is toch zo bij C++), de kans bestaat namelijk dat deze dezelfde classe namen gebruiken. Door eigen namespaces te introduceren gaat de kans op ambiguïteit naar beneden.

Terwijl jij het typt vind jij het handig

Maar wat als je het over een paar maand terug moet lezen omdat er een fout in zit? Of wanneer een andere programmeur er naar moet kijken omdat er iets mee is?

Wat is dan beter denk je? Jouw voorbeeld of iets als dit;

@Quitta

Oeps, zie het nu! Mijn fout

Citaat van Frenzo.Brouwer

Hoe gebruik ik PHP shorthand?

Code

if($select->rowCount() == 0){
                return 'This product does not exist!';
            } else {
                return $this->product = $select->fetchAll(PDO::FETCH_OBJ);
            }
            
            /* Wat dus ook kan in een regel
             * Alleen klopt mijn regel niet
             * wat doe ik fout?
             */
            
            return ($select->rowCount() == 0) ? 'This product does not exist!' : $this->product = $select->fetchAll(PDO::FETCH_OBJ);

Toon Meer

Je fout is return $this->product = $select->fetchAll(PDO::FETCH_OBJ);

je code moet dit zijn:

$this->product = ($select->rowCount() == 0) ? 'This product does not exist!' : $select->fetchAll(PDO::FETCH_OBJ);

@blaxus dit topic is bijna een jaar oud

Daarbij is het wellicht syntactisch correct wat je doet, maar ik zou daar wel wat semantische vragen bij hebben :).

Er staat $this->product. Dan verwacht ik eigenlijk precies één resultaat. Een fetchAll zou dan eigenlijk misplaatst zijn want dit levert dan mogelijk meerdere resultaten op.

Wat ik zou doen is controleren of er precies één resultaat is (rowCount() == 1) en dan dit ene resultaat retourneren (met simpelweg fetch()). Alle andere resultaten / aantallen zijn naar alle waarschijnlijkheid incorrect.

Ook zou ik geen string als foutmelding retourneren, maar simpelweg false retourneren (wat meestal aangeeft dat er iets mis is) of een exception throwen.

Wauw al een jaar. Ik had dat dus compleet niet door. Ik dacht dat ik op een notificatie had geklikt.
Wel mijn fout waarschijnlijk.

Ik heb de code gekopieerd zoals die aangegeven was in de topic starter.