C# — Говнокод.ру

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

+124

1
2
3
4
5
6
7

for (var i = 0; i < Collection.Count(); i++)
                {
                    if (i==x)
                    {
                        Collection.Remove(i);
                    }
                }

Классика

kegdan, 09 Августа 2013

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

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

+137

1
2

private static OfficerResponse ToPerson<TField, TTitle>(IEnumerable<TField> fields, IEnumerable<TTitle> titles, Func<TField, string> getName, Func<TField, string> getValue, Func<TTitle, string> getTitle, bool active, TextType1 bio, YMDType birth, ContactInformationType contactInformation, string tickers, DegreeType1[] educationHistory, CertificateType[] certifications, PositionInformationType positionInformation, AffilationType[] corporateAffiliations, string officerId, OfficerDirectorType info,string repNo)
{...}

нашел вот...

fade, 08 Августа 2013

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

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

+141

1
2
3

bool eventWasRaised = false;

eventWasRaised.Should().Be.False();

иногда удивляет до чего доходят .NET unit testing фреймворки.
пруф http://joseoncode.com/2010/04/29/event-aggregator-with-reactive-extensions/
эту конструкцию глядишь и эксепшеном вырвет если не false.

Irdis, 07 Августа 2013

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

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

+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

int a;
cout<<"Enter a\n";
cin>>a;

if (a==1 && a!=2 && a!=3 && a!=4 && a!=5 && a<=5)
{
	cout<<"Odin"<<endl;
}
else if (a==2 && a!=1 && a!=3 && a!=4 && a!=5 && a<=5)
{
	cout<<"Dva"<<endl;
}
else if (a==3 && a!=1 && a!=2 && a!=4 && a!=5 && a<=5)
{
	cout<<"Tri"<<endl;
}
else if (a==4 && a!=1 && a!=2 && a!=3 && a!=5 && a<=5)
{
	cout<<"Chetire"<<endl;
}
else if (a==5 && a!=1 && a!=2 && a!=3 && a!=4 && a<=5)
{
	cout<<"Pyat"<<endl;
}

как-то глянул в код соседа)

MoN, 05 Августа 2013

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

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

+105

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

class GeneticAlgorithm
    {
...
 private List<Individual> childs= new List<Individual>();
...
public List<int> Genetare(....)
{
......
                childs.Clear();
                Parallel.For(0, countOfCrossover,RandomCrossing);
                _population.AddRange(childs);
.....
}

 private void RandomCrossing(int stateInfo)
{
 .............
               var child = Сrossing(parents, counOfCrossoverPoints);
            
               childs.AddRange(child);   
 }

Классический пример моей невнимательности.
Приделывал параллельность в генетический алгоритм

kegdan, 04 Августа 2013

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

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# / Говнокод #13522

+114

1
2
3

//request.IsSecureConnection и TargetHttps - булевы переменные

if (!(request.IsSecureConnection ^ TargetHttps))

вместо if (request.IsSecureConnection == TargetHttps)

mozg_raka, 31 Июля 2013

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

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)

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

+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

private void Save(string ThreadID, string Board)
        {
            string pathL;
            if (cbGIF.Checked)
            {
                pathL = String.Format(path, Board, ThreadID, "-gif");
            }
            else
            {
                pathL = String.Format(path, Board, ThreadID, "");
            }
            string htmlPath = String.Format(threadPath, Board, ThreadID);

            WebClient wcli = new GZipWebClient();
            string cThread = wcli.DownloadString(htmlPath);
            
            var rx = new Regex(cbGIF.Checked ? regExGif : regEx);
            var ms = rx.Matches(cThread);
            imgSaved = 0;
            imgCount = ms.Count;
            saveProgress.Minimum = 0;
            saveProgress.Maximum = imgCount;
            saveProgress.Value = 0;
            if (!Directory.Exists(pathL))
            {
                Directory.CreateDirectory(pathL);
            }
            try
            {
                foreach (Match m in ms)
                {
                    WebClient ccl = new WebClient();
                    ccl.DownloadFileCompleted += new AsyncCompletedEventHandler(ccl_DownloadFileCompleted);
                    string[] v = m.Value.Split('"');
                    string sd = v[1].Split('/').Last();
                    string a = url + v[1];
                    string b = pathL + sd;
                    if (File.Exists(b))
                    {
                        imgSaved++;
                        saveProgress.Value = imgSaved;
                        lblSaveProgress.Text = imgSaved.ToString() + "/" + imgCount.ToString();
                        if (imgSaved == imgCount)
                        {
                            btnSave.Enabled = true;
                            lblSaveProgress.Text = "FUKKEN SAVED!";
                        }
                    }
                    else ccl.DownloadFileAsync(new Uri(a),b);
                }
            }
            catch (WebException e)
            {
                MessageBox.Show(e.Message + e.StackTrace);
            }          
        }

В пределах одного метода бросаемся из крайности в крайность в именовании переменных.

A1mighty, 26 Июля 2013

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

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

+120

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

if (IsSetPrinter)
                        {
                            if (Counter == 0)
                            {
                                printingIsOkay = PrintDocument(letterIDsString
                               , row.DocumentTemplateVersionID
                               , row.DocumentTemplate
                               , (row.IsHeader_DocumentTemplateVersionIDNull() ? (int?)null : row.Header_DocumentTemplateVersionID)
                               , (row.IsFooter_DocumentTemplateVersionIDNull() ? (int?)null : row.Footer_DocumentTemplateVersionID)
                               , true
                               , Counter
                               , Counter == documentTemplateVersions.Count - 1);
                            }
                            else
                            {
                                printingIsOkay = PrintDocument(letterIDsString
                              , row.DocumentTemplateVersionID
                              , row.DocumentTemplate
                              , (row.IsHeader_DocumentTemplateVersionIDNull() ? (int?)null : row.Header_DocumentTemplateVersionID)
                              , (row.IsFooter_DocumentTemplateVersionIDNull() ? (int?)null : row.Footer_DocumentTemplateVersionID)
                              , false
                              , Counter
                              , Counter == documentTemplateVersions.Count - 1);
                            }

                        }
                        else
                        {
                            printingIsOkay = PrintDocument(letterIDsString
                               , row.DocumentTemplateVersionID
                               , row.DocumentTemplate
                               , (row.IsHeader_DocumentTemplateVersionIDNull() ? (int?)null : row.Header_DocumentTemplateVersionID)
                               , (row.IsFooter_DocumentTemplateVersionIDNull() ? (int?)null : row.Footer_DocumentTemplateVersionID)
                               , false
                               , Counter
                               , Counter == documentTemplateVersions.Count - 1);
                        }

Я бы выложил весь класс. Он весь такой. Но, думаю, вы поняли кто писал?

kore_sar, 23 Июля 2013

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

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