Question

我试图以与命令行args int main( int argc, char *argv[] )类似的方式从键盘获取命令，但是在单独的函数中。当我在getCmd()函数的范围内解析并打印它们时，所有的外观和行为都按照预期进行，但是一旦它们返回到main函数，它们就变成了一堆垃圾。我的问题在代码之下。

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

void getCmd(char *cmd, char *args[])
{
    char input[81] = { 0 };
    char *next_token = NULL;
    printf("$ ");
    fgets(input, 81, stdin);
    input[strcspn(input, "\n")] = 0;
    cmd = strtok_s(input, " ", &next_token);
    if (!strcmp(cmd, "mv"))
    {
        args[0] = strtok_s(NULL, " ", &next_token);
        args[1] = strtok_s(NULL, " ", &next_token);
        printf("\n\n%s\n%s\n%s\n\n", cmd, args[0], args[1]);
    }
}

int main(void)
{
    char *cmd = NULL, *args[5];

    cmd = (char *)calloc(20,sizeof(char));
    for (size_t i = 0; i < (size_t)5; i++)
    {
        args[i] = (char *)calloc(20,sizeof(char));
    }
    getCmd(cmd, args);
    printf("\n\n%s \n%s\n%s", cmd, args[0], args[1]);
    return 0;
}

我不认为它是相关的，但我在64位处理器，Windows 7操作系统上使用VS 2015社区和Visual C ++编译器。

我的问题：

我应该如何通过引用传递cmd和args []？

是否有任何广泛接受的习语可以解决这种情况？

我已经看过一些类似的问题，并且无法找到适合这种情况的解决方案，如果问题是重复的，请告诉我，我会关闭它。我是stackoverflow的新手任何问题格式提示将不胜感激。干杯! （：

Answer 1

有许多不同的方法可以解决这个问题。虽然您可以自由地为cmd和args数组动态分配内存，但实际上没有必要，因为这个有限的内存量可以为所有内存使用静态声明。不需要单独的input数组，您可以使用cmd来实现此目的，然后对cmd进行标记。这样可以在调用cmd后将{em> nul-terminated 保留在strtok中。

注意：，在下面的示例中，使用strtok，strtok_s是C11中的可选编译器添加，不幸的是，我没有编译器实现该选项，因此我使用strtok进行测试。您可以轻松地对VS进行更改。

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

enum { NARGS = 5, MAXC = 128 };

size_t getcmd (char *cmd, char (*args)[MAXC]);

int main (void) {

    char cmd[MAXC] = "", args[NARGS][MAXC] = { "" };
    size_t i, n;

    if (!(n = getcmd (cmd, args))) return 1;

    printf ("  %s", cmd);
    for (i = 0; i < n; i++)
        printf(" %s", args[i]);
    putchar ('\n');

    return 0;
}

size_t getcmd (char *cmd, char (*args)[MAXC])
{
    char *delim = " ,.\t\n";
    char *p = NULL;
    size_t idx = 0;

    printf ("$ ");
    if (!fgets (cmd, MAXC, stdin)) {
        fprintf (stderr, "error: invalid input.\n");
        return 0;
    }

    strtok (cmd, delim);            /* terminate after 1st token */

    for (p = strtok (NULL, delim); p; p = strtok (NULL, delim)) {
        strncpy (args[idx++], p, MAXC);  /* limit to avail chars */
        if (idx == NARGS) break;    /* limit to available bounds */
    }

    return idx;
}

请注意，getcmd的返回类型为size_t。始终选择一个有意义的类型来返回成功/失败的指示，并返回一些所需的信息（这里的参数数量）。另请注意，C风格指南不喜欢 camelCase 变量/函数名称而不是所有小写。保留C ++的 camelCase 名称。参见例如NASA - C Style Guide, 1994

示例使用/输出

$ ./bin/getcmd
$ mv /this/here/file /that/there/file
  mv /this/here/file /that/there/file

$ ./bin/getcmd
$ mv -i --strip-trailing-slashes /this/here/file /that/there/file
  mv -i --strip-trailing-slashes /this/here/file /that/there/file

仔细看看，如果您有任何其他问题，请告诉我。

Answer 2

strtok_s()将指针返回到它正在解析的缓冲区（input这里）。

input生活在getCmd()堆栈上。它会在getCmd()返回时消失。从那时起，指向input的地址和存储在args元素中的地址不再指向有效的内存。

代码需要分配新内存并复制strtok_s()返回指针的内容。

看看这可以做到：

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

void getCmd(char **pcmd, char *args[], size_t s)
{
    char input[81] = { 0 };
    char *next_token = NULL;
    printf("$ ");
    fgets(input, 81, stdin);
    input[strcspn(input, "\n")] = 0;
    (*pcmd) = _strdup(strtok_s(input, " ", &next_token));
    if (!strcmp(*pcmd, "mv"))
    {
        args[0] = _strdup(strtok_s(NULL, " ", &next_token));
        args[1] = _strdup(strtok_s(NULL, " ", &next_token));
        printf("\n\n%s\n%s\n%s\n\n", *pcmd, args[0], args[1]);
    }
}


#define ARGS_MAX (5)

int main(void)
{
    char *cmd, *args[ARGS_MAX] = {0};

    getCmd(&cmd, args, ARGS_MAX);
    printf("\n\n%s \n%s\n%s", cmd, args[0], args[1]);

    /* Clean up. */
    free(cmd);
    for (size_t i = 0; i < ARGS_MAX; ++i)
    {
      free(args[i]);
    }

    return 0;
}

通过引用

2 个答案: