-
Юбилейный DevConfX пройдет 21-22 июня в Москве. Как всегда — Вы решаете, кто попадет в программу секции Backend — голосуйте за интересные доклады
Zend debugger и Pear не дружат
- Автор темы Rusmikle
- Дата начала
korchasa
LIMB infected
Полностью согласен. Но такие флажки и помогают жить. Правильное название метода тоже флажек, но как много он даетАвтор оригинала: jonjonson
korchasa, я понимаю, что тема несколько другая, но всё же хочу ответить на
private в php - это нечто декларотивное и не более того. Да, это флажок, но программисты на php не волки и не станут на него реагировать, если увидят простое решение с возможностью замены private на public.
cDLEON
От критики кода воздержусь, разговор сейчас не об этом. Все таки инкапcуюляцию вы отрицаете полностью, включая инкапсуляцию данных (__set/__get вас спасет?).
В классе XMLparser есть метод Load:
PHP:
function Load($filename)
{
if(!($file=@file_get_contents($filename)))
{
echo "file:<b>".$filename."</b> not found!";
return false;
}
$this->Parse($file);
}
PHP:
$parser = new XMLparser();
$el = $parser->Load('structure.xml');
unset($parser);
//дальше идет работа с элементами
PHP:
$el = XMLparser::parseFile('structure.xml');
//дальше идет работа с элементамиДля примера private-методов код слишком простой. Можно было бы, конечно, сделать надуманный пример, но судя по дискуссии, я знаю чем бы это закончилось.
cDLEON
Онанист РНРСlub
korchasa
Я же сказал, что скинул пример не проработанный, сырой, потому что не хотелось рыться в своих кучах наработок.
Руки ни как не дойдут навести там порядок =(
А где вы у меня в коде видели __set,__get ? Эта хрень, кстати, на пхп4 и пхп5 по разному работает (точно не помню что именно).
-~{}~ 30.09.07 21:19:
Я же сказал, что скинул пример не проработанный, сырой, потому что не хотелось рыться в своих кучах наработок.
Руки ни как не дойдут навести там порядок =(
А где вы у меня в коде видели __set,__get ? Эта хрень, кстати, на пхп4 и пхп5 по разному работает (точно не помню что именно).
Увидев, что обосновать ни чего не получается, мы перешли к обычной критике кода 1 летней давности 8) Логичное действие =)
Можно было бы, конечно, выложить свой пример, о великий гуру. И вообще, харе выражаться умными словами. Это мозгов не прибавляет 8).
-~{}~ 30.09.07 21:19:
Количества говнокода только увеличивается если говнокодер начнёт тыкать свои приватные методы везде 8)
korchasa
LIMB infected
В вашем сообщении не было ни слова о том, что он сырой
Перегрузка свойст - единственная возможность добавить "независимости" XMLObject'у в внутреннем представлении данных (не переписывая их обоих), т.к. во всем коде XMLPares'а вы напрямую работаете с его данными:
PHP:
$el=new XMLobject();
unset($el->childs);
unset($el->attributes);
unset($el->parent);Критика касалась все той же инкапсуляции, про остальное даже говорить не хочется.
Этому в институте учат...
Совершенно голосновное утверждение, т.к. смена с private на public никаких трудностей не вызывает. Наоборот гораздо сложнее, но зато очень редко приходится.
cDLEON
Онанист РНРСlub
tf
Хм. Вообще то мы с korchasa ругаемсо по этому поводу.
К чему тогда все твои выкрики?
korchasa
Хм. Вообще то мы с korchasa ругаемсо по этому поводу.
К чему тогда все твои выкрики?
korchasa
Почему. Есть ещё одна. Кода правда побольше станет. Зато появится обратная совместимость с ПХП4.
Когда работаешь с чужим кодом, без твоей супир-пупир ИДЕ(который обрезает приватные методы) натыкаться на чужие, ненужные приватные методы просто наколяет.
korchasa
LIMB infected
Автор оригинала: cDLEON
korchasa
Сам не мазохист, и вам не советую
Т.е. читать код:
вам приятнее, чемPHP:$link=&$this->prev->childs; $count=count($link); if($count && $link[$count-1]->type=="text")
?PHP:$previous = $this->getPreviousElement(); if($previous->hasChilds() && $previous->isText())
Вот getPrevious вообще не надо видеть снаружи, т.к. он зависит от состояния парсера, и вряд ли пригодится кому-нибудь.
А насчет хинтинга, который вы не пользуете, так это вы зря