为什么我的char数组长度不正确？

问题描述

我有一个可以输入密钥的程序。此键可移动纯文本，其中许多字符环绕z并保留大小写。仅当密文的容器大小不正确时才会出现问题。为什么ciphertext数组比我的planetext字符串大？

#import <cs50.h>
#import <stdio.h>
#import <math.h>
#import <string.h>
#import <ctype.h>
#import <stdlib.h>

bool is_number(string str);
void print_string(string call,string s);

// Your program must accept a single command-line argument,a non-negative integer. Let’s call it k for the sake of discussion.
int main(int argc,string argv[])
{
    //return error because there was no key given
    if (argc == 1 || !is_number(argv[1]))
    {
        printf("Usage: ./caesar key\n");
        return 1;
    }
    else if (argc != 2){
        printf("Usage: ./caesar key\n");
        return 1;
    }

    string plaintext = get_string("plaintext:");
    char ciphertext[strlen(plaintext)];
    int k = atoll(argv[1]) % 26;
    printf("plain Text: %s\n",plaintext);
    printf("Text len: %lu\n",strlen(plaintext));
    printf("Char len: %lu\n",strlen(ciphertext));
    printf("Char 1: %c\n",ciphertext[0]);
    printf("key length :%i\n",k);
    printf("ciphertext before: %s\n",ciphertext);
    
    for (int i = 0,n = strlen(plaintext); i < n; i++)
    {
        //isupper
        if (isupper(plaintext[i]))
        {
            //does it go past Z?
            if (plaintext[i] + k > 'Z')
            {
                ciphertext[i] = plaintext[i] + k - 'Z' + 'A' - 1;
            }
            // does it not go past Z
            else
            {
                ciphertext[i] = plaintext[i] + k;
            }
        }
        //is lower
        else if (islower(plaintext[i]))
        {
            //does it go past z?
            if (plaintext[i] + k > 'z')
            {
                ciphertext[i] = plaintext[i] + k - 'z' + 'a' - 1;
            }
            // does it not go past z
            else
            {
                ciphertext[i] = plaintext[i] + k;
            }
        }
        // if anything else don't change it
        else
        {
            ciphertext[i] = plaintext[i];
        }
    }
    printf("ciphertext after: %s",ciphertext);
    printf("\n");
    return 0;
}

对文本进行加扰就可以了。我只是不明白为什么在某些单元测试结束时会有一些垃圾值。

这是我的代码的输出：

plaintext:a
Plane Text: a
Text len: 1
Char len: 6
Char 1: 
    key length :1 //this is not a tabbing error. This was my output.
ciphertext:b*

plaintext:hello
plain Text: hello
Text len: 5
Char len: 6
Char 1: 
key length :12
ciphertext before: n2
ciphertext after: tqxxa

plaintext:asdfjdnghsidkwqd
plain Text: asdfjdnghsidkwqd
Text len: 16
Char len: 6
Char 1:  
key length :12
ciphertext before:  $
ciphertext after: meprvpzsteupwicp

plaintext:ashdngkdirheknshd
plain Text: ashdngkdirheknshd
Text len: 17
Char len: 0
Char 1: 
key length :12
ciphertext before: 
ciphertext after: metpzswpudtqwzetp'

我注意到的是char len为6，直到我在plaintext字符串中输入了16个以上的字符。然后它下降到0。我认为我的问题在这里，但我对计算机科学的了解不足，无法弄清发生了什么。你能启发我吗？

解决方法

您需要在strlen中添加一个，以便为终止字符串的'\ 0'留出空间。

,

char ciphertext[strlen(plaintext)];是一个可变长度数组，在C ++中为非标准，请参见Why aren't variable-length arrays part of the C++ standard?。

在任何情况下，该声明都不会为空终止符提供空间。您需要为此添加+1：

char ciphertext[strlen(plaintext)+1];

然后，确保char中最后一个未使用的ciphertext实际设置为'\0'，例如：

// copy some text into ciphertext,then...
ciphertext[LengthActuallyUsed] = '\0';

为此，在尝试打印之前，您不会在ciphertext中填充任何文本数据，因此printf()将打印垃圾（如果有的话），因为{ {1}}不能保证在其声明中以null结尾。