Frank
Frank

Reputation: 31086

How to create an array of string vectors in Java?

I use the following code try to create an array of string vectors, I hope to have an array of 3 items, each item is a string vector :

Vector<String> Result_Vector_Array[]=new Vector<String>[3];

But NB highlighted the line as error(generic array creation), what's wrong ? What's the correct way to do it ? I know there is also Arraylist, but it's not synchronized, so I want to use vector.

Upvotes: 4

Views: 36077

Answers (7)

OscarRyz
OscarRyz

Reputation: 199215

Should I understand you are going to use multithread on that array?...

If you are not, then you don't have to worry about the synchronization.

I would:

    List<List<String>> listOfLists = new ArrayList<List<String>>();

    List<String> firstVector = new ArrayList<String>();

    firstVector.add( "one" );
    firstVector.add( "two" );

    listOfLists.add(  firstVector );

    System.out.println( listOfLists.get(0).get(0) == "one" );
    System.out.println( listOfLists.get(0).get(1) == "two" );

Prints true, true

Upvotes: -1

Artem Barger
Artem Barger

Reputation: 41222

You can also create:

Vector<Vector<String>> Result_Vector_Array=new Vector<Vector<String>>( );

Or you can replace Vector with some other collection.

Upvotes: 1

gub
gub

Reputation: 5229

You cannot create an array like that, do this:

Vector<String> Result_Vector_Array[] = (Vector<String>[]) new Vector[3];

I would suggest a different approach - arrays of containers like that are often quite hard to use, and don't help in the understanding of your code.

PS Also worth noting that the java naming convention would be

 Vector<String> resultVectorArray[] = (Vector<String>[]) new Vector[3];

and it's not usual to include the type in the name (I suspect this will be contentious!), why not just call it 'result' and let the type system worry about the type?

Upvotes: 5

jqno
jqno

Reputation: 15520

Due to type erasure, the JVM doesn't know at runtime that you have a Vector of String. The best it can do is create a 'raw' Vector. It can't guarantee for you that all Vectors actually contain Strings. That's why you get a warning from your IDE.

One way to work around this, it cast it, as jgubby suggests. Another is to put a List into your Vectors, instead of an array.

But, more importantly, why can the array have only 3 items? Wouldn't it be better to create a class with three fields to put into your Vector? With three items, that's not too much work, and you get the added bonus that you can give each of the three elements a helpful name, which should make your code a lot clearer.

Also, since Java 6, there exist a number of useful new synchronized List implementations, which might perform better than Vector, such as CopyOnWriteArrayList, or wrap a regular List in a Collections.synchronizedList.

Upvotes: 6

Tom
Tom

Reputation: 45104

If you want to use an synchronized ArrayList, you could use the synchronizedList method in java.util.Collections.

ArrayList<String> a1 = new ArrayList<String>();
ArrayList<String> a2= new ArrayList<String>();
ArrayList<String> a3 = new ArrayList<String>();

ArrayList<String> array[] = (ArrayList<String>[]) new ArrayList[3];

array[0]= Collections.synchronizedList(a1);
array[1]= Collections.synchronizedList(a2);
array[2]= Collections.synchronizedList(a3);

Upvotes: 0

Andreas Dolk
Andreas Dolk

Reputation: 114767

I'd suggest to keep with collections, do something like

Collection<Vector<String>> resultVectorArray = new ArrayList<Vector<String>>(3);

Then you can use generics with the Constructor and practically spoken the same effect

Upvotes: 1

dfa
dfa

Reputation: 116334

using reflection it would be:

    Vector<String>[] arrayOfVectors = 
      (Vector<String>[]) java.lang.reflect.Array.newInstance(Vector.class, size);
    java.util.Arrays.fill(arrayOfVectors, new Vector<String>());
    for (Vector<String> v : arrayOfVectors) {
        System.out.println(v);
    }

alternatively you can use an ArrayList and then wrap it using Collections#synchronizedList(java.util.List)

Upvotes: 0

Related Questions