#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. Вызывает ли предупреждение исключение? Если бы я обработал исключение, оно все равно вызвало бы уведомление об отображении?