Question

在我的程序中，我正在接受用户输入并将其解析为2d char数组。该数组声明为：

char parsedText[10][255] = {{""},{""},{""},{""},{""},
            {""},{""},{""},{""},{""}};

我正在使用fgets来获取用户输入并使用sscanf解析它。这一切都像我认为的那样有效。

在此之后我想将parsedText传递给execvp，parsedText [0]应该包含路径，如果提供了任何参数，那么它们应该在parsedText [1]中通过parsedText [10]。

execvp有什么问题（parsedText [0]，parsedText [1]）？

可能值得一提的是，如果我只提供诸如“ls”之类的命令而没有任何参数，那么它似乎工作正常。

这是我的代码：

#include <stdio.h>
#include <string.h>
#include <unistd.h>
#include "308shell.h"

int main( int argc, char *argv[] )
{
char prompt[40] = "308sh";
char text[40] = "";
char parsedText[10][40] = {{""},{""},{""},{""},{""},
                           {""},{""},{""},{""},{""}};

// Check for arguments to change the prompt.
if(argc >= 3){
    if(!(strcmp(argv[1], "-p"))){
        strcpy(prompt, argv[2]);
    }
}

strcat(prompt, "> ");

while(1){
    // Display the prompt.
    fputs(prompt, stdout);
    fflush(stdout);

    // Grab user input and parse it into parsedText. 
    mygetline(text, sizeof text);
    parseInput(text, parsedText);

    // Check if the user wants to exit.
    if(!(strcmp(parsedText[0], "exit"))){
        break;
    }
    execvp(parsedText[0], parsedText[1]);
    printf("%s\n%s\n", parsedText[0], parsedText[1]);
}

return 0;
}

char *mygetline(char *line, int size)
{
if ( fgets(line, size, stdin) )
{
    char *newline = strchr(line, '\n'); /* check for trailing '\n' */
    if ( newline )
    {
        *newline =  '\0'; /* overwrite the '\n' with a terminating null */
    }
}

return line;
}

char *parseInput(char *text, char parsedText[][40]){
char *ptr = text;
char field [ 40 ];
int n;
int count = 0;

while (*ptr != '\0') {
    int items_read = sscanf(ptr, "%s%n", field, &n);
    strcpy(parsedText[count++], field);
    field[0]='\0';
    if (items_read == 1)
        ptr += n; /* advance the pointer by the number of characters read     */
    if ( *ptr != ' ' ) {
        strcpy(parsedText[count], field);
        break; /* didn't find an expected delimiter, done? */
    }
    ++ptr; /* skip the delimiter */
}

}

Answer 1

execvp获取指向指针（char **）的指针，而不是指向数组的指针。它应该是指向char *指针数组的第一个元素的指针，由空指针终止。

编辑：这是制作适合execvp的指针数组的一种（非常好的）方法：

char argbuf[10][256] = {{0}};
char *args[10] = { argbuf[0], argbuf[1], argbuf[2], /* ... */ };

当然，在现实世界中，您的参数可能来自用户输入的命令行字符串，并且它们之间可能至少有一个字符（例如空格），因此更好的方法是修改原始字符串就地填充字符串，或复制它，然后修改副本，在每个参数后添加空终止符，并设置args[i]指向字符串中的右侧偏移量。

您可以在每一步中执行大量动态分配（malloc），但是您必须编写代码来处理每个可能的故障点。： - ）

构建一个基本shell，更具体地说是使用execvp（）

1 个答案: