You are given two non-empty linked lists representing two non-negative integers. The digits are stored in reverse order and each of their nodes contain a single digit. Add the two numbers and return it as a linked list.
You may assume the two numbers do not contain any leading zero, except the number 0 itself.
Example
Input: (2 -> 4 -> 3) + (5 -> 6 -> 4) Output: 7 -> 0 -> 8 Explanation: 342 + 465 = 807.
虽然也做出来了这个题,但是一开始忽略了很多点,感觉自己还是做题做得少,基础不够扎实。
首先,有一点的事实我没有清晰地确认,每两个数字的和的进位(carray)只可能是0或者是1,因为最大的数是9+9+1=19,不可能进二的,所以在我的代码中,都是假设进位可能是任意数的。每次加都要是两个数加上进位,也是后来想起来的,初始把它给忽略了。
第二点,当最后一位也需要进位的时候,需要在最后补加一位数,这一点应该属于边界条件。
第三点,由于太长时间没有写算法的代码,我犯了一个致命的错误,应该先对list.next赋值,然后再移动list = list.next;我当时直接对list赋值,然后使list.next 为NULL,在list=list.next,脑洞也真够大的!对这一块要好好反省!
第四,编码习惯问题。对于链表问题,一定注意最后返回的都是一个链表节点,而且是特殊的头结点。所以自己定义或者使用已有的链表时,一定尽量保证已有的链表头结点的引用不变,而设置一个新的引用指向它。比如说我自己定义返回链表的头指针就不是很好,答案里面初始化的时候直接定义了一个val为0的节点,最后返回该节点的下一个值就可以。我当时是把循环中的逻辑给拿出来了一部分,进行了第一个节点的初始化,使得代码逻辑冗余。
去掉开始的冗余部分,我的答案如下:
public ListNode addTwoNumbers(ListNode l1, ListNode l2) {
int tmp = 0;
ListNode list = new ListNode(0);
ListNode head = list;
while(null != l1 && null != l2){
int num = (l1.val + l2.val + tmp) % 10;
list.next = new ListNode(num);
list = list.next;
tmp = (l1.val + l2.val + tmp) / 10;
l1 = l1.next;
l2 = l2.next;
}
if(null != l1) {
l2 = l1;
}
while(null != l2) {
int num = (l2.val + tmp) % 10;
list.next = new ListNode(num);
tmp = (l2.val + tmp) / 10;
list = list.next;
l2 = l2.next;
}
if(tmp!=0){
list.next = new ListNode(tmp);
list = list.next;
}
return head.next;
}
首先,根据链表的特点,可以设置新的头结点指向已有的链表指针,还有重要的一个逻辑点,就是当一个链表遍历完后,直接让它的下一个值为0即可,也就是别人答案中的若p!=null,就把当前值进行运算,若为空了,直接使他等于0即可。改进后为:
public ListNode addTwoNumbers(ListNode l1, ListNode l2) {
int tmp = 0;
ListNode list = new ListNode(0);
ListNode head = list;
ListNode p = l1;
ListNode q = l2;
while(null != p || null != q){
int x = (p != null) ? p.val : 0;
int y = (q != null) ? q.val : 0;
int num = (x + y + tmp) % 10;
list.next = new ListNode(num);
list = list.next;
tmp = (x + y + tmp) / 10;
if(p!=null)
p = p.next;
if(q!=null)
q = q.next;
}
if(tmp!=0){
list.next = new ListNode(tmp);
// list = list.next;
}
return head.next;
}