Question

我试图在Linux上用C语言构建一个进程记录器，但是无法正确完成它。我希望它有3个列：USER，PID，COMMAND。我正在使用ps aux的输出并尝试将其动态附加到数组中。也就是说，对于每行ps aux输出，我想在我的数组中添加一行。

这是我的代码。（为了保持输出简短，我只追求崇高。但这可能是任何东西。）

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

int main()
{
    char** processes = NULL;
    char* substr = NULL;
    int n_spaces = 0;
    int columns = 1;

    char line[1024];
    FILE *p;
    p = popen("ps -eo user,pid,command --sort %cpu | grep sublime", "r");
    if(!p)
    {
        fprintf(stderr, "Error");
        exit(1);
    }

    while(fgets(line, sizeof(line) - 1, p))
    {
        puts(line);
        substr = strtok(line, " ");
        while(substr != NULL)
        {
            processes = realloc(processes, sizeof(char*) * ++n_spaces);

            if(processes == NULL)
                exit(-1);

            processes[n_spaces - 1] = substr;

            // first column user, second PID, third all the rest
            if(columns < 2)//if user and PID are already in array, don't split anymore
            {
                substr = strtok(NULL, " ");
                columns++;
            }
            else
            {
                substr = strtok(NULL, "");
            }
        }
        columns = 1;
    }
    pclose(p);

    for(int i = 0; i < (n_spaces); i++)
        printf("processes[%d] = %s\n", i, processes[i]);

    free(processes);

    return 0;

}

最后的for循环输出如下所示。

processes[0] = user
processes[1] = 7194
processes[2] = /opt/sublime_text/plugin_host 27184

processes[3] = user
processes[4] = 7194
processes[5] = /opt/sublime_text/plugin_host 27184

processes[6] = user
processes[7] = 27194
processes[8] = /opt/sublime_text/plugin_host 27184

processes[9] = user
processes[10] = 27194
processes[11] = /opt/sublime_text/plugin_host 27184

但是，从puts(line)我得到的数组实际上应该包含这个：

user       5016 sh -c ps -eo user,pid,command --sort %cpu | grep sublime
user       5018 grep sublime
user      27184 /opt/sublime_text/sublime_text
user      27194 /opt/sublime_text/plugin_host 27184

所以，显然所有的价值都被覆盖了，我无法弄清楚为什么......（另外，我不会在7194中找到价值processes[0] = 7194的地方processes[4] = 7194来自）。

我在这里做错了什么？是否有可能使输出看起来像puts(line)的输出？

任何帮助将不胜感激！

Answer 1

strtok返回指向它处理的字符串的指针。该字符串始终存储在变量line中。因此，数组中的所有指针都指向内存中的相同位置。

@Lundin在评论中写道，这里没有二维数组，只是一维指针数组。

Answer 2

strtok的返回值是指向您标记的字符串的指针。（通过使用'\0'覆盖第一个分隔符后，令牌已被空终止。）

当您使用fgets读取新行时，您将覆盖此行的内容，并且所有标记（而不仅仅是上次解析中的标记）都指向该行的实际内容。（指向前一个令牌的指针仍然有效，但这些位置的内容会发生变化。）

有几种解决方法。

您可以使用令牌保存字符数组并strcpy解析内容。
您可以使用（非标准）strdup复制已解析的令牌，这会为堆上的字符串分配内存。
您可以阅读一系列行，以便令牌真的是唯一的。

为什么我的动态2D字符数组中的值被覆盖？

2 个答案: