c - 当我用 C 语言发送带有 char 数组的命令时，system() 调用或 exec() 函数不起作用

Question

我正在尝试在 Linux 中制作视频循环器。为此，我正在使用 mplayer 播放视频。我的程序从目录中获取视频和图像列表并播放它们。我的问题是当我使用 char 数组参数从 c 调用 mplayer 或图像查看器时它不起作用。例如，

system("eog -f /home/user/Desktop/Video/screenshot_0000.png");

或

system("mplayer -fs /home/user/Desktop/Video/video1.mov");

如果我这样调用它就可以了。但是当我从 directy 获取视频列表并使用 char 数组发送它们时，它不起作用。当我尝试调用 system() 时，它会给出“sh: 1: video1.mov not found”错误，而当我使用 exec() 函数时，什么也没有发生。

这是我的代码；

    #include <stdio.h>
    #include <sys/types.h>
    #include <unistd.h>
    #include <stdlib.h>
    #include <sys/wait.h>
    #include <dirent.h>
    #include <string.h>

    #define FILE_PATH "/home/user/Desktop/Video/"

    char *getExt (const char *fspec) {
        char *e = strrchr (fspec, '.');
        if (e == NULL)
            e = ""; // fast method, could also use &(fspec[strlen(fspec)]).
        return e;
    }

    void get_list(char liste[][150])
    {
        DIR *d;
        struct dirent *dir;
        char *path = ("%s.",FILE_PATH);
        d = opendir(path);
        int counter=0;
        /*if(d != NULL)
        {
            while((dir=readdir(d))!= NULL)
            {
    //          printf("%s\n", dir->d_name);
                if(dir->d_type==DT_REG)
                    file_numbers++;
            }
            closedir(d);

        }
        char liste[file_numbers][150];*/
        d = opendir(path);
        if(d != NULL)
        {
            while((dir=readdir(d))!= NULL)
            {
    //          printf("%s\n", dir->d_name);
                if(dir->d_type==DT_REG)
                {   
                    strncpy(liste[counter],dir->d_name,150);
    //              printf("%s\n", liste[counter]);
                    counter++;
                }
            }
            closedir(d);

        }

    }

    int main ()
    {
        int status;
        pid_t child_pid, pid_w;
        DIR *d;
        struct dirent *dir;
        char *path = ("%s.",FILE_PATH);
        d = opendir(path);
        int file_numbers=0;
        int counter = 0;
        if(d != NULL)
        {
            while((dir=readdir(d))!= NULL)
            {
    //          printf("%s\n", dir->d_name);
                if(dir->d_type==DT_REG)
                    file_numbers++;
            }
            closedir(d);

        }
        char liste[file_numbers][150];
        get_list(liste);
        /*for(int i = 0; i< file_numbers;i++)
            printf("%s\n", liste[i]);*/

        while(1)
        {
            char *ext = getExt(liste[counter]); 
            //printf("eog -f %s/%s\n",PATH,liste[counter]);
            child_pid = fork();
            if(child_pid==0)
            {
                //printf("-eog -f %s/%s\n",PATH,liste[counter]);
                if(!strcmp(ext,".JPG") || !strcmp(ext,".jpg") || !strcmp(ext, ".gif") || !strcmp(ext,".png") || !strcmp(ext, ".PNG") || !strcmp(ext, ".GIF"))
                {   
                    //printf("eog -f %s%s\n",FILE_PATH,liste[counter]);
                    char command[1000];
                    strcpy(command,("eog -f %s%s",FILE_PATH,liste[counter]));
                    //printf("%s", command);
                    execl(command,command,NULL,NULL);
                    sleep(5);
                }           
                else if (!strcmp(ext,".mov") || !strcmp(ext, ".mp4") || !strcmp(ext, ".avi") || !strcmp(ext,".wmv"))
                {
                    //printf("mplayer -fs %s%s\n",FILE_PATH,liste[counter]);
                    char command[1000];
                    strcpy(command,("mplayer -fs %s%s",FILE_PATH,liste[counter]));
                    //printf("%s",command);
                    execl(command,command,NULL,NULL);
//                  system(command);

                }
                else 
                    printf("--%s%s",FILE_PATH,liste[counter]);      
            }
            else
            {
                pid_w = waitpid(child_pid,&status,0);
    //          system("eog -f /home/user/Desktop/Video/screenshot_0000.png");
    //                  sleep(5);
    //                  exit(EXIT_SUCCESS);
            }
            counter++;
            if(counter >=file_numbers)
                counter =0;

        }
    }

Answer 1

我不确定这是唯一的错误，但这显然是您的直接问题:

strcpy(command,("mplayer -fs %s%s",FILE_PATH,liste[counter]));

我不知道你是怎么想到这会以某种方式格式化一个字符串。它所做的是将 liste[counter] 复制到 command。原因是你的 strcpy 的第二个参数是一个带括号的表达式:

("mplayer -fs %s%s",FILE_PATH,liste[counter])

它使用了两次逗号运算符 (,)。逗号运算符执行以下操作:

1.) 它评估逗号的两边 sequenced (首先是左侧，然后是右侧) 2.) 其评估结果是右侧的结果，其他结果被丢弃(*)

所以，这里

"mplayer -fs %s%s",FILE_PATH,liste[counter]

完全等同于

liste[counter]

---

你要找的是snprintf():

snprintf(command, sizeof(command), "mplayer -fs %s%s", FILE_PATH, liste[counter]);

这显然会达到您的预期。另外，还要注意在每个逗号后放置一个空格如何使整行很多更具可读性，因此我建议您也这样做。

---

^{(*) 但是副作用的顺序是明确定义的，这就是它的用途——当然，在这里你没有任何副作用}