122

I have the following php. However when I see the index.php I get the following error message.

Strict standards: Non-static method Page::getInstanceByName() should not be called statically in /var/www/webworks/index.php on line 12

I am hoping someone can tell me how to fix the problem.

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];
    }
1
  • 17
    Hmm, could it be that you're calling a method statically, and that method isn't defined as static? You know, pretty much exactly what the error says, on the line number it says... Commented Jan 13, 2011 at 19:48

7 Answers 7

197

Your methods are missing the static keyword. Change

function getInstanceByName($name=''){

to

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

if you want to call them statically.

Note that static methods (and Singletons) are death to testability.

Also note that you are doing way too much work in the constructor, especially all that querying shouldn't be in there. All your constructor is supposed to do is set the object into a valid state. If you have to have data from outside the class to do that consider injecting it instead of pulling it. Also note that constructors cannot return anything. They will always return void so all these return false statements do nothing but end the construction.

11
  • 2
    The codes are from this book...packtpub.com/cms-design-using-php-and-jquery/book. I think you should write a book, Gordon. :-)
    – shin
    Commented Jan 13, 2011 at 20:00
  • 6
    @shin Nah, I'd only repeat what others have said better than me before. But that's some really bad code for a book that was released Dec 2010. Do they give any reason for omitting any visibility keywords or not following PEAR coding convention? Let's hope the jQuery and general CMS architecture is more solid.
    – Gordon
    Commented Jan 13, 2011 at 20:26
  • 17
    @dzona that would be ignoring the problems with the code, not fixing it.
    – Gordon
    Commented Dec 4, 2013 at 6:10
  • 1
    Important NOTE: the public keyword is used only in function/variable declarations from within a class. See stackoverflow.com/questions/13341378/…
    – cssyphus
    Commented Feb 6, 2015 at 18:17
  • 1
    @Gordon, just curious -- why do you advocate changing the offending method to static, instead of (re)writing the code to use $p = new Page(); $p->getInstanceByName();?
    – Dennis
    Commented Jul 13, 2016 at 18:35
25

I think this may answer your question.

Non-static method ..... should not be called statically

If the method is not static you need to initialize it like so:

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

Or, in PHP 5.4+, you can use this syntax:

(new ClassName)->method();
3
  • Is (new ClassName)->method(); compatible with PHP 5.3 too?
    – Jeff
    Commented Mar 22, 2016 at 1:20
  • 1
    @Jeff, I'd use (new ClassName())->method();, and I believe it is compatible with PHP from 5 to 7
    – Dennis
    Commented Jul 13, 2016 at 20:01
  • 1
    (new ClassName)->method(); is not compatible with PHP 5.3. I just tried it.
    – Sonny
    Commented Jan 24, 2017 at 16:13
1

If scope resolution :: had to be used outside the class then the respective function or variable should be declared as static

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();
2
  • 1
    Can you provide examples for the OP and all future visitors?
    – B001ᛦ
    Commented Aug 23, 2016 at 11:47
  • <?php 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(); /*Hope this example helps you*/ Commented Aug 23, 2016 at 13:08
1

Try this:

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

It worked for me in a similar case.

1

use className->function(); instead className::function() ;

-1

return false is usually meant to terminate the object creation with a failure. It is as simple as that.

-2

Instead of using the instance with the scope resolution operator :: because it wasn't defined like static function.

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

change it to :

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

And it will work like a charm.

0

Not the answer you're looking for? Browse other questions tagged or ask your own question.