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

Posted 2023-02-22

【中文标题】为啥我的 char 数组长度不正确？

【英文标题】：Why is my char array not the correct length?为什么我的 char 数组长度不正确？

【发布时间】：2020-08-14 20:52:44

【问题描述】：

我有一个程序可以接收密钥。此键将许多​​字符环绕在 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。我认为我的问题出在某个地方，但我对计算机科学的了解还不够，无法弄清楚发生了什么。能给我解惑吗？

【问题讨论】：

在填写 ciphertext 之前，您已致电 strlen(ciphertext)。

您的环境中是否有string 和typedef？

@StephenNewell 显然是 CS50。

您还尝试在初始化之前打印ciphertext[0]。这就是为什么你会得到奇怪的标签。

您使用的是 C 还是 C++？您的问题标记为 C++，但代码看起来像 C。

【参考方案1】：

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

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

char ciphertext[strlen(plaintext)+1];

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

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

就此而言，您在尝试打印之前不会使用 任何 文本数据填充 ciphertext，因此 printf() 将打印垃圾（如果有的话），因为 ciphertext 不是保证在其声明中以空值终止。

【讨论】：

我怀疑这是一个 CS50 字符串，而不是 C++ std::string。

其实很明显，因为他写的是string argv[]。这是typedef char* string;

@Barmar 好的，这并不能改变ciphertext 仍然被错误处理的事实。

问题中显示的代码显然是假代码而不是真实代码。 plaintext 甚至没有在任何地方定义。 planetext 与 plaintext 不同。这个问题是一个失败的原因。

当然，我在上面的 cmets 中提到过。

【参考方案2】：

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