implementowanie wywołania systemowego cd przy użyciu C- jeśli warunek

Question

Oto kod, który implementuje wywołanie systemowe cd za pomocą C. Problem z tym kodem polega na tym, że nie wchodzi on w stan if stanu if(strcmp(buffer,"cd") == 0) i nie mogę zrozumieć, dlaczego.

#include<sys/stat.h> 
#include <stdio.h> 
#include <unistd.h> 
#include <string.h> 
#include<dirent.h> 
#include<error.h> 

#define BUFFERSIZE 20 
int main(){ 

char *args[80]; 
char buffer[BUFFERSIZE]; 
char *prompt = "OS"; 
char *a = ">"; 
printf("%s%s",prompt,a); 
fgets(buffer, BUFFERSIZE, stdin); 

char *tok; 
tok = strtok (buffer," "); 


while(buffer != NULL){ 
    buffer[strlen(buffer)-1] = '\0'; 
    pid_t pid; 
    pid = fork(); 
    if(pid < 0){ 
     fprintf(stderr, "Fork failed"); 
     return 1; 
    } 
    else if(pid == 0){ 

     if(strcmp(buffer,"cd") == 0){ 
     tok = strtok(NULL,"\n"); 
     cd(tok); 
     } 
     printf("%s%s",prompt,a); 
     fgets(buffer, BUFFERSIZE, stdin); 
    } 
    else{ 
    wait(NULL); 
    } 
} 
return 0; 
} 


int cd(char *pth){ 
    char path[1000]; 
    strcpy(path,pth); 

    static char *prompt = "OS"; 
    static char *a = ">"; 
    char *token; 

    char cwd[256]; 
    getcwd(cwd,sizeof(cwd)); 

    strcat(cwd,"/"); 
    strcat(cwd,path); 
    chdir(cwd);  

    printf("%s-%s%s",prompt,path,a); 
    return 0; 
    } 

Answer 1

Zaktualizowałem logikę po sugestiach innych osób.

Istnieje nie ma potrzeby, aby proces potomny tutaj. Jeśli chcesz wielozadaniowości, użyj wątków. Child process may be required for process running in background.

Poniższy program pracuje dla mnie:

#include <stdio.h> 

#include <sys/stat.h> 
#include <stdio.h> 
#include <unistd.h> 
#include <string.h> 
#include <dirent.h> 
//#include <error.h> 

int hasPrefix(char const *, char const *); 
int cd(char *pth); 

#define BUFFERSIZE 200 
int main(){ 

    char buffer[BUFFERSIZE]; 
    char *prompt = "OS"; 
    char *a = ">"; 

    char *tok; 
    tok = strtok (buffer," "); 


    while(buffer != NULL){ 
     bzero(buffer, BUFFERSIZE); 
     printf("%s%s",prompt,a); 
     fgets(buffer, BUFFERSIZE, stdin); 
     if(hasPrefix(buffer,"cd") == 0){ 
      tok = strchr(buffer,' '); //use something more powerful 
      if(tok) { 
       char *tempTok = tok + 1; 
       tok = tempTok; 
       char *locationOfNewLine = strchr(tok, '\n'); 
       if(locationOfNewLine) { 
        *locationOfNewLine = '\0'; 
       } 
       cd(tok); 
      } 
     }else{ 
      system("ls"); //for testing the CWD/PWD 
     } 
    } 
    return 0; 
} 

int hasPrefix(char const *p, char const *q) 
{ 
    int i = 0; 
    for(i = 0;q[i];i++) 
    { 
     if(p[i] != q[i]) 
      return -1; 
    } 
    return 0; 
} 

int cd(char *pth){ 
    char path[BUFFERSIZE]; 
    strcpy(path,pth); 

    char cwd[BUFFERSIZE]; 
    if(pth[0] != '/') 
    {// true for the dir in cwd 
     getcwd(cwd,sizeof(cwd)); 
     strcat(cwd,"/"); 
     strcat(cwd,path); 
     chdir(cwd); 
    }else{//true for dir w.r.t./
     chdir(pth); 
    } 

    return 0; 
} 

Answer 2

Myślę, że problem jest z powodu tej linii:

buffer[strlen(buffer)-1] = '\0'; 

ten zastępuje ostatni znak buffer z pustym znakiem. Więc jeśli buffer zawierał "cd", zawiera teraz tylko "c" (ponieważ znak null jest terminatorem ciągu w C).

Wydaje się, że to stwierdzenie nie jest potrzebne, po prostu je usuń.

Answer 3

Zastosowanie

... 
if(strncmp(buffer,"cd",2) == 0){ 
... 

zamiast. Dobrze jest porównywać przedrostki o dowolnej długości. Ustawia także limit rozmiaru łańcucha. Nie ma potrzeby tworzenia własnej procedury porównywania.

Masz inne problemy w innym miejscu kodu, ale można je rozwiązać osobno.