min
y max
no tienen en cuenta la longitud de las cadenas, tienen en cuenta el orden lexicográfico. En todo caso, si quieres usar esta aproximación, debes usar max
y min
sobre la salida de len
, cosa que puedes hacer usando el argumento key
:
mini = min(a, b, key=len)
maxi = max(a, b, key=len)
o puedes usar directamente sorted
:
mini, maxi = sorted((a, b), key=len)
key
permite indicar a min
, max
, sorted
y list.sort
que debe tener en cuenta para ordenar los ítems, en este caso a la hora de ordenar usa el retorno de len(cad)
, es decir la longitud de cada cadena.
La concatenación de cadenas es ineficiente dada su inmutabilidad (en cada concatenación se ha de crear una cadena nueva que luego a de ser eliminada por el GC), siempre que concatenes más de dos considera usas str.join
o literales de cadena formateados en Python >= 3.6
def combo_string(a, b):
mini, maxi = sorted((a, b), key=len)
return f"{mini}{maxi}{mini}" # return "".join((mini, maxi, mini))
Si ambas cadenas tiene por casualidad la misma longitud, se mantiene simplemente el orden inicial:
>>> combo_string("bbb", "cc")
'ccbbbcc'
>>> combo_string("cc", "bb")
'ccbbcc'
>>> combo_string("bb", "cc")
'bbccbb'
De cualquier manera, la forma más eficiente para el resultado concreto que quieres en este caso es usar un simple if
:
def combo_string(a, b):
if len(a) > len(b):
a, b = b, a
return f"{a}{b}{a}" # return "".join((a, b, a))
o directamente:
def combo_string(a, b):
return f"{b}{a}{b}" if len(a) > len(b) else f"{a}{b}{a}"