Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance Vigenère Cipher to Support Numeric Characters in Encoding and Decoding #1919

Closed
wants to merge 3 commits into from

Conversation

SaeeBiwalkar
Copy link

This pull request updates the Vigenère cipher implementation to support numeric characters during both encoding and decoding processes.

Changes Made:

  • Character Set: The alphabet now includes numeric characters (0-9) alongside the alphabetic characters (a-z).
  • Encoding Logic: Adjusted the encoding logic to correctly handle numeric characters, ensuring they are encrypted in accordance with the Vigenère cipher rules.
  • Preservation of Case: The implementation continues to preserve the case of alphabetic characters while leaving numeric characters unaffected.

Benefits:

  • Broader Usability: Users can now encrypt and decrypt strings containing both letters and numbers seamlessly.
  • Enhanced Functionality: This enhancement improves the utility of the Vigenère cipher for modern applications that often involve alphanumeric data.

Please review the changes and consider merging them to enhance the functionality of the Vigenère cipher operation. Thank you!

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ SaeeBiwalkar
❌ azeem30
You have signed the CLA already but the status is still pending? Let us recheck it.

@SaeeBiwalkar SaeeBiwalkar marked this pull request as draft October 1, 2024 04:02
@SaeeBiwalkar SaeeBiwalkar closed this by deleting the head repository Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants