我在做一个hackerrank problem的实验,似乎不管我尝试的溶液有什么变化,我都无法让循环检测工作。
这是我使用的代码

static boolean hasCycle(SinglyLinkedListNode head) {
        if (head == null) return false;

        SinglyLinkedListNode slow, fast;
        slow = head;
        fast = head;

        while (fast != null && fast.next != null) {
            slow = slow.next;
            fast = fast.next.next;
            if (slow == fast) return true;
        }
        return false;
    }

我可以调整解决方案使另一个测试通过,但不能同时通过在这种情况下,真理永远不会回来,即使它应该是我该怎么解决,我做错什么了?

最佳答案

这和Hackerrank本身有关一段时间前,我尝试了自己的解决方案,它通过了所有的测试,但对于存在周期的情况,它也失败了所以,我看了一下Hackerrank用于在测试用例中创建列表的insertNode方法:

public void insertNode(int nodeData) {
    // here a new node object is created each time a method is called
    SinglyLinkedListNode node = new SinglyLinkedListNode(nodeData);

    if (this.head == null) {
        this.head = node;
    } else {
        this.tail.next = node;
    }

    this.tail = node;
}

以及如何在他们的main中使用它:
public static void main(String[] args) throws IOException {
    BufferedWriter bufferedWriter = new BufferedWriter(new FileWriter(System.getenv("OUTPUT_PATH")));

    int tests = scanner.nextInt();
    scanner.skip("(\r\n|[\n\r\u2028\u2029\u0085])?");

    for (int testsItr = 0; testsItr < tests; testsItr++) {
        int index = scanner.nextInt();
        scanner.skip("(\r\n|[\n\r\u2028\u2029\u0085])?");

        SinglyLinkedList llist = new SinglyLinkedList();

        int llistCount = scanner.nextInt();
        scanner.skip("(\r\n|[\n\r\u2028\u2029\u0085])?");

        for (int i = 0; i < llistCount; i++) {
            int llistItem = scanner.nextInt();
            scanner.skip("(\r\n|[\n\r\u2028\u2029\u0085])?");

            // a new node is inserted each time so no cycle can be created whatsoever.
            llist.insertNode(llistItem);
        }

        boolean result = hasCycle(llist.head);

        bufferedWriter.write(String.valueOf(result ? 1 : 0));
        bufferedWriter.newLine();
    }

    bufferedWriter.close();

    scanner.close();
}

如您所见,调用每个llistItemllist.insertNode(llistItem)来向列表中添加一个项但是,这个方法不能创建循环,因为它每次都创建一个新的SinglyLinkedListNode因此,即使一些llistItem值相同,包含它们的节点也总是不同的。

10-06 13:01