在我的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] != '-')，但我不明白，为什么说法是错误的。

因此，举例来说，一个成功的输入可以是“约翰·史密斯”，或“坎洱辛普森”。在这两种情况下，函数应该返回0（成功）。 如果输入的是一样的东西：“丹尼白”，“-Alexa雷德菲尔德”，“伟业”或“1234 5647”的功能应该只是重新开始（再次提示用户）。 此刻，无论输入什么，功能都会重新开始。

Answer 1

在程序中的主要逻辑错误是，你是在同一时间两个阵列运行for循环与迭代的固定数目（20）的事实。您应该为第一个和最后一个名称数组运行两个单独的循环。另外，您应该只运行这两个循环，每个阵列只有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; 
}