未定义标识符/未声明

Question

我最近开始使用 C++，我想创建一个可以调用的简单的终止密码函数，但是因为我从我的 python 程序中复制了编码结构，我似乎收到了 4 个错误和 2 个警告。模式位是一个布尔值，其中 true 是加密，false 是解密（它在 python 上工作，所以嘿）:)。

我创建“int”所在的函数的第一个，它说“标识符“in”未定义”

同一行中的第二个说“预期为')'”

第三个是在 3 个 if 语句之后，表示“标识符“CharPos”未定义”，即使它已定义

和 Forth 在同一行说“'CharPos'：未声明的标识符”

#include <iostream>
#include <fstream>
#include <string>

std::string Encryption(std::string Password, int Key, bool Mode) {
    std::string Alphabet = "abcdefghijklmnopqrstuvwxyz0123456789";
    std::string EncryptPass = "";
    if (Key > 36) {
        Key = Key % 36;
    }
    for (int X = 0; X < Password.length(); X++) {
        if (Password.at(X) == ' ') {
            EncryptPass = EncryptPass + " ";
        }
        else {
            for (int Y = 0; Y < 36; Y++) {
                if (Password.at(X) == Alphabet.at(Y)) {
                    if (Mode == true) {
                        int CharPos = Y + Key;
                        if (CharPos > 35) {
                            CharPos = CharPos - 36;
                        }
                    }
                    if (Mode == false) {
                        int CharPos = Y - Key;
                        if (CharPos < 0) {
                            CharPos = CharPos + 36;
                        }
                    }
                    if (Mode != true and Mode != false) {
                        int CharPos = 0;
                    }
                    char CharPos2 = CharPos;
                    char EncryptChar = Alphabet.at(CharPos2);
                    EncryptPass = EncryptPass + EncryptChar;
                }
            }
        }
    }
    return EncryptPass;
}

任何帮助将不胜感激

Answer 1

如上所述，代码中的主要问题是 CharPos 在每个 if 子句中重新定义。

你把int CharPos = ...放在每一个地方，你都会创建一个new变量，尽管它的名字对你来说很相似，但是对于编译器来说，有三个unique变量这个名字的。为了在所有作用域 - 变量环境中使用相同的变量，您应该在第一个公共作用域中定义它一次这意味着，在if - else 子句之前的for 循环中。

另外，如上所述，Mode != true || Mode != false 等价于true！

我重写了您的代码以确保安全、实现所需的加密并且更具可读性 (IMO)。

std::string encrypt(const std::string& word, std::size_t shift_amount, bool should_shift_up)
{
    static const std::string alphabet = "abcdefghijklmnopqrstuvwxyz0123456789";

    assert (shift_amount <= alphabet.size()); // makes no sense having shift > alphabet size!

    std::string encrypted_word(word.size(), '\0');

    for (std::size_t idx = 0; idx < word.size(); ++idx)
    {
        char original_char = word[idx];
        std::size_t pos_in_alphabet = alphabet.find(original_char);

        std::size_t shifted_char = 'a';
        if (should_shift_up)
        {
            shifted_char = (pos_in_alphabet + shift_amount) % alphabet.size();
        }
        else
        {
            shifted_char = (pos_in_alphabet > shift_amount)  
                         ? pos_in_alphabet - shift_amount
                         : alphabet.size() - shift_amount + pos_in_alphabet;
        }

        encrypted_word[idx] = alphabet[shifted_char];
    }

    return encrypted_word;
}

Answer 2

您对某些变量的范围有一些问题，例如：char CharPos2 = CharPos; charPos 不再在范围内，因此实际上是无效分配。

因此，不要在每个 if else 中定义一个新的 CharPost，而是在之前声明它并在每个 if 检查中重新分配它，例如：

if (Password.at(X) == Alphabet.at(Y)) {
    int CharPos = 0;
    if (Mode == true) {
        CharPos = Y + Key;
        if (CharPos > 93) {
            CharPos = CharPos - 94;
        }
    }
    if (Mode == false) {
        CharPos = Y - Key;
        if (CharPos < 0) {
            CharPos = CharPos + 94;
        }
    }
    if (Mode != true or Mode != false) {
        CharPos = 0;
    }
    char CharPos2 = CharPos;
    char EncryptChar = Alphabet.at(CharPos2);
    EncryptPass = EncryptPass + EncryptChar;
}

但除此之外，你的代码被破坏了......所以只有转译在这里不起作用......

看这里：

std::string Alphabet = "abcdefghijklmnopqrstuvwxyz0123456789";
...
for (int Y = 0; Y < 94; Y++) {
    if (Password.at(X) == Alphabet.at(Y)) {

字母 at(Y) 将在值高于 36 时爆炸...