pset2 vigenere uppercase text not encrypted right

Question

I'm stuck at encrypting uppercase text. My code works fine for lowercase text, but will return wrong output when it comes to encrypting uppercase chars.

for (int c = 0; c < length_of_ptext; c++) {

        if ((isupper(ptext[c])) && (islower(shift[c])))
            ctext[c] = (((ptext[c] - 'A') + (shift[c] - 'A')) % 26) + 'A';//encryption of uppercase characters with lowercase key

        else if ((islower(ptext[c])) && (islower(shift[c])))
            ctext[c] = (((ptext[c] - 'a') + (shift[c] - 'a')) % 26) + 'a';//encryption of lowercase characters with lowercase key

        else
            ctext[c] = shift[c];//non-alphabetical characters remain as they are
    }

Answer 1

Are you sure that's your problem? Looking at your code, it appears that it only handles the two cases where the key and the plaintext are both the same case. It doesn't handle the two cases where the key and the plain text are different cases, one upper case and the other lower case.

It also appears that you are using the same index, c, for both the plaintext and the key. This will blow up on you when a non-alpha is processed, or when the length of the key is exceeded.

Without seeing the actual input and output, or more code, there's no way to accurately determine the problem, but one or both of these problems are likely to be the cause.

If this answers your question, please click on the check mark to accept. Let's keep up on forum maintenance. ;-)