java - 解决java算法的bug

Question

'以下代码对于某些输入可以正常工作，但对于其他输入则不能正常工作该代码是循环链表的实现给定 n = 列表长度和 k= 删除节点所需的步骤目的是删除增量为 k 的节点例如，如果 [1,2,3] 且 k =2，则要删除的第一个节点是 2，然后是 1因此最后一个节点将是 3代码上有 2 个测试用例，第一个测试用例是 n=12 k=4，结果应该是 1，但我得到 2在最后的消除步骤中由于某种原因它变得困惑第二个测试用例是 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();
}

}

Answer 1

我猜当只剩下 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 语句来了解删除的顺序，这可能有助于调试