我在做一个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();
}
如您所见,调用每个
llistItem
值llist.insertNode(llistItem)
来向列表中添加一个项但是,这个方法不能创建循环,因为它每次都创建一个新的SinglyLinkedListNode
因此,即使一些llistItem
值相同,包含它们的节点也总是不同的。