Python, лучший способ написать сумму из двух для циклов

Question

Обычно я знаю, что мы можем сделать sum([func(x,x) for x in i]), но я получил проверку if и два для циклов, так что самый пифонический способ написать код ниже. вы можете предположить, что сходство вернет число независимо от того, какой тип вы его дадите. Вы также можете предположить, что он действительно получит только ints и chars.

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

Может быть что-то вроде этого:

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

Или:

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

Или (в предположении, что строка, набор или какой-либо другой итерируемый тип по какой-либо причине не нарушает вашу функцию):

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

---

Если вы специально хотите проверить, если что-то итерация, вы можете сделать это следующим образом:

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

Если есть определенные Iterable типов вы не хотите, реагировать на те:

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

Answer 2

Поскольку if не внутри цикла, он не должен быть в списке понимания, когда вы преобразовать его:

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]) 

Между тем, вы действительно не нужен список понимание здесь, выражение генератор будет иметь тот же эффект, без необходимости создания списка:

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) 

---

Между тем, isinstance может принять tuple типов, чтобы проверить, так:

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

... но на самом деле, я не думаю, что вы хотите проверить в первую очередь. До тех пор, как a является любым итерабельным, код будет работать, поэтому ... почему я должен заставить его быть этими конкретными типами? Если вы не хотите исключение, просто обработать исключение:

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) 

да, типа не то же самое, как isistance, но это не драматический вопрос ..

, если бы OP захотел использовать этот фрагмент кода с чем-то похожим на dict, list или tuples (так что с подклассами и всеми), на самом деле нужно просто проверить, что a iserable.

что-то вроде:

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

Вы, вероятно, хотите что-то вроде:

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

Answer 5

В одной строке;)

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

Answer 6

Вы могли бы использовать некоторые функции из itertools, возможно:

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

и, как другие отметили, возможно, также передать кортеж типов в isinstance(), если вам действительно нужно проверить.