[已解决]第29讲 第1题，代码不知道怎么优化

11tadpole

try1.txt 内容：
ab
abc
abcd
abcdf

try2.txt 内容：
abc
abc
abcd
abcdf
666

我本意是程序判断出第1行和第5行不同，但没想到第4行也被判定为不同的了，观察发现是try1.txt文件末尾没有'\n'造成的
不知道怎么改进代码，能让程序判断得和人直观感受一样，第4行判断为相同

1. def compareTxt(file_name1, file_name2):
   
2.     f1 = open(file_name1, 'r')
   
3.     f2 = open(file_name2, 'r')
   
4.     difference = 0
   
5.     count = 0
   
6.     list1 = list()
   
7.     while True:
   
8.         one = f1.readline()
   
9.         print('f1:' + one)
   
10.         another = f2.readline()
    
11.         print('f2:' + another)
    
12.         count += 1
    
13.         if one == '' and another == '':
    
14.             break
    
15.         if one != another:
    
16.             difference += 1
    
17.             list1.append(count)
    
18. 
    
19.     print('两个文件共有【%d】处不同：' % difference)
    
20.     for each in list1:
    
21.         print('第 %d 行不一样' % each)
    
22.         
    
23.     f1.close()
    
24.     f2.close()
    
25. 
    
26. 
    
27. 
    
28. file_name1 = input('请输入需要比较的头一个文件名：')
    
29. file_name2 = input('请输入需要比较的另一个文件名：')
    
30. compareTxt(file_name1, file_name2)
    

复制代码

最佳答案

月排行榜 / 总排行榜

冬雪雪冬

2021-9-23 09:48:39

1. import itertools
   
2. def compareTxt(file_name1, file_name2):
   
3.     f1 = open(file_name1, 'r')
   
4.     f2 = open(file_name2, 'r')
   
5.     list1 = list(f1)
   
6.     list2 = list(f2)
   
7.     list3 = []
   
8.     for line, each in enumerate(itertools.zip_longest(list1, list2)):
   
9.         if None in each or each[0].replace('\n', '') != each[1].replace('\n', ''):
   
10.             list3.append(line + 1)
    
11.     print('两个文件共有【%d】处不同：'%len(list3))
    
12.     for each in list3:
    
13.         print('第 %d 行不一样'%each)
    
14.         
    
15.     f1.close()
    
16.     f2.close()
    
17. 
    
18. 
    
19. 
    
20. file_name1 = input('请输入需要比较的头一个文件名：')
    
21. file_name2 = input('请输入需要比较的另一个文件名：')
    
22. compareTxt(file_name1, file_name2)

复制代码

1. 请输入需要比较的头一个文件名：try1.txt
   
2. 请输入需要比较的另一个文件名：try2.txt
   
3. 两个文件共有【2】处不同：
   
4. 第 1 行不一样
   
5. 第 5 行不一样

复制代码

跳转到最佳答案楼层

newu

读进来之后用 strip()函数试试去除两头的空白符

冬雪雪冬

1. import itertools
   
2. def compareTxt(file_name1, file_name2):
   
3.     f1 = open(file_name1, 'r')
   
4.     f2 = open(file_name2, 'r')
   
5.     list1 = list(f1)
   
6.     list2 = list(f2)
   
7.     list3 = []
   
8.     for line, each in enumerate(itertools.zip_longest(list1, list2)):
   
9.         if None in each or each[0].replace('\n', '') != each[1].replace('\n', ''):
   
10.             list3.append(line + 1)
    
11.     print('两个文件共有【%d】处不同：'%len(list3))
    
12.     for each in list3:
    
13.         print('第 %d 行不一样'%each)
    
14.         
    
15.     f1.close()
    
16.     f2.close()
    
17. 
    
18. 
    
19. 
    
20. file_name1 = input('请输入需要比较的头一个文件名：')
    
21. file_name2 = input('请输入需要比较的另一个文件名：')
    
22. compareTxt(file_name1, file_name2)

复制代码

1. 请输入需要比较的头一个文件名：try1.txt
   
2. 请输入需要比较的另一个文件名：try2.txt
   
3. 两个文件共有【2】处不同：
   
4. 第 1 行不一样
   
5. 第 5 行不一样

复制代码

newu

冬雪雪冬 发表于 2021-9-23 09:48

版主真有耐心哇

11tadpole

冬雪雪冬 发表于 2021-9-23 09:48

1. if None in each or each[0].replace('\n', '') != each[1].replace('\n', ''):

复制代码

再请教一下，这个判断语句中，我没有想到什么情况下会None in each

11tadpole

冬雪雪冬 发表于 2021-9-23 09:48

1. zip_longest( iterable1, iterable2, fillval)

复制代码

因此，我觉得可以直接写

1. for line, each in enumerate(itertools.zip_longest(f1, f2)):

复制代码

不需要

1.     list1 = list(f1)
   
2.     list2 = list(f2)

复制代码

尝试了，没有报错

11tadpole

备注新知识点：

1. str.replace(old, new[, max])

复制代码

replace()方法仅适用于str，不可用于其他类型

冬雪雪冬

11tadpole 发表于 2021-9-25 01:08
再请教一下，这个判断语句中，我没有想到什么情况下会None in each

当短的文件已没有内容时，长文件还在循环。

冬雪雪冬

11tadpole 发表于 2021-9-25 01:20
因此，我觉得可以直接写

不需要

是的，不需要。

11tadpole

冬雪雪冬 发表于 2021-9-25 12:09
当短的文件已没有内容时，长文件还在循环。

当短文件没有内容时，长文件还有内容，此时each不为空
所以我感觉None in each好像也是多余的

冬雪雪冬

11tadpole 发表于 2021-9-28 01:25
当短文件没有内容时，长文件还有内容，此时each不为空
所以我感觉None in each好像也是多余的

当短文件没有内容时，长文件还有内容时each是一个元组，其中一个元素有值，一个元素为None

11tadpole

冬雪雪冬 发表于 2021-9-28 08:12
当短文件没有内容时，长文件还有内容时each是一个元组，其中一个元素有值，一个元素为None

我明白了，不过感觉each[0].replace('\n', '') != each[1].replace('\n', '')就已经可以包含短文件没有内容，长文件还有内容这种情况了