Question

我是编程新手，我正在尝试用C语言为Caesar Cipher编写一个程序。

输入包含一个等于字符串长度的整数ilength，后跟字符串str和整数加密。

我的意见是：

11
middle-Outz
2

输出：

okffng-Qwv@

必需的输出是：

okffng-Qwvb

以下是我编写的代码。有人可以帮我解释为什么我输出的最后一个字符错了！

我完全无能为力。

#include <stdio.h>
#include <string.h>
#include <math.h>
#include <stdlib.h>

int main()
{
  int ilength = 0, encrypt = 0, i = 0, j = 0;

  char alph_base[] = {'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'X', 'Y', 'Z', 'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z'};

  scanf("%d", &ilength);
  char str[ilength + 1];
  scanf("%s", str);
  scanf("%d", &encrypt);
  //printf("%c\n", str[5]);

  char outputString[ilength + 1];
  char temp[ilength + 1];

  for (j = 0; j <= ilength; j++)
  {
    temp[j] = str[j];
    i = 0;

    if (str[j] == '\0')
    {
      outputString[j] = '\0';
    }

    while ((i >= 0) && (i < 26))
    {
      if (temp[j] == alph_base[i])
      {
        if (i == 25 && encrypt == 0)
        {
          outputString[j] = alph_base[25];
        }

        if ((i + encrypt) == 26)
        {
          outputString[j] = alph_base[(i + encrypt) % 26];
        }
        else
          outputString[j] = alph_base[(i + encrypt) % 26];
      }
      if ((temp[j] < 65 || temp[j] > 90) && temp[j] < 97)
        outputString[j] = temp[j];

      if ((temp[j] < 97 || temp[j] > 122) && temp[j] > 90)
        outputString[j] = temp[j];

      i++;
    }

    while ((i > 25) && (i < 52))
    {
      if (temp[j] == alph_base[i])
      {
        if (i == 51 && encrypt == 0)
        {
          outputString[j] = alph_base[51];
        }

        if ((i + encrypt) == 51)
        {
          outputString[j] = alph_base[51];
        }

        if ((i + encrypt) > 51)
        {
          outputString[j] = alph_base[((i + encrypt) % 51) + 25];
        }
        else
          outputString[j] = alph_base[(i + encrypt) % 51];
      }
      if ((temp[j] < 65 || temp[j] > 90) && temp[j] < 97)
        outputString[j] = temp[j];

      if ((temp[j] < 97 || temp[j] > 122) && temp[j] > 90)
        outputString[j] = temp[j];
      i++;
    }
  }
  printf("%s\n", outputString);
  return 0;
}

Answer 1

您的代码太复杂，无法完成您的工作。

您的问题似乎与从'z'循环回'a'。

有关

像这样的简单函数可以完成角色的工作：

#include <ctype.h>

char caesar_encrypt(char input, int key)
{
    char output = input;
    char base, offset;
    // If not a letter, return the char unmodified
    if (! isalpha(input))
    {
        return output;
    }

    base = isupper(input) ? 'A' : 'a'; // Check if upper/lower case
    offset = input - base; // Take offset from 'a'
    offset += key; // Add key to offset
    offset %= 26; // Wrap offset to the 26 letters

    output = base + offset;
    return output;
}

这里有很多想法：

使用<ctype.h>（isalpha，isupper）中的函数，这可以避免代码中出现许多比较。
将您的角色视为＆＃39;偏移＆＃39;从字母A（大写或小写A）。因此，您正在处理范围[0;25]中的数字，并且您可以使用简单模数进行换行
字符是＆＃39;整数＆＃39;，因此您可以添加或减去它们。要获得大写字母的第三个字母，您可以执行char c = 'A' + 2;，这比您的大数组更简单。

免责声明：此处编写的代码未经测试，可能包含拼写错误;）

Answer 2

NiBZ的完美答案。要添加它，如果您仍想编写自己的检查而不是使用库函数，则可以执行以下操作：

int my_isalpha(char c){
    return ((c >= 'a' && c <= 'z')|| (c >= 'A' && c <= 'Z'));
}

int my_isupper(char c){
    return (c >= 'A' && c <= 'Z');
}

警告：上述实施适用于ASCII，但对ISO 8859-1或其亲属不太好。参考：https://stackoverflow.com/a/2169293/5183246

在Caesar Cipher in C中需要帮助

2 个答案: