1. Java / Говнокод #8710

    +75

    1. 1
    2. 2
    3. 3
    if(getFillColor().length() > 0){
		/***/
	}

    проверка на инициализацию переменной

    Запостил: JJunior, 02 Декабря 2011

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

    • ava guest 02.12.2011 19:33 # +2

      А что не так?
      isEmpty() появилось в 1.6, до нее это был единственный способ проверки, что строка не пустая.
      Ответить

    • ava JJunior 02.12.2011 19:40 # 0 

      private static String fillColor = "" ;


      код используется как проверка установления параметра
      Ответить

    • ava roman-kashitsyn 02.12.2011 19:50 # 0

      Предложите свой вариант.
      Ответить

      • ava absolut 02.12.2011 20:50 # 0

        !=""
        Ответить

        • ava guest 03.12.2011 19:03 # 0

          А сравнение строк в жабе не через хеш делается? Если так, то ваш вариант накладней получается...
          Ответить

          • ava Lure Of Chaos 03.12.2011 19:48 # 0

            вообще вариант абсолюта не совсем корректен, и может не всегда работать
            Ответить

            • ava absolut 03.12.2011 22:06 # 0

              это псевдокод. за конкретикой - к жабистам.
              Ответить

            • ava gegMOPO4 03.12.2011 23:02 # 0

              Вернее сказать — будет работать почти никогда.
              Ответить

              • ava Lure Of Chaos 04.12.2011 14:17 # 0

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

                • ava gegMOPO4 04.12.2011 17:09 # 0

                  Если пустая строка получилась не ссылкой на литерал, то не будет. Даже и насчёт литералов я не уверен.
                  Ответить

                  • ava tir 04.12.2011 20:16 # 0

                    насчет литералов - точно будет работать.

                    String literals-or, more generally, strings that are the values of constant expressions (§15.28)-are "interned" so as to share unique instances, using the method String.intern.

                    (http://java.sun.com/docs/books/jls/third_edition/html/lexical.html#3.10.5)
                    Ответить

          • ava roman-kashitsyn 03.12.2011 22:37 # 0

            != проверяет равенство ссылок. Пустая строка будет интернирована, поэтому приведённый код скорее всего будет работать правильно.
            Ответить

            • ava gegMOPO4 03.12.2011 23:00 # 0

              "Surprise!".substring(0,0)
              Ответить

              • ava roman-kashitsyn 04.12.2011 13:53 # 0

                ага, да вот только пустые строки в приведённых отрывках везде входят в качестве литералов.
                "Surprise!".substring(0,0).intern()
                Ответить

                • ava gegMOPO4 04.12.2011 17:10 # 0

                  Безусловно, в общем случае так сравнивать строки нельзя.
                  Ответить

      • ava eth0 02.12.2011 20:50 # 0

        Поясните.
        Ответить

      • ava JJunior 06.12.2011 10:58 # 0 

        fillColor = null ;

(getFillColor != null)
        Ответить

        • ava tir 06.12.2011 11:11 # 0

          тут важно, как этот getFillColor вызывают. Может в коде где есть заточка на то что он никогда не null.
          Ответить

    • ava guest8 09.04.2019 12:17 # −999

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

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