내 C 코드의 논리 오류

Question

데이터베이스의 두 항목의 성과 이름을 변경하기 위해 두 문자열을 읽는 다음과 같은 방법이 있습니다. 이름은 항상 소문자로 변환되며 하이픈이 문자열의 첫 번째 또는 마지막 문자가 아닌 경우 문자 또는 하이픈이있는 이름 만 가져야합니다. 오류가 발생하면 사용자에게 다시 묻습니다.

/* 
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; 
      } 
     } 
    } 
} 

}

나는 (!isalpha(buffer[i]) && buffer[i] != '-')와 라인에 오류를 추적했지만 문이 잘못된 이유를 알아낼 수 없습니다.

예를 들어, 성공적인 입력은 "john smith"또는 "hom-er simpson"이 될 수 있습니다. 이 두 경우 모두 함수는 0 (성공)을 반환해야합니다. 입력 내용이 "Denny 's White", "-alexa redfield", "albert"또는 "1234 5647"인 경우 함수를 다시 시작해야합니다 (사용자에게 다시 묻습니다). 현재 입력에 관계없이 함수가 다시 시작됩니다.

Answer 1

프로그램의 주요 논리 오류가 동시에에서 두 배열 에 고정 된 횟수의 반복 (20)와 for 루프를 실행하고 있다는 사실이다. 성 및 이름 배열에 대해 두 개의 별도 루프를 실행해야합니다. 또한 각 배열의 strlen에 대해서만이 두 루프를 실행해야합니다. 다음 예제를 참조하십시오

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

경고 : VHS의 주요 문제가있어. 그러나 이것은 또한 향상시킬 수있는 코드의 구조적 문제를 지적합니다. 거의 모든 코드가 각 배열에 대해 복제됩니다.

어레이에서 작동하는 기능을 만들면 문제를 더 쉽게 볼 수 있습니다. 나는이 [용서하십시오 무상 스타일 정리]를 추가하는 코드를 재 작업 한

:

#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; 
}