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

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

В номинации:

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

За время:

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

+109

1
2
3
4
5
6
7
8

BusinessLogic.Entity_InheritanceSetting.Save(
                    Session.SessionSettings.Settings.ContactID.Value,
                    Session.SessionSettings.Settings.SessionID.Value, copyID.Value,
                    true, true, true, true, true, true, true, true, true, true, true, true, true, true, true, true,
                    true, true, true, true, true, true, true, true, true, true, true, true, true, true, true, true,
                    true, true, true, true, true, true, true, true, true, true, true, true, true, true, true, true, true,
                    true, true, true, true, true, true, true, true, true, true, true, true, true, true, true,
                    true, true, true, true, true, true, true, true, true, true, true, true, true, true, true);

kore_sar, 30 Сентября 2013

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

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

+109

1
2
3
4
5

static void Main()
{
  int i = 0;
  Console.WriteLine(String.Format("{0} {1} {2}", i, i, i));
}

Я долго не видел там немного гoвна, но оказалось оно там есть

taburetka, 02 Августа 2013

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

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

+109

001
002
003
004
005
006
007
008
009
010
011
012
013
014
015
016
017
018
019
020
021
022
023
024
025
026
027
028
029
030
031
032
033
034
035
036
037
038
039
040
041
042
043
044
045
046
047
048
049
050
051
052
053
054
055
056
057
058
059
060
061
062
063
064
065
066
067
068
069
070
071
072
073
074
075
076
077
078
079
080
081
082
083
084
085
086
087
088
089
090
091
092
093
094
095
096
097
098
099
100

static void Main(string[] args)
        {
 
            Console.WriteLine("Как желаете заполнить?");
            Console.WriteLine("Цифра 1  = в столбик, цифра 2 = в строчку");
            int valMethod = Convert.ToInt32(Console.ReadLine());
 
            Console.Clear();
 
 
            Console.SetCursorPosition(1, 0);
            for (int index = 1; index != 22; index++)
            {
                Console.CursorLeft = 1;
                Console.WriteLine('&#9553;');
                if (index < 21)
                {
                    Console.CursorLeft = 19;
                    Console.Write('&#9553;');
                }
            }
 
 
 
 
            Console.SetCursorPosition(1, 0);
            for (int index = 1; index != 62; index++)
                Console.Write('&#9552;');
 
            Console.SetCursorPosition(35, 1);
 
            int valueRow = 0;
            for (int i = 0; i < 4; i++)
            {
                for (int index = 1; index != 20; index++)
                {
                    Console.SetCursorPosition(28 + valueRow, 1 + index);
                    Console.WriteLine('&#9553;');
                }
                valueRow += 8;
 
            }
 
            for (int index = 1; index != 20; index++)
            {
                Console.SetCursorPosition(29 + valueRow, 0 + index);
                Console.WriteLine('&#9553;');
            }
 
 
 
            Console.SetCursorPosition(20, 2);
            for (int index = 1; index != 43; index++)
                Console.Write('&#9552;');
 
            int value = 4;
            for (int i = 0; i < 9; i++)
            {
                Console.SetCursorPosition(1, value);
                for (int index = 1; index != 62; index++)
                    Console.Write('&#9552;');
                value += 2;
            }
 
            Console.SetCursorPosition(6, 2);
            Console.WriteLine("Спортсмен");
 
            Console.SetCursorPosition(25, 1);
            Console.WriteLine("Вид спорта");
 
 
            int value1 = 5;
            for (int i = 1; i < 9; i++)
            {
                Console.SetCursorPosition(6, value1);
                Console.WriteLine("{0}", i);
                value1 += 2;
            }
 
            int value2 = 2;
            for (int i = 1; i < 6; i++)
            {
                Console.SetCursorPosition(22 + value2, 3);
                Console.Write("{0}", i);
                value2 += 8;
            }
 
            Console.SetCursorPosition(1, 0);
            Console.Write("&#9556;");
 
            Console.SetCursorPosition(61, 0);
            Console.Write("&#9559;");
 
            Console.SetCursorPosition(1, 20);
            Console.Write("&#9562;");
             
            Console.SetCursorPosition(61, 20);
            Console.Write("&#9565;");

...

Мы так любим магические числа...

Psilon, 29 Июля 2013

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

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

+109

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

public function __get($name){          
        if(isset($this->fields[$name])){            
            if(class_exists($this->fields[$name]) && 
               strcasecmp($this->fields[$name], $this->vals[$name]['mdl']) == 0 && 
               strcasecmp(get_parent_class($this->fields[$name]), __CLASS__) == 0){    
                $class = $this->fields[$name];
                $id = (string)$this->vals[$name]['id'];
                if(!$this->refsCache[$id])
                    $this->refsCache[$id] = $class::findById($this->vals[$name]['id']);
                return $this->refsCache[$id];
            }else                
                return $this->vals[$name];
        }elseif(isset($this->fieldsRev[$name])) 
            return $this->vals[$name];
        else throw new Exception('Not found field "'.$name.'" when reading in model');
    }

    public function __set($name, $value){    
        if(isset($this->fields[$name])){  
            if(class_exists($this->fields[$name]) && 
               strcasecmp($this->fields[$name], get_class($value)) == 0 && 
               strcasecmp(get_parent_class($this->fields[$name]), __CLASS__) == 0)
                $this->vals[$name] = array('id' => $value->getId(), 'mdl' => strtolower($this->fields[$name]));
            else                
                $this->vals[$name] = $value;
        }elseif(isset($this->fieldsRev[$name])) 
            $this->vals[$name] = $value;
        else throw new Exception('Not found field "'.$name.'" when writing in model');        
    }

Пример реализации через жопу полей для базового класс модели, с ужасным оформлением кода.

haker, 11 Февраля 2013

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

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

+109

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

string New_naim = E.Remove(0, (NameGR(E.Trim()).Length)).Trim() + " " + Na;
                                New_naim = New_naim.Trim();
                                string check_naim =
                                    (E+Na).ToUpper().Replace(" ", "").Replace(",", ".").Replace("-", "").Replace("Е", "E")
                                    .Replace(";", "").Replace("*", "").Replace("Х", "X").Replace("О", "O").Replace("А", "A")
                                    .Replace("В", "B").Replace("М", "M").Replace("Н", "H").Replace("С", "C").Replace("Т", "T").Replace("З", "3")
                                    .Replace("Р", "P").Replace("Q", "G").Replace("К", "K").Replace("О", "O").Replace("0", "O").Replace("%", "").Replace("И", "U");
                                string check_gosttu = GU.ToUpper().Replace(" ", "").Replace(",", ".").Replace("Е", "E").Replace(".", "")
                                    .Replace("З", "3").Replace(";", "").Replace("*", "").Replace("Х", "X").Replace("О", "O").Replace("А", "A")
                                    .Replace("И", "U").Replace("В", "B").Replace("М", "M").Replace("Н", "H").Replace("С", "C").Replace("Т", "T").Replace("-", "")
                                    .Replace("Р", "P").Replace("Q", "G").Replace("К", "K").Replace("О", "O").Replace("0", "O").Replace("+", "");
                                /*quer = "SELECT PIN From dbo.Tb_Element Where (Id = '" + Id_el + "') AND "+
                                    "replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(UPPER(GOSTTU)" +
                                    ",' ',''),',','.') ,'Е','E'),'Х','X'),'Т','T'),'К','K'),'О','O'),';',''),'*',''),'А','A'),'В','B'),'М','M'),'Н','H'),'С','C'),'Р','P'),'Q','G'),'О','O'),'0','O'),'.','') = '"+ check_gosttu+   
                                    "' AND replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(UPPER(naim)" +
                                    ",' ',''),',','.') ,'Е','E'),'-',''),'Х','X'),'Т','T'),'К','K'),'О','O'),';',''),'*',''),'А','A'),'В','B'),'М','M'),'Н','H'),'С','C'),'Р','P'),'Q','G'),'О','O'),'0','O') = '" + check_naim + "'";*/
                                quer = "SELECT e.PIN From dbo.Tb_Element e, dbo.tb_elementname n Where e.id = n.id AND "+
                                    "replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(UPPER(GOSTTU)" +
                                    ",' ',''),',','.') ,'Е','E'),'Х','X'),'Т','T'),'К','K'),'О','O'),';',''),'*',''),'А','A'),'В','B'),'М','M'),'Н','H'),'С','C'),'Р','P'),'Q','G'),'О','O'),'0','O'),'.',''),'И','U'),'З','3'),'+',''),'-','') = '" + check_gosttu +
                                    "' AND replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(replace(UPPER(LTRIM(RTRIM(n.element))+RTRIM(LTRIM(e.naim)))" +
                                    ",' ',''),',','.') ,'Е','E'),'-',''),'Х','X'),'Т','T'),'К','K'),'О','O'),';',''),'*',''),'А','A'),'В','B'),'М','M'),'Н','H'),'С','C'),'Р','P'),'Q','G'),'О','O'),'0','O'),'%',''),'З','3'),'И','U') = '" + check_naim + "'";

Читаю старый код и радуюсь, что больше с ним не работаю. Правим результаты кривых рук операторов, заводящих данные.

Grover, 04 Декабря 2012

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

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

+109

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

public static MyResultData SqlReturnDataset(string sql, string connection)
            {
                MyResultData result = new MyResultData();
                try
                {
                    MySql.Data.MySqlClient.MySqlConnection connRC = new MySql.Data.MySqlClient.MySqlConnection(connection);
                    MySql.Data.MySqlClient.MySqlCommand commRC = new MySql.Data.MySqlClient.MySqlCommand(sql, connRC);
                    connRC.Open();

                    try
                    {
                        MySql.Data.MySqlClient.MySqlDataAdapter AdapterP = new MySql.Data.MySqlClient.MySqlDataAdapter();
                        AdapterP.SelectCommand = commRC;
                        DataSet ds1 = new DataSet();
                        AdapterP.Fill(ds1);
                        result.ResultData = ds1.Tables[0];
                    }
                    catch (Exception ex)
                    {
                        result.HasError = true;
                        result.ErrorText = ex.Message;
                    }
                    connRC.Close();
                }
                catch (Exception ex)//Этот эксепшн на случай отсутствия соединения с сервером.
                {
                    result.ErrorText = ex.Message;
                    result.HasError = true;
                }
            
                return result;

            }

Govnisti_Diavol, 26 Марта 2012

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

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

+109

01
02
03
04
05
06
07
08
09
10
11

private IEnumerable<CharacteristicValue> GetCharacteristicValues(IQueryable<CharacteristicValue> allCharacteristicValues, int characteristicId)
{
      var characteristicValues = new CharacteristicValue[userIds.Length];

       for (var i = 0; i < userIds.Length; i++)
       {
           characteristicValues[i] = allCharacteristicValues.FirstOrDefault(cv => cv.UserId == userIds[i] && cv.CharacteristicId == characteristicId); // Todo: Try to replace with a single query.
       }

       return characteristicValues;
}

Драсьте, нашёл в коде такую вот какашку. Как видите параметр allCharacteristicsValues реализует интерфейс IQuarable из EF. То бишь каждый раз, когда будет вызываться метод FirstOrDefault будет делаться запрос к базе. В идеале это должно было бы выглядеть так:

var characteristicValues = allCharacteristicValues.Where(cv => cv.CharacteristicId == characteristicId && userIds.Contains(cv.UserId));

Но при такой реализации размер масива пользователей и значение может разниться:

Values:
val1
val2
val3

Users:
1
2
3
4
5

Для первого же варианта результат будет выглядеть так:
Values:
val1
val2
val3
null
null

Users:
1
2
3
4
5

Собственно всё )))) Надеюсь подсоветуете чего-нибудь хорошего ;)

walash, 20 Марта 2012

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

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

+109

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

namespace ChemicalLabs
{
    public partial class MediaViewer : Form
    {
        public static string XPath;
        
        public MediaViewer(string ObjectMediaRef, string ObjectMediaName)
        {
            InitializeComponent();

            try
            {
                this.Text = ObjectMediaName;
                axWindowsMediaPlayer.settings.volume = 100;
                ObjectMediaRef = (Convert.ToInt16(ObjectMediaRef) + 1).ToString();
                XPath = ObjectMediaRef;

                StreamReader DataStream = new StreamReader("c://ChemicalData/" + ObjectMediaRef + "/" + ObjectMediaRef + ".txt", Encoding.GetEncoding(1251));
                _MEDIA_DATA_AREA.Text = DataStream.ReadToEnd();

                
                object[] MediaFiles = Directory.GetFiles(
                @"c://ChemicalData/" + ObjectMediaRef + "/" + "Media", "*.mp4", SearchOption.TopDirectoryOnly);

                for (int i = 0; i < MediaFiles.Length; i++)
                {FileInfo a = new FileInfo(MediaFiles[i].ToString());MediaFilesList.Items.Add(a.Name.Remove(a.Name.Length - 4));}
            }
            
            catch (Exception Ex)
            {MessageBox.Show(Ex.ToString(),(Ex.Message).ToString(),buttons: MessageBoxButtons.OK,icon: MessageBoxIcon.Information);}
        }

        private void MD_Click(object sender, EventArgs e)
        {

            if (MediaFilesList.SelectedItem == null)
            {MessageBox.Show("Вы не выбрали материал! Просмотр невозможен.","Сами не знаете, что хотите...",buttons: MessageBoxButtons.OK,icon: MessageBoxIcon.Information);}
            else
            {
                string MediaPath;
                MediaPath = MediaFilesList.SelectedItem.ToString();
                MediaPath = "c://ChemicalData/" + XPath + "/" + "Media" + "/" + MediaPath + ".mp4";
                axWindowsMediaPlayer.close();
                axWindowsMediaPlayer.URL = MediaPath;
            }
        }
        private void MediaStreamClose_Click(object sender, EventArgs e)
        {this.axWindowsMediaPlayer.close();}
        private void _MediaForm_Close_Click(object sender, EventArgs e)
        {this.axWindowsMediaPlayer.close(); MediaViewer.ActiveForm.Close();}
        private void MediaViewer_FormClosed(object sender, FormClosedEventArgs e)
        {this.axWindowsMediaPlayer.close();MediaViewer.ActiveForm.Close();}
        private void axWindowsMediaPlayer_MouseDownEvent(object sender, AxWMPLib._WMPOCXEvents_MouseDownEvent e)
        {MessageBox.Show("MediaPlayer - ver 0.0.1 for .NET Platform (By Kirill Sancharov)","MediaPlayer for Windows .NET",buttons: MessageBoxButtons.OK,icon: MessageBoxIcon.Information);}
 }
}

Удивляй меня полностью.

Govnisti_Diavol, 19 Марта 2012

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

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