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

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

В номинации:

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

За время:

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

0

1

do { code = randomCode(); } while (code.contains("FUCK"));

Пользователь с ником «Чапаев» опубликовал скриншот рекламного SMS-сообщения от
сбербанка, содержащего код «CRYVKILLJEWS», что можно перевести с английского
как «плачь, убивай евреев».

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

(проверочный код: 8bog)

Steve_Brown, 22 Ноября 2019

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

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

0

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

public class HttpQueryNameValueCollection : NameValueCollection
    {
        private ParallelLoopResult _parallelLoopResult;

        public override string ToString()
        {
            List<string> result;
            result = new List<string>();
            _parallelLoopResult = Parallel.ForEach(AllKeys, p =>
            {
                if (BaseGet(p) != null)
                    result.Add(p + "=" + Get(p));
            });
            return string.Join("&", result);
        }
    }

Формирование ссылки с query-параметрами недостаточно быстро, поэтому нужно это сделать в параллель :D

adoconnection, 24 Октября 2019

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

Lua / Говнокод #25829

0

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

if self.Weapon:GetNextPrimaryFire() < CurTime() then
	if GetConVarNumber( "ma2_togglechargeui" ) == 1 then
		if (self.Charge-self.DmgMin)/self.DmgMax*100<=10 then
			ply:SetNWString("chargebar","██")
		elseif (self.Charge-self.DmgMin)/(self.DmgMax-self.DmgMin)*100<=20 then
			ply:SetNWString("chargebar","████")
		elseif (self.Charge-self.DmgMin)/(self.DmgMax-self.DmgMin)*100<=30 then
			ply:SetNWString("chargebar","██████")
		elseif (self.Charge-self.DmgMin)/(self.DmgMax-self.DmgMin)*100<=40 then
			ply:SetNWString("chargebar","████████")
		elseif (self.Charge-self.DmgMin)/(self.DmgMax-self.DmgMin)*100<=50 then
			ply:SetNWString("chargebar","██████████")
		elseif (self.Charge-self.DmgMin)/(self.DmgMax-self.DmgMin)*100<=60 then
			ply:SetNWString("chargebar","████████████")
		elseif (self.Charge-self.DmgMin)/(self.DmgMax-self.DmgMin)*100<=70 then
			ply:SetNWString("chargebar","██████████████")
		elseif (self.Charge-self.DmgMin)/(self.DmgMax-self.DmgMin)*100<=80 then
			ply:SetNWString("chargebar","████████████████")
		elseif (self.Charge-self.DmgMin)/(self.DmgMax-self.DmgMin)*100<=90 then
			ply:SetNWString("chargebar","██████████████████")
		elseif (self.Charge-self.DmgMin)/(self.DmgMax-self.DmgMin)*100>90 then
			ply:SetNWString("chargebar","████████████████████")
			ply:SetNWBool("chargemaxxxed",true)
		end
	end
end

https://steamcommunity.com/sharedfiles/filedetails/?id=1825542758

amd, 11 Сентября 2019

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

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

0

1
2
3

We could replace the nonce with a deterministic value, but it's not entirely clear what the cryptographic implications are. At the very least, it allows attackers to obverse that a secret has changed, or that it has changed back to a previously observed value.

https://github.com/edolstra/rfcs/blob/nix-encryption/rfcs/0005-nix-encryption.md

Продолжаем наблюдения

CHayT, 04 Июня 2019

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

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

0

1
2

В "PHP" одинарные кавычки быстрее, чем двойные.
https://habr.com/ru/company/alfa/blog/447416/

Какой анскилл )))

BoeHHblu_nemyx, 10 Апреля 2019

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

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

+1

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

function preventSelection(element){
  var preventSelection = false;

  function addHandler(element, event, handler){
    if (element.attachEvent) 
      element.attachEvent('on' + event, handler);
    else 
      if (element.addEventListener) 
        element.addEventListener(event, handler, false);
  }
  function removeSelection(){
    if (window.getSelection) { window.getSelection().removeAllRanges(); }
    else if (document.selection && document.selection.clear)
      document.selection.clear();
  }
  function killCtrlA(event){
    var event = event || window.event;
    var sender = event.target || event.srcElement;

    if (sender.tagName.match(/INPUT|TEXTAREA/i))
      return;

    var key = event.keyCode || event.which;
    if (event.ctrlKey && key == 'A'.charCodeAt(0))  // 'A'.charCodeAt(0) можно заменить на 65
    {
      removeSelection();

      if (event.preventDefault) 
        event.preventDefault();
      else
        event.returnValue = false;
    }
  }

  // не даем выделять текст мышкой
  addHandler(element, 'mousemove', function(){
    if(preventSelection)
      removeSelection();
  });
  addHandler(element, 'mousedown', function(event){
    var event = event || window.event;
    var sender = event.target || event.srcElement;
    preventSelection = !sender.tagName.match(/INPUT|TEXTAREA/i);
  });

  // борем dblclick
  // если вешать функцию не на событие dblclick, можно избежать
  // временное выделение текста в некоторых браузерах
  addHandler(element, 'mouseup', function(){
    if (preventSelection)
      removeSelection();
    preventSelection = false;
  });

  // борем ctrl+A
  // скорей всего это и не надо, к тому же есть подозрение
  // что в случае все же такой необходимости функцию нужно 
  // вешать один раз и на document, а не на элемент
  addHandler(element, 'keydown', killCtrlA);
  addHandler(element, 'keyup', killCtrlA);
}

Долбоёбы борются с ветряными мельницами.

gost, 05 Января 2019

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

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

0

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

<?php
if(isset($_GET['action']) && $_GET['action']=="add"){
        $id=intval($_GET['id']);
        if(isset($_SESSION['cart'][$id])){
        $_SESSION['cart'][$id]['quantity']++;
        }else{
$sql_s="SELECT * FROM products
WHERE id_product={$id}";
$query_s=mysqli_query($sql_s);
if(mysqli_num_rows($query_s)!=0){
$row_s=mysqli_fetch_array($query_s);
                $_SESSION['cart'][$row_s['id_product']]=array(
                "quantity" => 1,
                "price" => $row_s['price']  );
                }else{
                $message="This product id it's invalid!"; } } } ?>
<h1>Product List</h1>
                <?php
               if(isset($message)){
               echo "<h2>$message</h2>";  }
               ?>
<table>
<tr>
            <th>Name</th>
            <th>Description</th>
            <th>Price</th>
            <th>Action</th>
</tr>
        <?php
$mysqli = new mysqli('……','root','……','tutorials');
$sql = 'SELECT name,description,price, id_product FROM products'; // select from mysql
$result = $mysqli->query($sql);  
while($row = $result->fetch_array()){
    ?>
        <tr>
            <td><?php echo $row['name'] ?></td>
            <td><?php echo $row['description'] ?></td>
            <td><?php echo $row['price'] ?>$</td>
<td><a href="index.php?page=products&action=add&id=<?php echo $row['id_product'] ?>">Add to cart</a></td>
        </tr>
<?php
    }
?>
</table>

строки:
2 - Входящую переменную так никто не проверяет. Есть функции filter_input
3 - $_GET['id'] без проверки на существовании
9 - mysqli_query - 1) аргумент, Идентификатор соединения. 2) сам запрос
10 - процесс проверки num_rows глупый. Есть 0 ( ложь ), либо в остальных случаях истина.
12 - мы выбивали из запроса id_product чтобы узнать и подставить значение как ключ $_SESSION['cart'][$row_s['id_product']] ??? Уверяю, я знаю уже ключ: $_SESSION['cart'][$id]
30 - Глупая, тупая ошибка новичков. Постоянно на каждой логике кода, устанавливают новое соединение с бд. Нравится в постели оргия, любите много и сразу?

Говнокодер: rita345
https://php.ru/forum/threads/this-product-id-its-invalid.74253/

MouseZver, 13 Декабря 2018

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

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

0

01
02
03
04
05
06
07
08
09
10

public void testGetClosedStatuses() {
    List<FooStatus> expectedStatuses = Arrays.asList(
            FooStatus.CANCELLED,
            FooStatus.COMPLETED,
            FooStatus.REJECTED);

    List<FooStatus> closedStatuses = fooService.getClosedStatuses();

    assertThat(closedStatuses, containsInAnyOrder(expectedStatuses.toArray());
}

Берём массив и преобразуем его в список. Чтобы использовать этот список, преобразуем его в массив.

wissenstein, 06 Декабря 2018

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

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