Question

我有一种方法可以根据患者的优先级将患者添加到正确的索引中。但是，当我添加一个以上的人时，似乎没有添加。我不太确定我的添加方法中缺少什么。我的猜测if-else语句出了点问题，但我无法查明问题出在哪里。

public void addPatient(Patient sickPerson)
{
    int lim = patients.size();

    if (patients.isEmpty()) //if empty
    {
        patients.add(sickPerson);
    }
    else //if not empty
    {
        for (int i = 0; i < lim; i++) 
        {
        if (patients.get(i).compareTo(sickPerson) > 0) 
            // if the sickperson is more important...
            {
                patients.add(i, sickPerson);
            } 

        else if (patients.get(i).compareTo(sickPerson) == 0)
            // if the sickperson's priority is the same, 
            {
                patients.add(i + 1, sickPerson);
            }
        else 
            {
                continue;
            }
        }
    }

}

@Test
public void getNumWaitingTest() {

    WaitingRoom a = new WaitingRoom();
    Patient b = new Patient("name", "bad", 1, 1);
    Patient c = new Patient("name2", "bad2", 2, 2);

    a.addPatient(b);
    a.addPatient(c);

    assertEquals(2, a.getNumWaiting());
}

如果运行此测试用例，由于某种原因，我将得到numWaiting 1。该代码在哪里出错？

Answer 1

在for周期之后，如果患者的优先级低于现有患者，则不会添加任何患者。

一旦您将患者添加到列表中，就应该从方法中return；如果您到达方法的末尾，则意味着新患者的优先级最低，应将其添加到列表的末尾。

使用for循环以正确的索引将对象添加到ArrayList

1 个答案: