Messaggio di errore Standard rigorosi: il metodo non statico non dovrebbe essere chiamato staticamente in php

Ho il seguente file php. Tuttavia, quando vedo index.php ricevo il seguente messaggio di errore.

Standard rigorosi: il metodo non statico Page :: getInstanceByName () non dovrebbe essere chiamato staticamente in /var/www/webworks/index.php alla riga 12

Spero che qualcuno possa dirmi come risolvere il problema.

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';
    }
}
...
...

page.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];
    }

Nei tuoi metodi manca la staticparola chiave . Modificare

function getInstanceByName($name=''){

per

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

se vuoi chiamarli staticamente.

Si noti che i metodi statici (e Singletons ) sono morte alla testabilità .

Nota anche che stai facendo troppo lavoro nel costruttore, specialmente tutte quelle query non dovrebbero essere lì. Tutto ciò che il tuo costruttore dovrebbe fare è impostare l'oggetto in uno stato valido. Se devi avere dati dall'esterno della classe per farlo, considera di iniettarli invece di estrarli. Notare inoltre che i costruttori non possono restituire nulla. Torneranno sempre nulle, quindi tutte queste return falsedichiarazioni non fanno altro che porre fine alla costruzione.

Penso che questo possa rispondere alla tua domanda.

Metodo non statico ..... non dovrebbe essere chiamato staticamente

Se il metodo non è statico, è necessario inizializzarlo in questo modo:

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

Oppure, in PHP 5.4+, puoi usare questa sintassi:

(new ClassName)->method();

Prova questo:

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

Ha funzionato per me in un caso simile.

usa className-> function (); invece className :: function ();

return falsedi solito ha lo scopo di terminare la creazione dell'oggetto con un errore. È così semplice.

Se la risoluzione dell'ambito :: doveva essere utilizzata al di fuori della classe, la rispettiva funzione o variabile dovrebbe essere dichiarata come statica

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();

Invece di utilizzare l'istanza con l'operatore di risoluzione dell'ambito :: perché non è stata definita come funzione statica.

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

cambiarlo in:

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

E funzionerà come un fascino.