Question

我在C中创建了一个Caesar密码，我在显示编码消息时遇到了问题。如果消息包含很少的字符，它就有效但是一旦消息超过一定数量的字符， printf 函数就会开始显示我认为是垃圾字节的未知字符。有什么想法吗？

代码：

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

char *concat(const char *s1, const char *s2)
{
    /*
        Remember to free allocated memory
    */
    char *result;
    if((result = malloc(strlen(s1)+strlen(s2)+1)) != NULL)
    {
        result[0] = '\0';
        strcat(result, s1);
        strcat(result, s2);
    }

    return result;
}

void encode(const char *alpha, const char *message)
{
    char result[(sizeof(message) / sizeof(char))];
    int x, y, z;

    memset(result, '\0', sizeof(result));
    for(x = 0; x < strlen(message); x++)
    {
        for(y = 0; y < 25; y++)
        {
            if(alpha[y] == message[x])
            {
                z = (y + 3);
                if(z > 25)
                {
                    z = z % 25;
                }

                result[x] = alpha[z];
            }
        }
    }

    printf("%s\n", result);

    //return result;
}

int main()
{
    char message[200];
    char *result;
    char array[26] = { 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z' };

    printf("Input your message:");
    scanf("%s", &message);

    encode(array, message);
}

结果：

Answer 1

更改

char result[(sizeof(message) / sizeof(char))];

到

char result[strlen(message) + 1];

这会在缓冲区中为terminating null-character留出空间。 sizeof也可以用于获取c字符串的长度，我们需要使用strlen。您还需要更新memset电话。

sizeof由编译器使用，因此在这种情况下，您将询问编译器＆＃34; message＆＃34;的大小是多少，在这种情况下将是8（（64位），因为指针是8个字节。

除此之外，您应该对scanf小心，因为您现在读取了一行输入，但是如果您尝试再读一行，则可以run into trouble。

正如评论中所提到的，for(x = 0; x < strlen(message); x++)每次循环迭代运行strlen一次，为您的程序增加了相当大的额外负载，随着message的长度增加，指数增长。< / p>

您的concat功能未使用，不确定为什么要包含它？

打印字符数组时显示未知字符

1 个答案: