1. C# / Говнокод #22042

    −34

    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
    if (name == "Паспорт гражданина Российской Федерации" || name == "Паспорт гражданина СССР")
{
	if (name == "Паспорт гражданина Российской Федерации")
	{
		return "Паспорт РФ";
	}
	else
	{
		return "Паспорт СССР";
	}
}
else
{
	return name;
}

    Описание метода: "Сокращает имя документа".

    Запостил: vladdy-moses, 26 Января 2017

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

    • ava Ukr-muslim-programer 26.01.2017 10:16 # +4

      В чем говно? Нормально же, код рабочий
      Ответить

      • ava Xom94ok 26.01.2017 10:29 # +10

        В порядке аргументов оператора "=="
        Обязательно нужно указывать константу в левой части выражения:
        "Паспорт гражданина Российской Федерации" == name
        В случае опечатки "=" вместо "==", программа не скомпилируется, потому что нельзя присвоить литералу. Компилятор проверяет программиста.
        Ответить

      • ava barop 26.01.2017 15:10 # +1

        Код не поддерживает ВНЖ, РВП и пасспорт моряка
        Ответить

    • ava dm_fomenok 26.01.2017 10:51 # −1

      Technical post. RSS fixed.
      Ответить

    • ava roman-kashitsyn 26.01.2017 12:52 # +9

      Сокращает жизнь того, кто читает этот метод.
      Ответить

      • ava dxd 26.01.2017 13:37 # +1

        Ну нет. Тут же можно было регулярку вкорячить, сокращало бы эффективнее.
        Ответить

        • ava Lokich 27.01.2017 05:38 # 0

          switch!
          кстати, бля, свитч бы тут норм зашел бы
          Ответить

          • ava inkanus-gray 27.01.2017 06:53 # +2

            Посчитать sha1 всех констант типа "Паспорт гражданина Российской Федерации" и сделать switch по sha1(name)?
            Ответить

            • ava dxd 27.01.2017 14:13 # +4

              Видел проект на плюсах, где люди, разбирая какой-то свой формат, кастовали четыре байта строки в инт и свитчились по нему.
              Ответить

              • ava roman-kashitsyn 27.01.2017 14:16 # +1

                > кастовали четыре байта строки в инт и свитчились по нему
                Отдалённо напомнило http://govnokod.ru/12611
                Ответить

              • ava inkanus-gray 27.01.2017 14:27 # +1

                Не вижу ничего криминального. Каст четырёх байтов в строку — скорее всего, вообще атомарная операция. А ко-ко-конпелятор автоматически оптимизирует свитч. В результате получится намного быстрее, чем тупо сравнивать все строки полностью.

                Правда, тут такой номер не пройдёт: тут же начала всех строк одинаковые.
                Ответить

          • ava guest 27.01.2017 13:42 # 0

            норм зашел в твоё очко, проверь.
            Ответить

    • ava nihau 27.01.2017 12:29 # +1

      Я все понял. Это критическая секция и неблокирующая синхронизация.
      Ответить

    Добавить комментарий