Шедевры в коде

[ALarik]

Здравствуйте други и подруги !
В данном разделе, хочу стартовать марафон, в котором нет ни победителей, ни проигравших, но есть уйма позитива и тонкий юмор, понимание которого может быть вызвано только у разработчика со стажем, затягивать не буде, начнем с небольших историй нашего уютного чатика:
Мастер:

полчаса не мог понять почему seopro не выводит ссылки на подкатегории
оказалось просто шаблон

$nurl=explode("/", $child['href']);
<a href="<?php echo '/'.end($nurl); ?>"

~~~​

"Удивительный алгоритм" от неменее удивительной команды разработчиков:

~~~
Новая порция, при просмотре модели парсера:

 

[SlaSoft]

$nurl=explode("/", $child['href']);
<a href="<?php echo '/'.end($nurl); ?>"

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

1. Если посмотреть в код генерации canonical (не сеопро)
то что мы увидим для ссылки на категорию?

$this->url->link('product/category', 'path=' . $category_id);

А это? h*p(S)://site/cat1

Что делает пеЙсатель?
он делает
ОТНОСИТЕЛЬНУЮ ссылку и !!!! каноническую

Т.е. уже и протокола нет, а это значит? что переход с http на https может пройти более спокойно

 

[ALarik]

@SlaSoft, если честно и откровенно писать, то где то в 50% случаев ioncube - кодирования, я подозреваю, что скрывают именно такой вот гениальный код, и лишь в 50% - считаю оправданным - защиту от ломки\внедрения в функционал, но это мое скромное мнение, которое основано на анализе немногих раскодирванных файлов, не имею возможности из своей коллекции добавить, так как не собирал такие шедевры ранее, но тем не менее, встречал подобные "ляпы" довольно часто И, осмелюсь высказаться, что сам грешил подобным, когда просматриваю на решения, многогодовой давности или принятые в моменты срочной спешки.

 

[Baco]

@ALarik, не все мы без греха, у меня тоже бывают не самые оригинальные костыли, каюсь :eek:

 

[ALarik]

 

[SlaSoft]

Это правильно, я тоже такое использую

и посмотри код
Все условия через AND
А если их нет, то когда "писать" WHERE?

 

[ALarik]

по синтаксису - логично, а возьми теперь какой то кривой модуль, который делает запрос, например, вот так:

        $filter_data = array(
            'sort'  => 'p.date_added'
        );
        $results = $this->model_catalog_product->getProducts($filter_data);

вроде и проверку прошло на:

if ($data) {

и прогрузило все товары, без отбора (не видел код дальше, но надеюсь есть условие на LIMIT) и получается откуда доп. нагрузка, это конечно утрирую, но есть всякие шедевры

 

[SlaSoft]

WHERE 1 не грузит базу от слов НИКАК

if ($data) {
Есть такое, и даже без лимита - например getLanguages

 

[SlaSoft]

 

[Baco]

обратите внимание на метод autocomplete

 

[Baco]

в одном из кодерских чатов:

ну, если уж так понеслась, то вот высокоуровневое:

инициализировать массив объектов (нож, картошка, рука, кастрюля), методы[брать, ложить, чистить]
свойства: чищенная = (bool) да\нет
алгоритм:
ЕСЛИ картошка != чищенная ТОГДА рука => брать[картошка]
кожура = 100%
{
    нож => чистить
    кожура --
    ЕСЛИ кожура == 0 ТОГДА рука => ложить[картошка]::кастрюля
}