По колено в коде — Говнокод.ру

SQL / Говнокод #4021

−866

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26

ALTER PROCEDURE [dbo].[spSearch]                  
	@Name nvarchar(256)
AS                  
BEGIN                  
 SET NOCOUNT ON                  
                   
	DECLARE @qry VARCHAR(MAX)
	DECLARE @Where VARCHAR(1000)                                
	              
	SET @Where = '1=1'                  
        
	IF ISNULL(@Name,'') <> ''                  
	BEGIN                  
		SET @Where = @Where + ' and a.Name = ''' + @Name + ''''                  
	END      

SET @Where = @Where + ' and a.Active = 1'     
      
	SET @qry ='    
select a.* from t_Table1 a   
	WHERE ' + @Where     + 
 ' order by a.Name'

	EXECUTE (@qry)                  
                   
END

zheka, 18 Августа 2010

Комментарии (5)

C++ / Говнокод #4020

+166

1
2
3
4
5
6
7

до ревью:

abcForm.Controls.FindControl<AbcDateControl>("MyDate").SetValue(DateTime.Parse(DateTime.Now.AddDays(1).ToString()));

после

abcForm.Controls.FindControl<AbcDateControl>("MyDate").SetValue(new DateTime(DateTime.Now.AddDays(1).ToString()));

Всегда думал, что ревью улучшает код... Похоже народ хотел сделать вот это:

abcForm.MyDate.SetValue(DateTime.Now.Add Days(1));

:)

pdk, 18 Августа 2010

Комментарии (7)

C++ / Говнокод #4019

+161

01
02
03
04
05
06
07
08
09
10
11
12
13

float size = xLabelItemHeight + xLabelItem2Height + xLabelItem3Height + xLabelItem4Height + xTickItem_->tickSize() + tickGap();
if(f1 * (powerAxis_->screenMin() - powerAxis_->screenMax()) < f2 * size)
{
	size = xLabelItemHeight + xLabelItem2Height + xLabelItem4Height + xTickItem_->tickSize() + tickGap();
	if(f1 * (powerAxis_->screenMin() - powerAxis_->screenMax()) < f2 * size)
	{
		size = xLabelItemHeight + xLabelItem2Height + xTickItem_->tickSize() + tickGap();
		if(f1 * (powerAxis_->screenMin() - powerAxis_->screenMax()) < f2 * size)
		{
			size = xLabelItemHeight + xTickItem_->tickSize() + tickGap();
			if(f1 * (powerAxis_->screenMin() - powerAxis_->screenMax()) < f2 * size)
			{
// а дальше ничего не важно.

Ну что, мои находки всё так же унылы?

Altravert, 18 Августа 2010

Комментарии (5)

Си / Говнокод #4018

+135

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17

#include <stdio.h>
#include <stdlib.h>
 
int main()
{
   static int val = 1;
   while(val /* vanish */ --> 0)
   {
      (++ /* do black magic ! */  val) /* very black magic !!! */ ++;
      (val == 2 ? printf("WTF ??!") : val =- 1);
   }
 
   if(val<-15)
           return ((1<0?/* magic */100500:500?233?33?321?2223?0:0:0:0:0:0));
 
   return ((!!val==!!false)?val ==- 100500?5:1:main());
}

Из разряда синтаксиса языка

ForEveR, 17 Августа 2010

Комментарии (74)

JavaScript / Говнокод #4017

+155

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75

<div id="header">
<script>
    function mShow (mEl) {
      if (document.getElementById(mEl)) {
        el = document.getElementById(mEl);
        if (el.style.display=='block') el.style.display = 'none';
        else el.style.display ='block';
        if (mEl != 'mFoto') document.getElementById('mFoto').style.display = 'none';
        if (mEl != 'mFriend') document.getElementById('mFriend').style.display = 'none';
        if (mEl != 'mMessage') document.getElementById('mMessage').style.display = 'none';
      }
    }
    function sOff (e) {
        if(!e){e=window.event;}
        var message="";
        document.getElementById('mFoto').style.display = 'none';
        document.getElementById('mFriend').style.display = 'none';
        document.getElementById('mMessage').style.display = 'none';
        
        function clickIE() {if (document.all) {(message);return false;}}
        function clickNS(e) {if (document.layers||(document.getElementById&&!document.all)) {
          if (e.which==2||e.which==3) {
            (message);
          return false;}}}
            if (document.layers) {
              document.captureEvents(Event.MOUSEDOWN);
              document.onmousedown=clickNS;
            }else{
              document.onmouseup=clickNS;
              document.oncontextmenu=clickIE;
            }
  //document.oncontextmenu=new Function("return false")
    }

if(document.addEventListener){
  document.addEventListener("click",sOff,true);
}else if(document.attachEvent){
  document.attachEvent("onclick",sOff);
}else{
  document.onclick = sOff;
}

  </script>
<ul class="menu_top">

<li><a href="/users/555355" style="font-weight:bold;">inkanus</a> (<a href="/logout">выйти</a>)
</li>
<li>
<a href="#" onClick="mShow('mFoto');return false;">Мои фотографии <small>▼</small></a>
  <ul id='mFoto' class="submenu_top" style="right:280px;">
  <li><a href="/users/555355">Моя страница</a></li>
    <li><a href="/upload">Загрузить фотографии</a></li>
    <li><a href="/create">Создать альбом</a></li>
    <li><a href="/comments">Последние комментарии</a></li>
    <li><a href="/options">Настройки</a></li>
  </ul>
</li>
<li>
<a href="#" onClick="mShow('mFriend');return false;">Мои друзья <small>▼</small></a>
  <ul id='mFriend' class="submenu_top" style="right:166px;">
    <li><a href="/friends">Фотки друзей</a></li>
    <li><a href="/friends/control">Друзья</a></li>
    <li><a href="/friends/banned">Игнорируемые</a></li>
  </ul>
</li>
<li>
<a href="#" onClick="mShow('mMessage');return false;">Мои сообщения <small>▼</small></a>
  <ul id='mMessage' class="submenu_top" style="right:5px;">
    <li><a href="/messages">Входящие</a></li>
    <li><a href="/messages/sent">Исходящие</a></li>
    <li><a href="/messages/send">Отправить сообщение</a></li>
  </ul>
</li>
</ul>
  </div>

Взято с одного фотосайта.

inkanus-gray, 17 Августа 2010

Комментарии (7)

JavaScript / Говнокод #4016

+151

1
2
3
4
5
6

<div id="stat" class="list_menu" onclick="
                        if ($('#stat_sub').is(':hidden')){
                            $('#stat_sub').slideDown('slow');
                        } else {
                            $('#stat_sub').slideUp('slow');
                        }">Статистика</div>

Тот же автор, что и #4015

scalar4eblo4no, 17 Августа 2010

Комментарии (17)

JavaScript / Говнокод #4015

+165

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30

$(document).ready(function(){
        if (2==0){
            $("#manage").hide();
            $("#manage_sub").hide();
        }
        if (2==0){
            $("#dfd").hide();
            $("#dfd_sub").hide();
        }
        if (2==0){
            $("#pfd").hide();
            $("#pfd_sub").hide();
        }
        if (2==0){
            $("#act").hide();
            $("#act_sub").hide();
        }
        if (2==0){
            $("#tests").hide();
            $("#tests_sub").hide();
        }
        if (2==0){
            $("#stat").hide();
            $("#stat_sub").hide();
        }
        if (2==0){
            $("#users").hide();
            $("#users_sub").hide();
        }
    })

Условия (2==0) выводятся на страницу PHP-скриптом, в зависимости от выходных параметров

scalar4eblo4no, 17 Августа 2010

Комментарии (5)

C# / Говнокод #4014

+121

1
2
3
4
5
6
7
8
9

/// <summary>
        /// Returns current UTC time
        /// </summary>
        /// <returns></returns>
        public static DateTime GetCurrentTimeUtc()
        {
            DateTime dt = DateTime.UtcNow;
            return dt;
        }

Индус-стайл метод

sem13, 17 Августа 2010

Комментарии (9)

PHP / Говнокод #4013

+158

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32

<?php
        switch (1)
        {
            case isset($args[2]) && isset($args[3]):
                if ( !$this->xml_client->query('service.get_stat',$args[0],$args[1],$args[2],$args[3]) )
                {
                    $this->log(ERR_QUERY);
                    return false;
                }
                break;


            case isset($args[2]) && !isset($args[3]):
                if ( !$this->xml_client->query('service.get_stat',$args[0],$args[1],$args[2]) )
                {
                    $this->log(ERR_QUERY);
                    return false;
                }
                break;

            default:
                if ( !$this->xml_client->query('service.get_stat',$args[0],$args[1]) )
                {
                    $this->log(ERR_QUERY);
                    return false;
                }
                break;

        }
        return $this->xml_client->getResponse();

?>

кусок вызова функции с переменным числом параметров...
и самое главное это штука уже давно-давно работает!....

nicolay-punin, 17 Августа 2010

Комментарии (0)

PHP / Говнокод #4012

+146

1

<? if (1==0) { ?>

сложно было комментарий написать(/**/)

ferry-very-good, 17 Августа 2010

Комментарии (12)

Говнокод: по колено в коде.

SQL / Говнокод #4021

C++ / Говнокод #4020

C++ / Говнокод #4019

Си / Говнокод #4018

JavaScript / Говнокод #4017

JavaScript / Говнокод #4016

JavaScript / Говнокод #4015

C# / Говнокод #4014

PHP / Говнокод #4013

PHP / Говнокод #4012