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

    Всего: 6

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

    +1

    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
    public class TransactionRequestViewModel
{
        public string X_login { get; set; }

        public double X_amount { get; set; }

        public int X_fp_sequence { get; set; }

        public int X_fp_timestamp { get; set; }

        public string X_fp_hash { get; set; }

        public string X_show_form { get; set; }

        public string X_receipt_link_method { get; set; }

        public string X_receipt_link_text { get; set; }

        public string X_receipt_link_url { get; set; }

        public string X_currency_code { get; set; }

        public string X_line_item { get; set; }
}

    Этот "Х" добавляет +80 к читаемости.

    Moses, 19 Октября 2018

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

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

    +1

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    public GlobalSetting ReadGlobalSettingById(string id)
{
	int totalItems;
	var globalSettings = ReadGlobalSettings(PageSetting.All, out totalItems);

	return globalSettings.FirstOrDefault(e => e.GlobalSettingID == id);
}

    Инновационный алгоритм считывания сущности по ID:
    1) считываем ВСЕ записи из базы в память;
    2) в памяти находим сущность с нужным ID;
    3) все остальное пусть Garbage Collector подберет;
    4) удобно же, не правда ли?)

    Moses, 11 Октября 2018

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

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

    +2

    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
    private int SortWObyAD(WorkOrderShort wo1, WorkOrderShort wo2)
{
	if (wo1.AD == wo2.AD)
	{
		return 0;
	}
	else if (wo1.AD > wo2.AD)
	{
		return 1;
	}
	else
	{
		return -1;
	}
}

    Сразу 2 достоинства:
    1) абсолютно не понятно, что такое AD;
    2) ах да, все это можно было бы заменить одной строкой "return wo1.AD - wo2.AD", т.к. все это будет использоваться в одной из стандартных функций сортировки

    Moses, 11 Октября 2018

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

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

    +1

    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
    public int GetWorkOrder(...)
{
	var wait = 0;
	const int stepWait = 30;
	var marker1 = false;

	do
	{
		if (marker1 == true) wait = wait + stepWait;
		...
		do
		{
			marker1 = false;
			...
			if (ChoosenWO.Count == 0)
			{
				if (counter == postalAreas.Count)
				{
					marker1 = true;
					break;
				}
				...
				counter = counter + 1;
			}
			else
			{
				...
			}
		} while (ChoosenWO.Count == 0);

	} while (wait < ((endDayTime.Hour - DateTime.UtcNow.Hour) * 60 + (endDayTime.Minute - DateTime.UtcNow.Minute)));

	return -1;
}

    Очень важный алгоритм, работающий на продакшене... Разработанный менеджером и написанный аналитиком...

    Moses, 11 Октября 2018

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

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

    +1

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    // todo '+1' is temporary
private string CorrectPhoneNumber(string phoneNumber, string countryCode = "+1")
{
	if (phoneNumber.Substring(0, 1) != "+")
	{
		return string.Format("{0}{1}", phoneNumber.Length == 10 ? countryCode : "+", phoneNumber);
	}

	return phoneNumber;
}

    Genious!

    Moses, 11 Октября 2018

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

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

    +2

    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
    [HttpPost]
        public string PayFromCreditCard(CreditCardViewModel cardInfo, double depositValue)
        {
            try
            {
                // todo: make real payment
                return AjaxResult.CreateSuccess();
            }
            catch (Exception exc)
            {
                return AjaxResult.CreateError(exc.Message);
            }
        }

    Этот код сейчас живет где-то на продакшене...

    Moses, 11 Октября 2018

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