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

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

+89

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

private void СтолкновениеСБонусом()// столкновение с бонусом
        {
            for (int i = 0; i < бонус.Count; i++)
            {
                if (Math.Abs(packMan.x - бонус[i].x) <= 19 && (packMan.y == бонус[i].y)
                    ||
                    (packMan.x == бонус[i].x) && Math.Abs(packMan.y - бонус[i].y) <= 19
                    ||
                    Math.Abs(packMan.x - бонус[i].x) <= 19 && Math.Abs(packMan.y - бонус[i].y) <= 19
                    )
                {
                    бонус.Remove(бонус[i]);//удаляем бонус
                    fl1 = false;//ставим в ложь и привидения прекрашают движение
                    //Labe2();
                    _tm1.Enabled = true;//таймер включен, хоть он и так включен
                    
                    _tm1.Start();//старт таймера
                }
            }
        }

Автор кода недоумевает, как код сделать еще читабельнее :facepalm:
Комментарии доставляют :)
Тема на форуме http://www.cyberforum.ru/csharp-beginners/thread586090.html

psina-from-ua, 04 Июня 2012

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

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

+84

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

public Boolean UnderMonitoring
        {
            get
            {
                checkUpdateMonitoring(ContractID);
                DataTable t0 = HespaWebServices.DynamicServices.SQLTable("CONTRACT", "CONTRACT", "ID=" + ContractID,
                                                                         "ID");
                if (t0.Rows.Count == 1)
                    return Core.RowBoolean(t0.Rows[0], "UNDER_MONITORING");
                else
                    return false;
            }
        }

В свойствах обосрать всю бизнес логику, такое надо уметь

__NoBody__, 31 Мая 2012

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

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

+82

1

bool IsClient = level == "Client" ? true : false;

startrack, 31 Мая 2012

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

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

+96

1
2
3
4
5

public bool IsStandart
{
    get { return IsStandart; }
    set { IsStandart = value; }
}

Maps, 31 Мая 2012

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

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

+101

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

private void FillLocalService(int SelectedPageIndex)
        {
            //Set category by search
            string search = inputSearch.Value == inputSearchValue ? string.Empty : inputSearch.Value;
            if (inputSearch.Value != inputSearchValue && !string.IsNullOrWhiteSpace(inputSearch.Value))
            {
                if (inputSearch.Value.StartsWith("landscap", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Landscapers";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("baby sitter", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Baby sitters";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("doctor", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Doctor";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("cleaning service", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Cleaning services";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("pet sitter", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Pet sitters";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("handyman", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Handyman";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("electrical service", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Electrical services";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("tutor", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Tutor";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("dog walker", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Dog walker";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("plumb", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Plumbing";
                    search = string.Empty;
                }
                else if (inputSearch.Value.StartsWith("Caterer", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Caterers";
                    search = string.Empty;
                }
                else if (inputSearch.Value.Equals("misc", StringComparison.InvariantCultureIgnoreCase))
                {
                    SelectedCategory = "Misc";
                    search = string.Empty;
                }
            }

200+ строк кода
}

тот же девелопер, та же .aspx страница)))))

mangyst, 30 Мая 2012

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

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

+112

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

switch (SelectedCategory)
            {
                case "Landscapers":
                    inputSearch.Value = "Landscapers";
                    break;

                case "Baby sitters":
                    inputSearch.Value = "Baby sitters";
                    break;

                case "Doctor":
                    inputSearch.Value = "Doctor";
                    break;

                case "Cleaning services":
                    inputSearch.Value = "Cleaning services";
                    break;

                case "Pet sitters":
                    inputSearch.Value = "Pet sitters";
                    break;

                case "Handyman":
                    inputSearch.Value = "Handyman";
                    break;

                case "Electrical services":
                    inputSearch.Value = "Electrical services";
                    break;

                case "Tutor":
                    inputSearch.Value = "Tutor";
                    break;

                case "Dog walker":
                    inputSearch.Value = "Dog walker";
                    break;

                case "Plumbing":
                    inputSearch.Value = "Plumbing";
                    break;

                case "Caterers":
                    inputSearch.Value = "Caterers";
                    break;

                case "Misc":
                    inputSearch.Value = "Misc";
                    break;
            }

обычный switch

mangyst, 30 Мая 2012

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

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

+102

1

return (m.Id == Id) ? (true) : (false);

походу платят за символы

MegaZver, 29 Мая 2012

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

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

+110

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

PropertyAdminFormModel data = new PropertyAdminFormModel
                { 
                    DateFinish = rep.DateFinish,
                    Name = rep.Name,
                    ReportingCircleId = rep.ReportingCircleId
                };

                if (data != null)
                {
                    Id = reportId;
                    Name = data.Name;
                    DateFinish = data.DateFinish;
                    ReportingCircleId = data.ReportingCircleId;
                }

А вдруг!? =)

DukeGonzo, 24 Мая 2012

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

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

+90

1

public string Материал_Clean { get; set; }

:[

nolka4, 24 Мая 2012

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

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

+93

1
2
3
4
5
6

for (int i = 0; i < wastePeriods.Length; i++)
    {
          wastePeriods = 0;
          docPeriods = 0;
          wastePeriodsYear = 0;
    }

Диггеры-рефакторщики раскопали очередной оригинальный цикл.
wastePeriods - массив с over300+ записями.
docPeriods, wastePeriodsYear - тоже массивы

UfoDriver, 22 Мая 2012

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

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