テストメソッド

質問

テストメソッドを使いプログラムを実行させたいのですが、テストメソッドにエラーが出て正常にプログラムが動くかどうか調べられません。

問題は、配列0の数字を、int型のnumShiftsで与えられた数字の回数だけ一番後ろへ移動させ、それ以外の配列の数字を左へシフトさせます。
つまり、一度配列0の中身を一番後ろに持っていくと、配列1の数字が配列0にくるので、それをまた一番後ろにもって行きます。
そしたら配列2の数字が配列0に来ます。それをまた一番後ろにもって行きます。それをnumShifts回繰り返します。
例）
({1,2,3,4,5,6,7}, 3) 3回移動→　{4,5,6,7,1,2,3}
({1,2,3,4,5,6,7}, 0) 0回移動→　{1,2,3,4,5,6,7}
({1,2,3}, 5) 5回移動→　{3,1,2}

プログラムは以下のように組みました。
public class ArrayFun {
public void shiftNTimes(int[] array, int numShifts) {

for (int i = 0; i < numShifts; i++) { //numShifts回繰り返す
for (int j = 0; j < array.length-1; j++) { //配列をシフト
int temp = array[j+1];
array[j+1] = array[j];
array[j] = temp;
}
}
}
}

テストメソッドは以下です。
import static org.junit.Assert.*;
import org.junit.Test;
public class ArrayFunTest {
@Test
public void testshiftNTimes() {
ArrayFun af = new ArrayFun();

int[] a1 = { 1, 2, 3, 4, 5, 6, 7 };//元の配列
int[] a2 = { 1, 2, 3, 4, 5, 6, 7 };//元の配列
int[] a3 = { 1, 2, 3 };//元の配列

int[] a4 = { 4, 5, 6, 7, 1, 2, 3 };//シフト後の配列
int[] a5 = { 1, 2, 3, 4, 5, 6, 7 };//シフト後の配列
int[] a6 = { 3, 1, 2 };//シフト後の配列

assertEquals(a4, af.shiftNTimes(a1, 3));
assertEquals(a5, af.shiftNTimes(a2, 0));
assertEquals(a6, af.shiftNTimes(a3, 5));
}
}

エラーは、assertEqualsに赤線が出てしまうことです。
このタイプのAssertでのassertEquals(Object, Object)は、(int[], void)に適切ではないと表示されます。
他のテストメソッドではこのようなエラーは出ないのですが・・・。どなたか解決方法をご存知であれば宜しくお願いします。

回答No.3

askaaska · ベストアンサー

assertEquals(a4, af.shiftNTimes(a1, 3));
→
af.shiftNTimes(a1, 3)
ArrayAssert.assertEquals(a4, a1);

こうすればいいよ。

お礼

ありがとうございました！

回答No.2

askaaska

メソッド
shiftNTimes
は戻り値が何もないよ。

ていうエラー。

それと最初のプログラムだけど
前の問いと同じで
引数の配列とは別にもう一個配列を用意すれば
もっと綺麗に書けるよ。

補足

確かに戻り値がありませんね・・・。しかしこの問題は
public void shiftNTimes(int[] array, int numShifts) {
でやると書いてあるので、戻り値はかけませんよね？

voidをint[]に変えて戻り値を加えれば正常に動作するんですが、voidだとどのようにテストすればいいんでしょうか？

回答No.1

ProKaseifu

テストしてるメソッドに戻り値がないですよ・・

それと、配列の場合中身が等価なのかまでチェックされないので以下のようにループにするか、他の手段を検討する必要があります。

ループ例
int[] result = af.shiftNTimes(a1, 3);

for (int i = 0 ; i < result.length ; i ++ ){
assertEquals(a4[i],result[i]);
}

その他の方法
http://www.itarchitect.jp/technology_and_programming/-/34161-3.html

お礼

ありがとうございます！指摘してくださった配列の比較で他の問題も解決できました！
ありがとうございます！