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

VisualBasic / Говнокод #12393

−110

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
76
77
78
79
80
81
82
83
84
85
86
87
88
89

Sub WriteCategories(number, choice)
    
    strSql = ""
    strSql2 = ""
    
    If number = 1 Then
        strSql = "SELECT name, category_type_id" '... the rest is skipped
    Elseif number = 2 Or number = 3 Then
        strSql = "SELECT goco.category_order, goco.category_type_id," '... the rest is skipped
    End If
    
    If number = 1 And choice = 1 Then
        strSql2 = "SELECT gosct.category_type_id, goct.operator_id," '... the rest is skipped
        
    Set RS2 = db.Execute(strSql2)
        
    ' --- No sub categories are displayed if choice = 3
    Elseif choice <> 3 Then
        strSql2 = "SELECT goco.category_type_id, goco.sub_category_order, ..." 'the rest is skipped
        
        Set RS2 = db.Execute(strSql2)
    End If
    
    Set RS = db.Execute(strSql)
    
    ' --- CategoryListCurrent should not display selected category in categoryListMoveTo and vice versa.
    If number = 2 And choice = 1 Then
        '... the real code is skipped
        
        Do Until RS.EOF
            '... the real code is skipped
        Loop
    End If
    
    If number = 2 And choice = 2 Then
        '... the real code is skipped
        
        Do Until RS.EOF
            '... the real code is skipped
        Loop
    End If
    
    
    ' --- Loop through the recordset and print categories and sub categories.
    Do Until RS.EOF
        '... the real code is skipped
        
        If number = 2 And choice = 1 Then
            '... the real code is skipped
        End If

        If number = 2 And choice = 2 Then
            '... the real code is skipped
        End If
        
        If number = 3 And choice = 3 And subConfigCatId <> -1 Then
            '... the real code is skipped
        End If
        
        ' --- Print main categories
        Response.Write ("<option style=""color:"& color &""" value=""" & RS("category_type_id") & "_0"" " & tempSel & " " & disabled & ">" & RS("name") &  "</option>" &vbCrLf)
        
        If choice <> 3 Then
            ' --- Print sub categories
            RS2.MoveFirst
            If Not RS2.EOF Then
                Do Until RS2.EOF
                    '... the real code is skipped
                    
                    If catId1 <> -1 And number = 2 And choice = 1 And subCatId1 > 0 Then
                        '... the real code is skipped
                    End If
                    
                    If catId2 <> -1 And number = 2 And choice = 2 And subCatId2 > 0 Then
                        '... the real code is skipped
                    End If
                    
                    ' --- Only print sub categories under a matching category
                    If CLng(RS2("category_type_id")) = CLng(RS("category_type_id")) Then
                        Response.Write ("<option value=""" & RS2("category_type_id") & "_" & RS2("sub_category_type_id") & """ " & tempSelSub & ">  - " & RS2("name") &  "</option>" &vbCrLf)
                    End If
                    RS2.MoveNext
                Loop
            End If
        End If
        
        '... the real code is skipped
    Loop
End Sub

Один параметр числовой. Назовём его number (C) Captain Obvious.
Второй параметр тоже числовой. Но так как имя number занято, назовём его choice.
Именованные константы?.. Анти-паттерн «Spaghetti Code»? Не, не слышал…

wissenstein, 06 Января 2013

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

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

+61

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

/**
	 * Convert unix timestamp into: (no leading zeros)
	 * array( 'day' => x, 'month' => x, 'year' => x, 'hour' => x, 'minute' => x );
	 * Written into separate static public function to allow for timezone to be used easily
	 *
	 * @param	integer	[$unix]	Timestamp
	 * @return	array 	Date parts
	 */
    static public function unixstamp_to_human( $unix=0 )
    {
    	$tmp = gmdate( 'j,n,Y,G,i', $unix );

    	list( $day, $month, $year, $hour, $min ) = explode( ',', $tmp );

    	return array( 'day'    => $day,
    				  'month'  => $month,
    				  'year'   => $year,
    				  'hour'   => $hour,
    				  'minute' => $min );
    }

IPB 3.x
Интересно сделали? А как бы сделали PRO :) ?

111111, 05 Января 2013

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

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

+129

1
2
3
4
5
6
7
8

static int multipart_buffer_eof(multipart_buffer *self TSRMLS_DC)
{
	if ( (self->bytes_in_buffer == 0 && fill_buffer(self TSRMLS_CC) < 1) ) {
		return 1;
	} else {
		return 0;
	}
}

nyaknyan, 05 Января 2013

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

Куча / Говнокод #12390

+132

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

Программа Елочки
Сместиться_В(4,11)
Треугольник
Треугольник
Треугольник
Сместиться_На(6,9)
Ветки
Ветки
Ветки
Сместиться_На(6,9)
Треугольник
Треугольник
Треугольник
Сместиться_На(6,9)
Ветки
Ветки
Ветки
Конец_Программы
Программа Треугольник
Опустить_Перо
Сместиться_На(2,-3)
Сместиться_На(-4,0)
Сместиться_На(2,3)
Поднять_Перо
Сместиться_На(-2,-3)
Сместиться_На(2,0)
Конец_Программы
Программа Ветки
Опустить_Перо
Сместиться_На(2,-3)
Сместиться_На(-2,3)
Сместиться_На(-2,-3)
Сместиться_На(2,3)
Сместиться_На(0,-3)
Поднять_Перо
Конец_Программы

Лежало в архиве вместе с тем самым могучим интерпретатором. Я аж прослезился.

Stallman, 05 Января 2013

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

Java / Говнокод #12389

+63

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

package javaapplication2;
import java.util.*;
public class JavaApplication2 {

    public static void main(String[] args) {
          int temp; byte n;
         
          Scanner in = new Scanner(System.in);
          System.out.print("Введите разрядность искомых чисел: "); n = in.nextByte();
          
          if (n < 1 | n > 5) {
            System.out.println("Всего чисел: 0");
            return;
          }
          
          int i = (int)Math.pow(10, (n - 1)); // i - минимальное число. Пр: 1, 10, 100, 1000, 10000
          int count = 0;
            
          while ( (int)Math.log10(i*2) + 1 == n) { // Цикл выполняется пока кол-во цифр в числе i равно числу n.
                 boolean unqie = false;                             
                 temp = i;
                 int[] g = new int[10];
                 
                 for (int j = 0; j < (n*2); j++) // Цикл выполниться вдвое больше раз, чем разрядность искомых чисел.  
                 { 
                    g[temp % 10]++; 
                    if (g[temp % 10] > 1) { 
                        unqie = true;
                        break;
                    }
                    
                    temp /= 10; // Отбрасываем последнюю проверенную цифру.
                    if (temp <= 0) { // После того, как проверили первое число на уникальность, 
                        temp = i*2; // проверяем второе, которое в 2 раза больше первого.
                    }                           
                 }
                
                 if ( unqie == false) { // Если число прошло проверку на уникальность, выводим его на экран. 
                  count++;
                  System.out.printf("%s %s \n", i, i*2);
                 }
             
             i++;
          }
          System.out.printf("Всего: %s numbers \n", count);           
    }                          
}

Моё. Программа выводит те числа, в которых, а также в умноженных на два числах, все цифры уникальные. Только начинаю изучать яву, не бейте, лучше обоссыте.

Govnocoder#0xFF, 04 Января 2013

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

1C / Говнокод #12388

−119

1

Торм. колодки передние[?(Докум.ТКП_Процент>0,",ост.толщина","")] [?(Докум.ТКП_Процент>0,Докум.ТКП_Процент,"")][?(Докум.ТКП_Процент>0,"мм","")]

это содержимое одной ячейки таблицы с видом вывода "Шаблон", мало того что автор вынуждает 3 раза обращаться к полю Докум.ТКП_Процент, так ещё и о конкатенации строк видимо ничего не слышал, упрощается до:
Торм. колодки передние [?(Докум.ТКП_Процент>0,",ост.толщина "+Докум.ТКП_Процент+" мм","")]

VladC, 04 Января 2013

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

Куча / Говнокод #12387

+125

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

<!-- ... -->
</body>
</html><noindex>
<table id="c_add2" name="c_add2" cellpadding="0" cellspacing="0" class="sh0"><tr><td><br></td></tr></table>
<table id="c_add" name="c_add" cellpadding="0" cellspacing="0" class="sh0 clcart ft11">
<tr>
<td width="100%" height="27px" class="c_h"><strong class="fs72">Товар добавлен в корзину</strong></td>
</tr>
<tr>
<td width="100%" height="90px">
	<table width="100%" cellpadding="2" cellspacing="0" class="fs b fcgr">
	<tr>
		<td width="50%" align="right">Товаров в корзине: </td><td> <span id="c_cart" name="c_cart" class="fcz"></span></td>
	</tr>
	<tr>
		<td width="50%" align="right">На сумму: </td><td> <span id="sum_cart" name="sum_cart" class="fcz"></span>&nbsp;р</td>
	</tr>
	</table>
</td>
</tr>
<tr>
<td width="100%" align="center" class="description">Окно будет закрыто через <span name="cl_time" id="cl_time">5</span> секунд</td>
</tr>
<tr>
<td width="100%" align="center">
<table width="100%" cellpadding="0" cellspacing="0">
<tr>
<td width="50%" align="center"><a href="cart.php"><img title="Перейти в корзину" alt="Перейти в корзину" src="img/btn/cartorder.gif" border="0"></a></td>
<td width="50%" align="center"><a href="#" onclick="clTCart(); return false;"><img border="0" src="img/btn/closewin.gif"></a></td>
</tr>
</table>
</td>
</tr>
</table>
</noindex>

I love my webmasterz!

Stallman, 04 Января 2013

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

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

+44

1
2
3
4
5
6

public function addHeadJS($src){
    $js_tag = '<script language="JavaScript" type="text/javascript" src="/'.$src.'"></script>';
	if($this->is_ajax) { echo $js_tag; return true; }
	$this->addHead($js_tag);
    return true;
}

Добавляет тег <script> с указанным путем
@param string $src - Первый слеш не требуется
@return true

Govnisti_Diavol, 04 Января 2013

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

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

+14

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

bool Item::IsBoundByEnchant() const
{
    // Check all enchants for soulbound
    for (uint32 enchant_slot = PERM_ENCHANTMENT_SLOT; enchant_slot < MAX_ENCHANTMENT_SLOT; ++enchant_slot)
    {
        if (enchant_slot > PRISMATIC_ENCHANTMENT_SLOT || enchant_slot < PROP_ENCHANTMENT_SLOT_0)    // not holding enchantment id
            continue;

        if (uint32 enchant_id = GetEnchantmentId(EnchantmentSlot(enchant_slot)))
            if (SpellItemEnchantmentEntry const* enchantEntry = sSpellItemEnchantmentStore.LookupEntry(enchant_id))
                if (enchantEntry->slot & ENCHANTMENT_CAN_SOULBOUND)
                    return true;
    }

    return false;
}

Взял из эмулятора World of WarCraft Trinity 4.3.4 ветка.
Все говно в 6 строчке
PERM_ENCHANTMENT_SLOT = 0
MAX_ENCHANTMENT_SLOT = 15

PRISMATIC_ENCHANTMENT_SLOT = 6
PROP_ENCHANTMENT_SLOT_0 = 10

Heisenberg, 03 Января 2013

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

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

+136

1
2
3
4
5
6
7
8
9

public override int GetHashCode()
    {
      if (this.FileName == null)
      {
        return base.GetHashCode();
      }

      return this.FileName.GetHashCode() + 13;
    }

почему 13?

taburetka, 02 Января 2013

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

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