Fehlermeldung Strenge Standards: Nicht-statische Methode sollte nicht statisch in PHP aufgerufen werden

Ich habe folgende php. Wenn ich jedoch die index.php sehe, erhalte ich die folgende Fehlermeldung.

Strenge Standards: Nicht-statische Methode Page::getInstanceByName() sollte nicht statisch in /var/www/webworks/index.php in Zeile 12 aufgerufen werden

Ich hoffe jemand kann mir sagen wie ich das Problem beheben kann.

index.php

// { common variables and functions
include_once('ww.incs/common.php');
$page=isset($_REQUEST['page'])?$_REQUEST['page']:'';
$id=isset($_REQUEST['id'])?(int)$_REQUEST['id']:0;
...

// { get current page id
if(!$id){
    if($page){ // load by name
        $r=Page::getInstanceByName($page);
        if($r && isset($r->id))$id=$r->id;
    }
    if(!$id){ // else load by special
        $special=1;
        if(!$page){
            $r=Page::getInstanceBySpecial($special);
            if($r && isset($r->id))$id=$r->id;
        }
    }
}

// { load page data
if($id){
    $PAGEDATA=(isset($r) && $r)?$r : Page::getInstance($id);
}
else{
    echo '404 thing goes here';
    exit;
}
...
...

ww.incs/common.php

<?php
require dirname(__FILE__).'/basics.php';
...
...

ww.incs/basics.php

session_start();
if(!function_exists('__autoload')){
    function __autoload($name) {
        require $name . '.php';
    }
}
...
...

Seite.php

class Page{
    static $instances             = array();
    static $instancesByName     = array();
    static $instancesBySpecial   = array();
    function __construct($v,$byField=0,$fromRow=0,$pvq=0){
        # byField: 0=ID; 1=Name; 3=special
        if (!$byField && is_numeric($v)){ // by ID
            $r=$fromRow?$fromRow:($v?dbRow("select * from pages where id=$v limit 1"):array());
        }
        else if ($byField == 1){ // by name
            $name=strtolower(str_replace('-','_',$v));
            $fname="page_by_name_".md5($name);
            $r=dbRow("select * from pages where name like '".addslashes($name)."' limit 1");
        }
        else if ($byField == 3 && is_numeric($v)){ // by special
            $fname="page_by_special_".$v;
            $r=dbRow("select * from pages where special&$v limit 1");
        }
        else return false;
        if(!count($r || !is_array($r)))return false;
        if(!isset($r['id']))$r['id']=0;
        if(!isset($r['type']))$r['type']=0;
        if(!isset($r['special']))$r['special']=0;
        if(!isset($r['name']))$r['name']='NO NAME SUPPLIED';
        foreach ($r as $k=>$v) $this->{$k}=$v;
        $this->urlname=$r['name'];
        $this->dbVals=$r;
        self::$instances[$this->id] =& $this;
        self::$instancesByName[preg_replace('/[^a-z0-9]/','-',strtolower($this->urlname))] =& $this;
        self::$instancesBySpecial[$this->special] =& $this;
        if(!$this->vars)$this->vars="{}";
        $this->vars=json_decode($this->vars);
    }
    function getInstance($id=0,$fromRow=false,$pvq=false){
        if (!is_numeric($id)) return false;
        if (!@array_key_exists($id,self::$instances)) self::$instances[$id]=new Page($id,0,$fromRow,$pvq);
        return self::$instances[$id];
    }
    function getInstanceByName($name=""){
        $name=strtolower($name);
        $nameIndex=preg_replace('#[^a-z0-9/]#','-',$name);
        if(@array_key_exists($nameIndex,self::$instancesByName))return self::$instancesByName[$nameIndex];
        self::$instancesByName[$nameIndex]=new Page($name,1);
        return self::$instancesByName[$nameIndex];
    }
    function getInstanceBySpecial($sp=0){
        if (!is_numeric($sp)) return false;
        if (!@array_key_exists($sp,$instancesBySpecial)) $instancesBySpecial[$sp]=new Page($sp,3);
        return $instancesBySpecial[$sp];
    }

Ihren Methoden fehlt das static Stichwort. Ändern

function getInstanceByName($name=""){

zu

public static function getInstanceByName($name=""){

wenn Sie sie statisch aufrufen möchten.

Beachten Sie, dass statische Methoden (und Singletons) sind Tod bis zur Prüfbarkeit.

Beachten Sie auch, dass Sie viel zu viel Arbeit im Konstruktor leisten, insbesondere all die Abfragen sollten nicht darin enthalten sein. Alles, was Ihr Konstruktor tun soll, ist, das Objekt in einen gültigen Zustand zu versetzen. Wenn Sie dazu Daten von außerhalb der Klasse benötigen, sollten Sie sie injizieren, anstatt sie zu ziehen. Beachten Sie auch, dass Konstruktoren nichts zurückgeben können. Sie werden immer leer zurückkehren, also all dies return false Anweisungen bewirken nichts anderes, als die Konstruktion zu beenden.

Ich denke, das könnte deine Frage beantworten.

Nicht statische Methode ….. sollte nicht statisch aufgerufen werden

Wenn die Methode nicht statisch ist, müssen Sie sie wie folgt initialisieren:

$var = new ClassName();
$var->method();

Oder Sie können in PHP 5.4+ diese Syntax verwenden:

(new ClassName)->method();

Wenn die Bereichsauflösung :: außerhalb der Klasse verwendet werden musste, sollte die entsprechende Funktion oder Variable als statisch deklariert werden

class Foo { 
        //Static variable 
        public static $static_var="static variable"; 
        //Static function 
        static function staticValue() { return 'static function'; } 

        //function 
        function Value() { return 'Object'; } 
} 



 echo Foo::$static_var . "<br/>"; echo Foo::staticValue(). "<br/>"; $foo = new Foo(); echo $foo->Value();

Versuche dies:

$r = Page()->getInstanceByName($page);

Bei mir hat es in einem ähnlichen Fall funktioniert.

benutze Klassenname->Funktion(); stattdessen className::function() ;

return false ist normalerweise dazu gedacht, die Objekterstellung mit einem Fehler zu beenden. So einfach ist das.

Anstatt die Instanz mit dem Bereichsauflösungsoperator :: zu verwenden, da sie nicht als statische Funktion definiert wurde.

$r=Page::getInstanceByName($page);

ändere es zu:

$r=Page->getInstanceByName($page);

Und es wird wie ein Zauber funktionieren.