Лучший говнокод за всё время

Лучший говнокод

В номинации:

Лучший рейтинг
Самый комментируемый

За время:

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

+163

1

$result = $db->query("update `" . $table_prefix . "options` set `option_value`='a:2:{i:0;b:0;s:8:" . '"auto_add"' . ";a:0:{}}' where `option_name`='nav_menu_options';");

unserialize "глазами на лету" - ЛЕГКО!!!!

taras_shs, 06 Ноября 2014

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

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

+170

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

<form id ='activate'>
    <div class ='form-control'>
        <label>Введите код подтверждения из SMS</label>
        <input type ='text' name ='code' class ='form-control' />
    </div>
    <button class = 'btn btn-success'>Активировать</button>
</form>
<script>
    $(function () {
        $('#activate').submit(function () {
            if ($('#activate [name="code"]').val != '<?php echo $secretCode; ?>') {
                alert('Вы ввели неверный код!')
                return false;
            }
        })
    })
</script>

Бог безопасности

Snickers, 15 Октября 2014

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

bash / Говнокод #16839

−115

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

ls -laF /usr/lib/x86_64-linux-gnu/libcurl.so.4.2.0
if [ "$?" -ne 0 ]; then
    current_location=$PWD
    gzip -dc < curl-7.22.0.tar.gz | tar -xf -
    cd curl-7.22.0
    ./configure --prefix=/usr
    make
    make install
    cd $current_location
else
    echo "libcurl.so.4.2.0 already exist!"
fi

Ключ -f? Нет, не слышали.

codemonkey, 12 Октября 2014

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

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

+140

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

@foreach (var groupTest in Model.Select(t => t.TestType).Distinct())
{
    <div class="form-column form-column2">
        <div class="static-height">
            <p class="content-subblock-title left upper-text">
                <label>
                    @if (@groupTest.GetEnumDescription().Length < 34)
                    {
                        <text>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</text>
                    }
                    @groupTest.GetEnumDescription()
                </label>
            </p>
            @foreach (var item in Model.Where(t => t.TestType == groupTest).OrderBy(t => t.Order))
            {
                <div class="donation-row">
                    <label>@item.TestName</label>
                    <span class="@item.TestClass blood-test-value-don">
                        @item.TestValue
                    </span>
                </div>
            }

        </div>
    </div>
}

Я пялюсь на этот кусок вот уже полчаса...

amatenkov, 17 Сентября 2014

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

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

+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
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

(closer-mop:defclass virtual-metaclass (closer-mop:standard-class) ())

(closer-mop:defclass virtual-slot-definition
    (closer-mop:standard-slot-definition)
  ((function :initarg :function 
             :accessor virtual-slot-definition-function)))

(defmethod slot-definition-allocation ((slotd virtual-slot-definition))
  :virtual)

(defmethod (setf slot-definition-allocation) 
    (allocation (slotd virtual-slot-definition))
  (unless (eq allocation :virtual)
    (error "Cannot change the allocation of a ~S"
           'virtual-direct-slot-definition)) allocation)

(closer-mop:defclass virtual-direct-slot-definition 
    (closer-mop:standard-direct-slot-definition
     virtual-slot-definition) ())

(defmethod closer-mop:direct-slot-definition-class
    ((class virtual-metaclass) &rest initargs)
  ;; Use virtual-direct-slot-definition if appropriate.
  (if (eq (getf initargs :allocation) :virtual)
      (find-class 'virtual-direct-slot-definition)
      (call-next-method)))

(closer-mop:defclass virtual-effective-slot-definition 
    (closer-mop:standard-effective-slot-definition
     virtual-slot-definition) ())

(defmethod closer-mop:effective-slot-definition-class 
    ((class virtual-metaclass) &rest initargs)
  ;; Use virtual-effective-slot-definition if appropriate.
  (let ((slot-initargs (getf initargs :initargs)))
    (if (member :virtual-slot slot-initargs)
        (find-class 'virtual-effective-slot-definition)
        (call-next-method))))

(defmethod closer-mop:compute-effective-slot-definition 
    ((class virtual-metaclass) name direct-slot-definitions)
  ;; Copy the function into the effective slot definition
  ;; if appropriate.
  (let ((effective-slotd (call-next-method)))
    (dolist (slotd direct-slot-definitions)
      (when (typep slotd 'virtual-slot-definition)
        (setf (virtual-slot-definition-function effective-slotd) 
              (virtual-slot-definition-function slotd))
        (return)))
    effective-slotd))

(defmethod closer-mop:slot-value-using-class 
    ((class virtual-metaclass) object slot-name)
  (let ((slotd (find slot-name (closer-mop:class-slots class) 
                     :key 'closer-mop:slot-definition-name)))
    (if (typep slotd 'virtual-slot-definition)
        (funcall (cadr (virtual-slot-definition-function slotd)) :get object)
        (call-next-method))))

(defmethod (setf closer-mop:slot-value-using-class) 
    (value (class virtual-metaclass) object slotd)
  (if (typep slotd 'virtual-slot-definition)
      ;; This is ugly and probably not portable, but what if?
      (funcall (cadr (virtual-slot-definition-function slotd))
               :set object value)
      (call-next-method)))

(defmethod closer-mop:slot-boundp-using-class 
    ((class virtual-metaclass) object slot-name)
  (let ((slotd (find slot-name (closer-mop:class-slots class) 
                     :key 'closer-mop:slot-definition-name)))
    (if (typep slotd 'virtual-slot-definition)
        (funcall (cadr (virtual-slot-definition-function slotd)) :is-set object)
        (call-next-method))))

(defmethod closer-mop:slot-makunbound-using-class 
    ((class virtual-metaclass) object slot-name)
  (let ((slotd (find slot-name (closer-mop:class-slots class) 
                     :key 'closer-mop:slot-definition-name)))
    (if (typep slotd 'virtual-slot-definition)
        (funcall (virtual-slot-definition-function slotd) :unset object)
        (call-next-method))))

О простоте объектно-ориентированого программирования, или страшная правда, которую от вас так долго скрывали.
Написано по мотивам: http://www.lispworks.com/documentation/lw50/LWUG/html/lwuser-173.htm В попытке сделать это, по возможности, портабельным (на SBCL вроде даже завелось).

Я понимаю, что читать это никто не будет, поэтому, краткий пересказ событий:
Захотелось мне виртуальных свойств, ну тоесть так, чтобы при обращении к Сипипишной библиотеке, для которой я ваяю оберкту не было различий между обычными Лисповыми объектами и Сипипишными.
Простой вариант - скопировать значения, но перформанс же!
И вот родился этот вариант. (Использование не показано, т.как не влезло).

wvxvw, 12 Сентября 2014

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

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

+138

1

p.fANT = Convert.ToUInt16(((devStat & (1 << 13)) != 0 ? "1" : "0") + ((devStat & (1 << 12)) != 0 ? "1" : "0"), 2);

Вот сижу думаю так задуманно или автор сам не понял что написал

Smekalisty, 12 Сентября 2014

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

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

+75

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

package com.seriouscompany.business.java.fizzbuzz.packagenamingpackage.impl;

import com.seriouscompany.business.java.fizzbuzz.packagenamingpackage.interfaces.factories.FizzBuzzSolutionStrategyFactory;
import com.seriouscompany.business.java.fizzbuzz.packagenamingpackage.impl.factories.EnterpriseGradeFizzBuzzSolutionStrategyFactory;
import com.seriouscompany.business.java.fizzbuzz.packagenamingpackage.interfaces.strategies.FizzBuzzSolutionStrategy;

public class FizzBuzz {
	public void fizzBuzz(int nFizzBuzzUpperLimit) {
		final FizzBuzzSolutionStrategyFactory mySolutionStrategyFactory =
			new EnterpriseGradeFizzBuzzSolutionStrategyFactory();
		final FizzBuzzSolutionStrategy mySolutionStrategy =
			mySolutionStrategyFactory.createFizzBuzzSolutionStrategy();
		mySolutionStrategy.runSolution(nFizzBuzzUpperLimit);
	}
}

FizzBuzz Enterprise Edition на Java от уважаемых разработчиков.

http://habrahabr.ru/company/abbyy/blog/173885/
https://github.com/EnterpriseQualityCoding/FizzBuzzEnterpriseEdition

gost, 10 Сентября 2014

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

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

+134

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

public static class GlobalFunc
{
	// bla-bla-bla ...
 
	public static bool isDouble(string input)
	{
		Double dec;
		return Double.TryParse(input, out dec);
	}

	public static bool isUShort(string input)
	{
		ushort dec;
		return ushort.TryParse(input, out dec);
	}

	public static bool isShort(string input)
	{
		short dec;
		return short.TryParse(input, out dec);
	}

	public static bool IsDate(string input)
	{
		DateTime date;
		return DateTime.TryParse(input, out date);
	}
}

face palm

Smekalisty, 26 Августа 2014

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

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

+76

1
2
3
4
5

private Boolean active = false;
...
synchronized (active) {
...
}

Чудо синхронизации. Блокируется раз и навсегда.

borka, 25 Августа 2014

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

Objective C / Говнокод #16450

−405

01
02
03
04
05
06
07
08
09
10

CGFloat ageInYears = self.user.age;
             
if (ageInYears < 1) {
    CGFloat ageInSeconds = [[NSDate date] timeIntervalSinceDate:self.user.birthday];
    ageInYears = floorf(ageInSeconds / (365.25 * 24.0 * 60.0 * 60.0));
}

// ...

profileInfoCell.schoolLabel.text = [NSString stringWithFormat:@"%.0f", ageInYears];

нет слов

realsugar, 30 Июля 2014

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

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