时间戳

首页 > 解决方案 > 使用 strcat 在一行中连接多个 char*

c - 使用 strcat 在一行中连接多个 char*

问题描述

我目前正在尝试在我的服务器程序中使用strcat. 代码的相关部分如下：我成功获取了客户端发送的年月日和文件名，因为它打印得很好。

我的字符串初始化如下：

    char username[MAX_USERNAME_SIZE];
    char filename[MAX_FILENAME_SIZE];
    char path[MAX_FILEPATH_SIZE];
    char buff[BUFFSIZE];
    char year[4], month[2], day[2];
    ...
    if ((numbytes = recv(new_fd, &username, MAX_USERNAME_SIZE, 0)) == -1)
    {
        perror("Serveur: recv username");
        return EXIT_FAILURE;
        }
        username[numbytes] = '\0';
        printf("Serveur: username: %s\n", username); 
        /* create user's repository if it doesn't exist yet*/

        // 3) get date from client
        if ((numbytes = recv(new_fd, year, 4, 0)) == -1)
        {
            perror("Serveur: recv year");
            return EXIT_FAILURE;
        }
        year[numbytes] = '\0';
        printf("year: %s\n", year);
        if ((numbytes = recv(new_fd, month, 2, 0)) == -1)
        {
            perror("Serveur: recv month");
            return EXIT_FAILURE;
        }
        month[numbytes] = '\0';
        printf("month: %s\n", month);
        if ((numbytes = recv(new_fd, day, 2, 0)) == -1)
        {
            perror("Serveur: recv day");
            return EXIT_FAILURE;
        }
        day[numbytes] = '\0';
        printf("day: %s\n", day);
        // get filename from client
        if ((numbytes = recv(new_fd, filename, MAX_FILENAME_SIZE, 0)) == -1)
        {
            perror("Serveur: recv filename");
            return EXIT_FAILURE;
        }
        filename[numbytes] = '\0';
        printf("Serveur: filename: %s\n", filename);

但是当我尝试正确连接所有字符串时，我遇到了问题，如图所示。

        // create user repository
        strcpy(path, argv[1]);
        printf("Serveur: Path: %s\n", path);
        strcat(path, "/");
        printf("Serveur: Path: %s\n", path);

        strcat(path, username);
        strcat(path, "/");
        printf("Serveur: Path: %s\n, username:%s\n", path, username);
        my_mkdir(path, MODE);

        strcat(path, year);
        strcat(path, "/");
        printf("Serveur: Path: %s\n, year: %s\n", path, year);
        my_mkdir(path, MODE);

        strcat(path, month);
        strcat(path, "/");
        printf("Serveur: Path: %s\n, month: %s\n", path, month);
        my_mkdir(path, MODE);

        strcat(path, day);
        strcat(path, "/");
        printf("Serveur: Path: %s\n, day: %s\n", path, day);
        my_mkdir(path, MODE);

        strcat(path, filename);
}

在我这样做之后，用户名、年份和月份的打印效果令人惊讶。这是我执行代码时的输出（我确定文件名是可以的，因为我从客户端检索的文件保存在正确的名称下）：

Serveur: connection recieved from client 127.0.0.1
Serveur: username: student
year: 95
month: 5
day: 11
Serveur: filename: tux.png
Serveur: Path: ./Test0/Test1
Serveur: Path: ./Test0/Test1/
Serveur: Path: ./Test0/Test1//
, username:
Serveur: Path: ./Test0/Test1///
, year: 
Serveur: Path: ./Test0/Test1////
, month: 
Serveur: Path: ./Test0/Test1////11/
, day: 11

我真的很想清楚我错在哪里。提前致谢

标签： cstrcat

解决方案

问题是您没有为字符串提供足够的空间。你仍然没有确切地展示它们是如何声明的，但是有足够的例子来证明这个问题，比如这个。

char year[4]

上面只能容纳 3 个字符 - 第 4 个是存储\0终止字符的空间。

在这里，您告诉您的代码从套接字读取 4 个字节，它将执行此操作，然后numbytes将等于 4...

    if ((numbytes = recv(new_fd, year, 4, 0)) == -1)

...然后您使用numbytes添加 NUL 终止符...

    year[numbytes] = '\0';

...但是year[4]超出了数组的范围，这会导致未定义的行为。它可能看起来year有一个有效的年份，但随后你的代码的其余部分发生了并且发生了奇怪的事情，因为你也对所有其他字符串也犯了同样的错误。

要容纳 4 个字符加上 NUL，您需要year至少声明 5 个字符，像这样......

char year[5]

...但是您的计算机内存不足以至于您需要如此节俭吗？最好给它比它需要的更多，或者将它读入一个有足够空间的缓冲区，并分配足够的内存来创建正确大小的字符串，如果空间非常宝贵，就像这个例子所示。

char *year;
char buffer[100];
if ((numbytes = recv(new_fd, buffer, 4, 0)) == -1)
{
    perror("Serveur: recv year");
    return EXIT_FAILURE;
}
buffer[numbytes] = '\0';
year=malloc(numbytes+1);
strcpy(year,buffer);

推荐阅读