c++ - 凯撒密码/文件输入不起作用

Question

我正在开发一个使用凯撒密码加密和解密 .txt 文件的程序。 (键盘输入是可选的)我已经努力了，但还是被难住了。

我现在拥有的是一个基本的或者我猜已经有些复杂的菜单结构。我已经到了我应该从文件中调用信息的地方，它只是调用垃圾或者是一种或另一种方式的垃圾。

这是我的代码，它运行到输入部分，然后在输入之后结束。

它的选项 1 选项 1 然后是任意数字然后它运行文件输入然后结束。

提前致谢。我几乎到处都看了，尝试了 3 或 4 种不同的方法，但都无济于事。

所以我想知道的是我应该如何修复文件输入以及关于如何获取该数组或字符串的建议，无论您如何建议并通过移位值更改每个字符的 ascii 值。

提前致谢。

#include <iostream>
#include <fstream>
#include <cmath>
#include <cctype>
#include <cstring>
#include <string>
#include <sstream>
#include <dos.h>
#include <stdio.h>
#include <windows.h>
#include <cstdlib>

using namespace std;

int decryptce();
int encryptcefile();
int encryptce();
int outputce();
int encmenu();
int decmenu();

int main()
{
    int t;
      //main menu
    cout << "-----------------------------------------------------"<<endl;
    cout << "This is a Caesar Cipher, please select [1] or [2]" << endl;
    cout << "-----------------------------------------------------"<<endl;
    cout << "[1] Encrypt"<<endl;
    cout << "[2] Decrypt"<<endl;
    cout << "What is your choice:\t";
    cin  >> t;
    cout << "\n-----------------------------------------------------"<<endl;

    // menu switch statement
    switch (t)
    {
        case 1:
        {
            cout << "\n running encryption:\n\n";
            encmenu();
            break;

        }
        case 2:
        {
            cout << "\n running decryption:\n\n";
            decmenu();
            break;
        }
        default:
        {
            cout<< "Thats not a 1 or 2"<<endl;
            cout<< "Relaunch the program"<<endl;
            system ("pause");
            return 0;
        }
    }

    return 0;
}

int encmenu()
{

    int t;
    cout << "-----------------------------------------------------"<<endl;
    cout << "You selected: Encrypt with Caesar Cypher"<<endl;
    cout << "-----------------------------------------------------"<<endl;
    cout <<"Do you want to enter from a file or the keyboard?"<<endl;
    cout <<"Enter [1] From file, or [2] From Keyboard"<<endl;
    cout <<"Run option:\t";
    cin  >> t;
    cout << "\n-----------------------------------------------------"<<endl;
    // encrypt menu switch

    switch (t)
    {
        case 1:
        {
            encryptcefile();
            break;
        }
        case 2:
        {
            encryptce();
            break;
        }
        default:
        {
            cout<< "Thats not a 1 or 2"<<endl;
            cout<< "Relaunch the Program"<<endl;
            system ("pause");
            return 0;
        }
    }
    return 0;
}

// decrypt main menu
int decmenu()
{
    int h;
    cout << "-----------------------------------------------------"<<endl;
    cout << "You selected: Decrypt with Caesar Cypher"<<endl;
    cout << "-----------------------------------------------------"<<endl;
    cout << "Reading from Data.txt in main file directory:";
    cout << "\n"<<endl;
    cout << "Ofset is:\t";
    cin  >> h;
    cout << "\n-----------------------------------------------------"<<endl;
    return h;
}

int decryptce()
{
    return 0;
}


int encryptcefile()
{
    // for figuring out what to do with negatives
    /*
    letter = x;
    letter = ( letter + shift + 26 ) % 26;
    // add 26 in case the shift is negative
    letter += x; // back to ascii code

    */
    char c,i;

    int num=0;
    int shift;
    ofstream ofile;
    ifstream ifile;
    ifile.open("data.txt");
    ofile.open("Encrypted.txt");
    if(!ifile)
    {
        cout << "Error Opening File" << endl;
        return 0;
    }
    if(!ofile)
    {
        cout << "Error Opening File" << endl;
        return 0;
    }

    while (ifile.good())
    {
        c = ifile.get();
        if (c=='\n')num++;
        {
            cout << "Is: " << num << " Long. \r";
            Sleep(1);
            fflush ( stdin );
        }
    }

    cout << "File is: " << num << " characters Long.\n"<<endl;
    cout << "\n-----------------------------------------------------"<<endl;
    cout << "What is the shift for the encryption:\t";

    // ---------------------working here-------------------
    cin >> shift;

    const int xx = num+1;
    char *arraye;
    char *arrayce;
    int j;

    arraye = new char [xx];
    arrayce = new char [xx];
    arrayce[xx];
    ifile.read(arrayce,xx);

    for(j=0;j<xx;j++)
    {
        arraye[j]=arrayce[j];
        cout << arrayce[j]<< endl;
    }

    return 0;
}

int encryptce()
{
    return 0;
}

int outputce()
{
    return 0;
}

Answer 1

这通常是一种不好的文件解析方式。 ifile.good() 将一直为真，直到您尝试读取过去 文件末尾，这意味着它将读取一次太多次。

while (ifile.good())
    {
        c = ifile.get();
        if (c=='\n')num++;
        {
            cout << "Is: " << num << " Long. \r";
            Sleep(1);
            fflush ( stdin );
        }
    }

你的 if 语句在那里是错误的。现在，它在条件为真时执行 num++;，但 block 部分在每个 while 迭代中执行。您将换行符 ('\n') 的数量存储在 num 中，然后在代码中说“文件长度为 num 个字符”。所以我不确定你是要计算行数还是确定文件的大小。我也不知道 fflush(stdin); 的目的是什么。

更简单、更简洁、更正确的方法是将读取语句放在while条件中，这样当读取失败时就不会进入循环。所以如果你想逐行读取文件，做这样的事情:

std::vector<std::string> file_by_lines; // pick a better name :)
std::string line;
while (std::getline(ifile, line))
{
    num += line.length();
    file_by_lines.push_back(line);
}
std::cout << "File is " << num << " characters Long." << std::endl;

在那里，您已经将文件存储在一个 vector 中，不再需要自己为文件动态分配空间，vector 会处理这件事。

或者，如果你需要一个字符一个字符地读取:

// get the size of the file

ifile.seekg (0, ios::end);
size_t length = ifile.tellg();
ifile.seekg (0, ios::beg);  // move read pointer back to the beggining

char* buffer = new char[length];
ifile.read(buffer, length);

// free the allocated memory once you're done.

delete [] buffer;

还有一件事。将来，考虑将代码缩小到实际问题。当您只有(在您的情况下)输入有问题时，您不需要发布整个代码。200 行格式不佳的代码使您不太可能得到人们的帮助。在此处阅读有关简短、自包含、可编译示例的信息:http://sscce.org/

希望对您有所帮助，抱歉我的英语笨拙 :)