CS50 如何修复错误：使用未声明的标识符“i”？

Question

我正在处理我的 PSet2 凯撒问题。 完成我的代码后，这里是我得到的错误。 任何建议如何解决它们？ 非常感谢。

caesar.c:46:10: error: use of undeclared identifier 'i'
    for (i = 0; i < strlen(plaintext); i++)

         ^
caesar.c:46:17: error: use of undeclared identifier 'i'
    for (i = 0; i < strlen(plaintext); i++)

                ^
caesar.c:46:40: error: use of undeclared identifier 'i'
    for (i = 0; i < strlen(plaintext); i++)

                                       ^
caesar.c:48:31: error: use of undeclared identifier 'i'
        if (isupper(plaintext[i]))

                              ^
caesar.c:50:39: error: use of undeclared identifier 'i'
            printf("%c", (((plaintext[i] - 65) + k) %26) + 65);

                                      ^
caesar.c:52:36: error: use of undeclared identifier 'i'
        else if (islower(plaintext[i]))

                                   ^
caesar.c:54:39: error: use of undeclared identifier 'i'
            printf("%c", (((plaintext[i] - 97) + k) %26) + 97);

                                      ^
caesar.c:58:36: error: use of undeclared identifier 'i'
            printf("%c", plaintext[i]);

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

int main(int argc, string argv[])
{
    //Check that there is one command-line argument
    
    if (argc != 2)
    {
        printf ("Usage: ./caesar key\n");
        return 1;
    }
    else printf ("Success!\n");
    
    //Define the key 
    
    string key = argv[1];
    
    //Check if input is a digit
    
    for (int i = 0; i < strlen(argv[1]); i++)
    {
        if (!isdigit(argv[1][i]))
        {
            printf ("Usage: ./caesar key\n");
            return 1;
        }
        else printf ("Success!\n%s\n", key);
    }
    
    //Get plain text from user
    
    string plaintext = get_string("Plaintext: ");
    
    //Define key
    
    int k = atoi(key);
    printf("ciphertext: ");
    
    //Obtain ciphertext
    
    for (i = 0; i < strlen(plaintext); i++)
    {
        if (isupper(plaintext[i]))
        {
            printf("%c", (((plaintext[i] - 65) + k) %26) + 65);
        }
        else if (islower(plaintext[i]))
        {
            printf("%c", (((plaintext[i] - 97) + k) %26) + 97);
        }
        else
        {
            printf("%c", plaintext[i]);
        }
    }
    printf("\n");
}

Answer 1

“使用未声明的标识符'i'”意味着您在使用之前没有声明标识符i - 编译器无法识别该名称。

在这种情况下，代码应该是for (int i = 0; ...

Answer 2

很高兴看到你在做 cs50。 你错过了一点，但很重要的事情。 由于您没有在任何循环之外声明变量i ，因此，它仅适用于该循环。 所以，当你在循环之外的任何地方使用它时，它实际上是未声明的变量，这就是这里发生的事情。 因此，您也需要在最后一个for循环中声明i 。 我附上了修改后的代码。

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

int main(int argc, string argv[])
{
//Check that there is one command-line argument

if (argc != 2)
{
    printf ("Usage: ./caesar key\n");
    return 1;
}
else printf ("Success!\n");

//Define the key 

string key = argv[1];

//Check if input is a digit

for (int i = 0; i < strlen(argv[1]); i++)
{
    if (!isdigit(argv[1][i]))
    {
        printf ("Usage: ./caesar key\n");
        return 1;
    }
    else printf ("Success!\n%s\n", key);
}

//Get plain text from user

string plaintext = get_string("Plaintext: ");

//Define key

int k = atoi(key);
printf("ciphertext: ");

//Obtain ciphertext

for (int i = 0; i < strlen(plaintext); i++) // declared i
{
    if (isupper(plaintext[i]))
    {
        printf("%c", (((plaintext[i] - 65) + k) %26) + 65);
    }
    else if (islower(plaintext[i]))
    {
        printf("%c", (((plaintext[i] - 97) + k) %26) + 97);
    }
    else
    {
        printf("%c", plaintext[i]);
    }
}
printf("\n");
}