Logischer Fehler in meinem C-Code

Question

Ich habe die folgende Methode, die in zwei Zeichenfolgen liest, um den Vor- und Nachnamen von zwei Einträgen in einer Datenbank zu ändern. Der Name wird immer in Kleinbuchstaben konvertiert und sollte nur Namen mit Buchstaben oder Bindestrichen enthalten, es sei denn, der Bindestrich ist das erste oder letzte Zeichen in der Zeichenfolge. Bei einem Fehler wird der Benutzer nur erneut aufgefordert.

/* 
Prompts for a student name, and checks if input is correct. 
Returns 0 on success, -1 onEOF. 
If the user inputs an invalid entry, the error is ignored and the user is simply asked again. 
*/ 
int get_name(char *first, char *last) 
{ 
char input[LINESIZE]; 
char buffer[LINESIZE]; 
char buffer2[LINESIZE]; 
int i; 

while (1) 
{ 
    printf("\n%s", "Enter the Student Name "); 
    if (!fgets(input, LINESIZE, stdin)) 
    { 
     clearerr(stdin); 
     return -1; 
    } 
    if (sscanf(input, "%s %s", buffer, buffer2) == 2) 
    { 
     if (strlen(buffer) < 20 && strlen(buffer2) < 20 && strlen(buffer) >= 2 && strlen(buffer2) >= 2) /*checks to see if the length is at least 2, but less than 20*/ 
     { 
      for (i = 0; i < 20; i++) /*checks every character on the string, converts it to lower and then checks if it's a letter or a hyphen, restarts if not.*/ 
      { 
       if (isalpha(buffer[i])) 
       { 
        buffer[i] = tolower(buffer[i]); 
       } 
       if (isalpha(buffer2[i])) 
       { 
        buffer2[i] = tolower(buffer2[i]); 
       } 
       if (!isalpha(buffer[i]) && buffer[i] != '-') 
       { 
        break; 
       } 
       if (!isalpha(buffer2[i]) && buffer2[i] != '-') 
       { 
        break; 
       } 
      } 
      if (buffer[0] == '-' || buffer[(strlen(buffer) - 1)] == '-') 
      { 
       i = 0; 
      } 
      if (buffer2[0] == '-' || buffer2[(strlen(buffer2) - 1)] == '-') 
      { 
       i = 0; 
      } 
      if (i == 20) 
      { 
       sscanf(buffer, "%s", first); 
       sscanf(buffer2, "%s", last); 
       return 0; 
      } 
     } 
    } 
} 

}

Ich habe den Fehler auf die Linien mit (!isalpha(buffer[i]) && buffer[i] != '-') aufgespürt, aber ich kann nicht herausfinden, warum die Aussage falsch ist.

So kann beispielsweise eine erfolgreiche Eingabe "John Smith" oder "Hom-er Simpson" sein. In beiden Fällen sollte die Funktion 0 (erfolgreich) zurückgeben. Wenn die Eingabe etwas wie: "Denny's White", "-alexa Redfield", "Albert" oder "1234 5647" ist, sollte die Funktion einfach von vorn beginnen (den Benutzer erneut auffordern). Im Moment, unabhängig von der Eingabe, beginnt die Funktion neu.

Answer 1

Die Haupt Logikfehler in Ihrem Programm ist die Tatsache, dass Sie eine for Schleife mit einer festen Anzahl (20) von Wiederholungen auf zwei Arrays zugleich ausgeführt werden. Sie sollten zwei separate Schleifen für die Vor- und Nachnamen-Arrays ausführen. Außerdem sollten Sie diese beiden Schleifen nur für strlen jedes Arrays ausführen. Siehe das folgende Beispiel:

for (i = 0; i < strlen(buffer); i++) 
{ 
    /* your existing stuff for the "buffer" */ 
} 

for (i = 0; i < strlen(buffer1); i++) 
{ 
    /* your existing stuff for the "buffer1" */ 
} 

Answer 2

Caveat: VHS das Hauptproblem bekam. Dies weist jedoch auch auf ein strukturelles Problem in Ihrem Code hin, das verbessert werden kann. Fast der gesamte Code ist für jedes Array doppelt vorhanden.

Das Erstellen einer Funktion zum Arbeiten mit dem Array hätte [wahrscheinlich] das Erkennen des Problems erleichtert.

ich Ihren Code überarbeitet habe diesen hinzufügen [bitte unentgeltlichen Stil Bereinigungs verzeihen]:

#include <stdio.h> 
#include <string.h> 
#include <ctype.h> 

#define LINESIZE 80 

int 
bufdo(char *buf) 
{ 
    int len; 
    int idx; 
    int chr; 
    int ret; 

    len = strlen(buf); 

    do { 
     // assume non-conforming 
     ret = -1; 

     // length too great 
     if (len >= 20) 
      break; 

     // length too small 
     if (len < 2) 
      break; 

     // begins with hyphen 
     if (buf[0] == '-') 
      break; 

     // ends with hyphen 
     if (buf[len - 1] == '-') 
      break; 

     // assume it will be okay 
     ret = len; 

     // convert to lowercase and check for bad chars 
     for (idx = 0; idx < len; ++idx) { 
      chr = buf[idx]; 

      if (isalpha(chr)) { 
       chr = tolower(chr); 
       buf[idx] = chr; 
       continue; 
      } 

      // non-hyphen char in name 
      if (chr != '-') { 
       ret = -1; 
       break; 
      } 
     } 
    } while (0); 

    return ret; 
} 

int 
get_name(char *first, char *last) 
{ 

    char input[LINESIZE]; 
    char buffer[LINESIZE]; 
    char buffer2[LINESIZE]; 
    int len1; 
    int len2; 
    int ret; 

    ret = -1; 

    while (1) { 
     printf("\n%s", "Enter the Student Name "); 
     fflush(stdout); 

     if (!fgets(input, LINESIZE, stdin)) { 
      clearerr(stdin); 
      break; 
     } 

     if (sscanf(input, "%s %s", buffer, buffer2) != 2) 
      continue; 

     len1 = bufdo(buffer); 
     len2 = bufdo(buffer2); 

     if ((len1 > 0) && (len2 > 0)) { 
      strcpy(first,buffer); 
      strcpy(last,buffer2); 
      ret = 0; 
      break; 
     } 
    } 

    return ret; 
}