Different keys will output different messages with the same ciphered text. To Decrypt a ciphered Text/File, the Key for Decryption must match the Key given to Encrypt the original version of the ciphered Text/File. To Decrypt there's need of a Key and Text (or File). For Files, the user must give a name for the output File (which will be created), where the ciphered text will be saved once the encryption is done. The ciphered text doesn't have any trailing or leading spaces if copy-paste used, please mind this. The ciphered text will be shown when the Key and the Text are given. Different Keys will output different ciphered texts for the same Text/File. To Encryp there's need of a Key and Text (or File). Requirementsīrowse the Menu to find the different options: Encrypt, Decrypt or the Smart Mode Options (Special Features, see below). This application encrypts text and decrypts ciphered text using an ADFGVX cipher, a fractionating transposition cipher which combines a modified Polybius square with a single columnar transposition. Again, as a user I'd much rather be able to take input from a file and/or write output to a file (and this still allows me to run it interactively in the rare circumstance that I really want to).įor implementation of the cipher, I'd consider using a little math instead of the tables you're currently using.POLYBIUS SQUARE CYPHER Author: Andres Penas Palmeiro Final Project for the Module "Object Oriented Programming" of the H.Dip in Computer Science in Software Development - GMIT Version: 1.0 (Aug 2019) Summary Likewise, each would act as a filter, taking its input from cin, and writing its output to cout (or possibly read/write other files if you specify them on the command line). As a user, I'd much rather type decrypt or encrypt than have to interactively enter an 'e' or 'd' to tell it whether to encrypt or decrypt. I think I'd start by splitting it into two pieces, one to encrypt and one to decrypt. If you expect every character to be upper case, either (1) tell the user (not recommended) or (2) upper case the string yourself: std::transform(input.begin(), input.end(), input.begin(), ::toupper) For example, for(int i = 0 i for std::isspace. Use ranged-based loop instead of index loop. Use auto to simply some variable definitions if you want to: // std::vector coords = getCoords() Don't allocate memory if you don't have too (this might actually not allocate memory for the std::string, due to SSO actually). SquareIJ doesn't have to be a std::string, it can be a simple char. Your choice variable really wants to be an enum instance: enum class mode When compiled with -Wall, your code produces 2 same warnings: warning: comparison between signed and unsigned integer expressions It's not that messy IMO, but there are a few things which you could have done better:Īlways compile code with every warning turned on, and fix them. This code, in my opinion, is quite messy. The getCoords() function gets a string, and then tokenizes the string into a vector of strings, each corresponding to the coordinates of a character.Įxample of "encryption": Encrypt or decrypt = eĮxample of "decryption": Encrypt or decrypt = dĬoordinates (separate with spaces): 12 11 44 The getChoice() function simply gets the e or d character to choose encryption or decryption. As shown in the unordered map and char array, the numbers correspond to the row and column on a 5x5 square: The Polybius square is a simple way to assign characters numbers, and then "encrypt" and "decrypt" based off of those numbers. Since it is an uncommon cipher, it is nowhere on Code Review.Īlthough the program technically works, it ended up very messy. I attempted to recreate the Polybius square, also called the Polybius checkerboard, which was used in Ancient Greece for cryptography.
0 Comments
Leave a Reply. |
AuthorWrite something about yourself. No need to be fancy, just an overview. ArchivesCategories |