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

    Всего: 6

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

    +159

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    var lines = content.Split(new[] { Environment.NewLine }, StringSplitOptions.RemoveEmptyEntries).ToList();

// remove first and last tags
lines = lines.Skip(2).Take(lines.Count - 3).ToList();           //    <------------   ОНО
for (var i = 0; i < lines.Count; i++)
{
        // remove one indent from each line
        lines[i] = lines[i].Substring(indentation, lines[i].Length - indentation);
}

    Покоробило от такого подхода...
    Я бы написал for от 1 до lines.Count-1 :)

    ddv_demon, 17 Декабря 2013

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

  3. C# / Говнокод #12080

    +111

    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
    // todo
/// <summary>
/// Генерация пароля из GUID
/// </summary>
/// <param name="guid">GUID</param>
/// <returns>пароль</returns>
public string PasswordByGuid(string guid)
{
    return guid[33].ToString()
        + guid[28].ToString()
        + guid[2].ToString()
        + guid[10].ToString()
        + guid[21].ToString()
        + guid[15].ToString();
}

    Коммерческий проект :)

    ddv_demon, 08 Ноября 2012

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

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

    +140

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    private void SendFile(string root, string url, DateTime dateTimeBegin)
{
	if ( ... isSend ... )
	{
		...MoveToArchive();
		...Dispose();
	}
	else
	{
		SendFile(root, url, dateTimeBegin);
	}
}

    Ну тут всё ясно...
    Если файлики отсылаются нормально, то всё работает хорошо.
    Зато если файлики не отсылаются, то начинается злая рекурсия, которая постепенно сжирает всю память :).

    ddv_demon, 13 Сентября 2012

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

  5. C# / Говнокод #11362

    +112

    1. 1
    int cutPosition = sb.ToString().IndexOf("                                          \r\n                                          ");

    Коммерческий проект :)

    ddv_demon, 05 Июля 2012

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

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

    +131

    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
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    private static bool state;

public static bool InWork
{
    get
    {
        return state;
    } 
    internal set
    {
        switch (value)
        {
            case true:
                {
                    try
                    {
                        // попытка запуска сервиса
                        ...
                    }
                    catch (Exception ex)
                    {
                        throw;
                    }
                }
                break;
            case false:
                {
                    if (!state) return;
                    // попытка остановить сервис
                    ...
                } 
                break;
        }
        state = value;
    }
}

public static void Start(...)
{
    ...
    InWork = true;
}

public static void Stop()
{
    ...
    InWork = false;
}

    Интересный ход, правда?

    ddv_demon, 01 Декабря 2011

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

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

    +121

    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
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    private void textBox1_TextChanged(object sender, EventArgs e)
{
	if ((textBox1.Text + textBox2.Text).Length + 1 > 255)
	{
		textBox1.BackColor = Color.LightPink;
		textBox2.BackColor = Color.LightPink;
	}
	else
	{
		textBox1.BackColor = Color.White;
		textBox2.BackColor = Color.White;
	}

	if ((textBox3.Text + textBox6.Text).Length + 1 > 255)
	{
		textBox3.BackColor = Color.LightPink;
		textBox6.BackColor = Color.LightPink;
	}
	else
	{
		textBox3.BackColor = Color.White;
		textBox6.BackColor = Color.White;
	}
}

private void bntSave_Click(object sender, EventArgs e)
{
	if (textBox1.BackColor == Color.LightPink)
	{
		MessageBox.Show("Длинна полей От и Адрес в сумме не должна превышать 255");
		return;
	}
	if (textBox3.BackColor == Color.LightPink)
	{
		MessageBox.Show("Длинна полей Кому и Адрес в сумме не должна превышать 255");
		return;
	}

	Properties.Settings.Default.Save();
	Navigator.Navigate(new ConfigMenuPage());
}

    ТЗ: "Суммарная длина полей X и Y не должна превышать 255 символов"
    Решение шедеврально как по вычислению длинны суммы строк, так и по цветовой идентификации :)

    ddv_demon, 10 Ноября 2011

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