java - 解决Java算法的错误

'以下代码对某些输入有效，而对其他输入则无效
该代码是循环链表的实现
给定n =列表的长度，k =删除节点要采取的步骤
目的是删除以k为增量的那个节点
例如，如果[1,2,3]且k = 2，则要删除的第一个节点为2，然后为1
因此最后一个节点将是3
代码上有2个测试用例，第一个是n = 12 k = 4，结果应该是1，但我得到2
在最后的消除步骤中，由于某种原因，它弄乱了
sec测试用例为n = 12 k = 3，结果应为10，可以正常工作
所以我无法弄清楚是否有人可以识别购买，我很感激
谢谢'

 public static int whoIsElected(int n, int k) {
    LinkedList<Integer> circle = new LinkedList<>();
    for(int i=1;i<=n;i++)
    {
        circle.add(i);
    }
    int posToDel = k;

    if(n==1)
    {
        return 1;
    }
    if(posToDel>n)
    {
        posToDel=posToDel%n;
    }
    while(circle.size()>1)
    {
        if(posToDel==0)// becasue the posToDel-1 when posToDel ==0 is the last index
        {
            circle.remove(n-1);
        }
        else
        {
            circle.remove(posToDel-1);
        }
        n--; // decrease the length of n by 1 when u delete one from the list
        // after removing the posToDel in circle

        // this if else set the new posToDel
        if(posToDel==0)
        {
            posToDel=n-1;
        }
        else
        {
            posToDel=posToDel-1;
        }
        if(posToDel+k>n)
        {
            posToDel=(posToDel+k)%n;
        }
        else
        {
            posToDel=posToDel+k;
        }
    }
    return circle.get(0);
}

/**
 * bool doTestsPass()
 * Runs various tests. Returns true if tests pass. Otherwise,
 * returns false.
 */
public static boolean doTestsPass() {
    // todo: implement more tests, please
    // feel free to make testing more elegant
    // test cases are structered as {n, k, expected answer}
    int[][] testCases = {
            {12, 4, 1},// give wrong answer
            {12, 3, 10}// give right answer
    };

    for (int[] testCase : testCases) {
        int answer = whoIsElected(testCase[0], testCase[1]);
        if (answer != testCase[2]) {
            System.out.println("test failed!");
            System.out.printf("n:%d, k%d, answer got: %d, should be: %d\n", testCase[0], testCase[1], answer, testCase[2]);
            return false;
        }
    }
    System.out.println("All tested passed");
    return true;
}

/**
 * Execution entry point.
 */
public static void main(String args[]) {
    doTestsPass();
}

}

最佳答案

我想问题来了，只剩下3个元素[1,2,9]

因此，当手动尝试时，发现要删除元素9时posToDel应该为3，但是当我们在代码中运行时，posToDel的值为0。

您可以尝试在不同时间点添加控制台日志和跟踪值。但是它很快就会变得混乱，您可能最终会花费比所需更多的时间。

但是，我建议您在循环逻辑中尝试以下算法：

在循环分配posToDel= k-1之前，这样您就不必每次都在循环中执行posToDel-1（这只是在较少特殊情况下，使在循环中构建不同的逻辑变得容易一些）

我已经尝试过以下循环逻辑，它可以很好地用于您的测试用例：

while( circle.size() > 1 )
        {
            int temp = circle.remove( posToDel );
            System.out.println( temp );
            n--;
            posToDel = posToDel - 1;
            System.out.println( "after deletion of " + temp + ", pos is: " + posToDel );
            if( posToDel + k >= n )
            {
                posToDel = ( posToDel + k ) % n;
            }
            else
            {
                posToDel = posToDel + k;
            }
        }

添加了2条sysout语句以了解删除的顺序，这可能有助于调试