isalpha()不能正常工作

Question

我尝试使用for循环逐个字符地检查两个字符串，看看它们在哪里相等，在哪里不同，如果是，那么我做了一个条件，但如果一个是字母，一个是下划线，那么我就做其他的事情。循环的第一部分工作正常；它可以判断两个值何时相等。但是，当我使用isalpha来查看它们是否都是不同的字母时，它不会执行此操作。

#include <stdio.h>
#include <string.h>
#include <ctype.h>
char s1[] = "v_intner_";
char s2[] = "wri_t_ers";


char* getTranscript(char* s1, char* s2){
    int i;
    char sOneTemp[9];
    char sTwoTemp[9];
    char eTranscript[9];
    strcpy(sOneTemp, s1);
    strcpy(sTwoTemp, s2);

    for(i=0; i<9; i++){
        if(sOneTemp[i]==sTwoTemp[i]){
            printf("Those two characters are equal.\n");
        }
        if(sOneTemp[i]!=sTwoTemp[i]){


            if(isalpha(sOneTemp[i]) && isalpha(sTwoTemp[i])){
                printf("Both are letters\n");
            }
            else{
                printf("One is a dash\n");
            }
        }
    }
printf("The value of s1 is: %s\n", sOneTemp);

return s1;}
main()
{
printf("The main method returns: %s", getTranscript(s1,s2));
return 0;
}

Answer 1

您的isalpha逻辑是正确的。但是，您有一个缓冲区溢出：

char sOneTemp[9];
char sTwoTemp[9];

但是您可以使用strcpy将10个字符复制到中(请记住，C字符串有一个空终止符)。如果你修复了这个错误，那么代码就会给出预期的输出。

一个很好的修复方法是在函数中使用s1和s2，而不是冗余地复制输入字符串。

Answer 2

您的程序会受到未定义行为的影响。要使用strcpy字符串"v_intner_"，目标系统至少需要10个字符。您创建的数组只有9个字符。

更改行：

char sOneTemp[9];
char sTwoTemp[9];

至

char sOneTemp[N]; // Where N is 10 or greater.
char sTwoTemp[N];

Answer 3

char sOneTemp[9];
char sTwoTemp[9];
char eTranscript[9];
strcpy(sOneTemp, s1);
strcpy(sTwoTemp, s2);

您的数组大小不足以容纳包括nul字符在内的10个字符。因此，增加数组大小以包含nul字符