CS50 pset6 Server Parse-Funktion - CS50-Server 1 check

Question

Können Sie nicht einige Hinweise darüber, was genau das geben, übergeben sind die Fehler in CS50 Check server1 und einige Hinweise darauf hingewiesen, wie sie lösen:

meine Parse-Funktion wie folgt aussieht :

bool parse(const char* line, char* abs_path, char* query) 

{ 
    // create copy of line 
    char linecopy[strlen(line)+1]; 
    strcpy(linecopy, line); 
    linecopy[strlen(line)] = '\0'; 



int spaces = 0; 
    for (int i = 0, lenght = strlen(line); i < lenght; i++) 
    { 
    if (line[i] == ' ') 
    spaces++; 

    if (spaces > 2) 
    { 
     error(400); 
     return false; 
    } 
    } 
    char* methodcopy; 
    char* targetcopy; 
    char* httpcopy; 

    // extract method, request-target, and http version to tokens 
    methodcopy = strtok(linecopy, " "); 
    targetcopy = strtok(NULL, " "); 
    httpcopy = strtok(NULL, "\r\n"); 

    //copy tokens into method, target and http 

    char method[strlen(methodcopy) + 1]; 
    strcpy (method, methodcopy); 
    char target[strlen(targetcopy) + 1]; 
    strcpy(target, targetcopy); 
    char http[strlen(httpcopy) + 1]; 
    strcpy(http, httpcopy); 

    // add null terminators 

    method[strlen(method)] = '\0'; 
    target[strlen(target)] = '\0'; 
    http[strlen(http)] = '\0'; 

    // ensure method is GET 

    if(strncmp(method, "GET", 4)!= 0) 
    { 
     error(405); 
    } 
    // ensure request-target begins with "/" 
    if(strchr(target, '/') == NULL) 
    { 
     error(501); 

    } 
    // ensure request-target does not contain '"' 
    if (strchr(target, '"') != NULL) 
     { 
      error(400); 
     } 
    // ensure HTTP version is 1.1 
    if(strcmp(http, "HTTP/1.1") != 0) 
    { 
     error(505); 

    } 

    // extract query from request-target 
    char* abs_pathcopy; 
    char* querycopy; 
    abs_pathcopy = strtok(target, "?"); 
    querycopy = strtok(NULL, "\0"); 

    //if query isn't null, copy to query 
    if (querycopy != NULL) 
    { 
     strcpy(abs_path, abs_pathcopy); 
     strcpy(query, querycopy); 

     abs_path[strlen(abs_pathcopy)] = '\0'; 
     query[strlen(querycopy)] = '\0'; 
    } 
    //if query is null, set query to null 
     else 
     { 
      strcpy(abs_path, abs_pathcopy); 
      abs_path[strlen(abs_pathcopy)] = '\0'; 
      query[0] = '\0'; 
     } 
    // ensure absolute path does not contain "?" 
    if(strchr(abs_path, '?') != NULL) 
    { 
     error(400); 

    } 
    // ensure query does not contain '"' 

    if(strchr(abs_path, '"') != NULL) 
    { 
     error(400); 

    } 
    return true; 
} 

Wenn ich das CS50-Check-Server 1, habe ich die folgenden Fehler angezeigt:

:(Anfrage Ziel von abc/hello.php Fehlercode 501 \ erwartete Ausgabe zurückgibt, aber nicht "HTTP/1.1 404 nicht gefunden \ r \ nInhaltstyp: ...“

:(cat.exe Anforderung gibt Fehlercode 501 \ erwartete Ausgabe, kein Exit-Code von 0

:(Zwei Leerzeichen nach GET kehrt Fehlercode \ erwartete Ausgabe, kein Ausgang Code von 0

nach meinem Verständnis ist die Anforderungsleitung nicht gemäß der Definition:

Methode SP Anfrage-Ziel SP HTTP-Version CRLF

aber nicht sicher, wie es zu definieren ... .

Alle Tipps und Hinweise sind willkommen! Ich will keine Lösung, aber nur ein paar Hinweise;)

Danke an alle, die sich die Zeit genommen haben, diese lange Frage zu lesen!

Answer 1

Ihr Algorithmus nimmt zuerst an, dass die Eingabe, Linie, eine wohlgeformte Anfrage ist, als sie in Stücke zu teilen und dann die geteilten Teile zu prüfen. Wie findest du jetzt delimeter "" (SP) genau an richtigen Stellen und in korrekter Anzahl? Was ist, wenn ich niemals einen SP verwende oder doppelten SP überall verwende?

Kurz gesagt, ich denke nicht, dass Sie strtok() mit einer Zeichenfolge verwenden sollten, die Sie nicht über die Richtigkeit ihrer Form wissen. Analysieren Sie die Eingabe char durch char oder verwenden Sie Funktionen wie strchr(), um das Auftreten von Delimetern selbst zu erkennen.

Bearbeiten auf den Einsatz von strchr():

Das Dokument strchr() ich für strchr verknüpft() sagt: "Gibt einen Zeiger auf das erste Vorkommen von Zeichen in der C-String str.". Also, wenn eine Zeichenfolge wie "eins * zwei * drei * vier \ 0" hat, wobei '*' das Delimeter ist;

char* firstStar =strchr(myString, '*') 

einen Zeiger zurück, die auf den ersten Stern weist,

char* secondStar =strchr(firstStar + 1, '*') 

den zweiten Stern zurück, weil Sie von der First seaching sagte starten. Diese Funktion gibt null zurück, wenn das von Ihnen angegebene Zeichen nicht gefunden werden kann. (Siehe Beispiel bei bestimmtem Link)

Jetzt kann ich mithilfe von strncpy() Funktion zweite Parameter um „zwei“ abrufen, weil ich weiß, wo es beginnt (bei First +1) und wie lange ist es (second - First) Beachten Sie, dass Nachdem Sie strncpy() verwendet haben, sollten Sie den Null-Terminator '\ 0' manuell am Ende der kopierten Zeichenfolge setzen.