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

    +69

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    public static Integer valueOf(int i) {
        if(i >= -128 && i <= IntegerCache.high)
            return IntegerCache.cache[i + 128];
        else
            return new Integer(i);
    }

    Код java.lang.Integer

    из этого как минимум следует что: Integer.valueOf(-130) != Integer.valueOf(-130).

    Запостил: KoderZ, 08 Декабря 2010

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

    • ava KoderZ 08.12.2010 17:41 # −1

      А еще смущает магичность числа 128.
      Ответить

      • ava 3.14159265 08.12.2010 17:43 # 0

        >Iteger.valueOf(-130) != Integer.valueOf(-130)
        омич чтоле?
        джважды сработает return new Integer(-130);
        ты еще строки через == посравнивай
        Ответить

        • ava Анонимус 08.12.2010 17:49 # +1

          так пул же
          правда, для 130 не сработает

          а вот
          Iteger.valueOf(1) == Integer.valueOf(1)

          Ваш КО
          Ответить

          • ava KoderZ 08.12.2010 17:51 # 0

            Я о разном поведении для разных значений. Может старались кешировать наиболее использованные значения, но почему именно 128?
            Ответить

          • ava 3.14159265 08.12.2010 17:54 # 0

            а у меня такие сырцы
            private static class IntegerCache {
	private IntegerCache(){}
	static final Integer cache[] = new Integer[-(-128) + 127 + 1];
	static {
	    for(int i = 0; i < cache.length; i++)
		cache[i] = new Integer(i - 128);
	}
    }

    /**
     * Returns a <tt>Integer</tt> instance representing the specified
     * <tt>int</tt> value.
     * If a new <tt>Integer</tt> instance is not required, this method
     * should generally be used in preference to the constructor
     * {@link #Integer(int)}, as this method is likely to yield
     * significantly better space and time performance by caching
     * frequently requested values.
     *
     * @param  i an <code>int</code> value.
     * @return a <tt>Integer</tt> instance representing <tt>i</tt>.
     * @since  1.5
     */
    public static Integer valueOf(int i) {
	final int offset = 128;
	if (i >= -128 && i <= 127) { // must cache 
	    return IntegerCache.cache[i + offset];
	}
        return new Integer(i);
    }

            но это херня. в java.awt вещи поинтереснее встречаются

            это написано не для быдла не знающего жабы и сравнивающего Integerы через "==".

            подчеркиваю
            as this method is likely to yield significantly better space and time performance by caching frequently requested values.
            КОММЕНТЫ ДЕВОВ НЕ ЧИТАЙ
            @
            НА ГОВНОКОД ЗАЛИВАЙ
            Ответить

            • ava KoderZ 08.12.2010 18:12 # −2

              Никто не сравнивает, то я про разность поведения хотел сказать, а вот оптимизания сомнительная.
              Ответить

              • ava 3.14159265 08.12.2010 18:28 # 0

                думаю девам видней.
                оптимизация - нормал.
                реализация - попахивает.
                хотя конечно когда я впервые увидел new Integer[-(-128) + 127 + 1];
                тоже просмеялся.

                >но почему именно 128?
                0_0
                Ответить

                • ava gegMOPO4 08.12.2010 22:55 # 0

                  > хотя конечно когда я впервые увидел new Integer[-(-128) + 127 + 1];
                  > тоже просмеялся.

                  Артефакты. В Sun JDK в этом месте переменные. В OpenJDK решили, что достаточно констант по умолчанию (и верно решили).
                  Ответить

              • ava Lure Of Chaos 08.12.2010 22:37 # −1

                > ДЕВОВ
                старых?
                Ответить

    • ava ingenuus 08.12.2010 18:03 # −1

      Про кеш Integer никогда не слышали? ;)
      Ответить

    • ava pingw33n 08.12.2010 18:29 # 0

      http://www.owasp.org/index.php/Java_gotchas#Immutable_Objects_.2F_Wrapp er_Class_Caching
      Ответить

    • ava 3.14159265 09.12.2010 03:12 # 0

      вот кто ИТТ автоголосовалку зла тестил, а? признавайтесь.
      Ответить

      • ava inkanus-gray 09.12.2010 03:18 # +4

        Смотрите выше. У одного из участников спора нет ни одного минуса. Только тихо! Это рабочая гипотеза.
        Ответить

        • ava bugmenot 09.12.2010 03:35 # +3

          скандалы, интриги, расследования
          Ответить

        • ava bugmenot 09.12.2010 04:16 # 0

          Палево Professional 
          javascript:alert($('span.comment-vote strong').each(function(n){var s=document.createElement('span'),a=/^(\d+) за и (\d+) против$/g.exec(this.title);s.innerHTML=(a[1].fontcolor('green')+'/'+a[2].fontcolor('red')).sup();this.appendChild(s)}))
          Ответить

        • ava Oleg_quadro 09.12.2010 12:40 # +1

          Если про меня, то это не я.
          Я сам попадал под эти минусы не раз.
          Ответить

          • ava inkanus-gray 09.12.2010 13:38 # 0

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

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