You’re returning early in the loop, you need to build up a string and return that. Either just return a joint string or build it up
"".join(chr(ord(s)+4) for s in pas)
def cip(pas):
ret_val = ""
for i in pas:
asci = ord(i)
encryption = asci + 4
ret_val += chr(encryption)
return ret_val
solved It only return one letter