Какой лучший способ конкатенировать строку в python?

Question

Поймите «лучше», как более быструю, изящную и удобочитаемую.

У меня есть две строки (a и b), которые могут быть нулевыми или нет. И я хочу объединить их разделенных дефисом, только если оба не равно нулю:

a - b

a (если б равна нулю)

b (где нулевая)

Answer 1

# Concatenates a and b with ' - ' or Coalesces them if one is None 
'-'.join([x for x in (a,b) if x]) 

Редактировать
Вот результаты этого алгоритма (Заметим, что ни будет работать так же, как «»):

>>> '-'.join([x for x in ('foo','bar') if x]) 
'foo-bar' 
>>> '-'.join([x for x in ('foo','') if x]) 
'foo' 
>>> '-'.join([x for x in ('','bar') if x]) 
'bar' 
>>> '-'.join([x for x in ('','') if x]) 
'' 

* Также обратите внимание, что оценка Рафаэля в его сообщении ниже показала разницу в 0,0002 секунды над 1000 итерациями метода фильтра, можно предположить, что такая небольшая разница может быть вызвана несогласованностью в ava во время запуска скрипта. Я выполнил его timeit-реализацию на нескольких итерациях и обнаружил, что любой алгоритм будет быстрее примерно в 50% случаев, ни с большим отрывом. Таким образом, они в основном эквивалентны.

Answer 2

Вот один опция:

("%s - %s" if (a and b) else "%s%s") % (a,b) 

EDIT: Как указано на mgilson, этот код будет неудачной с None «са лучший способ (но менее читаемым один) будет:

"%s - %s" % (a,b) if (a and b) else (a or b) 

Answer 3

Как о чем-то простым, как:

# if I always need a string even when `a` and `b` are both null, 
# I would set `output` to a default beforehand. 
# Or actually, as Supr points out, simply do `a or b or 'default'` 
if a and b: 
    output = '%s - %s' % (a, b) 
else: 
    output = a or b 

Edit: Много интересных решений в этой теме. Я выбрал это решение, потому что я подчеркивал удобочитаемость и быстроту, по крайней мере, с точки зрения реализации. Это не самое масштабируемое или интересное решение, но для этой области действия оно работает и позволяет быстро перейти к следующей проблеме.

Answer 4

Попробуйте это:

def myfunc(a,b): 
    if not b: 
     return a 
    elif not a: 
     return b 
    else: 
     return a+' - '+b 

Или

def myfunc(a,b): 
    if a and b: 
     return a+' - '+b 
    else: 
     return a or b 

Answer 5

Ого, кажется, как горячий вопрос: р Мое предложение:

' - '.join(filter(bool, (a, b))) 

Что дает:

>>> ' - '.join(filter(bool, ('', ''))) 
'' 
>>> ' - '.join(filter(bool, ('1', ''))) 
'1' 
>>> ' - '.join(filter(bool, ('1', '2'))) 
'1 - 2' 
>>> ' - '.join(filter(bool, ('', '2'))) 
'2' 

Очевидно, None ведет себя как '' с помощью этого кода.

Answer 6

Там много ответов здесь :)

Две лучшие ответы (производительность и чистый код в одной строке) являются ответы @icecrime и @Hoopdady

Оба asnwers результаты одинаково, с той лишь разницей это производительность.

cases = [ 
(None, 'testB'), 
('', 'testB'), 
('testA', 'testB'), 
('testA', ''), 
('testA', None), 
(None, None) 
] 

for case in cases: print '-'.join(filter(bool, case)) 
'testB' 
'testB' 
'testA-testB' 
'testA' 
'testA' 

for case in cases: print '-'.join([x for x in case if x]) 
'testB' 
'testB' 
'testA-testB' 
'testA' 
'testA' 

Так что давайте делать тест :)

import timeit 

setup = ''' 
cases = [ 
    (None, "testB"), 
    ("", "testB"), 
    ("testA","testB"), 
    ("testA", ""), 
    ("testA", None), 
    (None, None) 
] 
''' 

print min(timeit.Timer(
    "for case in cases: '-'.join([x for x in case if x])", setup=setup 
).repeat(5, 1000)) 
0.00171494483948 

print min(timeit.Timer(
    "for case in cases: '-'.join(filter(bool, case))", setup=setup 
).repeat(5, 1000)) 
0.00283288955688 

Но, как сказал @mgilson, используя None вместо bool как функция filter производит тот же результат и имеют достаточно большую производительность:

print min(timeit.Timer(
    "for case in cases: '-'.join(filter(None, case))", setup=setup 
).repeat(5, 1000)) 
0.00154685974121 

Таким образом, лучший результат ответ дал на @icecrime с предложением от @mgilson:

'-'.join(filter(None, (a,b))) 

Разница рабочих характеристик составляет миллисекунды на 1000 итераций (микросекунды на итерацию). Таким образом, эти два метода имеют совершенно равную производительность, и для практически любого проекта вы можете выбрать любой; В случае, если ваш проект должен иметь более высокую производительность, учитывая микросекунд, вы могли бы следовать этому бенчмарк :)

Answer 7

Что-то pythonian, читаемый и элегантную:

strings = string1, string2 

'{0}{1}{2}'.format(
    # output first string if it's not empty 
    strings[0] if strings[0] else '', 

    # join with hyphen if both strings are not empty  
    '-' if all(strings) else '', 

    # output second string if it's not empty 
    strings[1] if strings[1] else '' 
    ) 

И слишком быстро;)

Answer 8

Я хотел бы сделать это нравится:

def show_together(item1=None, item2=None, seperator='-'): 
    return '%s%s%s' % (item1,seperator,item2) if item1 and item2 else item1 or item2 



>>> show_together(1,1) 
'1-1' 

>>> show_together(1) 
1 

>>> show_together() 
>>> show_together(4,4,'$') 
'4$4' 

Answer 9

Я просто хотел предложить решение toxotes' переписан как один лайнер с использованием format.

output = "{0} - {1}".format(a, b) if (a and b) else (a or b)