1

我有一个关于extern char **environ. 我正在尝试制作一个 C 程序来计算环境列表的大小，将其复制到字符串数组（字符数组数组），然后使用冒泡排序按字母顺序对其进行排序。它将根据格式值打印name=value或排序。value=name

我尝试使用strncpy将字符串从 environ 获取到我的新数组，但字符串值显示为空。我怀疑我正在尝试以我无法使用的方式使用 environ，所以我正在寻求帮助。我试图在网上寻求帮助，但这个特定的程序非常有限。我不能使用system()，但我在网上找到的唯一帮助告诉我编写一个程序来进行这个系统调用。（这没有帮助）。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
extern char **environ;
int main(int argc, char *argv[])
{
    char **env = environ;
    int i = 0;
    int j = 0;
    printf("Hello world!\n");
    int listSZ = 0;
    char temp[1024];
    while(env[listSZ])
    {
        listSZ++;
    }
    printf("DEBUG: LIST SIZE = %d\n", listSZ);
    char **list = malloc(listSZ * sizeof(char**));
    char **sorted = malloc(listSZ * sizeof(char**));
    for(i = 0; i < listSZ; i++)
    {
        list[i] = malloc(sizeof(env[i]) * sizeof(char));        // set the 2D Array strings to size 80, for good measure
        sorted[i] = malloc(sizeof(env[i]) * sizeof(char));
    }
    while(env[i])
    {
        strncpy(list[i], env[i], sizeof(env[i]));
        i++;
    }           // copy is empty???

    for(i = 0; i < listSZ - 1; i++)
    {
        for(j = 0; j < sizeof(list[i]); j++)
        {
            if(list[i][j] > list[i+1][j])
            {
                strcpy(temp, list[i]);
                strcpy(list[i], list[i+1]);
                strcpy(list[i+1], temp);
                j = sizeof(list[i]);                    // end loop, we resolved this specific entry
            }
            // else continue
        }
    }

这是我的代码，非常感谢帮助。为什么这个话题这么难找？是缺乏必要性吗？

编辑：粘贴错误的代码，这是同一主题的单独 .c 文件，但我从另一个文件开始。

4

3 回答 3

1

您的代码存在多个问题，包括：

  • list为and分配“错误”大小sorted（您乘以sizeof(char **)，但应该乘以，sizeof(char *)因为您分配的是 数组char *。这个错误这次实际上不会伤害您。使用sizeof(*list)可以避免问题。
  • list为和中的元素分配错误的大小sorted。您需要使用strlen(env[i]) + 1大小，记住允许终止字符串的空值。
  • 您不检查内存分配。
  • 您的字符串复制循环正在使用strncpy()并且不应该使用（实际上，您应该很少使用strncpy()），尤其是因为它只复制每个环境变量的 4 或 8 个字节（取决于您使用的是 32 位还是 64 位系统），并且不能确保它们是空终止的字符串（只是不使用strncpy().
  • 您的“排序”代码的外循环是可以的；您的内部循环是 100% 虚假的，因为您应该使用一个或另一个字符串的长度，而不是指针的大小，并且您的比较是在单个字符上进行的，但是您使用的strcpy()只是需要移动指针的地方.
  • 您分配但不使用sorted.
  • 您不会打印 sorted 环境来证明它已排序。
  • 您的代码缺少最终的}.

这是一些使用标准 C 库函数进行排序的简单代码，并 以名称qsort()模拟 POSIX —如果您有可用的 POSIX，则可以使用。strdup()dup_str()strdup()

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

extern char **environ;

/* Can also be spelled strdup() and provided by the system */
static char *dup_str(const char *str)
{
    size_t len = strlen(str) + 1;
    char *dup = malloc(len);
    if (dup != NULL)
        memmove(dup, str, len);
    return dup;
}

static int cmp_str(const void *v1, const void *v2)
{
    const char *s1 = *(const char **)v1;
    const char *s2 = *(const char **)v2;
    return strcmp(s1, s2);
}

int main(void)
{
    char **env = environ;
    int listSZ;

    for (listSZ = 0; env[listSZ] != NULL; listSZ++)
        ;
    printf("DEBUG: Number of environment variables = %d\n", listSZ);

    char **list = malloc(listSZ * sizeof(*list));
    if (list == NULL)
    {
        fprintf(stderr, "Memory allocation failed!\n");
        exit(EXIT_FAILURE);
    }

    for (int i = 0; i < listSZ; i++)
    {
        if ((list[i] = dup_str(env[i])) == NULL)
        {
            fprintf(stderr, "Memory allocation failed!\n");
            exit(EXIT_FAILURE);
        }
    }

    qsort(list, listSZ, sizeof(list[0]), cmp_str);

    for (int i = 0; i < listSZ; i++)
        printf("%2d: %s\n", i, list[i]);

    return 0;
}

其他人指出，您可以main()使用原型通过第三个参数来获取环境int main(int argc, char **argv, char **envp)。请注意，Microsoft 明确支持这一点。它们是正确的，但您也可以通过 获得环境environ，即使在 . 以外的功能中也是如此main()。该变量environ在 POSIX 定义的全局变量中是唯一的，没有在任何头文件中声明，因此您必须自己编写声明。

请注意，内存分配是错误检查的，错误报告在标准错误上，而不是标准输出上。

显然，如果你喜欢编写和调试排序算法，你可以避免使用qsort(). 请注意，字符串比较需要使用 来完成，但在对指针数组进行排序时strcmp()不能strcmp()直接使用 with ，因为参数类型错误。qsort()

我的部分输出是：

DEBUG: Number of environment variables = 51
 0: Apple_PubSub_Socket_Render=/private/tmp/com.apple.launchd.tQHOVHUgys/Render
 1: BASH_ENV=/Users/jleffler/.bashrc
 2: CDPATH=:/Users/jleffler:/Users/jleffler/src:/Users/jleffler/src/perl:/Users/jleffler/src/sqltools:/Users/jleffler/lib:/Users/jleffler/doc:/Users/jleffler/work:/Users/jleffler/soq/src
 3: CLICOLOR=1
 4: DBDATE=Y4MD-
…
47: VISUAL=vim
48: XPC_FLAGS=0x0
49: XPC_SERVICE_NAME=0
50: _=./pe17

如果要对值而不是名称进行排序，则必须做一些更艰苦的工作。您需要定义您希望看到的输出。有多种方法可以处理这种排序。

于 2018-04-28T08:38:03.030 回答
1

在 unix 环境中，环境是 main 的第三个参数。

尝试这个：

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

int main(int argc, char *argv[], char **envp)
{

   while (*envp) {
   printf("%s\n", *envp);
   *envp++;
   }
 }
于 2018-04-28T01:50:17.567 回答
0

要获取环境变量，您需要main像这样声明：

int main(int argc, char **argv, char **env);

第三个参数是NULL- 终止的环境变量列表。看：

#include <stdio.h>

int main(int argc, char **argv, char **environ)
{
    for(size_t i = 0; env[i]; ++i)
        puts(environ[i]);

    return 0;
}

这个的输出是：

LD_LIBRARY_PATH=/home/shaoran/opt/node-v6.9.4-linux-x64/lib:
LS_COLORS=rs=0:di=01;34:ln=01;36:m
...

另请注意，sizeof(environ[i])在您的代码中不会得到字符串的长度，它会得到指针的大小，所以

strncpy(list[i], environ[i], sizeof(environ[i]));

是错的。同样的重点是基于目标strncpy限制，而不是源限制，否则如果源大于目标，你仍然会溢出缓冲区。正确的调用是

strncpy(list[i], environ[i], 80);
list[i][79] = 0;

请记住，如果目标不够大，strncpy可能不会写入'\0'-terminating 字节，因此您必须确保终止字符串。另请注意，79 个字符可能太短，无法存储环境变量。例如，我的LS_COLORS变量很大，至少有 1500 个字符长。您可能希望list[i] = malloc基于strlen(environ[i])+1.

另一件事：你的交换

strcpy(temp, list[i]);
strcpy(list[i], list[i+1]);
strcpy(list[i+1], temp);
j = sizeof(list[i]);

仅当所有都list[i]指向相同大小的内存时才有效。由于list[i]是指针，因此更便宜的交换方式是交换指针：

char *tmp = list[i];
list[i] = list[i+1];
list[i+1] = tmp;

这更有效，是一个 O(1) 操作，如果内存空间大小不同，您不必担心。

我不明白的是，你打算做什么j = sizeof(list[i])？这不仅会sizeof(list[i])返回指针的大小（对于 all 来说都是常量list[i]），为什么还要弄乱j块内的运行变量？如果你想离开循环， do break. 你正在寻找 strlen(list[i])：这会给你字符串的长度。

于 2018-04-28T01:55:35.687 回答