Question

请耐心等待我，我只用了一个星期的时间。

这个代码是我在完成打开和读取文件以及RegEx的章节后想要做的。我总觉得我过度复杂了代码。请看看，如果这是合理的，可以改进的地方让我知道。我还没有学到很多概念。我正在教自己一本书。

此代码用于打开日志文件（最终将实时检索），使用RegEx逐行查找“失败”，将报告失败的行添加到列表中并返回完成的列表。

谢谢！

#!/usr/bin/python

import os
import re

myFile = '/home/nick/python/chap8ReadingWritingFiles/15Jan2016_000000.txt'

def RegEx(file):
    openFile = open(myFile, 'r')
    myResList = []
    myList = openFile.readlines()
    myReg = re.compile(r'Fail')
    for i in (myList):
            #print(i, end='')
            result = myReg.search(i)
            if result != None:
                    #print(i, end='')
                    myResList.append(i.rstrip())
                    final = '\n'.join(myResList[:])
    return final

print()
print('Results:')
print(RegEx(myFile))
print()

Answer 1

你应该将它发布到codereview SE网站，但是，有更简单的方法可以做到这一点。例如：

with open(file) as f:
  return [line for line in f if myReg.search(line)]

我是不是太复杂了？ Python练习 - 初学者

1 个答案: