我在尝试让代码运行时遇到了一些问题。它打印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
答案 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;
}