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

    +120

    1. 1
    if (ViewData["partialViewName"].ToString() == "" ||  ViewData["partialViewName"] == null)

    кратко и лаконично

    Запостил: sergfreest, 09 Ноября 2011

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

    • ava MAGnit 09.11.2011 16:17 # −4

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

      • ava absolut 09.11.2011 16:40 # +4

        представьте, что ViewData["partialViewName"]==null и сделайте выводы.
        Ответить

        • ava MAGnit 09.11.2011 18:21 # 0

          А если поменять местами?

          if (ViewData["partialViewName"] == null || ViewData["partialViewName"].ToString())
          Ответить

          • ava martin 10.11.2011 12:24 # −1

            то тебя отправлят искать ошибку в том, что ты только что написал =)
            Ответить

      • ava martin 09.11.2011 16:59 # 0

        Читаем в msdn, что такое NullReferenceException. Я правда кстати не сильно представляю, что там может преобразовыватся в пустую строку, хотя может как-то хитро переопределено. Нужен контекст, что посмаковать в полной мере.
        Ответить

        • ava Eugene 09.11.2011 17:07 # −1

          > что там может преобразовыватся в пустую строку
          например пустая строка
          if((ViewData["partialViewName"] ?? string.Empty).ToString() == string.Empty)
          Ответить

          • ava martin 09.11.2011 17:12 # 0

            Ну вы, батенька, и монстр...
            Ответить

          • ava absolut 09.11.2011 17:25 # +3

            if( string.Empty.ToString() == string.Empty ) // тянет на отдельный ГК
            Ответить

          • ava abatishchev 09.11.2011 17:27 # 0

            C.O.
            if (String.IsNullOrEmpty(ViewData["partialViewName"]))

            / C.O.
            Ответить

            • ava martin 09.11.2011 17:34 # +1

              а откуда у тебя уверенность что там строка? =)
              ViewData["partialViewName"].ToString() очень может быть это автор не просто так написал...
              Ответить

              • ava abatishchev 09.11.2011 17:38 # 0

                тогда ToString() == "" вообще не имеет смысла
                ничто кроме строки (или объекта с перегруженным методом) не вернет true на такую проверку!
                Ответить

                • ava martin 09.11.2011 17:43 # 0

                  буду краток: "или".
                  Ответить

                • ava martin 09.11.2011 17:50 # +1

                  Вобще мне, что кажется автор кода сначала проверял только на null. Потом по какой-то причине, кто-то (может и сам автор) понял, что этого мало и решил "залатать" проблемный участок тем, что в голову пришло.
                  Ответить

                  • ava abatishchev 09.11.2011 17:54 # 0

                    или наоборот)
                    Ответить

                    • ava martin 09.11.2011 18:00 # 0

                      Наоборот врядли ибо заплатка тогда своей функции невыполняет =)
                      Ответить

                      • ava SmackMyBitchUp 09.11.2011 18:02 # +1

                        > своей функции не выполняет
                        так-то ничего удивительно

                        мы же на говнокоде
                        Ответить

    • ava guest8 09.04.2019 11:57 # −999

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

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