2

我编写了一个实用程序来扫描包含字母字符的所有空格分隔字段的文本文件，它工作得很好但非常慢，因为我将每一行分成单词并扫描每个单词，有没有更快的方法来做到这一点？

谢谢。

这是代码：

#!/bin/python
import argparse
import sys
import time

parser = argparse.ArgumentParser(description='Find all alpha characters in 
an input file')
parser.add_argument('file', type=argparse.FileType('r'), 
help='filename.txt')

args = parser.parse_args()

def letters(input):
    output = []
    for character in input:
        if character.isalpha():
            output = input
    return output

def main(argv):

    start = time.time()
    fname = sys.argv[1]

    f = open(fname)
    for line in f:
        words = line.rstrip().split()
        for word in words:
            alphaWord = letters(word)
            if alphaWord:
                print(alphaWord)
    f.close()

    end = time.time()
    elapsed = end - start
    print "%s secs elapsed" % elapsed

if __name__ == "__main__":
    main(sys.argv)
4

2 回答 2

5

您的程序存在以下错误letters()：

def letters(input):
    output = []
    for character in input:
        if character.isalpha():
            output = input # after we get here we'll keep iterating
                           # even though the result will not change
    return output

您正在做的是迭代所有字母，即使其中一个是字母数字，您也会将输入保存到其中，output但您还将继续迭代其余字符 - 这不会添加任何内容。

要么你想检查所有的字符（然后程序返回错误的结果），要么，如果程序返回正确的结果，你可能想break在行之后：output = input.

于 2017-11-10T16:35:02.647 回答
0 
for character in input:
   if character.isalpha(): 
         output = input 
   return output

这不会返回单词它只返回一个字母，因为它不会添加到数组中

即使确实返回了一个单词，它也会从单词中删除数字，所以如果你有类似“12ab34”的东西，它仍然会将它视为一个单词，但如果这是你的意图，那就没问题了

于 2017-11-11T11:09:08.977 回答