74

コードをきれいにするためにSonarを使用していましたが、new Integer(1)代わりに を使用していると指摘されましたInteger.valueOf(1)valueOf新しいオブジェクトをインスタンス化しないように見えるため、よりメモリに優しいです。valueOf新しいオブジェクトをインスタンス化できないのはなぜですか? それはどのように機能しますか?これはすべての整数に当てはまりますか?

4

4 に答える 4

73

Integer.valueOf-128への値のキャッシュを実装します+127。Java 言語仕様の最後の段落のセクション 5.1.7 を参照してください。このセクションでは、ボックス化 (通常は.valueOfメソッドの観点から実装されます) の要件について説明しています。

http://docs.oracle.com/javase/specs/jls/se7/html/jls-5.html#jls-5.1.7

于 2010-06-04T13:29:54.377 に答える
28

JavaDocから:

public static Integer valueOf(int i)指定されたint値を表す整数インスタンスを返します。新しいIntegerインスタンスが必要ない場合は、コンストラクターInteger(int)よりもこのメソッドを使用する必要があります。このメソッドは、頻繁に要求される値をキャッシュすることで、スペースと時間のパフォーマンスを大幅に向上させる可能性があります。

ValueOfは一般的にオートボクシングに使用されるため、(オートボクシングに使用される場合)オートボクシングの仕様に従うために少なくとも-128から127の値をキャッシュします。

これがSunJVM1.5のvalueOf実装です。クラス全体を見て、キャッシュがどのように初期化されるかを確認してください。

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);
}
于 2010-06-04T14:06:38.193 に答える
2

彼らはあなたにvalueOf()代わりに使用するように促してnew Integer()いるので、メソッドvalueOf()はあなたのためにそれを行い、将来同じ番号を再び取得したい場合に備えて値をキャッシュします。その場合、メソッドは新しい整数をインスタンス化しませんが、キャッシュされた整数を提供します。これにより、新しい整数の「作成」がはるかに高速でメモリに優しいプロセスになります。

Integer.valueOf(342)==Integer.valueOf(342)このように、経験の浅いJavaプログラマーである場合、2つの整数に対して同じポインターを持っている(または持っていない)可能性があるため、多くの問題を引き起こす可能性があります 。あなたはC#で学んだので、時々バグが表示され、それらがどのように、どこから来たのかわかりません...

于 2011-10-31T13:55:20.550 に答える
2

java.lang.Integer ソース コードから。整数キャッシュは構成可能です。Sun 以外の整数キャッシュ サイズを構成するにはjava.lang.Integer.IntegerCache.high、ソース コードに従って System プロパティを使用する必要があります。

/**
 * Cache to support the object identity semantics of autoboxing for values between 
 * -128 and 127 (inclusive) as required by JLS.
 *
 * The cache is initialized on first usage. During VM initialization the
 * getAndRemoveCacheProperties method may be used to get and remove any system
 * properites that configure the cache size. At this time, the size of the
 * cache may be controlled by the vm option -XX:AutoBoxCacheMax=<size>.
 */

// value of java.lang.Integer.IntegerCache.high property (obtained during VM init)
private static String integerCacheHighPropValue;

static void getAndRemoveCacheProperties() {
    if (!sun.misc.VM.isBooted()) {
        Properties props = System.getProperties();
        integerCacheHighPropValue =
            (String)props.remove("java.lang.Integer.IntegerCache.high");
        if (integerCacheHighPropValue != null)
            System.setProperties(props);  // remove from system props
    }
}

private static class IntegerCache {
    static final int high;
    static final Integer cache[];

    static {
        final int low = -128;

        // high value may be configured by property
        int h = 127;
        if (integerCacheHighPropValue != null) {
            // Use Long.decode here to avoid invoking methods that
            // require Integer's autoboxing cache to be initialized
            int i = Long.decode(integerCacheHighPropValue).intValue();
            i = Math.max(i, 127);
            // Maximum array size is Integer.MAX_VALUE
            h = Math.min(i, Integer.MAX_VALUE - -low);
        }
        high = h;

        cache = new Integer[(high - low) + 1];
        int j = low;
        for(int k = 0; k < cache.length; k++)
            cache[k] = new Integer(j++);
    }

    private IntegerCache() {}
}

java.lang.Short、java.lang.Byte、および java.lang.Long から 127 から -128 のキャッシュを作成します

private static class LongCache {
    private LongCache() {
    }

    static final Long cache[] = new Long[-(-128) + 127 + 1];

    static {
        for (int i = 0; i < cache.length; i++)
            cache[i] = new Long(i - 128);
    }
}

private static class ShortCache {
    private ShortCache() {
    }

    static final Short cache[] = new Short[-(-128) + 127 + 1];

    static {
        for (int i = 0; i < cache.length; i++)
            cache[i] = new Short((short) (i - 128));
    }
}

private static class ByteCache {
    private ByteCache() {
    }

    static final Byte cache[] = new Byte[-(-128) + 127 + 1];

    static {
        for (int i = 0; i < cache.length; i++)
            cache[i] = new Byte((byte) (i - 128));
    }
}
于 2013-01-12T19:30:04.273 に答える