1. Список говнокодов пользователя O_O

    Всего: 8

  2. C# / Говнокод #5826

    +122

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    var product=dataContext.Products.Where(p=>p.Name == "SomeProductName")

if (products.Count() != 0)
            {
                return new Product
                {
                    ProductId = products.First().AM_PRODUCT_ID,
                    ProductName = products.First().PRODUCT_NAME,
                    IsActive = products.First().IS_ACTIVE
                };
            }

    this way of selecting product from table by its name generate 4 queries instead 1 ...

    O_O, 28 Февраля 2011

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

  3. SQL / Говнокод #3691

    −864

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    Гавнокод --

SELECT * FROM  Table1
WHERE [Id] not in 
		(
		SELECT min([Id]) FROM Table1
		WHERE [Code] in 
				(
				SELECT [Code] FROM Table1
				GROUP BY [Code]									HAVING count([Code]) > 1
				)
					GROUP BY [Code]
		)
	and [Id] in 
		(
				SELECT [Id] FROM Table1
				WHERE [Code] in (
						SELECT [Code] FROM Table1
						GROUP BY [Code]
						HAVING count([Code]) > 1
						)
		)


Нормальний варіант --

SELECT * FROM Table1 AS a
WHERE [Id] not in 
		(
			SELECT min([Id]) FROM a_ListParam
			WHERE a_ListParam.[Code] = a.[Code]
		)

    O_O, 12 Июля 2010

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

  4. C# / Говнокод #3621

    +102

    1. 1
    2. 2
    3. 3
    long First = 2* ... ;
long Second = 2* ... ;
long Average = (First + Second) / 2

    1.як не треба рахувати середнє арифметичне.
    2.БАЯН.

    O_O, 02 Июля 2010

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

  5. SQL / Говнокод #3615

    −858

    1. 1
    IF ((SELECT 1 FROM DELETED AS D INNER JOIN INSERTED AS I ON (I.[id_Status]=D.[id_Status])) IS NULL )

    такий от тригер стоїть на апдейт однієї таблички ...
    ось так -- IF Insertes.id_Status != Deleted.Id_Status було б значно краще і лаконічніше.

    O_O, 01 Июля 2010

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

  6. C# / Говнокод #3545

    +120

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    protected void GridView1_PreRender(object sender, EventArgs e)
    {         
         System.Drawing.Color OddGridColor = new System.Drawing.Color();
         OddGridColor = System.Drawing.Color.FromArgb(0xD3, 0xDE, 0xEF);
         System.Drawing.Color EvenGridColor = new System.Drawing.Color();
         EvenGridColor = System.Drawing.Color.FromArgb(0xEC, 0xE9, 0xD8);

            for (int i = 0; i < GridView1.Rows.Count; i++)
            {
                  if (i % 2 == 1)
                  {
                      GridView1.Rows[i].BackColor = OddGridColor;
                  }
                  else
                  {
                      GridView1.Rows[i].BackColor = EvenGridColor;
                  }
             }
    }

    the way of change GridView color through one ...
    Why do not use <AlternatingRowStyle> ?!!

    O_O, 23 Июня 2010

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

  7. C# / Говнокод #3378

    +122

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    SqlCommand SomeCommand = new SqlCommand(SelectString, ConnectionString);
int result;
if(SomeCommand.ExecuteScalar() == null)
    {
        result = 0;
     }
else
     {
         result = (int)SomeCommand.ExecuteScalar();
      }

    Why bad ?! The method execute twice ...

    O_O, 02 Июня 2010

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

  8. C# / Говнокод #3331

    +131

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    object FirstElement = null;
foreach(var item in SomeCollection)
   {
       FirstElement = item;
       break;
   }

    Ось так от береться перший елемент з колекції :)

    O_O, 26 Мая 2010

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

  9. SQL / Говнокод #3330

    −854

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    Select  *
From Table1
Where ( 
               (id = 2000)
                    or
                id = ( Select max(id) from Table1 )
              )

    а я дивувався, чому цей код в табличці > 300 000 записів виконується так довго ...

    O_O, 26 Мая 2010

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