因此,我正在创建一个名为“ 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
我的测试就在这里,我不能修改我的测试,只能提供所提供的代码,因为那里存在错误,我需要我之前所说的内容:
grab方法应随机从麻袋中取出物品并返回。如果没有物品 目前,它应该返回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");
}
}
让我知道您是否对我的获取方法有任何解决方案,以了解如何完成此任务。
答案 0 :(得分: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;
}
}
这种获取方法背后的想法是选择一个随机索引 rand (从0到有效大小-1)并返回此元素,但是在我们必须将此元素与最后一个元素(索引有效尺寸为1的元素)并减小有效尺寸。我之所以使用Math.rand(),是因为它在首次调用时会创建一个新的伪随机数生成器,然后将此生成器用于此方法的所有调用。
注意:我还添加了一个重新分配因子,以避免在数组饱和后为每个元素重新分配数组。
答案 1 :(得分:0)
这里的答案更简单了。我感谢@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;
}
答案 2 :(得分:0)
这是解决此方法的另一种方法
public E grab()
{
if(isEmpty())
{
return null;
}
Random rand = new Random();
int i = rand.nextInt(size);
E item = elementData[i];
remove(i);
return item;
}