Подавлять предупреждение о неустановленном свойстве при использовании метода magic __get

#php #class #codeigniter #warnings

#php #класс #codeigniter #предупреждения

Вопрос:

У меня есть библиотека CI, которая загружает переводимый контент из XML-файла в класс, и у класса есть метод magic __get, который проверяет, определено ли это свойство при ссылке на него, возвращая локализованный ключ, если да, или ключевое имя плюс ‘_ #’, чтобы сообщить мне, чтоперевод отсутствует, если нет.

Все круто.

Однако, поскольку все это зависит от свойств объекта, я получаю много предупреждений «notice: undefined и т. Д.», Когда я запускаю debug (E_ALL), и это раздражает. Я не хочу отключать уведомления, но я хочу знать, как отключить это внутри этой конкретной библиотеки (если это возможно). Я мог бы ставить @ перед каждым вызовом класса, но опять же, это тоже довольно ужасно.

Какие-нибудь советы?

Упрощенные фрагменты кода ниже:

 class MY_Translation
{

    function _get_keys($lang) {
        // load xml translations, could split this into different files..

        $translations = new DOMDocument();
        $translations->load($_SERVER['DOCUMENT_ROOT']."/xml/translations.xml");
        if ($translations) {
            $words = $translations->getElementsByTagName("word");
            $count = 0;
            foreach( $words as $word ){

                $name = $word->getAttribute('name');
                $trans = $word->childNodes;

                if ($trans->length > 0) {
                    for($i = 0; $i < $trans->length; $i  ) {
                        $child = $trans->item($i); 

                        if ($child->nodeName == $lang) {
                            $this->$name = $child->nodeValue;
                        }
                    }
                }
            }   
        }
    }

    function __get($key){
        if (property_exists('MY_Translation',$this->$key)) {
            return $this->$key;
        } else {
            return $key."_#";
        }
    }

}
  

XML (просто для справки, чтобы было понятно, что происходит):

 <?xml version="1.0" encoding="UTF-8"?>
<words>
    <word name="thing">
        <en>thing en</en>
        <pt>thing pt</pt>
    </word>
</words>
  

Ответ №1:

Поскольку вы используете $this для хранения переводов, __get вызывается только для несуществующих свойств. Измените его, предупреждение, которое вы получаете, уведомляет вас только о том, что вы делаете что-то неправильно:

 function __get($key)
{
    return $key."_#";
}
  

Это действительно все, что вам нужно сделать.


Вы имели в виду

 $this->t->$key
  

вместо

 $this->$key
  

?

Тогда вам также нужно будет исправить назначение:

 $this->t->$name = ...
  

В общем, когда вы проверяете свойство в __get функции, вы не должны видеть никаких предупреждений. Предупреждения показывают вам, что вы допустили какую-то ошибку в логике своей программы, поэтому они полезны, и вы не должны отключать их даже для короткой части вашего кода. Отключение предупреждений — это не решение, а исправление кода.

Дайте мне знать, если это было полезно.

Комментарии:

1. Извините, я по ошибке оставил этот t = new stdClass. Но вы помогли мне сосредоточиться на том, где срабатывало предупреждение — на ‘property_exists’, а не на __get или ссылке, которая инициирует этот вызов. Указание @ перед property_exists подавило предупреждение.

2. Использование @ для подавления ошибок — плохая практика. Вам следует избегать его использования.

3. Если вы ссылаетесь $this на сохранение переводов как свойств, __get будет вызываться только в том случае, если оно не существует. Так что вам даже не нужно проверять его. Вместо этого просто верните строку, что она не будет существовать. Как написано, вы должны исправить ошибку, а не подавлять ее.

4. @danp: я отредактировал ответ и добавил вам функцию, как вам это должно понадобиться.

5. Вызывает ли предупреждение исключение? Если бы я обработал исключение, оно все равно вызвало бы уведомление об отображении?