タプルを値として辞書の値をあらかじめ計算する

Question

次のコード例では、idの辞書にその値の2つの数値を持つ辞書resultを作成しています。

# populate the ids list (contents of the current directory which is for a speicif id name) 
ids_list = [filename for filename in glob('*' + .txt) 

def some_numerical_calc(filename): 
    # calculates and returns some number as string 
def size_of_file(filename): 
    # calculates and returns size number as string 

def count_stuff(id, filename): 
    result = { id: (some_numerical_calc(filename), size_of_file(filename)) } 

for id in ids_list: 
    for f in files_list: 
     count_stuff(id, f) 

アイデアは私が最終的に1つの辞書（おそらくこの部品は再設計必要。）の下で、これらすべての辞書のキーと値のペアを集約することです。

私が扱っている問題は、特定のidのfiles_listが1より大きい場合です。これらのケースでは、それぞれfilenameのタプル内の2つの数値に、それと同じ数値の前の数値を加算したいとします。filename例として

、

ids_list = ['001', '002', '003']id='001'ことが files_list=['file1.txt', 'file2.txt', 'file3.txt']

とを有する

some_numerical_calc('file1.txt')は10と を与える場合がその後、

some_numerical_calc('file2.txt')は、150と size_of_file('file2.txt')は35を与える与え

some_numerical_calc('file3.txt')が30と size_of_file('file3.txt')は120を与える与え、

、80を与え、私はid='001'の出力はresult = { '001': (190, 235) }

ことを期待します

私はtuplesが不変であることを知っています。私は各IDのすべてのファイルの2つの数値を事前に計算して、その特定の辞書項目を作成する実装を考え出すのに苦労しています。あるいは、おそらく、私はnamedtuplesを使用し、set（？）に2つの数値を格納することを望んでいたとしても、タプル構造を削除する必要があります。どんな提案も高く評価されます。

効率的で非平凡な提案のために願っています。

Answer 1

問題の一部は、コードをひどく整理したことです。あまりにも早くあなたの辞書を作成しています。

def count_stuff(id, filename): 
    return (some_numerical_calc(filename), size_of_file(filename)) 

for id in ids_list: 
    nums = 0 
    sizes = 0 
    for f in files_list: 
     num, size = count_stuff(id, f) 
     nums += num 
     sizes += sizes 
    result = { id: (nums, sizes) } 

、あなたのデータを集計した後今、あなたの辞書が作成されます。

は、あなたがそれをこのような何かを再編成した場合に考えてみましょう。合計が含まれているタプルを作成するには

Answer 2

、あなたは*このような何かを行うことができます。あなたの現在のコードを使用して、

result[id] = (sum(some_numerical_calc(filename) for filename in files_list), 
       sum(size_of_file(filename) for filename in files_list)) 

しかし、単にヘッドアップなどを、これがための辞書で同じタプル値を格納しますあなたのidキーのすべて特定のfiles_listと特定のidとを関連付ける方法は現在ありません。

* files_listを2回でなく1回だけ繰り返したい場合は、Python element-wise tuple operations like sumから回答の1つを適応させることができます。

Answer 3

あなたがマップとの和を混合試みることができる：

resultDict = {} 

for id in ids_list: 
    resultDict[id] = (sum(map(some_numerical_calc(files_list[id]))), sum(map(size_of_file(files_list[id])))) 

編集：あなたの特定の状況与え

より詳細な例。いくつかの部分は、角括弧内のコメントに記述されます。

#<Get list of ids as strings> 
files_list = {} #Initialize the files dictionary. 

for id in ids_list: 
    #<Switch to directory based on id> 
    files_list[id] = [filename for filename in glob('*' + .txt)] 

def some_numerical_calc(filename): 
    # calculates and returns some number as string 
def size_of_file(filename): 
    # calculates and returns size number as string 

result_dict = {} #Init results. 

for id in ids_list: 
    resultDict[id] = (sum(map(some_numerical_calc(files_list[id]))), sum(map(size_of_file(files_list[id])))) 

Answer 4

あなたのコードについての奇妙ないくつかのものがありますが、1のために、あなたは新しい辞書にあなたがcount_stuffを呼び出すたびに作成しているが、それを使って何をやったことがないか、それを返します。あなたはすべてが1つの辞書に追加されたように思える質問から。

このような何かがよりよく働くかもしれない：

def some_numerical_calc(filename): 
    # calculates and returns some number as string 
def size_of_file(filename): 
    # calculates and returns size number as string 

def count_stuff(id, file_list): 
    some_number = 0 
    size = 0 
    for filename in file_list: 
     some_number += some_numerical_calc(filename) 
     size += size_of_file(filename) 
    return (some_number, size) 

results = {} 
for id in ids_list: 
    results[id] = count_stuff(id, file_list)) 
print results