Python, il modo migliore per scrivere una somma di due per loop

Question

Normalmente so che possiamo fare sum([func(x,x) for x in i]), ma ho ottenuto un controllo if e due loop, quindi qual è il modo più poderoso di scrivere il codice qui sotto. puoi presumere che la somiglianza restituirà un numero indipendentemente dal tipo che gli dai. Puoi anche presumere che in realtà otterrà solo interi e caratteri.

x = 0 
if isinstance(a, dict) or isinstance(a, list) or isinstance(a, tuple): 
    for i in a: 
     for j in b: 
      x += similarity (i,j) 

Answer 1

Forse qualcosa di simile:

x=0 
if isinstance(a,(dict,list,tuple)): 
    x=sum(similarity(i,j) for i in a for j in b) 

Oppure:

x=(sum(similarity(i,j) for i in a for j in b) if isinstance(a,(dict,list,tuple)) 
    else 0) 

O (supponendo che una stringa, set o qualche altro tipo iterabile non interrompe la funzione per qualche motivo):

try: 
    x=sum(similarity(i,j) for i in a for j in b) 
except TypeError: 
    x=0 

---

Se sono specificamente cercando di verificare se qualcosa è iterabile, si può fare in questo modo:

from collections import Iterable 
if isinstance(e, Iterable): 
    ... 

Se ci sono alcuni tipi iterabili che non si desidera, reagiscono a quelle:

if isinstance(e, Iterable) and not isinstance(el, str): 
    # an iterable that is not a string... 

Answer 2

Dal momento che il if non è all'interno di un ciclo, non ha bisogno di essere dentro la lista di comprensione quando si converte:

x = 0 
if isinstance(a, dict) or isinstance(a, list) or isinstance(a, tuple): 
    x = sum([similarity(i, j) for i in a for j in b]) 

Nel frattempo, davvero non hanno bisogno di una lista la comprensione qui, un generatore di espressione avrà lo stesso effetto senza dover costruire una lista:

x = 0 
if isinstance(a, dict) or isinstance(a, list) or isinstance(a, tuple): 
    x = sum(similarity(i, j) for i in a for j in b) 

---

Nel frattempo, isinstance può prendere una tuple di tipi per controllare, in modo da:

x = 0 
if isinstance(a, (dict, list, tuple)): 
    x = sum(similarity(i, j) for i in a for j in b) 

... ma in realtà, non credo che si desidera controllare in primo luogo. Finché a è qualsiasi iterabile, il codice funzionerà, quindi ... perché dovrei costringerlo a essere quei tipi specifici? Se non si vuole un'eccezione, basta gestire l'eccezione:

try: 
    x = sum(similarity(i, j) for i in a for j in b)) 
except TypeError: 
    x = 0 

Answer 3

if type(a) in [dict, list, tuple]: 
    x += sum(similarity(i, j) for i in a for j in b) 

sì, tipo non è lo stesso di isistance ma non è un problema drammatico ..

se l'OP volesse usare quel pezzo di codice con qualcosa che assomiglia a dict, list o tuples (quindi con sottoclassi e tutto), dovrebbe davvero controllare che l'a sia iterable.

qualcosa di simile:

def isiterable(obj): 
    try: 
     iter(obj) 
     return True 
    except TypeError: 
     return False 

if isiterable(a): 
    x += sum(similarity(i, j) for i in a for j in b) 

Answer 4

probabilmente si desidera qualcosa di simile:

if isinstance(a, (dict, list, tuple)): 
    x = sum(similarity(i, j) for i in a for j in b) 
else: 
    x = 0 

Answer 5

In una riga;)

x = sum(similarity(i, j) for i in a for j in b) if isinstance(a, (dict, list, tuple)) else 0 

Answer 6

Si potrebbe utilizzare alcune funzioni da itertools, forse:

from itertools import starmap, product 
x = sum(starmap(similarity, product(a, b))) 

e come altri hanno notato, può anche passare una tupla di tipi di isinstance() se si ha realmente bisogno di controllare.