OOP mit PHP 5(.5)

The Scout · 20. Januar 2014

Heyho @ll,
ich scheitere grad erfolgreich an der OO-Programmierung in PHP (bzw. allgemein OOP ). Die größte Schwierigkeit bereitet mir im Moment aber die Vererbung. Ich kriegs nicht gebacken

Mein Hauptproblem steht als Kommentar im Code

riotapi.inc.php

PHP

<?php
class RiotApi {
    /* für jede Abfrage benötigte Attribute */
    protected $Base_Url = 'http://prod.api.pvp.net/api/lol/';
    protected $Api_Key = 'd1es-is7-3in-4pi-k3y';
    protected $region = 'euw';
}




class Summoner extends RiotApi {
    private $ExtUrl = '/v1.3/summoner/';

    function getNames(array $SummonerIds) {
        /* Hier ist mein Problem:
         * Ich möchte die unten abgefragte $url aus verschiedenen Dingen zusammenbauen
         *
         * Base_Url (parent) . region (parent) . Ext_Url (this) . $SummonerIds (als Array gegeben, kommasepariert in die Url zu bauen) . '/name' (benötigter Anhang) . '?api_key=' . Api_Key (parent)
         * 
         * Beispiel:
         * http://prod.api.pvp.net/api/lol/euw/v1.3/summoner/123456,654321/name?api_key=d1es-is7-3in-4pi-k3y
         *
         * Meine Frage: Wie geht das mit dem Einbinden der vererbten Attribute???
         */




        /* Das Array umbauen krieg ich hin ^^ */
        $ids = implode(',',$SummonerIds);




        /* API Abfrage und Ergbnisaufbereitung */
        $names_json = file_get_contents($url);
        $names = json_decode($names_json, TRUE);

        /* Ergebnisausgabe */
        return $names;
    }




    /* Hier stehen andere Methoden für Abfragen den Summoner betreffend */
}




/* Hier stehen andere Klassen für andere Abfragen */
?>

Alles anzeigen

Abfrage sollte dann so laufen

PHP

<?php
require("riotapi.inc.php");




$summoner = new Summoner();




$ids = array(123456,654321,24680,13579)
$namen = $summoner->getNames($ids)

Alles anzeigen

Ich will das halt so machen, da es für die API viele verschiedene Infos gibt, die alle eine andere URL ansprechen. ABER die Basis der URLs bleibt jedes Mal gleich.

cottton · 20. Januar 2014

einfach $this->

Du erbst mit der subclass die Eigenschaften (sofern nicht private).

siehe Methode test()

PHP

<?php
class RiotApi {
    /* für jede Abfrage benötigte Attribute */
    protected $Base_Url = 'http://prod.api.pvp.net/api/lol/';
    protected $Api_Key = 'd1es-is7-3in-4pi-k3y';
    protected $region = 'euw';




    protected static $test_val = 'das hier ist static';
    const TEST_CONST = 'const können nicht protected (ect) werden';
}




class Summoner extends RiotApi {
    private $ExtUrl = '/v1.3/summoner/';




    public function test()
    {
        echo '
            <pre>
                $this->Base_Url : '.$this->Base_Url.'
                parent::$Base_Url : == Fatal error: Access to undeclared static property: RiotApi::$Base_Url
                parent::$test_val : '.parent::$test_val.'
                parent::TEST_CONST : '.parent::TEST_CONST.'
                $this::TEST_CONST : '.$this::TEST_CONST.'
            </pre>
        ';
    }
    function getNames(array $SummonerIds) {
        /* Hier ist mein Problem:
         * Ich möchte die unten abgefragte $url aus verschiedenen Dingen zusammenbauen
         *
         * Base_Url (parent) . region (parent) . Ext_Url (this) . $SummonerIds (als Array gegeben, kommasepariert in die Url zu bauen) . '/name' (benötigter Anhang) . '?api_key=' . Api_Key (parent)
         *
         * Beispiel:
         * http://prod.api.pvp.net/api/lol/euw/v1.3/summoner/123456,654321/name?api_key=d1es-is7-3in-4pi-k3y
         *
         * Meine Frage: Wie geht das mit dem Einbinden der vererbten Attribute???
         */




        /* Das Array umbauen krieg ich hin ^^ */
        $ids = implode(',',$SummonerIds);




        /* API Abfrage und Ergbnisaufbereitung */
        $names_json = file_get_contents($url);
        $names = json_decode($names_json, TRUE);




        /* Ergebnisausgabe */
        return $names;
    }




    /* Hier stehen andere Methoden für Abfragen den Summoner betreffend */
}




/* Hier stehen andere Klassen für andere Abfragen */







$a = new Summoner();
echo $a->test();
?>

Alles anzeigen

Würde noch was vorschlagen:
da sich die class RiotApi nicht zu ändern scheint, würde ich diese static aufbauen.
Und ich hab mir angelesen, dass man Eigenschaften nie direkt von einer anderen Stelle aus ansprechen sollte.
Also innerhalb der gleichen Klasse : $this->eigenschaft
aber von "extern" : $this->getEigenschaft()

Somit kannst Du jeder Zeit die Rückgabewerte kontrollieren, wenn zB irgentwas neues hinzu kommt.
Bsp:

PHP

<?php
class RiotApi {
    /* für jede Abfrage benötigte Attribute */
    private static $Base_Url = 'http://prod.api.pvp.net/api/lol/';
    private static $Api_Key = 'd1es-is7-3in-4pi-k3y';
    private static $region = 'euw';




/* angenommen das hier war die "alte" methode zur ausgabe:
    public function getBaseUrl()
    {
        return self::$Base_Url;
    }
*/
/*
später fällt dir plötzlich ein, dass (einfaches Bsp) beim testen auf localhost etwas angehängt werden soll.
um nun nicht alle anfragen an diese klasse zu ändern, brauchst du hier nur die methode ändern:
*/
    public function getBaseUrl()
    {
        if($_SERVER['SERVER_NAME'] == 'localhost'){
            return self::$Base_Url.'?x=y';
        }
        return self::$Base_Url;
    }
    public function getApiKey()
    {
        return self::$Api_Key;
    }
    public function getRegion()
    {
        return self::$region;
    }
}




class Summoner extends RiotApi {
    private $ExtUrl = '/v1.3/summoner/';




    public function test()
    {
        echo '
            <pre>
                $this->getBaseUrl() : '.$this->getBaseUrl().'
                $this->getApiKey() : '.$this->getApiKey().'
                $this->getRegion() : '.$this->getRegion().'
            </pre>
        ';
    }
}
$a = new Summoner();
echo $a->test();
?>

Alles anzeigen

The Scout · 20. Januar 2014

Hey, danke für die schnelle Antwort. Ich war da einfach zu kompliziert und hab $elternklasse->attribut gemacht

Jetzt klappt die grundlegende API-Abfrage schonmal. Hier ein Beispiel (mit Champs):

riotapi.inc.php

PHP

class RiotApi {
    /* für jede Abfrage benötigte Attribute */
    protected $Base_Url = 'http://prod.api.pvp.net/api/lol/';
    protected $Api_Key = 'd1es-is7-3in-4pi-k3y';
    protected $region = 'euw';

    protected function getExtURL($ext) {
        return $this->Base_Url . $this->region . $ext;
    }
}




class Champion extends RiotApi {
    private $Ext_Url = '/v1.1/champion';

    function getChamps($f2p) {
        $url = $this->getExtURL($this->Ext_Url);
        $url .= ($f2p) ? '?freeToPlay=true&api_key=' : '?api_key=';
        $url .= $this->Api_Key;

        $champs_json = file_get_contents($url);
        $champs = json_decode($champs_json, TRUE);

        return $champs;
    }
}

Alles anzeigen

test.php

PHP

<?php
include("riotapi.inc.php");




echo '<!doctype html>




<html lang="de">




<head>
	<meta charset="UTF-8" />
	<title>League of Legends Champions</title>
</head>




<body>';




$champs = new Champion();
$champinfo = $champs->getChamps(false); //Hier auch gerne mal durch ein bool'sches "true" ersetzen und gucken was passiert ;)




foreach ($champinfo["champions"] as $champ) {

	echo '<img src="http://ddragon.leagueoflegends.com/cdn/3.15.5/img/champion/' . $champ["name"] . '.png" title="' . $champ["name"] . '" /> ';
}
echo '</body>




</html>';
?>

Alles anzeigen

Zwei Dateien im selben Ordner und unter XAMPP oder so ausführen.

Allerdings bin ich noch nicht zufrieden, mit der Art, wie die URL generiert wird Hat da wer eine Lösung, wie man das noch verallgemeinern könnte

Roland · 20. Januar 2014

vll in deiner RiotApi eine Methode:

PHP

protected function sendRequest($params = '?'/*, $this->Ext_Url könntest auch als parameter übergeben */){
        $url = $this->getExtURL($this->Ext_Url) . $params . '&api_key=' . $this->Api_Key;

        $response_json = file_get_contents($url);
        return json_decode($response_json, TRUE);
}

cottton · 20. Januar 2014

$params = '?'
wenn nix reingegeben wird, dann kommt sowas raus:

?&api_key=...

EDIT: konnts nicht lassen
(btw : eine Klasse pro File egtl)

riotapi.inc.php

PHP

<?php







class RiotApi
{
    /*private eigenschaften : nur die klasse selbst darf diese direkt ansprechen*/
    private static $Base_Url = 'http://prod.api.pvp.net/api/lol/'; /*default*/
    private static $Api_Key = '****'; /*default*/
    private static $region = 'euw'; /*default*/




    /*die "gets" und "sets" ... (könnten auch alle protected sein ...)*/
    public static function setBaseUrl($val)
    {
        /*evtl check ob $val ok ist ...*/
        self::$Base_Url = $val;
        return true;
    }
    public static function setApiKey($val)
    {
        /*evtl check ob $val ok ist ...*/
        self::$Api_Key = $val;
        return true;
    }
    public static function setRegion($val)
    {
        /*evtl check ob $val ok ist ...*/
        self::$region = $val;
        return true;
    }
    public function getBaseUrl()
    {
        return self::$Base_Url;
    }
    public function getApiKey()
    {
        return self::$Api_Key;
    }
    public function getRegion()
    {
        return self::$region;
    }




    protected function getURL($ext='',$f2p=false)
    {
        $url = self::getBaseUrl() . self::getRegion() . $ext;
        $param = false;
        if($f2p){
            $param[] = 'freeToPlay=true';
        }
        if( ($key = self::getApiKey()) ){
            $param[] = 'api_key=' . $key;
        }
        if(false /*noch ein param ...*/){
            $param[] = 'soundso';
        }
        $url .= ($param) ? '?' . implode('&',$param) : '';
/*test:*/test($url,'debug : url');
        return $url;
    }
    protected function sendRequest($ext='',$f2p=false)
    {
        $response_json = file_get_contents(self::getURL($ext,$f2p));
        if($response_json !== false){
            return json_decode($response_json,true);
        }
        return false;
    }
}




class Champion extends RiotApi
{
    private $Ext_Url = '/v1.1/champion'; /*default*/




    /*die "gets" und "sets" ...*/
    public function setExtUrl($val)
    {
        /*evtl check ob $val ok ist ...*/
        $this->Ext_Url = $val;
        return true;
    }
    public function getExtUrl()
    {
        return $this->Ext_Url;
    }




    public function getChamps($f2p)
    {
        $data = $this->sendRequest($this->getExtUrl(),$f2p);
        return $data;
    }
}













function test($val,$title='')
{
    $br = '<br />';
    echo '<pre># <b>debug</b>: <u>'.$title.'</u>'.$br;
    echo '<i>type:</i> '.gettype($val).$br.'<i>val:</i> ';
    if(is_bool($val)){
        if($val === true){
            echo 'true';
        }else{
            echo 'false';
        }
    }elseif(is_null($val)){
        echo 'NULL';
    }elseif(!is_numeric($val)and empty($val)){
        echo 'empty';
    }elseif(is_resource($val)){
        echo get_resource_type($val);
    }else{
        print_r($val);
    }
    echo $br.'#</pre>';
}







?>

Alles anzeigen

OOP mit PHP 5(.5)

Jetzt mitmachen!

Teilen