🚀 go-pugleaf

On 2011-11-15, Artur M. Piwko <milusi.pysiaczek@buziaczek.pl> wrote:
> In the darkest hour on Tue, 15 Nov 2011 03:48:34 +0100,
> hubert depesz lubaczewski <depesz@depesz.com> screamed:
>> def calculate_hash( filename ):
>>     f = open( filename, 'r' )
>>     mail = email.message_from_file( f )
>>     keep_headers =  ( 'subject', 'from', 'to', 'cc', 'in-reply-to', 'references', 'date', 'message-id', 'content-type', 'mime-version' )
>>     for h in mail.keys():
>>         if h.lower() not in keep_headers:
>>             del mail[ h ]
>>     return sha256( mail.as_string() ).hexdigest()
>>
>
> W obecnych Pythonach możesz stosować konstrukcję (automagicznie zamknie uchwyt):
>
> def calculate_hash(filename):
>     with open(filename) as f:
>         mail = .....

a to powyżej nie zamknie uchwytu po wyjściu z funkcji?

> f = open(filename, 'r') to to samo co f = open(filename)

pewnie tak, ale akurat wolę explicite podawać tryb otwarcia pliku, takie
zboczenie.

>> def visit_fn( hash_dict, path, files ):
>>     for i in [ 'new', 'tmp' ]:
>>         if files.count ( i ) > 0:
>>             files.remove( i )
>>
> Tu lepiej użyć tupli ('new', 'tmp').

czemu lepiej?

>>         checksum = calculate_hash( full_path )
>>         if not hash_dict.has_key( checksum ):
>>             hash_dict[ checksum ] = list()
>>         hash_dict[ checksum ].append( full_path )
>>
>> hash_dict = dict()
>>
>
> Alternatywnie (z tym, że trzeba uważać):
> hash_dict[checksum] = []
> hash_dict = {}

na co uważać?

> Oprócz konstrukcji:   if not d.has_key(k):
> można też używać:     if k not in d:
> Dla mnie ta druga jest czytelniejsza.

czy one są równoważne? w szczególności - czy ich wydajność będzie taka
sama jak d ma np. 10M elementów?

>> walk( sys.argv[1], visit_fn, hash_dict )
>>
>> for i in hash_dict.keys():
>>     if len( hash_dict[i] ) == 1:
>>         del hash_dict[i]
>>
>> for sha in hash_dict.keys():
>>     print "# %s" % ( sha )
> Alternatywnie, po prostu:   print '#', sha

dzięki za wszystkie uwagi.

depesz

-- 
The best thing about modern society is how easy it is to avoid contact with it.
                                                             http://depesz.com/