为什么 'a' 提前结束这个循环？

Question

如果您愿意，请指点而不是直接回答。

这个循环对字符进行一些操作并根据密钥 k 和一些明文 p.

输出密文 c

当明文中出现 'a' 或 'A' 时，程序将按预期输出该字母，但随后过早结束循环。

p突然变成一个字符长，这个字符是1.

while (i < strlen(p))
{
    char stdp = p[i];
    char stdk = k[j];

    if (isalpha(stdp))
    {
        if (islower(stdp))
        {
            p[i] = stdp - 'a';
            Aa = 0;
        }
        else
        {
            p[i] = stdp - 'A';
            Aa = 1;
        }

        if (islower(k[j]))
        {
            k[j] = stdk - 'a';
        }
        else
        {
            k[j] = stdk - 'A';
        }
    }

    if (isalpha(stdp))
    {
        c[i] = ((p[i] + k[j]) % 26);
    }
    else
    {
        c[i] = p[i];
    }

    if (isalpha(stdp))
    {
        if (Aa == 1)
        {
            c[i] = c[i] + 'A';
        }
        else if (Aa == 0)
        {
            c[i] = c[i] + 'a';
        }
    }

    if (isalpha(stdp))
    {
        if (j + 1 == kk)
        {
            j = (j + 1) % kk;
            strcpy(k, argv[1]);
        }
        else
        {
            j++;
        }
    }

    i++;
}

Answer 1

我现在已经解决了程序提前终止所面临的问题。

大多数问题似乎已经通过停止对 strlen(p) 的常量引用并将其分配给静态变量来解决。我已尝试按照建议整理代码。

int main(int argc, char *argv[])
{
if (argc != 2 || argv[1] == NULL)//no more than 1 arg  and not empty
{
    printf("Command requires one argument to run.\n");
    return 1;
}

char *k = malloc(100);
strcpy(k, argv[1]);
int kInt = strlen(k);

for (int i = 0, n = kInt; i < n ; i++)// iterating though the key to check it is alphabetic
{
    if (isalpha(k[i]) == false)
    {
        printf("Make sure key is alphabetic!\n");
        return 1;
    }
}

printf ("plaintext: ");  

char *p = malloc(100);
strcpy(p, get_string());
int pInt = strlen(p);

int i = 0;
int j = 0;
int Aa = 0;

char *c = malloc(100);

while (j < kInt)
{
    if (islower(k[j]))
    {
        k[j] = k[j] - 'a';
    }
    else                
    {
        k[j] = k[j] - 'A';
    }
    j++;
}

j = 0;

while (i < pInt)
{
    if (isalpha(p[i]))
    {
        if (islower(p[i]))
        {
            p[i] = p[i] - 'a';
            Aa = 0;
        }
        else
        {
            p[i] = p[i] - 'A';
            Aa = 1;
        }

        c[i] = ((p[i] + k[j]) % 26);

        if (Aa == 1)
        {
            c[i] = c[i] + 'A';
        }
        else if (Aa == 0)
        {
            c[i] = c[i] + 'a';
        }

        if (j + 1 == kInt)
        {
            j = (j+1) % kInt;
        }
        else
        {
            j++;
        }
    }
    else if (isalpha(p[i]) != true)
    {
        c[i] = p[i];
    }

    i++;
}

printf("ciphertext: %s\n", c);

free(c);
free(k);
free(p);
}

为什么 'a' 提前结束这个循环？

Why does 'a' prematurely end this loop?

c

cs50