Question

我的程序应该接受一个char，如果它是一个字母，则使用ROT13对其进行编码，否则保持相同，然后打印结果。

我的代码适用于所有小写字母和大写字母A-M，但在大写字母N-Z和其他符号/数字上失败。任何帮助表示赞赏：）

#include <stdlib.h>
#include <stdio.h>
#include <assert.h>

#define TRUE 1
#define FALSE 0

#define UPPER_START 65
#define UPPER_END 90
#define LOWER_START 97
#define LOWER_END 122

#define UPPER_MID 77
#define LOWER_MID 109


void testEncode (void); int isValid (char cipherChar); char encode (char letter);

int main (int argc, char* argv[]) {

    char cipherChar;

    scanf("%c", &cipherChar);
    if (isValid(cipherChar) == TRUE) {
        printf("%c", encode (cipherChar));
    } else if (isValid(cipherChar) == FALSE) {
       printf("%c", cipherChar);
    }
       return EXIT_SUCCESS;

}

int isValid (char cipherChar) {

    int valid;

    if ((cipherChar >= UPPER_START) &&
        (cipherChar <= UPPER_END)) {
        valid = TRUE;
    } else if ((cipherChar >= LOWER_START) &&
            (cipherChar <= LOWER_END)) {
        valid = TRUE;
    } else {
        valid = FALSE;
    }

    return valid;
}

char encode (char letter) {

    if ((letter <= UPPER_MID) || (letter <= LOWER_MID)) {
        letter = letter + 13;
    } else {
        letter = (letter - 13);
    }

    return letter;
}

Answer 1

ROT13通常不会影响[A-Za-z]以外的字符，所以我只想忽略它们。除非你有一些理由旋转它们。例如，有时0-9用ROT5处理。

在任何情况下，带有if条款的(letter <= UPPER_MID) || (letter <= LOWER_MID)语句基本上都会说，“'a'之前的任何字符都会添加13”，这就是为什么[NZ]无法正常工作的原因。相反，尝试打破大写和小写逻辑并处理每个案例的环绕，如下所示：

if (letter >= UPPER_START && letter <= UPPER_END) {
    letter = letter + 13;

    if (letter > UPPER_END) {
        letter -= 13;
    }
} else if (letter >= LOWER_START && letter <= LOWER_END) {
    letter = (letter + 13);

    if (letter > LOWER_END) {
        letter -= 13;
    }
}

ROT13代码中的错误（C）

1 个答案: