Как правильно работает __get() ?

[niko42]

Добрый день столкнулся с такой проблемой при реализации класса:

class GET {

    public static $_vars, $_instance;

    /**
    * @return GET
    */
    public static function getInstance(){
        if(!isset(self::$_instance))
            self::$_instance = new GET();
        return self::$_instance;
    }

    public function __construct(){
        if(empty(self::$_vars))
            self::$_vars = $_GET;
        unset($this->pushState);
    }

    public function __get($name){
        return self::$_vars[$name];
    }

    public function __set($name, $value){
        self::$_vars[$name] = $value;
    }

    public function __isset($name){
        return isset(self::$_vars[$name]);
    }

    public function __unset($name){
        unset(self::$_vars[$name]);
    }
}

Вызываем: GET::getInstance()->trulyalya
Получаем: Notice: Undefined index: (ну понятно - нет индекса)
И почему оно так себя ведет?

P.S. использую PHP Version 5.5.17 (настройки самые мин., т.к. компилил руками)

 

[c0dex]

niko42, а ничего, что сингтон используется не так как у тебя? Почему конструктор публичный?

ЗЫ: потому что вызывать надо этот файл с ?trulyalya=xxx

 

[niko42]

niko42, а ничего, что сингтон используется не так как у тебя? Почему конструктор публичный?
ЗЫ: потому что вызывать надо этот файл с ?trulyalya=xxx

Конструктор закроем.
Смысле сингл использует не так, как у меня - не много не понятно.

?trulyalya=xxx Да, но если взять такую конструкцию в доп.:

switch(GET::getInstance()->sort){
                case "recommend" :
                    $sort = "`c`.`position` ASC";
                    break;
                case "like" :
                    $sort = "`p`.`like` DESC";
                    break;
                case "high_price" :
                    $sort = "`p`.`price` DESC";
                    break;
                case "low_price" :
                    $sort = "`p`.`price` ASC";
                    break;
                default:
                    $sort = "`c`.`position` ASC";
                    break;
            }

Т.е. я не хочу писать isset(GET::getInstance()->sort) а потом бежать по свитчу, т.к. не попаду в default

 

[Вурдалак]

niko42,

И почему оно так себя ведет?

По определению.

 

[c0dex]

niko42, ну так допиши проверку, если нет такого ключа - возвращай null в _get

 

[AmdY]

проверку не надо, для isset есть магический __isset, но я думаю здесь ничего не поможет.

 

[niko42]

niko42, ну так допиши проверку, если нет такого ключа - возвращай null в _get

Лишняя операция

проверку не надо, для isset есть магический __isset, но я думаю здесь ничего не поможет.

К сожалению, что да

 

[c0dex]

niko42, правда что ли?

 

[niko42]

niko42, правда что ли?

public function __get($name){
      if(!isset(self::$_vars[$name]))
          return null;
      return self::$_vars[$name];
  }