构建一个基本shell,更具体地说是使用execvp()

在我的程序中,我正在接受用户输入并将其解析为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("%sn%sn", 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 =  ''; /* 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 != '') {
    int items_read = sscanf(ptr, "%s%n", field, &n);
    strcpy(parsedText[count++], field);
    field[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 */
}

}
    
已邀请:
execvp
指向指针(
char **
),而不是指向数组的指针。它应该是指向
char *
指针数组的第一个元素的指针,由空指针终止。 编辑:这是制作适合
execvp
的指针数组的一种(不是很好的)方法:
char argbuf[10][256] = {{0}};
char *args[10] = { argbuf[0], argbuf[1], argbuf[2], /* ... */ };
当然,在现实世界中,您的参数可能来自用户输入的命令行字符串,并且它们之间可能至少有一个字符(例如空格),因此更好的方法是修改原始字符串 - 放置或复制它然后修改副本,在每个参数后添加空终止符并设置
args[i]
指向字符串中的右偏移量。 相反,您可以在每一步中进行大量动态分配(
malloc
),但是您必须编写代码来处理每个可能的故障点。 :-)     

要回复问题请先登录注册