为什么代码只显示“1 2”?

时间:2015-10-17 18:01:09

标签: java linked-list nodes

我想测试我的程序并测试它我只是将一个ListNode的整数转换为String并连接这些转换。例如,如果我有:

ListNode object1;
object1 = new ListNode(2);
object1 = new ListNode(4);
object1 = new ListNode(3);

addTwoNumbers()的输出应该是“243”(方法的目标是不同的,我只想测试它)但是它给了我“1 2”。 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));

  }
  }
}

2 个答案:

答案 0 :(得分:1)

而不是:

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);

在原始代码中,您将覆盖object1object2的值。 这相当于您的原始代码,当然不是您想要的:

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但从不访问它。 它似乎完全没用,毫无意义。 该方法简单地连接第一个列表中的值并返回它, 所以它没有做任何顾名思义。

答案 1 :(得分:0)

每次为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