pythonのutilファイルでグローバル変数を避ける

Question

pythonプロジェクトにはutilities.pyというファイルがあります。ファイルから

1. 読み取り、valid.txt、すべての数字が含まれています。それは、など

   は、今私が持っている機能is_valid(number)、へを持ちたい、たとえばis_float(string)、is_empty(file)ため、関数のみUTIL含まれていそれらは有効であり、それらをマップ/セットにロードする。

2. numberの存在をマップで確認し、TrueまたはFalseを返します。

この関数は頻繁に呼び出され、実行時間はできるだけ小さくする必要があります。私はオープンを読んで、その関数が呼び出されるたびにvalid.txtと読むのは嫌です。私が思いついた唯一の解決策は、valid.txtをインポートしたときにグローバル変数valid_dictを1回ロードすることです。ローディングコードは、utilities.pyにメインとして書かれています。

私の質問は、それが悪い習慣とみなされるので、グローバル変数を使用せずにこれをどうやって行うのですか？グローバルを使用せずにこのようなタスクを実行するための良い設計パターンは何ですか？また、このファイルはutilファイルであることに注意してください。

Answer 1

以下は、クロージャの簡単な例です。辞書cacheは、外部関数（load_func）内にカプセル化されていますが、返された場合でも内部のスコープ内に残ります。 load_funcは内部関数をオブジェクトとして返し、オブジェクトを呼び出さないことに注意してください。 utilities.pyで

：__main__で

def _load_func(filename): 

    cache = {} 

    with open(filename) as fn: 
     for line in fn: 
      key, value = line.split() 
      cache[int(key)] = value 

    def inner(number): 
     return number in cache 

    return inner 

is_valid = _load_func('valid.txt') 

：

from utilities import is_valid     # or something similar 

if is_valid(42): 
    print(42, 'is valid') 
else: 
    print(42, 'is not valid') 

辞書（cache）作成は辞書内包表記を使用して行われている可能性が、私はあなたが閉鎖に集中したかったです。

Answer 2

変数valid_dictはグローバルではなく、utilities.pyのローカルです。もしあなたがfrom utilities import *のようなことをしたら、それはグローバルになるでしょう。今度ははとなり、パッケージを開発しているときに悪い習慣とみなされています。

しかし、このような場合は基本的に静的変数が必要です：valid_dict={}をis_valid()に追加してください。この辞書は、一度だけインスタンス化され、関数が呼び出されるたびにdictがvalid_dictで使用可能になります。

def is_valid(number, valid_dict={}): 
    if not valid_dict: 
     # first call to is_valid: load valid.txt into valid_dict 
    # do your check 

がif -clauseにvalid_dictに割り当てたが、だけにそれを変更しないの操作を行います。例えば、キーvalid_dict[x] = yを設定したりvalid_dict.update(z)のようなものを使用することによって。

（PS：これは「ダーティ」または「非神託」と見なされた場合、私に教えてください）