1

Мне нужно переместить данные из Vector в float[]. Как это сделать? Я написал вот такой вариант, но он долгий... Подскажите готовый вариант если есть.

private void convert_Float_To_float_data(float out[], Vector <Float> in)
{
    for (int i = 0; i < in.size(); i++)
    {
        out[i] = in.get(i);
    }
}
4
  • В каком смысле долгий? Работает долго? 3 сен 2015 в 8:49
  • Да. Долговато. Но так как другого решения нет, то вопрос закрыт
    – CJ1
    3 сен 2015 в 9:11
  • 2
    а вам точно Vector нужен? ArrayList может побыстрее работать. Vector вообще как бы устарел и не рекомендуется к использованию. Ну и вообще тот, кто изначально создаёт этот самый вектор, мог бы создавать массив float[]. Тогда бы и преобразовывать не надо было. 3 сен 2015 в 9:12
  • @CJ1: долго он работает в первую очередь из-за устаревшего Vector. Рекомендую использовать ответ Tagir, он надежнее, т.к. сам создает массив.
    – Nick Volynkin
    3 сен 2015 в 9:35

2 ответа 2

2

Стандартного метода для преобразования boxed-значений в примитивный массив в Java нет. Ваш способ нормальный, хотя я бы создавал новый массив и сделал поддержку любых входных коллекций:

static float[] toPrimitiveFloat(Collection<Float> in) {
    float[] result = new float[in.size()];
    int i=0;
    for(Float val : in) {
        result[i++] = val;
    }
    return result;
}

Подобный метод есть, например, в библиотеке Apache Common Lang: ArrayUtils.toPrimitive(). Только он преобразует не коллекцию, а массив. Внутри там аналогичный цикл. Проще никак.

2
  • Можно ли надеяться на ускорение, если писать в несколько потоков?
    – Nick Volynkin
    3 сен 2015 в 9:36
  • @NickVolynkin, можно надеяться, но уверены ли вы, что это самое узкое место в программе? :-) Насколько я понимаю, тормозить может из-за того, что в андроиде туповатая JVM (я под андроид не пишу). На Hotspot это очень быстрый код, который выиграет разве что при очень огромных массивах (тогда в Java-8 можно сделать IntStream.range(0, in.size()).parallelStream().forEach(i -> result[i] = in.get(i));). Ну и я на самом деле не очень уверен, что у автора это самое узкое место. Неизвестно ведь, как он профилировал код. 3 сен 2015 в 9:41
-1
 public Float[] convertVector(Vector<Float> foo)
{ 
    return foo.toArray(new Float[0]);
}
4
  • У вас массив Float, а не float. А еще, почему массив нулевой длины? Разве войдет в него содержимое вектора?
    – Nick Volynkin
    3 сен 2015 в 9:38
  • @NickVolynkin, если передать массив нулевой длины, toArray() его проигнорирует и создаст новый массив нужной длины. Работать будет, только лишний объект сборщику мусора подбирать. А про float замечание верное. 3 сен 2015 в 9:44
  • @TagirValeev: т.е всё это ещё можно обернуть в if (true) {...}? )
    – Nick Volynkin
    3 сен 2015 в 9:45
  • Да, все верно, но автор попросил другой способ, а это наверное один из немногих "других" способов. А с массивом Float можно работать так же как и смасивом float. 3 сен 2015 в 9:56

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.