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

    +126

    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
    public static bool ValidateEmailAddress(string emailAddress)
        {
            try
            {
                string TextToValidate = emailAddress;
                // test email address with expression
                if (emailExpression.IsMatch(TextToValidate))
                {
                    // is valid email address
                    return true;
                }
                else
                {
                    // is not valid email address
                    return false;
                }
            }
            catch (Exception)
            {
                throw;
            }
        }

    public static bool ValidateEmailAddress(string emailAddress)
    {
    if(string.IsNullOrEmpty(emailAddress))
    return false;
    return emailExpression.IsMatch(TextToValidate);
    }

    Запостил: jenik15, 14 Июня 2011

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

    • ava Lure Of Chaos 14.06.2011 18:05 # +3

      классический if(true) return true; else return false;
      Ответить

    • ava Lure Of Chaos 14.06.2011 18:06 # +5

      того и гляди, как сишарп-быдло перещеголяет похапе-быдло.
      Ответить

      • ava bugmenot 14.06.2011 18:52 # 0

        как показала практика - такие заявления вызывают баттхёрт
        Ответить

      • ava 3.14159265 14.06.2011 21:00 # 0

        так все к этому и идет.
        все больше мух слетается на сей продукт.
        и все больше мне на глаза попадается русских студий.
        Ответить

        • ava Lure Of Chaos 14.06.2011 21:14 # 0

          связано ли это с тем, что в сем продукте переработки появляется все больше рюшечек и няшечек?
          Ответить

        • ava bugmenot 14.06.2011 23:14 # 0

          > русских студий
          казалось бы, откуда в студиях взяться программистам? разве что натурщица с собой приведет...
          Ответить

    • ava daemon_master 14.06.2011 23:09 # −4 

      return false;
 return emailExpression.IsMatch(TextToValidate);

      а сами слышали про то, что "модуль должен иметь одну точку входу, и одну точку выхода"? Э.Дейкстра
      Ответить

      • ava jenik15 15.06.2011 15:28 # +4

        Может еще сам бил Гейтс чего сказал, так что теперь утопиться?
        Ответить

      • ava inser 16.06.2011 10:49 # 0

        Не знаю как Дейкстра, но по мне, такая конструкция лучше.
        http://sourcemaking.com/refactoring/replace-nested-conditional-with-guard-clauses
        Ответить

        • ava 3.14159265 16.06.2011 14:16 # 0

          То что по ссылке
          double getPayAmount() {
    double result;
    if (_isDead) result = deadAmount();
    else {
        if (_isSeparated) result = separatedAmount();
        else {
            if (_isRetired) result = retiredAmount();
            else result = normalPayAmount();
        };
    }
  return result;
  };
   ------>
  double getPayAmount() {
    if (_isDead) return deadAmount();
    if (_isSeparated) return separatedAmount();
    if (_isRetired) return retiredAmount();
    return normalPayAmount();
  };

          А вот всякие извращенцы вроде меня иногда пишут так
          return  (_isDead) ? deadAmount():
                 (_isSeparated) ? separatedAmount():
                 (_isRetired) ? retiredAmount():
                 normalPayAmount();
          Ответить

          • ava inser 16.06.2011 16:20 # 0

            Вот эти все конструкции ? : ? : ? : ? : ? ... я вообще не воспринимаю.
            Ответить

            • ava jenik15 16.06.2011 18:19 # 0

              я одобряю только одну ?.
              Если две нужно на кол!
              Ответить

              • ava 3.14159265 17.06.2011 14:34 # 0

                >Вот эти все конструкции ? : ? : ? : ? : ? ... я вообще не воспринимаю.
                >Если две нужно на кол!

                Вам обоим срочно нужно пописать на AviSynthe.
                http://govnokod.ru/3689
                Ответить

                • ava absolut 17.06.2011 14:40 # +1

                  >Вам обоим срочно нужно пописать на AviSynthe оголенный провод.
                  Ответить

      • ava carsten 19.06.2011 13:13 # 0

        >а сами слышали про то, что "модуль должен иметь одну точку входу, и одну точку выхода"? Э.Дейкстра

        может он другое имел в виду?
        переделка кода, чтобы возвращалось всё ровно через один return, делает его как раз-таки всё нечитаемым boilerplate
        Ответить

    • ava jenik15 15.06.2011 10:38 # 0

      Ребята! Код из Норвегии родом. Не нужно гнать на рашку.
      Ответить

      • ava bugmenot 15.06.2011 10:56 # +1

        сразу вспоминается другой норвежский код...

        <yossarian> and anyway, punishing opera users is like farting in an empty room
        <yossarian> they only punish themselves
        Ответить

    • ava carsten 20.06.2011 10:02 # 0

      > catch (Exception)
      > {
      > throw;
      > }

      философично

      стесняется чего?
      Ответить

    • ava guest8 09.04.2019 10:57 # −999

      показать все, что скрытоvanished
      Ответить

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