Как конвертировать int [] в Integer [] в Java?

167

Я новичок в Java и очень смущен.

У меня большой набор данных длиной 4 int[] и я хочу подсчитать, сколько раз встречается каждая конкретная комбинация из 4 целых чисел. Это очень похоже на подсчет частот слов в документе.

Я хочу создать Map<int[], double> который отображает каждое значение int [] на счетчик при переборе списка, но Map не принимает примитивные типы.

так что я сделал Map<Integer[], Double>

мои данные хранятся в виде, ArrayList<int[]>так что мой цикл должен быть что-то вроде

ArrayList<int[]> data = ... // load a dataset`

Map<Integer[], Double> frequencies = new HashMap<Integer[], Double>();

for(int[] q : data) {

    // **DO SOMETHING TO convert q from int[] to Integer[] so I can put it in the map

    if(frequencies.containsKey(q)) {
    frequencies.put(q, tfs.get(q) + p);
    } else {
        frequencies.put(q, p);
    }
}

Я не уверен, какой код мне нужен в комментарии, чтобы сделать эту работу для преобразования int[]в Integer[]. Или, может быть, я в корне не понимаю, как сделать это правильно.

Jonik
источник
6
«Я хочу создать карту <int [], double> ... но карта не принимает примитивные типы». Как указано в одном из постов ниже, int [] не является примитивным типом, так что это не является реальной проблемой. Реальная проблема заключается в том, что массивы не переопределяют .equals () для сравнения элементов. В этом смысле преобразование в Integer [] (как говорит ваш заголовок) вам не поможет. В приведенном выше коде частот. Ports.containsKey (q) все равно не будет работать так, как вы ожидаете, поскольку для сравнения он использует .equals (). Реальное решение состоит в том, чтобы не использовать массивы здесь.
Newacct
Не хочу спамить с другим ответом, но стоит отметить, что сейчас есть пример документации по этому поводу.
Мурейник

Ответы:

189

Родная Java 8 (одна строка)

С помощью Java 8 int[]можно легко преобразовать Integer[]:

int[] data = {1,2,3,4,5,6,7,8,9,10};

// To boxed array
Integer[] what = Arrays.stream( data ).boxed().toArray( Integer[]::new );
Integer[] ever = IntStream.of( data ).boxed().toArray( Integer[]::new );

// To boxed list
List<Integer> you  = Arrays.stream( data ).boxed().collect( Collectors.toList() );
List<Integer> like = IntStream.of( data ).boxed().collect( Collectors.toList() );

Как говорили другие, Integer[]обычно это не очень хороший ключ карты. Но что касается преобразования, у нас теперь есть относительно чистый и нативный код.

Sheepy
источник
3
Между тем это правильный путь. Если вам нужен Integer-Array в виде списка, вы можете написать:List<Integer> list = IntStream.of(q).boxed().collect(Collectors.toList());
aw-think
для преобразования в формат Integer[]я бы на самом деле предложил использовать следующий синтаксис: Integer[] boxed = IntStream.of(unboxed).boxed().toArray();аналогично @NwDx
YoYo
1
@JoD. Да, это работает. Но все равно IntStream.ofзвонит Arrays.stream. Я думаю, что это сводится к личным предпочтениям - я предпочитаю одну переопределенную функцию, некоторые любят использовать более явный класс.
Sheepy
2
Это сумасшедшее, но отличное решение! Спасибо!
Ругал
1
@VijayChavda Новый синтаксис, называемый ссылками на методы , был введен в 2014 году в Java 8 как часть лямбды ( JSR 335, часть C ). Значит the "new" method (constructor) of the Integer[] class.
Sheepy
79

Если вы хотите преобразовать a int[]в a Integer[], в JDK нет автоматизированного способа сделать это. Тем не менее, вы можете сделать что-то вроде этого:

int[] oldArray;

... // Here you would assign and fill oldArray

Integer[] newArray = new Integer[oldArray.length];
int i = 0;
for (int value : oldArray) {
    newArray[i++] = Integer.valueOf(value);
}

Если у вас есть доступ к библиотеке Apache lang , вы можете использовать такой ArrayUtils.toObject(int[])метод:

Integer[] newArray = ArrayUtils.toObject(oldArray);
Эдди
источник
71
Это неприятный способ сделать цикл.
Джеймс Беквар
9
Это совершенно нормально для каждого ... Я не вижу неприятной части.
Дахака
18
@Dahaka, гадость заключается в использовании итератора, в valueто время как переменная индексации iесть.
13
1
@Icn, я понимаю, почему некоторые люди не предпочитают этот стиль, но что делает его таким нежелательным, чтобы быть противным?
Эдди
14
@icn, @Dahaka: Гадость заключается в том, что, поскольку вы уже используете переменную индекса с приращением, нет необходимости также использовать for-each; хотя это выглядит красиво и просто в коде, за кулисами, что связано с ненужными накладными расходами. Традиционный for (int i...)цикл был бы более эффективным здесь.
daiscog
8

Предположительно, вы хотите, чтобы ключ на карте соответствовал значению элементов, а не идентичности массива. В этом случае вы хотите какой-то объект, который определяетequals и hashCodeкак вы ожидаете. Проще всего конвертировать в List<Integer>, ArrayListили лучше использовать Arrays.asList. Более того, вы можете ввести класс, представляющий данные (аналогично, java.awt.Rectangleно я рекомендую сделать переменные private final, а также final класс).

Том Хотин - Tackline
источник
О, вот вопрос преобразования массива, наоборот: stackoverflow.com/questions/564392/…
Том Хотин - tackline
8

Использование обычного цикла for без внешних библиотек:

Преобразуйте int [] в Integer []:

int[] primitiveArray = {1, 2, 3, 4, 5};
Integer[] objectArray = new Integer[primitiveArray.length];

for(int ctr = 0; ctr < primitiveArray.length; ctr++) {
    objectArray[ctr] = Integer.valueOf(primitiveArray[ctr]); // returns Integer value
}

Преобразовать целое число [] в int []:

Integer[] objectArray = {1, 2, 3, 4, 5};
int[] primitiveArray = new int[objectArray.length];

for(int ctr = 0; ctr < objectArray.length; ctr++) {
    primitiveArray[ctr] = objectArray[ctr].intValue(); // returns int value
}
Серебряный
источник
6

Я был неправ в предыдущем ответе. Правильное решение состоит в том, чтобы использовать этот класс в качестве ключа в карте, оборачивая фактическое int [].

public class IntArrayWrapper {
        int[] data;

        public IntArrayWrapper(int[] data) {
            this.data = data;
        }

        @Override
        public boolean equals(Object o) {
            if (this == o) return true;
            if (o == null || getClass() != o.getClass()) return false;

            IntArrayWrapper that = (IntArrayWrapper) o;

            if (!Arrays.equals(data, that.data)) return false;

            return true;
        }

        @Override
        public int hashCode() {
            return data != null ? Arrays.hashCode(data) : 0;
        }
    }

и измените свой код следующим образом:

   Map<IntArrayWrapper, Double > freqs = new HashMap<IntArrayWrapper, Double>();

    for (int[] data : datas) {
        IntArrayWrapper wrapper = new IntArrayWrapper(data);

        if ( freqs.containsKey(wrapper)) {
            freqs.put(wrapper, freqs.get(wrapper) + p);
        }

        freqs.put(wrapper, p);
    }
Михай Тоадер
источник
1
Я бы сделал финал класса, а полевой финал. Сделайте защитную копию массива в конструкторе (клон). И просто возвращайте значение из Arrays.equals, а не используйте это своеобразное выражение if. toString было бы хорошо.
Том Хотин -
1
Да, и бросить NPE из конструктора (возможно, вызывающего клон) для нулевых данных, и не иметь проверки в hashCode.
Том Хотин -
Верно .. Но код для equals, hashCode генерируется IDEA :-). Работает правильно.
Михай Тоадер
6
  1. Конвертировать int [] в Integer []

    public static Integer[] toConvertInteger(int[] ids) {
    
      Integer[] newArray = new Integer[ids.length];
         for (int i = 0; i < ids.length; i++) {
           newArray[i] = Integer.valueOf(ids[i]);
         }
       return newArray;
    }
  2. Преобразовать целое число [] в int []

    public static int[] toint(Integer[] WrapperArray) {
    
       int[] newArray = new int[WrapperArray.length];
          for (int i = 0; i < WrapperArray.length; i++) {
             newArray[i] = WrapperArray[i].intValue();
          }
       return newArray;
    }
жесткий
источник
2
Это довольно неточно из-за автобокса. В первом примере вы можете просто сделать, newArray[i] = ids[i];а во втором newArray[i] = WrapperArray[i]:)
Eel Lee
3

Вместо того, чтобы писать свой собственный код, вы можете использовать IntBuffer для переноса существующего int [] без необходимости копировать данные в массив Integer.

int[] a = {1,2,3,4};
IntBuffer b = IntBuffer.wrap(a);

IntBuffer реализует сопоставимые возможности, поэтому вы можете использовать уже написанный код. Формально карты сравнивают ключи так, что a.equals (b) используется, чтобы сказать, что два ключа равны, поэтому два IntBuffers с массивом 1,2,3 - даже если массивы находятся в разных местах памяти - называются равными и поэтому будут работать на свой частотный код.

ArrayList<int[]> data = ... // load a dataset`

Map<IntBuffer, Double> frequencies = new HashMap<IntBuffer, Double>();

for(int[] a : data) {

    IntBuffer q = IntBuffer.wrap(a);

    if(frequencies.containsKey(q)) {
        frequencies.put(q, tfs.get(q) + p);
    } else {
        frequencies.put(q, p);
    }

}

надеюсь, это поможет

Крис
источник
2

Не уверен, почему вам нужен двойник на вашей карте. С точки зрения того, что вы пытаетесь сделать, у вас есть int [], и вы просто хотите подсчитать, сколько раз встречается каждая последовательность? В любом случае, для чего это нужно?

Я хотел бы создать оболочку для массива int с соответствующими методами .equals и .hashCode, чтобы учесть тот факт, что сам объект int [] не учитывает данные в своей версии этих методов.

public class IntArrayWrapper {
    private int values[];

    public IntArrayWrapper(int[] values) {
        super();
        this.values = values;
    }

    @Override
    public int hashCode() {
        final int prime = 31;
        int result = 1;
        result = prime * result + Arrays.hashCode(values);
        return result;
    }

    @Override
    public boolean equals(Object obj) {
        if (this == obj)
            return true;
        if (obj == null)
            return false;
        if (getClass() != obj.getClass())
            return false;
        IntArrayWrapper other = (IntArrayWrapper) obj;
        if (!Arrays.equals(values, other.values))
            return false;
        return true;
    }

}

А затем используйте мультисет Google Guava, который предназначен именно для подсчета вхождений, при условии, что тип элемента, который вы в него вставили, имеет надлежащие методы .equals и .hashCode.

List<int[]> list = ...;
HashMultiset<IntArrayWrapper> multiset = HashMultiset.create();
for (int values[] : list) {
    multiset.add(new IntArrayWrapper(values));
}

Затем, чтобы получить счет для любой конкретной комбинации:

int cnt = multiset.count(new IntArrayWrapper(new int[] { 0, 1, 2, 3 }));
Matt
источник
Это IntArrayWrapperопределенно правильный подход для использования int[]массива в качестве хеш-ключа, но следует отметить, что такой тип уже существует ... Вы можете использовать его для обёртывания массива, а не только иметь его, hashCodeи equalsон даже сопоставим.
Хольгер
2

Обновление: хотя приведенная ниже компиляция, она выдает ArrayStoreExceptionво время выполнения. Жаль. Я оставлю это на будущее.


Преобразование int[], в Integer[]:

int[] old;
...
Integer[] arr = new Integer[old.length];
System.arraycopy(old, 0, arr, 0, old.length);

Я должен признать, что я был немного удивлен, что это компилируется, учитывая System.arraycopyнизкоуровневость и все такое, но это так. По крайней мере, в Java7.

Вы можете конвертировать другой способ так же легко.

Томас Але
источник
2
Я не вижу особой ценности в сохранении этого ответа «для справки», когда он не работает. В документации для System.arraycopy () даже говорится, что она не будет работать для этой цели.
Ноль3
2

Это работает как шарм!

int[] mInt = new int[10];
Integer[] mInteger = new Integer[mInt.length];

List<Integer> wrapper = new AbstractList<Integer>() {
    @Override
    public int size() {
        return mInt.length;
    }

    @Override
    public Integer get(int i) {
        return mInt[i];
    }
};

wrapper.toArray(mInteger);
Итамар Борхес
источник
Привет Tunaki, не могли бы вы объяснить мне приведенный выше код? когда я запускаю отладчик, я вижу, что метод get и size вызывается автоматически, то есть без них вызывается явно !! как так?! не могли бы вы объяснить?
forkdbloke
0

Преобразуйте int [] в Integer []:

    import java.util.Arrays;
    ...

    int[] aint = {1,2,3,4,5,6,7,8,9,10};
    Integer[] aInt = new Integer[aint.length];

    Arrays.setAll(aInt, i -> aint[i]);
zemiak
источник
-1

тебе не нужно int[]является объектом и может использоваться в качестве ключа внутри карты.

Map<int[], Double> frequencies = new HashMap<int[], Double>();

является правильным определением карты частот.

Это было неправильно :-). Правильное решение тоже выложено :-).

Михай Тоадер
источник
Большое спасибо за ваш ответ. Сначала я попробовал это, и он прекрасно скомпилировался, но, похоже frequencies.containsKey(q), всегда будет ложным, даже если у меня putдважды один и тот же массив - есть ли здесь какой-то сбой, связанный с определением java равенства с int []?
1
(Подсказка, (new int [0]). Equals (new int [0]) - false; (new ArrayList <Integer> ()). Equals (new ArrayList <String> ()) - true.
Том Хотин - tackline
1
Плохая идея, так как сравнение массивов основано на равенстве ссылок . Вот почему Arrays.equals () существует ...
sleske
Черт :-). Я надеялся, что был прав. Объявление правильное, но семантика неверна из-за равенства ссылок.
Михай Тоадер
-3

Просто используйте:

public static int[] intArrayToIntegerArray(int[] a) {
    Integer[] b = new Integer[a.length];
    for(int i = 0; i < array.length; i++){
        b[i] = a[i];
    }
    return g;
}
Lurzapps
источник