Question

我想知道是否有人可以帮我解决问题。它似乎正常工作，但当我去运行它时，它开始删除我的数据，即使我从来没有调用任何东西删除它。

例如，我的文件来自：

=======================
Rotated by 11 positions
=======================
Lnwypeyw lala sqz xa na lnwypeyw eb pdau sqz pwa w eppa
na pea bn znawec.
              -- F. L. IyAru**
to 
=======================
Rotated by 12 positions
=======================
Moxzqzx mbmb tra yb ob moxzqzx c qebv tra qxb x qqb
ob qb co aobxd.
              -- G. M. JzBsv

这是我的代码，我尝试了几次，这一切都在逻辑上有意义，并且在丢失数据时不知道。

#include <iostream>
#include <cctype>
#include <fstream>

using namespace std;

bool isUpper(char character){

    //this will test and see if it's upper or lowercase
    bool upper;

    if(character>= 'A' && character <= 'Z'){
        upper=true;
    }
    else {
        upper= false;
    }

   return upper;
}

bool isLower(char character){

    //this will test and see if it's upper or lowercase
    bool lower;

    if(character >= 'a' && character <= 'z'){
        lower=true;
    }

    else {
        lower= false;
    }

   return lower;
}

//Actual function that will rotate the character
char rotate(char character, int offset) {

    char next_letter;

    //Changes it if it's a lower case
    if (isLower(character)) {

        next_letter = character + offset;

        if (next_letter > 'z'){
            next_letter = (next_letter - 26);
            return next_letter;
        }

        return next_letter;
    }

    else if(isUpper(character)) {

        next_letter = character + offset;

        if (next_letter > 'Z'){
            next_letter = (next_letter - 26);
            return next_letter;
        }

        return next_letter;
    }

    else {
       return character;
    }
}


int main() {

    //variables for program
    char character = 'a';
    int offset = 0;

    while(offset < 26){

        //opens the file  
        ifstream fin;
        fin.open("secretMessage.txt");

        if(!fin.good()) {
            cout << "Please check your file name!!!";
            return 0; 
        }

        //report for reading       
        cout << "=======================" << endl  
        << "Rotated by " << offset << " positions" << endl
        << "=======================" << endl; 

        //Reads until it's at the end of the file
        while ((! fin.eof())){
            character = fin.get();

            cout << rotate(character,offset);

        }

    //makes it run 26 times  
    ++offset;
    fin.close();
    cout << endl << endl;
    }
    //Closes the file output.
    return 0;
}

Answer 1

在这个逻辑中：

    if (next_letter > 'z') {
        next_letter = (next_letter - 26);
        return next_letter;
    }

你没有想到如果签名的话会发生什么。测试一下：

assert(rotate('z', 0) == 'z');
assert(rotate('z', 1) == 'a');
assert(rotate('z', 2) == 'b');
assert(rotate('z', 3) == 'c');
assert(rotate('z', 4) == 'd');
assert(rotate('z', 5) == 'e');

到目前为止很好，但现在：

assert(rotate('z', 6) == 'f'); // WHOOPS

Ascii＆＃39; z＆＃39; +5是127，＆＃39;＆＃39; +6是128.但是，z + = 6导致它为负数，因此检查> 'z'失败

建议的简化：

bool isUpper(char ch) { return (ch >= 'A' && ch <= 'Z'); }
bool isLower(char ch) { return (ch >= 'a' && ch <= 'z'); }

// Actual function that will rotate the character
char rotate(char ch, int offset) {
    if (isLower(ch)) return (ch - 'a' + offset) % 26 + 'a';
    if (isUpper(ch)) return (ch - 'A' + offset) % 26 + 'A';
    return ch;
}

由于您添加了<cctype>，因此您也可以使用std::islower和std::isupper。

<强> Live On Coliru

#include <cctype>
#include <fstream>
#include <iostream>
#include <algorithm>

// Actual function that will rotate the character
char rotate(char ch, int offset) {
    if (std::islower(ch)) return (ch - 'a' + offset) % 26 + 'a';
    if (std::isupper(ch)) return (ch - 'A' + offset) % 26 + 'A';
    return ch;
}

std::string readFile(std::string const& fname) {
    std::ifstream fin;
    fin.exceptions(std::ios::failbit);
    fin.open(fname);
    return { std::istreambuf_iterator<char>(fin), {} };
}

int main() {

    auto const original = readFile("main.cpp");

    // makes it run 26 times
    for (int offset = 0; offset < 26; ++offset) {

        std::cout 
            << "\n"
            << "=======================\n"
            << "Rotated by " << offset << " positions\n"
            << "=======================\n";

        // Reads until it's at the end of the file
        std::transform(
                original.begin(), original.end(),
                std::ostreambuf_iterator<char>(std::cout),
                [offset](char ch) { return rotate(ch, offset); });
    }
}

奖金

实际上，你可以通过26次就地旋转来完成主要的复杂性：

<强> Live On Coliru

auto text = readFile("main.cpp");

for (int offset = 0; offset < 26; ++offset) {
    std::cout 
        << "\n"
        << "=======================\n"
        << "Rotated by " << offset << " positions\n"
        << "=======================\n";

    std::cout << text;
    for (auto& ch : text) ch = rotate(ch, 1);
}

Caesar Cipher删除数据

1 个答案:

建议的简化：

奖金