2014-09-27 71 views
0

因此,我正在尝试制作一个ROT13解码器,而这正是我目前为止所做的。只有一些字母改变,但我不知道为什么。我对编程非常陌生。我只是想弄清楚如何读入文件,并写入文件。到目前为止,该部分的作品,但是它并没有改变原始文件中的所有字母,只是其中的一部分。我真的很感激任何反馈。为什么我的程序只改变了一些字母

#include <iostream> 
#include <fstream> 
//the letters in the secretMessage file are "Lbh unir gb fgnl va funcr. Zl tenaqzbgure, fur fgnegrq jnyxvat svir zvyrf n qnl jura fur jnf 60. Fur’f 97 gbqnl naq jr qba’g xabj jurer gur uryy fur vf 
// 
// 
//and this is what it outputs to the decodedMessage file "Lbh haie gb fgal ia fhace. Ml geaadmbghee, fhe fgaeged jalkiag fiie milef a dal jhea fhe jaf 60. Fhe’f 97 gbdal aad je dba’g kabj jheee ghe hell fhe if. 

using namespace std; 


int main(){ 
    ofstream fout; 
    ifstream fin; 
    fin.open("secretMessage.txt"); 
    fout.open("decodedMessage.txt"); 


    char c = 0; 



    while (!fin.eof()){ 
     c = fin.get(); 
     if (c == 'a')c = 'n'; 
     if (c == 'b')c = 'o'; 
     if (c == 'c')c = 'p'; 
     if (c == 'd')c = 'q'; 
     if (c == 'e')c = 'r'; 
     if (c == 'f')c = 's'; 
     if (c == 'g')c = 't'; 
     if (c == 'h')c = 'u'; 
     if (c == 'i')c = 'v'; 
     if (c == 'j')c = 'w'; 
     if (c == 'k')c = 'x'; 
     if (c == 'l')c = 'y'; 
     if (c == 'm')c = 'z'; 
     if (c == 'n')c = 'a'; 
     if (c == 'o')c = 'b'; 
     if (c == 'p')c = 'c'; 
     if (c == 'q')c = 'd'; 
     if (c == 'r')c = 'e'; 
     if (c == 's')c = 'f'; 
     if (c == 't')c = 'g'; 
     if (c == 'u')c = 'h'; 
     if (c == 'v')c = 'i'; 
     if (c == 'w')c = 'j'; 
     if (c == 'x')c = 'k'; 
     if (c == 'y')c = 'l'; 
     if (c == 'z')c = 'm'; 


     if (c == 'A')c = 'N'; 
     if (c == 'B')c = 'O'; 
     if (c == 'C')c = 'P'; 
     if (c == 'D')c = 'Q'; 
     if (c == 'E')c = 'R'; 
     if (c == 'F')c = 'S'; 
     if (c == 'G')c = 'T'; 
     if (c == 'H')c = 'U'; 
     if (c == 'I')c = 'V'; 
     if (c == 'J')c = 'W'; 
     if (c == 'K')c = 'X'; 
     if (c == 'L')c = 'Y'; 
     if (c == 'M')c = 'Z'; 
     if (c == 'N')c = 'A'; 
     if (c == 'O')c = 'B'; 
     if (c == 'P')c = 'C'; 
     if (c == 'Q')c = 'D'; 
     if (c == 'R')c = 'E'; 
     if (c == 'S')c = 'F'; 
     if (c == 'T')c = 'G'; 
     if (c == 'U')c = 'H'; 
     if (c == 'V')c = 'I'; 
     if (c == 'W')c = 'J'; 
     if (c == 'X')c = 'K'; 
     if (c == 'Y')c = 'L'; 
     if (c == 'Z')c = 'M'; 
     cout << c; 
     if (!fin.eof())fout << c; 
    } 


    fin.close(); 
    fout.close(); 



    return 0; 
} 

回答

4

您示例中的大多数字母将翻转两次。您可能需要添加大量的“其他”命令或使用switch语句。

if (c == 'a')c = 'n'; 
else if (c == 'b')c = 'o'; 

有一个更好的数学方法;但我会把它作为练习给读者。

3

你有一个逻辑错误。如果以“a”开头,则第一个IF将其变为“n”,随后的IF将其变回“a”。

看看switch声明。

2

你不应该到处使用if条件,你有一个新的行中的每个人都会进行检查。假设你的性格是l那么你的if语句改变信y但还检查了y的条件,使该字符被改回l。所以实际上你的代码不是而是改变了一些字母,它改变了他们两次

您应该能够使用else if为后续条目解决此问题,以便在其中一个条件成立时其他条目不会被检查,或者您可以使用switch case

2

以ROT13编码的单个字母。如果将它编码两次,则会再次得到原始字母,这使得对ROT13进行编码和解码几乎相同。

现在以例如字母'a'为例。什么你在你的代码上面做的是

if (c == 'a') c = 'n'; 
... 
if (c == 'n') c = 'a'; 

因此,对于上面的字母上半年(A至M),你编码两次,再次收到同样的信。仅对字母的下半部分编码只进行一次。

一种解决方法是改变你的语句像

if (c == 'a') { c = 'n'; continue; } 

这将跳过代码的其余部分匹配并continue下一个循环迭代的情况下。

相关问题