我一直在研究cs50 pset2,我认为在使用它几天之后我已经将vigenere密码降低了 . 此代码用于获取用户给出的字母参数(argv []),并将其用作密钥,用于将用户(字符串)给出的短语按字母索引中的数字加密 . 例如,如果你给出参数'abc'和字符串'cat',那么输出应该是'cbv'(移动0,b移动1,c移动2)参数也应该环绕,这样如果字符串是更长时间,参数将换行到第一个字符并继续直到字符串结束 .
这就是我对代码的看法:
#include <cs50.h>
#include <stdio.h>
#include <string.h>
#include <ctype.h>
int main(int argc, string argv[])
{
if(argc != 2)
{
printf("Try again\n");
return 1;
}
string k = (argv[1]);
int klen = strlen(k);
for(int x = 0; x < klen; x++)
{
if(isalpha(k[x]))
{
if(isupper(k[x]))
{
k[x] = tolower(k[x]);
}
k[x] -= 'a';
}
else
{
printf("Try again\n");
return 1;
}
}
string code = GetString();
int clen = strlen(code);
for(int a = 0, b = 0; a < clen; a++)
{
if(isalpha(code[a]))
{
int key = k[b%klen];
if(isupper(code[a]))
{
printf("%c", (((code[a] - 'A') + key)%26) + 'A');
b++;
}
else
{
printf("%c", (((code[a] - 'a') + key)%26) + 'a');
b++;
}
}
else
{
printf("%c", code[a]);
}
}
printf("\n");
}
代码似乎适用于密钥1的长度 . 例如,我输入了'aaaa'的参数
然后输入一串'bbbbb'并正确接收'bbbbb' .
但是,如果我输入相同的'aaaa'
然后输入一个比键1'bbbbbbb'更长的字符串'我收到'bbbbbNN'
我相信我的操作顺序有问题,但是尝试移动括号无效 . 我希望有人可以指出我正确的方向,为什么我的钥匙没有正确包装 .
1 回答
像这样的代码你最大的风险是所有类似的,重复的条款 . 只有一个中的错误很难跟踪完成 . 在处理代码时对密钥进行任何处理都是低效的 .
这是一个返工,在处理代码之前完全处理密钥,并尝试将处理简化为一个案例 . 看看它是否适合你: