Unas cuantas cosas:
La función no imprime nada ni retorna nada, bueno None
que es lo que toda función retorna por defecto. Por lo tanto, nunca vas a ver resultado alguno.
La línea len(letter)%2!=0
en el else
sobra, aparte de no hacer nada por no estar asociada a ningún condicional o variable, si el if
no se cumple la palabra es "impar", no hay que comprobar nada.
No uses *kwargs
, por convención kwargs
se usa para que la función acepte un número indeterminado de argumentos de tipo keyword, por tanto será un diccionario. Para argumentos posicionales se usa *args
por convención. No es incorrecto en cuanto al lenguaje, pero si en cuanto a convenciones de estilo, puede confundir.
def mayusculas_y_minusculas (*args):
for word in args:
if len(word) % 2 == 0:
print(word.upper())
else:
print(word.lower())
mayusculas_y_minusculas ("hola", "ADIÓS")
HOLA
adiós
Usas letter
como nombre de variable, lo cual contradice un poco lo que expresas en la pegunta y el propio código que intentas aplicar, si quieres que lo que se evalúe como par o impar sea la posición del carácter en la palabra, entonces no debes usar len sino aplicar el módulo sobre el índice:
def mayusculas_y_minusculas (*args):
for word in args:
for i, letter in enumerate(word):
if i % 2 == 0:
print(letter.upper(), end="")
else:
print(letter.lower(), end="")
print()
mayusculas_y_minusculas ("hola", "ADIÓS")
HoLa
AdIóS