我正在尝试用c ++编写一个caesar密码程序。我使用四个函数,一个用于选择shift键,两个用于加密和解密,最后一个用于实现caesar密码,使用inputfile读取文本并将加密或解密的文本输出到输出文件中。我正在尝试运行代码,它正在崩溃。我认为问题出在Caesar函数内部。但我不知道到底是什么。有任何想法吗?这是我的代码:
#include <iostream>
#include <fstream>
using namespace std;
int chooseKey()
{
int key_number;
cout << "Give a number from 1-26: ";
cin >> key_number;
while(key_number<1 || key_number>26)
{
cout << "Your number have to be from 1-26.Retry: ";
cin >> key_number;
}
return key_number;
}
void encryption(char *w, char *e, int key){
char *ptemp = w;
while(*ptemp){
if(isalpha(*ptemp)){
if(*ptemp>='a'&&*ptemp<='z')
{
*ptemp-='a';
*ptemp+=key;
*ptemp%=26;
*ptemp+='A';
}
}
ptemp++;
}
w=e;
}
void decryption (char *e, char *w, int key){
char *ptemp = e;
while(*ptemp){
if(isalpha(*ptemp))
{
if(*ptemp>='A'&&*ptemp<='Z')
{
*ptemp-='A';
*ptemp+=26-key;
*ptemp%=26;
*ptemp+='a';
}
}
ptemp++;
}
e=w;
}
void Caesar (char *inputFile, char *outputFile, int key, int mode)
{
ifstream input;
ofstream output;
char buf, buf1;
input.open(inputFile);
output.open(outputFile);
buf=input.get();
while(!input.eof())
{
if(mode == 1){
encryption(&buf, &buf1, key);
}else{
decryption(&buf1, &buf, key);
}
output << buf;
buf=input.get();
}
input.close();
output.close();
}
int main(){
int key, mode;
key = chooseKey();
cout << "1 or 0: ";
cin >> mode;
Caesar("test.txt","coded.txt",key,mode);
system("pause");
}
以下是我的程序的一些细节。我的代码模板位于我的帖子之上。
参数 w是开头的文本,而参数是解密的文本。正在加密,如果我们在w *文本中添加shift键,我们得到* e。反向过程使解密功能,我有加密文本(* e),我想采取解密文本(* W),从我对selectKey函数的键移位进行子跟踪。在Caesar函数中,我使用包含一些数据的输入文件,我想在输出文件中提取加密或解密的文本。我已经实现了加密和解密功能,我的问题是如何在Caesar函数中使用它们。我没有采取任何理想的结果,我的程序正在崩溃。请提一下我的错误。任何解决方案?
答案 0 :(得分:4)
您将单个字符的地址传递给decryption
buf=input.get();
// ...
encryption(&buf, &buf1, key);
然后将其视为指向以null结尾的字符串。
while(*ptemp){
// ...
ptemp++;
}
这不会很好用,因为ptemp
首先没有指向以null结尾的字符串。你做ptemp++
的那一刻,你处于未定义的行为之地。
除此之外还有许多其他陷阱(不要在.eof()
上循环,例如)和潜在的改进。
修改哦,以及作业
w = e;
和
e = w;
没有任何效果(你将指针值赋给本地函数参数;返回时,这些变量甚至不再存在)。
这是一个清理后的内容,我期望做你想做的事: Live on coliru
#include <iostream>
#include <fstream>
using namespace std;
int chooseKey() {
cout << "Give a number from 1-26: ";
int key_number;
while((!(cin >> key_number)) || (key_number < 1 || key_number > 26)) {
cout << "Your number have to be from 1-26 (" << key_number << "). Retry: ";
cin.clear();
}
return key_number-1;
}
template <typename InIt, typename OutIt>
void encryption(InIt f, InIt l, OutIt out, int key) {
for (InIt ptemp = f; ptemp != l; ++ptemp)
{
if(isalpha(*ptemp))
{
char base = islower(*ptemp)? 'a' : 'A';
*out++ = base + (*ptemp - base + key) % 26;
} else
*out++ = *ptemp;
}
}
void Caesar(const char *inputFile, const char *outputFile, int key, int mode) {
ifstream input(inputFile);
ofstream output(outputFile);
istreambuf_iterator<char> init(input), end;
ostreambuf_iterator<char> outit(output);
encryption(init, end, outit, mode? key : 26-key);
}
int main() {
int key, mode;
key = chooseKey();
cout << "1 or 0: ";
cin >> mode;
Caesar("test.cpp", "coded.txt", key, mode);
}