ok now that you say that...then my encrypt function is wrong. I need to take each character to see if it is a letter, then change it to lower case, then encrypt by mapping using 'a' as index 0 and so on. I don't think I am implemeenting it right and am stuck.