为什么 snprintf 在 %s 之后不读取？

Question

我正在尝试编写一个程序来检查 /home/user 目录下是否存在某个位置。为此，我必须使用whoami 命令获取用户名，并将其输出添加到缓冲区以使用locate 命令。

然而，即使snprintf 读取了whoami 命令，它也没有读取其余部分。我进行了几次搜索，结果NULL 可能不会在字符串末尾终止。不过，我不知道如何手动终止它。我不确定问题是什么，所以，我在这里。

这是更好地演示我的问题的代码：

#include <stdio.h>
#include <stdlib.h>
#include <stdbool.h>
#include <assert.h>
#include <string.h>
    
char *readFile(char *);
bool check();
    
bool check() {
    char path[200] = { 0 };
    
    snprintf(path, 200, "/home/%s/.example", readFile("whoami"));
    
    char lll[300] = { 0 };
    
    snprintf(lll, 300, "locate %s", path);
    
    char *buffer = readFile(lll);
    
    if (strcmp(buffer, path) == 0) {
        return true;
    } else {
        return false;
    }
}

char *readFile(char cmd[200]) {
    char cmd1[99999] = { 0 };
    system("touch cmd");  
    snprintf(cmd1, 99999, "%s >> cmd", cmd);
    system(cmd1); 
        
    FILE *f = fopen("cmd", "rt");
    assert(f);
    fseek(f, 0, SEEK_END);
    long length = ftell(f);
    fseek(f, 0, SEEK_SET);
    char *buffer = (char *)malloc(length + 1);
    buffer[length] = '\0';
    fread(buffer, 1, length, f);
    fclose(f);
    system("rm cmd");
    return buffer;
}
    
int main() {
    int x = check();
    
    if (x == 1)
        printf("There is a location like that");
    else
        printf("There isn't");
    
    return 0;
}

Answer 1

whoami 命令打印出当前用户的名称，以换行符结束。将文件cmd 读回buffer 将包含相同的换行符。

path 字符串将是 "/home/USERNAME\n/.example"。试图执行 locate /home/USERNAME\n/.example 可能会混淆system 函数。

解决方案应该是去掉readFile 中的最后一个换行符。

Answer 2

你的readFile函数有一些问题：

• 名称具有误导性，应为 runCommand 或类似名称。

• 您将命令输出附加到cmd 文件。如果在您运行程序之前存在cmd 文件，这将导致意外结果。你应该写：

  snprintf(cmd1, 99999, "%s > cmd", cmd);
  

• 命令输出可能有一个尾随换行符。你应该修剪输出。

这是修改后的版本：

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

char *readFile(const char *cmd) {
    /* construct the command */
    size_t size = 4 + strlen(cmd) + 1;
    char *cmd1 = malloc(size);
    assert(cmd1);
    snprintf(cmd1, size, "%s > cmd", cmd);

    /* run the command */
    system(cmd1);
    free(cmd1);
        
    /* read the output file */
    FILE *f = fopen("cmd", "rt");
    assert(f);
    fseek(f, 0, SEEK_END);
    long length = ftell(f);
    fseek(f, 0, SEEK_SET);
    char *buffer = malloc(length + 1);
    buffer[length] = '\0';
    fread(buffer, 1, length, f);
    fclose(f);
    unlink("cmd");

    /* trim trailing white space from the output */
    while (length > 0 && isspace((unsigned char)buffer[length - 1])) {
        buffer[--length] = '\0';
    }
    /* trim leading white space from the output */
    size_t i = 0;
    while (isspace((unsigned char)buffer[i]) {
        i++;
    }
    if (i > 0) {
        memmove(buffer, buffer + i, length - i + 1);
    }
    return buffer;
}

Answer 3

由于 Linux 为您的主目录 ("~") 提供了一个快捷方式，因此只需使用以下命令就可以避免整个 "whoami"/"getuid" 混乱：

char *buffer = readFile("locate ~/.example");

您仍然可能需要修剪尾随的换行符和/或 CR 字符，可能使用：

strtok(buffer,"\r\n\t ");

或者更好，正如@chqrlie 下面建议的那样。