使用C ++的Vigenere Cipher存在输出问题

时间:2014-02-04 06:25:10

标签: c++ encryption vigenere

我在尝试让代码运行时遇到了一些问题。它打印ALMOST正确的数据,但它可能没有正确循环?我不认为它通过字母表重复密钥。它全部小写,不超过26。

void vigenereEncrypt( const char  plaintext[], char ciphertext[], const char key[] )

{
int idx;
int j;

for( idx = 0, j = 0; idx <= strlen(plaintext); idx++ )
{
    if ( CHAR_OUT_OF_RANGE(plaintext[idx]) )
    {
        ciphertext[idx] = plaintext[idx];
    }
    else
    {
        ciphertext[idx] = plaintext[idx];
        ciphertext[idx] += key[j] - MIN_ASCII_VALUE;

        if (ciphertext[idx] >= MAX_ASCII_VALUE) ciphertext[idx] += -MAX_ASCII_VALUE + MIN_ASCII_VALUE - 1;
    }
    j = (j + 1) % strlen(key);
}

ciphertext[idx] = 0;    
}

例如:如果我用一把钥匙输入明文墨粉,输出将是csevé。它应该改为csevp

2 个答案:

答案 0 :(得分:0)

你的循环是一对一的。你应该使用&lt;而不是&lt; =。而且我认为你应该测试&gt; MAX_ASCII_VALUE,不是&gt; =(但你没有显示MAX_ASCII_VALUE是什么)。

但是你的基本问题是签名与未签名的char问题。使用带符号的字符,当它超过127时,它会回绕并变为负数,因此&gt;它应该通过时测试失败。

void vigenereEncrypt(const char plaintext[], char ciphertext[], const char key[])
{
    size_t i, j;
    for(i = 0, j = 0; i < strlen(plaintext); ++i )
    {
        ciphertext[i] = plaintext[i];
        if (!CHAR_OUT_OF_RANGE(plaintext[i]))
        {
            ciphertext[i] += (uchar)key[j] - (uchar)MIN_ASCII_VALUE;

            if ((uchar)ciphertext[i] > (uchar)MAX_ASCII_VALUE)
                ciphertext[i] -= (uchar)MAX_ASCII_VALUE - (uchar)MIN_ASCII_VALUE + 1;
        }
        j = (j + 1) % strlen(key);
    }

    ciphertext[i] = 0;
}

答案 1 :(得分:0)

让所有人(特别是你自己)受宠,并使用std::string代替C风格的字符串。然后使用标准算法而不是来自行搞乱循环。

#include <iostream>
#include <iterator>
#include <algorithm>

class crypt {
    std::string key;
    size_t pos;
public:
    crypt(std::string const &k) : key(k), pos(0) { }

    char operator()(char input) { 
        char ret = input ^ key[pos]; 
        pos = (pos + 1) % key.size(); 
        return ret; 
    }
};

int main() {
    std::string input("This is some input to be encrypted by the crappy encryption algorithm.");

    std::transform(input.begin(), input.end(),
        std::ostream_iterator<char>(std::cout),
        crypt("This is the key"));
    return 0;
}