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

    +125

    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
    // ****************************************************************************************
        // checkArguments
        // 
        // 
        // ****************************************************************************************
        private bool checkArguments()
        
        {

            bool boolSourceUserNameOK      = false;
            bool boolPathOK                = true;

            // Check the user name has been provided.
            if (this.UserName != null)
            {
                if (this.UserName.Length > 0)
                {
                    boolSourceUserNameOK = true;
                }
            }

            // Check the source path has been provided.
            if (this.SourcePath == null)
            {
                boolPathOK = false;
            }
            else
            {
                if (this.SourcePath.Length == 0)
                {
                    boolPathOK = false;
                }
            }

            return boolSourceUserNameOK && boolPathOK;

        }

    Очень поэтичная проверка на две пустые строки =)

    Запостил: victorko, 25 Января 2011

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

    • ava abatishchev 25.01.2011 20:08 # −7

      показать все, что скрытодоморощеный <i>String.IsNullOrEmpty</i> при чём первая проверка была добавлена после того как на второй стал вылетать <i>NullRefEx</i>
      Ответить

    • ava abatishchev 25.01.2011 20:09 # +3

      доморощеный String.IsNullOrEmpty при чём первая проверка была добавлена после того как на второй стал вылетать NullRefEx
      Ответить

    • ava Nigma143 25.01.2011 23:53 # −1

      Где здесь говно код?
      Ответить

      • ava abatishchev 26.01.2011 13:44 # 0

        где здесь нормальный код?

        return new[] { this.UserName, this.SourcePath }.All(s => !String.IsNullOrEmpty(s));
        Ответить

    • ava Nigma143 25.01.2011 23:59 # +1

      Это скорее быдлокод
      Ответить

    • ava victorko 26.01.2011 12:04 # +1

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

      • ava mrvlmor 26.01.2011 14:43 # +3

        Первая проверка на Null строку, а вторая на строку с 0-й длиной, это разные вещи
        Но у нас есть классный механизм String.IsNullOrEmpty
        Ответить

        • ava gegMOPO4 27.01.2011 22:28 # +1

          Имеется в виду, что для двух переменных проверка двух этих вариантов сделана по-разному.
          Ответить

      • ava artureg 26.01.2011 16:49 # 0

        прикинь, это не пхп :)
        Ответить

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