Skip to content

Conversation

@stelaki50
Copy link
Contributor

I added some error handling, included the necessary functions, and replaced the while loops with if statements to avoid unnecessary breaks (it still works the same). The code is better now, but there are still a few issues left.
Changes:

  1. If the user enters a string for decryption in hex or base64 that isn’t in the correct encrypted format, the code used to throw an error and crash. I’ve now added a proper function to check whether the string is in a valid format for decryption, along with more user-friendly error messages.
    2)No while loop is needed here using if works better.
    3)In the Morse code encryption, lowercase text couldn’t be added before. I updated it so that lowercase letters can now be encrypted as well.

Suggested changes:
1)Improve the menu to allow the user to choose whether they want to encrypt or decrypt.
2)Add more error handling in some methods.

…placed the while loops with if statements to avoid unnecessary breaks (it still works the same). The code is better now, but there are still a few issues left.
@github-actions
Copy link

👋 This pull request has been automatically marked as stale because it has not had recent activity.

It will be closed in 7 days if no further activity occurs.

If you're still working on this:

  • Push new commits
  • Add a comment with an update
  • Request a review

Thank you for your contribution! 🙏

@github-actions github-actions bot added the stale label Nov 24, 2025
@josharsh josharsh merged commit ac8c683 into josharsh:master Dec 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants