如何在Java中删除Array Generic对象Sack

Question

所以，我正在创建一个名为“Sack”的通用数据结构。在这里我添加项目到一个麻袋，抓取一个随机项目，看它是否为空，或倾倒其内容等。另外我正在创建它以扩展为根据需要保留尽可能多的项目。

remove helper方法应该删除指定索引处的基础数据结构中的项。它应该通过将其替换为底层数组中的“last”元素来实现。它还应确保任何未使用的元素无效。

目前，我正在使用remove方法，我的删除方法遇到了麻烦。我在运行测试时收到错误，说它没有正确的修饰符。因此我的代码是

public class Sack<E>
{
public static final int DEFAULT_CAPACITY = 10;
private E [] elementData;
private int size;

@SuppressWarnings("unchecked")
public Sack()
{
    elementData = (E[]) new Object[DEFAULT_CAPACITY];
}
@SuppressWarnings("unchecked")
public Sack(int capacity)
{
    if(capacity < 0)
    {
        throw new IllegalArgumentException("capacity " + capacity);
    }
    this.elementData = (E[]) new Object[capacity];
}

public boolean isEmpty()
{
    if(size == 0)
    {
        return true;
    }
    else
    {
        return false;
    }
}
public void add(E item)
{
    int index = size++;
    if(size >= elementData.length-1)
    {
        elementData = Arrays.copyOf(elementData, size);
    }
    elementData[index] = item;
}
  public E [] dump()
{
   E [] E2 = Arrays.copyOf(elementData, size);
   for(int i = 0; i < size; i++)
   {
      elementData[i] = null;

   }
   size = 0;
    return E2;
}

我在此处收到错误

 public void remove(int index)
{
    for (int i = index; i < size; i++)
    {
        elementData[i] = elementData[i + 1];
    }
    size--;
}
}

它期望获得的价值是2，但它收到的是1.下载任何确保这一点的方法。

所以这是我的测试。我不能修改我的测试，只能修改我的代码。我在第一行收到错误。我会评论它，所以你可以看到。

@Test
public void testRemove()
{
    assertEquals(2, remove.getModifiers(), "remove does not have the correct modifiers"); // I receive an error here. 
    try
    {
        Random rand = new Random();
        Integer[] setElementData = new Integer[10];
        ArrayList<Integer> expectedElements = new ArrayList<Integer>();
        int randElement;
        for(int i=0; i<10; ++i)
        {
            randElement = rand.nextInt(50) + 1;
            setElementData[i] = randElement;
            expectedElements.add(randElement);
        }
        elementData.set(s, setElementData);
        size.set(s, 10);

        int randIndex;
        int numEe;
        while(!expectedElements.isEmpty()) {
            numEe = expectedElements.size()-1;
            randIndex = rand.nextInt(expectedElements.size());
            expectedElements.set(randIndex, expectedElements.get(numEe));
            expectedElements.remove(numEe);
            remove.invoke(s, randIndex);
            assertEquals(expectedElements.size(), size.get(s), "remove is not working correctly (check size usage)"); //Error now takes place here. 
            for(int i=0; i<expectedElements.size(); ++i) {
                assertEquals(expectedElements.get(i), ((Object[])elementData.get(s))[i], "remove is not working correctly (sack array element order incorrect)");
            }
            assertNull(((Object[])elementData.get(s))[expectedElements.size()], "remove is not working correctly (sack array element not nullified correctly)");
        }
        assertEquals(0, size.get(s), "remove is not working correctly (check size usage)");

    } catch (Exception e) {
        fail("remove is not working correctly");
    }
}

Answer 1

这将保持订单：

public void remove(final int index) {

    if (index >= size || 0 > index) {

        // TODO better to throw 'new ArrayIndexOutOfBoundsException(index)'
        // when remove illegal index
        return;
    }

    if (index < size - 1) { // move whole array

        System.arraycopy(
                elementData, index + 1, // copy from
                elementData, index, // copy to
                size - index - 1); // copy length
    }

    elementData[--size] = null;
}

Answer 2

我想到了。它最终通过了所有测试，但这非常困难。这是代码：

private void remove( int index)
{
    if(index >= 0)
    {
        E last = elementData[size - 1];
        elementData[index] = last;
        elementData[--size] = null;
    }
}

Answer 3

尝试将其替换为：

public void remove(int index)
{
    if(index >= 0) {
     E last = elementData[size - 1];
     elementData[index] = last;
     size--;
    }
}

Answer 4

您收到错误是因为remove变量的修饰符不是修饰符类型private。

``getModifiers（）```方法返回和修饰符类型的int值。有关更多信息，请参阅this。

由于你的断言函数声明修饰符应该是2，那么你应该将类修饰符更改为private。

Answer 5

这是解决这个问题的另一种方法，更简单一些。

private void remove(int index)
{
    elementData[index] = elementData[size-1];
    elementData[size -1] = null;
    --size;
}