C++内联汇编有什么问题吗？

Question

首先，请使用简单的单词，因为我的英语不太好；）

现在的问题是：

我想编写一个程序，可以使用简单的凯撒算法加密我的文本。这意味着字母字符成为字母表中较晚的字符，但我的程序不会从 Z 之后的字母表开头开始。现在的代码：

void Entschlüsseln(char Text[50], int Schlüssel)
{
    char Entschlüsselt[sizeof(Text)];
    for (int x = 0; x < sizeof(Text)-1; x++)
    {
        Entschlüsselt[x] = '\0';
    }
        char Zeichen;
    for (int i = 0; i < sizeof(Text)-1; i++)
    {
        if (Text[i] != '\0')
        {
            Zeichen = Text[i];
            for (int j = 0; j < Schlüssel; j++)
            {
                _asm
                {
                        mov al, Zeichen   
                        cmp al, 90
                        jb Großbuchstabe
                        mov al, Zeichen
                        sub al, 32
                        mov Zeichen, al
                    Großbuchstabe:
                        inc Zeichen
                        mov al, Zeichen
                        cmp al, 90
                        ja Anfang_Alphabet
                        jmp Ende
                    Anfang_Alphabet:
                        mov Zeichen, 65
                    Ende:
                }   
            }
            Entschlüsselt[i] = Zeichen;
        }
    }
    cout << endl << Entschlüsselt;
}

我希望它没问题，你可以帮助我

Answer 1

这在纯 C++ 中更容易实现，无需汇编。

Zeichen = tolower(Text[i]);
Zeichen += Schlussel; // Note, the C++ standard does not guarantee that non-ASCII characters such as U-umlaut are allowed in identifiers
if (Zeichen > 'Z') Zeichen -= 26;
Entschlusselt[i] = Zeichen;

Answer 2

也许您只想更改字符（如果它确实是字母），以便保留空格和其他字符。而且您使用的是 C++，因此使用 std::string 而不是 char[] 更容易。

#include <iostream>
#include <string>

using std::string;

void caesar_encode(std::string &text, int schluessel) {

  for (size_t i = 0; i < text.length(); i++) {
    char zeichen = text[i];

    int buchstabe = -1; // -1 == unbekannt, 0 = a, 1 = b, ... 25 = z
    if ('A' <= zeichen && zeichen <= 'Z') {
      buchstabe = zeichen - 'A';
    } else if ('a' <= zeichen && zeichen <= 'z') {
      buchstabe = zeichen - 'a';
    }

    char codiertes_zeichen = zeichen;
    if (buchstabe != -1) {
      int codierter_buchstabe = (buchstabe + schluessel) % 26;
      char alphabet_anfang = zeichen - buchstabe; // 'A' oder 'a'
      codiertes_zeichen = alphabet_anfang + codierter_buchstabe;
    }

    text[i] = codiertes_zeichen;
  }
}

int main() {
  string s("Hallo, Welt");
  caesar_encode(s, 13);
  std::cout << s << "\n";
  return 0;
}

只要尝试理解这段代码，应该是可以的。

Answer 3

我认为错误的代码是这一行：

cmp al, 90
jb Großbuchstabe

由于 90（在 ASCII 中是 Z）是 Großbuchstabe，因此您不应该从中减去 32。做到：

cmp al, 90
jbe Großbuchstabe