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

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

В номинации:

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

За время:

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

+136

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

// где-то там ...  
private int Kapital = 777;

// ...

void Save()
{
...
            int kapital;
            try { kapital = (int)Double.Parse(Kapital.ToString()); }
            catch { kapital = 0; }
...
}

Проект из солнечной Швейцарии...
И таких перлов много :)

CodeKiller, 18 Апреля 2014

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

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

+136

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

private Action<Action, Action<Exception>> ToCallbackAction(Action a)
        {
            return (sc, ec) =>
            {
                try
                {
                    a();
                    sc();
                }
                catch (Exception e)
                {
                    ec(e);
                }
            };
        }

толково или говно? Мнения разделились

taburetka, 01 Апреля 2014

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

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

+136

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

#include <stdio.h>
#include <inttypes.h>

inline uint8_t mid_ch (uint8_t a, uint8_t b, uint8_t res)
{
  if (b == 0){ if (a >= 2) return mid_ch (a-2, b  , res+1); else return res;}
  if (a == 0){ if (b >= 2) return mid_ch (a  , b-2, res+1); else return res;}
  return mid_ch (a-1, b-1, res+1);
}

uint64_t mid_0_ch (uint64_t a, uint64_t b)
{
  return mid_ch(a, b, 0);
}


int main(void)
{
  printf("%u %u", mid_0_ch (253, 123), (253+123)/2);
  return 0;
}

Нахождение среднего арифметического двух чисел через рекурсию. Сначала сделал для uint64_t чтобы это имело смысл, ведь сложение двух 64-битных чисел с записью результата в третье может привести к целочисленному переполнению (для 64-битных чисел, сложение которых может привести к переполнению, этот код работал чрезвычайно медленно, поэтому я ограничился 8-битными). При таком наркоманско-рекурсивном алгоритме этого переполнения не будет. 128-битные типы есть только как нестандартное расширение, но тогда еще возникает вопрос, как найди среднее арифметическое двух таких 128-битных чисел? А если есть 256-битные, то как тогда их них находить среднеарифметическое... ну и так далее.
Можно эту проблему еще решать через битовые маски т.е. убрать из обеих чисел самые старшие биты (предварительно сохранив их), сложить эти два числа, поделить на два, потом уже эти сохраненные биты вида 10000... или 0000... оба поделить на 2(сдвинуть на один разряд) и прибавить. Наркоманство какое-то.
Почему бы не сделать в С некий особый целочисленный тип, в котором любая фигня влезет, но который бы использовался только временно для промежуточных вычислений, чтобы не делать бэкапы битиков всяких?

j123123, 31 Марта 2014

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

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

+136

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

private AseConnection con;

public void CloseConnection()
{
    if (this.con != null && this.con.State == ConnectionState.Open)
    {
        CloseConnection(this.con);
    }
}

public void CloseConnection(AseConnection con)
{
    if (con == null)
        return;

    if (con.State == ConnectionState.Closed)
        return;

    con.Close();
}

А кому ещё враппэровъ? У меня много ихъ!

wissenstein, 18 Февраля 2014

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

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

+136

1
2
3
4
5

//...
float a = 7;
printf("%d", *(unsigned int *)(&a) >> 23);
// Что напечатает?
//...

На экзамене как-то задали такой вопрос. А ну-ка, кто без компилятора ответит?

GonZaleZ, 18 Февраля 2014

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

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

+136

1
2
3
4
5

public static string CreateBrowserCacheExtension(object key)
{
    //...Остальной код опущен для ясности
    return "Cache=" + Math.Abs(key.GetHashCode());
}

Legacy code из проекта над которым я работаю.
Косяк в том, что GetHashCode() иногда возвращает значение, равное System.Int32.MinValue.
А это в свою очередь приводит к OverflowException, в случае с Math.Abs(...);

pikowatt, 14 Февраля 2014

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

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

+136

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

public int GetModuleId(int userId)
{
    return moduleIdGet(userId);
}

protected int moduleIdGet(int userId)
{
    int moduleId;
    // calculate moduleId
    // ...

    return moduleId;
}

Дал открытый доступ, но в то же время как бы сохранил защищённый.

wissenstein, 29 Января 2014

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

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

+136

01
02
03
04
05
06
07
08
09
10

int file_exist (char *filename)
{
    char s[200];
    sprintf(s, "test -e %s", filename);
    if (system(s) == 0){
        return 1;
    }else{
        return 0;
    }
}

http://stackoverflow.com/questions/3828192/checking-if-a-directory-exists-in-unix-system-call

rat4, 23 Декабря 2013

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

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

+136

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

countDigits :: (Integral a) => a -> Int
{-# INLINE countDigits #-}
countDigits v0 = go 1 (fromIntegral v0 :: Word64)
  where go !k v
           | v < 10    = k
           | v < 100   = k + 1
           | v < 1000  = k + 2
           | v < 1000000000000 =
               k + if v < 100000000
                   then if v < 1000000
                        then if v < 10000
                             then 3
                             else 4 + fin v 100000
                        else 6 + fin v 10000000
                   else if v < 10000000000
                        then 8 + fin v 1000000000
                        else 10 + fin v 100000000000
           | otherwise = go (k + 12) (v `quot` 1000000000000)
        fin v n = if v >= n then 1 else 0

Хаскельная магия из исходников Data.Text.

Yuuri, 12 Декабря 2013

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

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

+136

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

/*
 * Just a wifi driver hooker.
 *
 * Yongle Lai @ 2009-05-10 @ Rockchip
 */

#include <linux/init.h>
#include <linux/module.h>
#include <linux/kernel.h>

MODULE_LICENSE("Dual BSD/GPL");

#ifdef CONFIG_MODVERSIONS
#define MODVERSIONS
#include <linux/modversions.h>
#endif 

//#define OLD_WIFI_IFACE

#ifdef OLD_WIFI_IFACE
extern int mv88w8686_if_sdio_init_module(void);
extern void mv88w8686_if_sdio_exit_module(void);
#else
extern int rockchip_wifi_init_module(void);
extern void rockchip_wifi_exit_module(void);
#endif

static int wifi_launcher_init(void) 
{
  int ret;

  printk("=======================================================\n");
  printk("==== Launching Wi-Fi driver! (Powered by Rockchip) ====\n");
  printk("=======================================================\n");

#ifdef OLD_WIFI_IFACE
  ret = mv88w8686_if_sdio_init_module();
  if (ret) /* Try again */
        ret = mv88w8686_if_sdio_init_module();
#else
  ret = rockchip_wifi_init_module();
  //if (ret) /* Try again */
  //    ret = rockchip_wifi_init_module();
#endif

  return ret;
}

static void wifi_launcher_exit(void) 
{
  printk("=======================================================\n");
  printk("== Dis-launching Wi-Fi driver! (Powered by Rockchip) ==\n");
  printk("=======================================================\n");

#ifdef OLD_WIFI_IFACE
  mv88w8686_if_sdio_exit_module();
#else
  rockchip_wifi_exit_module();
#endif
}

module_init(wifi_launcher_init);
module_exit(wifi_launcher_exit);

Вот так вот, просто и незатейливо на RK3188 включается и выключается вайфай.

Necromant, 25 Ноября 2013

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

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