问题描述
因此，我正在创建一个名为“ Sack”的通用数据结构。 在此方法中，我将项目添加到麻袋中，抓取一个随机项目，看看它是否为空，或者倾销其内容等。此外，我正在创建它以扩展为容纳所需的任何项目。
我需要创建一个抓取方法，该方法应从麻袋中随机删除并返回一个项目。 如果不存在任何项目，则应返回null。
我的代码如下：
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 E [] grab()
{
E [] E2 = Arrays.copyOf(elementData, size);
return elementData;
}
这是不正确的，在运行测试时出现错误，指出AssertionFailedError：抓斗无法正常工作（检查空麻袋）==>
Expected: null
Actual : [Ljava.lang.Object;@76908cc0
我的测试就在这里，我无法修改我的测试，只能提供所提供的代码，因为那里存在错误，我需要我之前所说的内容：
抓斗方法应随机从麻袋中取出物品并将其返回。 如果不存在任何项目，则应返回null
因此，这是我的测试：
@Test
public void testGrab()
{
assertNull(s.grab(), "grab is not working correctly (check with empty sack)");
Random rand = new Random();
int numElements = rand.nextInt(9) + 1;
Integer[] setElementData = new Integer[10];
ArrayList<Integer> expectedElements = new ArrayList<Integer>();
int randElement;
for(int i=0; i<numElements; ++i) {
randElement = rand.nextInt(50) + 1;
if(!expectedElements.contains(randElement)) {
setElementData[i] = randElement;
expectedElements.add(randElement);
} else {
--i;
}
}
try {
elementData.set(s, setElementData);
size.set(s, numElements);
for(int i=0; i<numElements; ++i) {
expectedElements.remove(s.grab());
assertEquals(numElements-i-1, size.get(s), "grab is not working correctly (check size usage)");
}
assertEquals(0, expectedElements.size(), "grab is not working correctly (check size usage)");
} catch (Exception e) {
fail("grab is not working correctly");
}
}
让我知道您是否对我的抓取方法有任何解决方案，以了解如何完成此任务。
1楼
我不明白为什么您的grab方法返回一个数组而不是单个元素。 我可以建议您使用其他方法
public class Sack<E> {
private final static int DEFAULT_CAPACITY = 10;
private final static float REALLOC_FACTOR = 1.5f;
private E[] elementData;
private int size;
public Sack() {
this(DEFAULT_CAPACITY);
}
@SuppressWarnings("unchecked")
public Sack(int capacity) {
if(capacity <= 0)
throw new IllegalArgumentException("capacity " + capacity);
elementData = (E[]) new Object[capacity];
}
public boolean isEmpty() {
return size == 0;
}
public void add(E item) {
int index = size++;
if(size > elementData.length-1)
elementData = Arrays.copyOf(elementData, (int)(size*REALLOC_FACTOR));
elementData[index] = item;
}
public E [] dump() {
E [] E2 = Arrays.copyOf(elementData, size);
Arrays.fill(elementData, null);
size = 0;
return E2;
}
public E grab() {
if(size == 0)
return null;
int index = (int)(Math.random()*size);
E element = elementData[index];
elementData[index] = elementData[size-1];
elementData[size-1] = null;
size--;
return element;
}
}
(from 0 to effectiveSize-1) and to return this element, but before we have to swap this element with the last element (the element with index effectiveSize-1) and reduce the effective size. 这个抓取方法的思想是选择一个随机索引 （从0到有效大小1）并返回此元素，但是在我们必须将此元素与最后一个元素（索引有效大小为1的元素）交换并减小之前有效尺寸。 我之所以使用Math.rand（），是因为它在首次调用时会创建一个新的伪随机数生成器，然后将此生成器用于此方法的所有调用。
注意：我还添加了一个重新分配因子，以避免在数组饱和后为每个元素重新分配数组。
2楼
这是更简单分解的答案。 我感谢@Mirko Alicastro。
public E grab()
{
if(size == 0)
return null;
int index = (int) (Math.random() * size);
E element = elementData[index];
elementData[index] = elementData[size - 1];
elementData[size - 1] = null;
size--;
return element;
}
3楼
这是解决此方法的另一种方法，
public E grab()
{
if(isEmpty())
{
return null;
}
Random rand = new Random();
int i = rand.nextInt(size);
E item = elementData[i];
remove(i);
return item;
}