【重发】第29课作业有个比较文件的作业，请问我的代码有什么问题？

haomingzi

刚才发了问题后手贱点了个已解决，请允许在发一次

这段代码总是得不到正确的结果，而下面小甲鱼老师的方法可行，为什么啊？

我的代码：

1. def file_compare(file_name1, file_name2):
   
2.         list1 = []
   
3.         count = 0
   
4. 
   
5.         fp1 = open(file_name1)
   
6.         fp2 = open(file_name2)
   
7. 
   
8.         while True:
   
9.                 count += 1
   
10.                 line1 = fp1.readline()
    
11.                 line2 = fp2.readline()
    
12.                 if line1 != line2:
    
13.                         print(line1+ '+' +line2+ '+%d' %count)
    
14.                         list1.append(count)
    
15.                 elif (fp1.readline() == '') and (fp2.readline() == ''):
    
16.                         break
    
17.         
    
18.         fp1.close()
    
19.         fp2.close()
    
20.         return list1
    
21.                         
    
22.                
    
23. file_name1 = input('输入一个文件名：')
    
24. file_name2 = input('输入另一个文件名：')
    
25. 
    
26. list_out = file_compare(file_name1, file_name2)
    
27. if len(list_out) == 0:
    
28.         print('相同')
    
29. else:
    
30.         for each in list_out:
    
31.                 print('第%d行不同' % each)

复制代码

小甲鱼老师方法：

1. def file_compare(file_name1, file_name2):
   
2.         list1 = []
   
3.         count = 0
   
4. 
   
5.         fp1 = open(file_name1)
   
6.         fp2 = open(file_name2)
   
7. 
   
8.         for line1 in fp1:
   
9.                 line2 = fp2.readline()
   
10.                 count += 1
    
11.                 if line1 != line2:
    
12.                         print(line1+ '+' +line2+ '+%d' %count)
    
13.                         list1.append(count)
    
14.        
    
15.         fp1.close()
    
16.         fp2.close()
    
17.         return list1
    
18.                        
    
19.                
    
20. file_name1 = input('输入一个文件名：')
    
21. file_name2 = input('输入另一个文件名：')
    
22. 
    
23. list_out = file_compare(file_name1, file_name2)
    
24. if len(list_out) == 0:
    
25.         print('相同')
    
26. else:
    
27.         for each in list_out:
    
28.                 print('第%d行不同' % each)

复制代码

冬雪雪冬

你是用while循环代替了for循环，问题应该出在15行的elif判断上，如果文本文件的最后没有空行，就会出错。

小火木

1. def file_compare(file_name1, file_name2):
   
2.         list1 = []
   
3.         count = 0
   
4. 
   
5.         fp1 = open(file_name1)
   
6.         fp2 = open(file_name2)
   
7. 
   
8.         while True:
   
9.                 count += 1
   
10.                 line1 = fp1.readline()
    
11.                 line2 = fp2.readline()
    
12.                 if line1 != line2:
    
13.                         #print(line1+ '+' +line2+ '+%d' %count)
    
14.                         list1.append(count)
    
15.                # elif (fp1.readline() == '') and (fp2.readline() == ''):#虽然是判断，这里会执行一次readline()会读取一行，又根据短路逻辑，and前面为假的话，后面的不会执行，这样两个文件之后读取的行就错开了
    
16.                 elif (line1 == '') and (line2== ''):
    
17.                         break
    
18.         
    
19.         fp1.close()
    
20.         fp2.close()
    
21.         return list1
    
22.                         
    
23.                
    
24. file_name1 = input('输入一个文件名：')
    
25. file_name2 = input('输入另一个文件名：')
    
26. 
    
27. list_out = file_compare(file_name1, file_name2)
    
28. if len(list_out) == 0:
    
29.         print('相同')
    
30. else:
    
31.         for each in list_out:
    
32.                 print('第%d行不同' % each)

复制代码

haomingzi

小火木 发表于 2016-4-13 22:15

非常感谢！最开始的时候我第一个if判断也是直接read的，以为那里出错了改成了line1和line2，没有注意到后面这里...