Попробуйте разбить оператор if и проверять каждый по отдельности.
def username_generator(first_name, last_name):
if len(first_name) > 3:
username = first_name[:3]
else:
username = first_name
if len(last_name) > 4:
username += last_name[:4]
else:
username += last_name
return username
print(username_generator("Abe", "Simpson"))
Приведенный выше код представляет собой реструктуризацию вашего кода. Однако в python все немного проще.
скажем:
first_name = 'Ab'
last_name = 'Sim'
username = first_name[:3] + last_name[:4]
Это выведет:
AbSim
Так что вам не нужно сделать много из приведенного выше кода. Python сделает это за вас.