ClassCastException toArray（）メソッドの使用

Question

ArrayBasedStackのメソッドの1つであるtoArrayメソッドのjunitテストを記述しています。このクラスは内部に配列を持ち、toArrayメソッドは要素をコピーして返します。メソッドはクラスによって与えられ、配列の各要素を出力しようとしています。

このメソッドはローカル配列をコピーし、オブジェクトの型を持つコピー配列を返します。テストのために

public T[] toArray() { 
    T[] copy = (T[]) new Object[this.size()]; 
    for (int i = 0; i < this.size(); i++) { 
     copy[i] = (T) this.myArray[i]; 
    } 

    return copy; 
} 

、私は以下のようなとしてString配列を設定します。

プライベートArrayBasedStackスタックを、 toArray（）メソッドをテストする

public void setUp(){ 
    stack = new ArrayBasedStack<String>(); 
} 

は、私が試し：のtoArray（）メソッド以外

public void testToArray(){ 
     stack.push("000"); 
     stack.push("111"); 
     assertEquals(2, stack.toArray().length); 
     assertEquals("111", stack.toArray()[0]); 
    } 

を、他のメソッドのテストのようなよく渡さ：peek()、push()、clear()、equals() ...など

しかし、これだけのテストでは、両方のassertEqualsのためのエラーを返します。

assertEquals(2, stack.toArray().length); 
    assertEquals("111", stack.toArray()[0]); 

エラーは次のとおりです。ここで

>java.lang.ClassCastException: [Ljava.lang.Object; cannot be cast to [Ljava.lang.String; 
     at arraystack.ArrayBasedStackTest.testToArray(ArrayBasedStackTest.java:82) 
     at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
     at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) 
     at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) 
     at java.lang.reflect.Method.invoke(Unknown Source) 
     at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) 
     at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) 
     at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) 
     at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17) 
     at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26) 
     at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) 
     at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:298) 
     at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:292) 
     at java.util.concurrent.FutureTask.run(Unknown Source) 
     at java.lang.Thread.run(Unknown Source) 

Answer 1

T[] copy = (T[]) new Object[this.size()]; 

あなたはコンパイラに嘘をついています。 T[]がObject[]でない限り、T[]ではありません。

このメソッドを作成する正しい方法は、あなたが幸運ならば、プロトタイプを供給することにより、Collectionsはそれを行うのと同じ方法で、すでに適切なサイズである：

import java.lang.ref.Array; 

public T[] toArray(T[] a) 
{ 
    if (a.length != this.size()) 
    { 
     a = Array.newInstance(a.getClass().getComponentType(), this.size()); 
    } 
    for (int i = 0; i < this.size(); i++) 
    { 
     a[i] = (T) this.myArray[i]; 
    } 
    return a; 
} 

または代わり：

import java.util.Arrays; 

public T[] toArray(T[] a) 
{ 
    return Arrays.copyOf(this.myArray, this size()); 
}