我的字符比较条件有什么问题？

Question

我在这里有一个简单的子程序，它接收一个 char 数组并返回一个布尔值。

我的条件有问题。我通过传递单个字符串“A”来测试它，因此 check_char = 97 并使用短路条件应该为 check_char 提供 false ！='a'。子程序应该返回 1。但它总是返回 0。这是为什么呢？这里有什么我没有注意到的问题？

/*
 * Checks string of characters and returns 1 if string is a valid DNA sequence and
 * returns 0 if string is not a valid DNA sequence.
 */
int is_valid_dna(char *sequence){
        int i;
        for( i = 0; i < ( sizeof(sequence) / sizeof(sequence[0]) ); i++ ){
                int check_char = tolower(sequence[i]);
                if( check_char != 'a' || check_char != 'c'   
                        || check_char != 'g' || check_char != 't' ){
                        return 0;
                }
        }
        /* If the subroutine made it this far, then the sequence is a valid DNA sequence */
        return 1;
}

编辑：谢谢大家。传入数组的大小并在条件中使用 AND 效果很好。

Answer 1

使用短路条件应该为 check_char 提供 false ！= 'a'

是的，但是由于您使用的是||，因此如果答案为真，短路将中止其余的条件：我们只需要其中一个子句为真即可完成整个事情是的，所以我们可以阻止一个我们发现一个是真的。但它是错误的，所以你继续 check_char != 'c' 这是正确的，所以我们将其余条件短路并返回 0。

您的意思是在您的条件之间使用&&，而不是||？

Answer 2

也许这是你想要的：

int is_valid_dna(char *sequence, int numChars){
    int i;
    for (i = 0; i < numChars; i++){
        int check_char = tolower(sequence[i]);
        if (check_char != 'a' && check_char != 'c'
            && check_char != 'g' && check_char != 't'){
            return 0;
        }
    }
    /* If the subroutine made it this far, then the sequence is a valid DNA sequence */
    return 1;
}

Answer 3

我会这样编码：

int is_valid_dna(char *sequence)
{
    for ( ; ; sequence++)
    {
        switch (*sequence)
        {
            case 'a':
            case 'A':
            case 'c':
            case 'C':
            case 'g':
            case 'G':
            case 't':
            case 'T':
            continue;

            case '\0':
            return 1;

            default:
            return 0;
        }
    }
    return 1;
}