Говнокод

Здесь я хочу поделиться с вами тремя примерами неадекватного кода. И в то же время постараюсь разобрать и классифицировать каждый случай. Тем самым расскажу не только «что такое плохо?», но и «почему?»

Вот такой интересный код можно встретить в HttpClient'e Rss Grabber'а для DLE

while (true)
{
$this->errormsg = 'Connection failed (' . $errno . ')';
$this->errormsg .= ' ' . $errstr;
$this->debug ($this->errormsg);
if (true)
{
return false;
}
}


мне так и не удалось разгадать тайный смысл этого инопланетного послания .))

Говорят что программистам из Индии деньги платят за количество строчек кода которые они написали(редко обращая внимание на качество).
Вот пример одной из конструкций:
if ($x == 0) {
// Do 1....
}
elseif ($x!=0) {
// Do 2.....
}
else {
//
// А вот здесь чистые деньги :)
//
}



P.S. В каждой шутке есть дуля шутки.

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

Наткнулся на интересную реализацию капчи.
По привычке стал дергать картинку курсором в опере, каково же было удивление когда изображение оказалось текстом.


Ссылка попала в руки по наводке Alert.

Вам нравится NetCat? Нам с NickMitin тоже — его реализация достойна призовых мест в шитпараде.

Посмотрим что внутри?
if ($f_Checked && $admin_mode)
{
$IsChecked = $f_Checked + 0;
}
WTF?!..

1.

string[] strs = new List<string>().ToArray();* This source code was highlighted with Source Code Highlighter.

2.

class MyClass
{
   int a;
   public MyClass(int a)
   {
      try
      {
         this.a = a;
      }
      catch (Exception ex)
      {
         logger.Error(ex);
      }
   }
} * This source code was highlighted with Source Code Highlighter.

Правильная подстановка нуля перед месяцами:

//dt — инициализированный объект Date()
$('declinedate').value = dt.getDate() + '.' + («00»+(dt.getMonth()+1)).substring((«00»+(dt.getMonth()+1)).length-2) + '.' + dt.getYear();

Сегодня на работе встретил в коде на JS довольно забавный кусок:

obj_.checked=(obj_.checked==true)?false:true;

вот таким способом я считаю количество новостей на сайте:

<?
$previev_amount=mysql_query("SELECT * FROM news;");
while ($previev_amount_inquiry=mysql_fetch_array($previev_amount)){
$amount+=$previev_amount_inquiry['amount'];
}
?>
Общее количество новостей размещенных на сайте: <?echo "$amount";?>

где в столбце amount для каждой записи стоит 1

Вопрос: можно ли это сделать к.л. другим способом или этот способ единственный?

P.S. Это цитата с форума, я всегда поражался изворотливости человеческого ума, но радует что человек понял что что-то не так...

lock («Place Order»)
{
// some code
}

c = document.getElementById(objid+'_main');
if {
    c.id=objid+'_main';
}

или такое:

if (160>450) {
    resize_height=450;
} else {
    resize_height=160;
}

Просматривал исходники одного проекта, зарубежный портал с возможностью публикации аудио-видео и т.д., не суть важно. И вот так там реализована регистрация пользователей, вернее — получение id пользователя при его регистрации.

if VerifyStructure(avDetails) == TRUE
	bStructure = TRUE
else
	bStructure = FALSE

if VerifyStatement(avDetails) == TRUE
	bStatement = TRUE
else
	bStatement = FALSE

Обе функции по спецификации возвращают строго boolean

habrahabr.ru/blog/code_wtf
Для публикования самизнаетечего