DexizeR Sep 25 2008 at 15:26

Rss grabber для DLE жжот

1 min

3.6K

Delirium coding

+38

Comments 41

Bobos Sep 25 2008 at 15:34

Может быть индус переопределил true и использует её в качестве переменной?

Vas3K Sep 25 2008 at 17:44

в РНР переменные начинаются с $. Без «бакса» только константы. true — глобальная константа.

FeoOne Sep 25 2008 at 22:11

define('true', false); вот и переопределена глобальная константа

DexizeR Sep 25 2008 at 22:27

define('true', false);

if(! true)
echo 'работает';
else
echo 'не работает';

выводит «не работает». PHP не дает переопределять предопределенные(глобальные как тут выражаются) константы.

lasc Sep 26 2008 at 01:19

Совершенно не удивлюсь, если под какойто платформой будет выводить «работает»

что то типа такого, работает под виндой/линухом по разному
${1}=«s»;
$a=1;
if($$a==${1}) echo 'работает';

p01nt Sep 26 2008 at 03:19

а вы уверенны что отличается только платформа, а не версия php?

Bobos Sep 26 2008 at 02:04

2 Vas3K: либо у вас нет чувства юмора, либо вы плохо готовите индусов — они и не на такое способны.

pietrovich Sep 25 2008 at 15:45

скорее всего это результат работы «механического оптимизатора». вполне возможно что раньше вместо true там стояла переменная, которой еще раньше присвоили TRUE и больше значение в ней не меняли. «Оптимизатор» заменил переменную ее значением, а вот блоки типа if(true) он выкоцывать не умел…

ну или вместо «механики» использовали «search and replace» не удосужившись провести ревизию кода после замены :)

Zordhauer Sep 25 2008 at 17:43

странный способ оптимизации… Тут помоему стоит обратить внимание не на
блоки типа if(true) он выкоцывать не умел
а скорее на вечный цикл while… эт помоему пострашнее ифа)

serega011 Sep 26 2008 at 10:05

Нет там бесконечного цикла, там цикл всегда выполняется однажды.

Zordhauer Sep 26 2008 at 20:02

я просто не вижу насильного выхода из цикла, а логическая операция в описании цикла имеет значение константы…
Конструкция return false для неопределенного цикла мне к сожалению неизвестна

NoN Sep 27 2008 at 16:45

return — выход из текущей функции, в т. ч. сразу из всех возможных циклов.

Zordhauer Sep 27 2008 at 22:23

Спасибо… буду знать… Поидее можно было додуматься… Но не старался видимо=) Теперь все ясно)

Sanchous Sep 25 2008 at 22:48

Как вариант — во время разработки было какое-то «человеческое» условие, от которого решили избавиться, закомментировав само условие и поставив вместо него такую вот «заглушку». Потом про него благополучно забыли, а перед выходом проекта в свет порезали все комментарии.

Sashok Sep 25 2008 at 17:03

это дзен-код, служащий для поднятия кармы

Joka Sep 25 2008 at 21:39

хм… а как же тогда работает граббер ?? он волшебный?

Vas3K Sep 25 2008 at 21:50

он просто не работает: D

wdk Sep 25 2008 at 21:58

Хабр порвал Вам смайлик!

Vas3K Sep 25 2008 at 22:30

UFO landed and left these words here

romanstr Sep 25 2008 at 22:01

надеюсь, Вы их научили писать по-другому

UFO landed and left these words here

FatalErr Sep 26 2008 at 15:20

Обычное дело, часто такое вижу в студенческих прогах: просто механическая запись условия задания в код, когда не выспался да не понимаешь, ещё не то напишешь. А ещё бывают перлы типа

if (l[0]+l[1]+l[2]+l[3]+l[4]=='i'+'n'+'p'+'u'+'t')

вместо strncmp и sizeof вместо strlen :), но это немного из другой оперы…

kossnocorp Sep 25 2008 at 22:02

На самом деле это шутка. Ну правда. Не бывает конченых кретинов которые пишут неплохие CMS.

SenSoft Sep 25 2008 at 22:41

Интересно, как же граббер тогда работает.

cwer Sep 26 2008 at 00:30

а что там не должно работать?
return отрабатывается, цикл прерывается.

zxzxzx Sep 25 2008 at 22:50

вы бы название файла написали, чтобы проверить.

DexizeR Sep 25 2008 at 23:00

у меня это 364 строка в

engine/inc/plugins/rss.classes.php

calg0n Sep 25 2008 at 23:40

это называется обфускация

nchaly Sep 26 2008 at 02:02

Причем, ручная.

prudis Sep 25 2008 at 23:54

Удалось мне как то связаться с этим движком. Я вообще в ужасе остался как он работает, но работает. Хотя я какраз таки и познакомился когда надо было сделать чтобы он работал :)

lasc Sep 26 2008 at 01:22

ИМХО остатки от дебага, забыли почистить

nchaly Sep 26 2008 at 02:06

Точно, это отладка какая-то. Кодревью надо.

А (фантастичная версия) цикл по идее нужен для обработки массива сообщений (или вложенных ошибок, если там такие бывают). Но потом оставили один сообщение, и хитровывернуто сделали один проход в цикле.

p01nt Sep 26 2008 at 03:18

а оно внутри функции лежит или так? просто если в функции, то оно прервет и выполнение бесконечного цикла и функции… а если просто в коде то на «return false;» отстановится выполнение скрипта

no_smoking Sep 26 2008 at 08:15

судя по $this это происходит в методе класса

p01nt Sep 26 2008 at 14:14

действительно… я и не обратил внимание… вот что значит в три ночи каммент писал…

dime Sep 26 2008 at 08:49

Экстремальное программирование, вырожденный случай. И успешно забытый.
Пишем функцию вычисления синуса. Сначала тест:
assert_equals 0, sin(0)
функция:
def sin { 0 }
срабатывает!
Добавляем в тест:
assert_equals 1, sin(PI/2)
ломается, рефакторим…
Тут, видимо, остановились на пол-пути :)

EzS Sep 26 2008 at 11:26

К сожалению (может быть), ДЛЕ зарекомендовал себя как движок для «варезных» помоек, нормальных сайтов на нем очень мало.

P.S. Из DLE и WP выбрал последний, хотя из из проектов работает на дле.

PingMe Sep 26 2008 at 15:01

DLE весь так написан. =)

olekhy Dec 5 2008 at 00:26

А чем плох код?
Работает? Да!!!
А тот кто написал просто умница!
Посмеялся над многими «ботанами»
Желаю Вам не забывайте что вы не роботы а
Люди!

kardens Jul 26 2024 at 22:36

Чисто юзерский интерес через смену технологий:

Этот код представляет собой бесконечный цикл, который всегда будет возвращать false при каждом выполнении итерации. Давайте разберем его пошагово:

Бесконечный цикл: while (true) указывает на то, что цикл будет выполняться бесконечно, так как условие всегда истинно.
Установка сообщения об ошибке:
```
$this->errormsg = 'Connection failed (' . $errno . ')';
$this->errormsg .= ' ' . $errstr;
```
Эти строки кода устанавливают сообщение об ошибке, объединяя строку 'Connection failed (' с переменной $errno, затем добавляют значение $errstr к сообщению об ошибке.
Отладка:
```
$this->debug($this->errormsg);
```
Эта строка вызывает метод debug, передавая ему сообщение об ошибке для отладки.
Возврат значения:
```
if (true)
{
    return false;
}
```
Внутри цикла есть условие if (true), которое всегда истинно, и оно заставляет функцию возвращать false и выходить из текущей итерации цикла. Однако, поскольку это внутри бесконечного цикла, цикл будет повторяться и снова возвращать false.
Цикл начинается заново: После возврата false, цикл начинает новую итерацию и снова выполняет те же действия.

Итог

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