我想测试我的程序,为了测试它,我只是将一个ListNode的整数转换为String,并将这些转换连接起来。例如,如果我有:
ListNode object1;
object1 = new ListNode(2);
object1 = new ListNode(4);
object1 = new ListNode(3);
addtwonnumbers()的输出应该是"243"(该方法的目标是不同的,我只是想测试它),但它给了我"12"。而且Eclipse也不会在这个程序中运行调试器,我不知道为什么。
public class ListNode {
int val;
ListNode next;
ListNode(int x) {
val = x;
}
public String addTwoNumbers(ListNode l1, ListNode l2) {
String l1Digits = "";
String l2Digits = "";
while(l1 != null) {
l1Digits += Integer.toString(l1.val) + "";
l1 = l1.next;
}
while(l2 != null) {
l2Digits += Integer.toString(l2.val) + "";
l2 = l2.next;
}
return l1Digits;
}
class Tester {
public void main(String[] args) {
ListNode object1;
object1 = new ListNode(2);
object1 = new ListNode(4);
object1 = new ListNode(3);
ListNode object2;
object2 = new ListNode(5);
object2 = new ListNode(6);
object2 = new ListNode(4);
System.out.println(addTwoNumbers(object1, object2));
}
}
}
而不是:
ListNode object1; object1 = new ListNode(2); object1 = new ListNode(4); object1 = new ListNode(3); ListNode object2; object2 = new ListNode(5); object2 = new ListNode(6); object2 = new ListNode(4);
看来你是认真的:
ListNode object1;
object1 = new ListNode(2);
object1.next = new ListNode(4);
object1.next.next = new ListNode(3);
ListNode object2;
object2 = new ListNode(5);
object2.next = new ListNode(6);
object2.next.next = new ListNode(4);
在原始代码中,您覆盖了object1
和object2
的值。这相当于你的原始代码,当然不是你想要的:
ListNode object1 = new ListNode(3);
ListNode object2 = new ListNode(4);
要创建更长的列表,这可能会很繁琐。您可以创建一个帮助器方法来简化此操作,例如:
ListNode createList(int...values) {
if (values.length == 0) {
return null;
}
ListNode head = new ListNode(values[0]);
ListNode node = head;
for (int i = 1; i < values.length; ++i) {
node.next = new ListNode(values[i]);
node = node.next;
}
return head;
}
这将允许您用以下代码替换顶部的第一个代码:
ListNode object1 = createList(2, 4, 3);
ListNode object2 = createList(5, 6, 4);
顺便说一句,你的程序中还有其他问题。在addTwoNumbers
中,您分配给l2Digits
,但永远不会访问它。它似乎完全没有被使用过,也毫无意义。该方法只是将第一个列表中的值连接起来并返回它,所以它没有做任何它名字所暗示的事情
每次为object1
分配新值时,都隐式地删除了之前分配的所有值。您应该通过分配next
字段来将节点附加到LinkNode的后面。
你应该在ListNode里面有一个类似addToTail
的方法。
public class ListNode {
// ... all the fields and methods you already have
public void addToTail(ListNode newNode) {
if (this == null) {
this = newNode;
return;
}
ListNode tmp = this;
// traverse to the tail node
while(tmp.next) { tmp = tmp.next; }
tmp.next = newNode;
return;
}
}
ListNode object1;
object1.addToTail(new ListNode(2)); // (2)
object1.addToTail(new ListNode(4)); // (2) -> (4)
object1.addToTail(new ListNode(3)); // (2) -> (4) -> (3) .. and so on