Question

我已经盯着这个问题好几个星期了，我一无所有！它不起作用，我知道的很多，但我不知道为什么或什么是错的。我知道开发人员吐出了关于我突出显示的那一行的“错误：预期表达”，但实际上这只是冰山一角。如果有人知道如何修复任何一小部分，我会非常感激！

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <cs50.h>
#include <ctype.h>

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

    //Get the key
    if (argc != 2 || atoi(argv[1]) < 0)
    {
        printf("Usage: ./caesar k");
        return 1;
    }

    int key = atoi(argv[1]);
    string plaintex;
    string plaintext = GetString();

    for (int i = 0, n = strlen(plaintext); n < i; i++)
    {
        if (plaintext[i] > 'A' && plaintext[i] <= 'Z')
        {
            plaintext[i] = (plaintext[i] - 'A' + key) % 26 + 'A';
        }
    }  
    for (int i = 0, n = strlen(plaintext); n < i; i++)
    {           
        if (plaintext[i] >= 'A' && plaintext[i] >= 'Z')  // Highlighted line
        {
            plaintext[i] = (plaintext[i] - 'A' + key) % 26 + 'A';
        }
        else if (plaintext[i] >= 'a' && plaintext[i] < 'z')
        {
            plaintext[i] = (plaintext[i] - 'a' + key) % 26 + 'a';
        }
        else
        {
            printf("%c\n", plaintext[i]);
        }
    }
    return 0;
}

Answer 1

if (plaintext[i] >= 'A' && plaintext[i] >= 'Z')

应该是

if (plaintext[i] >= 'A' && plaintext[i] <= 'Z')

Answer 2

A的符号表索引不如Z。最有可能A == 65和Z == 90。

if (plaintext[i] >= 'A' && plaintext[i] >= 'Z')

你说的是“如果某物大于65且大于90”。逻辑没有任何意义，这与说“如果某事物大于90”是完全一样的。

你可能意味着

if (plaintext[i] >= 'A' && plaintext[i] <= 'Z')

CS50的Caesar Cipher

2 个答案: