Ç - 分段故障

Question

这里是我的一点ç片断生成段故障：

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

int checkTiime(char time[]); 

int main(int argc, char** argv) { 

    char time1[6]; 
    char time2[6]; 

    printf("Tempo 1 [hh:mm]: "); 
    fgets(time1, sizeof time1, stdin); 
    printf("Tempo 2 [hh:mm]: "); 
    fgets(time2, sizeof time2, stdin); 

    printf("Checktime: %d", checkTime(time1)); 

    return (EXIT_SUCCESS); 
} 

int checkTime(char time[]){ 

    long hours = 0; 
    long minutes = 0; 

    if(time[2] == ':') { 

     if(isdigit(time[3]) && isdigit(time[4]) && isdigit(time[0]) && isdigit(time[1])) { 
      hours = strtol(time[0], NULL, 10) + strtol(time[1], NULL, 10); 
      minutes = strtol(time[3], NULL, 10) + strtol(time[4], NULL, 10); 

      if((hours >= 0 && hours <= 23) && (minutes >= 0 && minutes <= 59)){ 

       return 0; 

      } else { 

       return 1; 

      } 

     } else { 

      return 1; 

     } 

    } else { 

     return 1; 

    } 

} 

有人可以帮助我。我真的不知道为什么给我的问题。我也注意到，当我输入时，例如，“12:34”它要求我输入第二个输入，但是当我编写“12:34”时，然后使用退格删除“34”并输入“34 “它再次写入第二个printf，但不允许我输入第二个输入并退出程序。

个人点评：

我注意到，这是更好地使用gets()功能，输入字符串东阳它不计算\n字符。

Answer 1

使用fgets时要小心。您的来电要求它在输入换行符或EOF前最多读入6个字符。 fgets认为换行符是一个有效的字符并存储它，如果遇到这种情况。现在，在你的情况下，最后一个字符不是NULL，因为它应该是C风格的字符串，但是是\n。您需要将其替换为NULL，然后才能将其传递到strtol，该文件需要C样式NULL终止的字符串。所以：

if (fgets(time1, sizeof time1, stdin) != NULL) { 
    time1[ sizeof time1 - 1 ] = NULL; 
} 

，或者甚至更好：

if (fgets(time1, sizeof time1 - 1, stdin) != NULL) { 
    // hurray we have the time! 
} 

或者，你可以有自己的换行符安全fgets包装：

char *chomp(char *buf, size_t sz, FILE *fp, size_t *outsz) { 
    char *p = fgets(buf, sz - 1, fp) ; /* make sure there is room for terminating NULL */ 
    if (p != NULL) { 
     char *nl = strchr(buf, '\n'); 
     if (nl) *nl = NULL; /* replace trailing newline */ 
     *outsz = strlen(buf); /* recompute */ 
    } 
    return p; 
}