2

Mark Pilgrim の Dive into Python 3 を読んでいます。彼は、名詞を複数形にするための「涅槃」コードを提供しました。私の質問は: 30 行目と 31 行目のコードは必要ですか? それらがなくてもプログラムは正常に動作するようです。それらを追加する利点は何ですか？ありがとうございました

    import re
    def build_match_and_apply_functions(pattern, search, replace):
        def matches_rule(word):
            return re.search(pattern, word)
        def apply_rule(word):
            return re.sub(search, replace, word)
        return (matches_rule, apply_rule)

def plural(noun):
    for matches_rule, apply_rule in rules:
        if matches_rule(noun):
            return apply_rule(noun)
    raise ValueError('no matching rule for {0}'.format(noun))

class LazyRules:
    rules_filename = 'plural6-rules.txt'
    def __init__(self):
        self.pattern_file = open(self.rules_filename, encoding='utf-8')
        self.cache = []

    def __iter__(self):
        self.cache_index = 0
        return self

    def __next__(self):
        self.cache_index += 1
        if len(self.cache) >= self.cache_index:
            return self.cache[self.cache_index - 1]

##Line 30:        if self.pattern_file.closed:
##Line 31:            raise StopIteration

        line = self.pattern_file.readline()
        if not line:
            self.pattern_file.close()
            raise StopIteration

        pattern, search, replace = line.split(None, 3)
        funcs = build_match_and_apply_functions(pattern, search, replace)
        self.cache.append(funcs)
        return funcs

rules = LazyRules()
nouns=['fox', 'horse', 'cheetah','ad', 'quay', 'vacancy']
for noun in nouns:
    print (noun)
    print(plural(noun)+'\n')

## plural6-rules.txt:
##[sxz]$ $ es
##[^aeioudgkprt]h$ $ es
##[^aeiou]y$ y$ ies
##$   $   s
4

2 に答える 2

1

はい、本が説明しているように、これらの行は必要です（注2を参照）。

前の反復では、パターンファイルはすでに閉じられている可能性があります。

を上げるStopIterationと、forループが正常に終了します（注5を参照） 。

そうしないraise StopIterationと、次の行は閉じたファイルから読み込もうとします。

line = self.pattern_file.readline()

これにより、が発生しValueErrorます。

これらの行がいつ実行されるかを示すために、plural6-rules.txtファイルの最後の行（すべてに一致する）を削除してから、次のように入力してください。

>>> from plural6 import plural
>>> plural("fox") # matches the first rule
'foxes'
>>> plural("foo") # does not match any rules. file is closed.
>>> plural("foo") # We have removed the check, we will try to read from a closed file
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "plural6.py", line 50, in plural
    for matches_rule, apply_rule in rules:
  File "plural6.py", line 36, in __next__
    line = self.pattern_file.readline()
ValueError: I/O operation on closed file.
于 2012-05-30T23:16:05.723 に答える
-1

数行下のコード (ファイルreadline()の終わりを示す空の文字列を返すとファイルを閉じます) が発生するため、それらは必要ありませんStopIteration。ファイルが前の反復から閉じられる状況はありません。ファイルが閉じられた後もそれを呼び出し続けていたのだと思いnext()ますが、使用しているコードはそれを行いません。

そのコードは、一般的に言えばひどいものです。これはその本の典型ですか？もしそうなら、それを捨てて別のものを試してください。

于 2012-05-30T23:39:48.897 に答える