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

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

+135.2

1

Continue = (Continue) ? true : false;

CADakaRK, 17 Августа 2009

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

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

+139

1

dgridPFF1.Rows[1].Cells[1].Value = UtilHelper.Number2String(Convert.ToDouble(UtilHelper.StringToInvariantCulture(dgridPFF1.Rows[0].Cells[1].Value), UtilHelper.getCultureInfo()) / Convert.ToDouble(UtilHelper.StringToInvariantCulture(dgridPFF2.Rows[0].Cells[0].Value),UtilHelper.getCultureInfo()), 2); // String.Format(UtilHelper.getCultureInfo(),UtilHelper.NUMBERIC_FORMAT_2,Convert.ToDouble(dgridPFF1.Rows[0].Cells[1].Value) / Convert.ToDouble(dgridPFF2.Rows[0].Cells[0].Value, UtilHelper.getCultureInfo()));

Виетнамский jungle code :)

bugotrep, 16 Августа 2009

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

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

+138.9

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

public static void loadDataToDataGridView(DataGridView objDataGridView, List<object> objResultSet, IList objects)
        {
            if (objDataGridView == null || objResultSet == null) return;

            objDataGridView.Rows.Clear();
            for (int i = 0; i < objResultSet.Count; i++)
            {
                for (int columnIndex = 0; columnIndex < ((List<object>)objResultSet[i]).Count; columnIndex++)
                {
                    if (columnIndex == 0)
                    {
                        objDataGridView.Rows.Add();
                    }
                    objDataGridView.Rows[i].Cells[columnIndex].Value = ((List<object>)objResultSet[i])[columnIndex];
                }
                if (objects != null)
                {
                    objDataGridView.Rows[i].Tag = objects[i];
                }
            }
        }

        public static void loadDataToDataGridView(DataGridView objDataGridView, List<object> objResultSet)
        {
            loadDataToDataGridView(objDataGridView, objResultSet, null);
        }
...
use of this code:
...
            List<object> values = new List<object>();
            List<object> value = new List<object>();
            if (_objPriceFormationFormula.RateVsEuro == 0)
            {
                value.Add(UtilHelper.Number2String(1.00,2));// String.Format(UtilHelper.getCultureInfo(),UtilHelper.NUMBERIC_FORMAT_2,1.00));//1.00
            }
            else 
            {
                value.Add(UtilHelper.Number2String(_objPriceFormationFormula.RateVsEuro,2));// String.Format(UtilHelper.getCultureInfo(),UtilHelper.NUMBERIC_FORMAT_2,_objPriceFormationFormula.RateVsEuro));
            }
            
            values.Add(value);
            UtilHelper.loadDataToDataGridView(this.dgridPFF2, values);

Instead of databind :)

bugotrep, 16 Августа 2009

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

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

+139

01
02
03
04
05
06
07
08
09
10

public bool IsEmpty()
        {
            int x = 0;
            x += EntityType.Length == 0 ? 1 : 0;
            x += EntityTypeName.Length == 0 ? 1 : 0;
            x += EntityDisplayName.Length == 0 ? 1 : 0;
            x += EntityId == Guid.Empty ? 1 : 0;

            return x == 0 ? false : true;
        }

Аритметичный if :)

bugotrep, 16 Августа 2009

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

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

+137

1
2
3
4
5
6

if (messages[i].description != "")
                            if (messages[i].description.StartsWith("<![CDATA["))
                            {
                                messages[i].description = messages[i].description.Substring(9);
                                messages[i].description = messages[i].description.Substring(0, messages[i].description.Length - 3);
                            }

для строк типа <![CDATA[******]!>
Возникает устойчивое ощущение говнокода. Как нормально написать? :)

ikovalyov, 16 Августа 2009

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

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

+133.8

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

public static XmlNode FindNodeXPath(XmlNode root, string xPath)
        {
            string[] paths = xPath.Split(new char[] { '/' });
            XmlNode node = root;
            for (int i = 0; i < paths.Length; i++)
            {
                XmlNode childNode = null;
                for (int j = 0; j < node.ChildNodes.Count; j++)
                {
                    if (node.ChildNodes[j].Name == paths[i])
                    {
                        childNode = node.ChildNodes[j];
                        node = childNode;
                        break;
                    }
                }
                if (childNode == null)
                {
                    return null;
                }
            }
            return node;
        }

        public static XmlNode FindNodeXPath(XmlNode root, string nodeName, string xPath)
        {
            XmlNode node = FindNodeXPath(root, xPath);
            if (node != null)
            {
                for (int i = 0; i < node.ChildNodes.Count; i++)
                {
                    if (node.ChildNodes[i].Name == nodeName)
                    {
                        node = node.ChildNodes[i];
                    }
                }
            }
            return node;
        }

        public static void UpdateBaseAddress(string url, string fileConfig)
        {
            // Create config file to create 
            XmlDocument xmlDom = new XmlDocument();
            xmlDom.Load(fileConfig);
            XmlNode root = xmlDom.DocumentElement;
            // Get XML node 
            XmlNode xmlNode = FindNodeXPath(root, "endpoint", "system.serviceModel/services/service");
            xmlNode.Attributes["address"].Value = url;

            xmlNode = FindNodeXPath(root, "add", "system.serviceModel/services/service/host/baseAddresses");
            xmlNode.Attributes["baseAddress"].Value = url;

            xmlDom.Save(fileConfig);
        }

Виетнамский XPath эквивалент :)

bugotrep, 15 Августа 2009

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

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

+107.5

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

/// <summary>
        /// Check if this char is digit
        /// </summary>
        /// <param name="symbol">Some char</param>
        /// <returns>True if is digit</returns>
        private static bool IsDigit(char symbol)
        {
            List<char> digits = new List<char>();
            digits.Add('0');
            digits.Add('1');
            digits.Add('2');
            digits.Add('3');
            digits.Add('4');
            digits.Add('5');
            digits.Add('6');
            digits.Add('7');
            digits.Add('8');
            digits.Add('9');
            return digits.Contains(symbol);
        }

так сказать код от велосипедиста, сделал свой IsDigit() хотя уже есть char.IsDigit()

sv219, 13 Августа 2009

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

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

+133.1

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

/// <summary>
/// General handler for all buttons
/// </summary>
private void FormButtons_Click(object sender, EventArgs e)
{
    Control control = (Control) sender;

    if (control.Handle == btnCreateInvoices.Handle)
        ExportOrders();
    else if (control.Handle == btnFirstUsageInvoices.Handle)
        ExportFirstUsageInvoices();
    else if (control.Handle == btnImportCustomers.Handle)
        ImportCustomers();
    else if (control.Handle == btnImportProdcuts.Handle)
        ImportProducts();
    else if // и так далее...
}

WinForms приложение, на все кнопки навешен 1 обработчик события OnClick.
А внутри вот....

vleschenko, 12 Августа 2009

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

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

+142.9

1
2
3
4
5

public Аккаунт НайтиУчётнуюЗаписьПользователя(string логин, string пароль)
{
	lock (_пользователи)
		return _пользователи.Find(пользователь => пользователь.НеТыЛиЭто(логин, пароль));
}

Dimarius, 12 Августа 2009

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

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

+138

1

txtCollimator.parentNode.parentNode.parentNode.parentNode.parentNode.style.display = "none";

Прапрапрадедушка можно уже не показывать.
Дикая вложенность UserContol в ASP.Net дает о себе знать.

vaceknt, 11 Августа 2009

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

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

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

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

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

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

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

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

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

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

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

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