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

11tadpole

try1.txt 内容：
ab
abc
abcd
abcdf

try2.txt 内容：
abc
abc
abcd
abcdf
666

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

def compareTxt(file_name1, file_name2):
    f1 = open(file_name1, 'r')
    f2 = open(file_name2, 'r')
    difference = 0
    count = 0
    list1 = list()
    while True:
        one = f1.readline()
        print('f1:' + one)
        another = f2.readline()
        print('f2:' + another)
        count += 1
        if one == '' and another == '':
            break
        if one != another:
            difference += 1
            list1.append(count)

    print('两个文件共有【%d】处不同：' % difference)
    for each in list1:
        print('第 %d 行不一样' % each)
        
    f1.close()
    f2.close()



file_name1 = input('请输入需要比较的头一个文件名：')
file_name2 = input('请输入需要比较的另一个文件名：')
compareTxt(file_name1, file_name2)

最佳答案

月排行榜 / 总排行榜

冬雪雪冬

2021-9-23 09:48:39

import itertools
def compareTxt(file_name1, file_name2):
    f1 = open(file_name1, 'r')
    f2 = open(file_name2, 'r')
    list1 = list(f1)
    list2 = list(f2)
    list3 = []
    for line, each in enumerate(itertools.zip_longest(list1, list2)):
        if None in each or each[0].replace('\n', '') != each[1].replace('\n', ''):
            list3.append(line + 1)
    print('两个文件共有【%d】处不同：'%len(list3))
    for each in list3:
        print('第 %d 行不一样'%each)
        
    f1.close()
    f2.close()



file_name1 = input('请输入需要比较的头一个文件名：')
file_name2 = input('请输入需要比较的另一个文件名：')
compareTxt(file_name1, file_name2)

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

跳转到最佳答案楼层

newu

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

冬雪雪冬

import itertools
def compareTxt(file_name1, file_name2):
    f1 = open(file_name1, 'r')
    f2 = open(file_name2, 'r')
    list1 = list(f1)
    list2 = list(f2)
    list3 = []
    for line, each in enumerate(itertools.zip_longest(list1, list2)):
        if None in each or each[0].replace('\n', '') != each[1].replace('\n', ''):
            list3.append(line + 1)
    print('两个文件共有【%d】处不同：'%len(list3))
    for each in list3:
        print('第 %d 行不一样'%each)
        
    f1.close()
    f2.close()



file_name1 = input('请输入需要比较的头一个文件名：')
file_name2 = input('请输入需要比较的另一个文件名：')
compareTxt(file_name1, file_name2)

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

newu

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

版主真有耐心哇

11tadpole

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

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

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

11tadpole

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

zip_longest( iterable1, iterable2, fillval)

因此，我觉得可以直接写

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

不需要

    list1 = list(f1)
    list2 = list(f2)

尝试了，没有报错

11tadpole

备注新知识点：

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', '')就已经可以包含短文件没有内容，长文件还有内容这种情况了