无法在 C 函数中返回“真”值

Question

如果我尝试使用以下函数 (e2CheckPINoverride) 来检查输入的 5 字节数组 (p) 与存储在闪存中的 5 字节数组（数据），以简单地返回 true 或 false 值。 但似乎无论我尝试什么，它都只会返回“false”。

我在这里调用函数：

if (e2CheckPINoverride(pinEntry) == 1){
  PTDD_PTDD1 = 1; 
}
else{
  PTDD_PTDD1 = 0; 
}

函数如下：

BYTE e2CheckPINoverride(BYTE *p)
{
    BYTE i;
    BYTE data[5];

if(e2Read(E2_ENABLECODE, data, 5)) {
    if(data[0] != p[0]) return FALSE;
    if(data[1] != p[1]) return FALSE;
    if(data[2] != p[2]) return FALSE;
    if(data[3] != p[3]) return FALSE;
    if(data[4] != p[4]) return FALSE;
}
return TRUE;
}

我已经在defines.h文件中赋值了真假：

#ifndef TRUE
    #define TRUE ((UCHAR)1)
#endif

#ifndef FALSE
    #define FALSE ((UCHAR)0)
#endif

在哪里

typedef unsigned char   UCHAR;

当我单步执行代码时，它正确执行了所有检查，传入正确的值，正确比较它，然后在正确的点中断，但无法处理 true 的返回值？

请帮忙？

Answer 1

#define TRUE 1
#define FALSE 0

忘记无符号字符。您可以假设在 c 0 中为假，其他一切都为真

Answer 2

可能不会解决你的问题，但你应该写：

PTDD_PTDD1 = e2CheckPINoverride(pinEntry) ? 1 : 0;

此外，您正在混合 BYTE 和 UCHAR（即使它们可能相同）

Answer 3

尝试通过省略#define 并直接说来缩小范围

return 1;

如果这有效，那么您的#define 有什么问题。

Answer 4

如果您返回TRUE 或FALSE，您还应该检查它们。像这样重写if 子句：

   if (e2CheckPINoverride(pinEntry) == TRUE) { // instead of '== 1'

Answer 5

IMO，您正在制造大量不必要的复杂性。我会写这样的函数：

int e2CheckPINoverride(BYTE *p) {
    BYTE data[5];

    return e2Read(E2_ENABLECODE, data, 5) && 
        data[0] == p[0] &&
        data[1] == p[1] &&
        data[2] == p[2] &&
        data[3] == p[3] &&
        data[4] == p[4];
}

而调用代码就变得简单了：

PTDD_PTDD1 = e2CheckPINoverride(pinEntry);

Answer 6

只是我测试工作的一个小例子。我不知道 e2Read 的内容所以我只是做了一个假人

#ifndef UCHAR
    typedef unsigned char   UCHAR;
#endif
#ifndef BYTE
    typedef  char   BYTE;
#endif
#ifndef TRUE
    #define TRUE ((UCHAR)1)
#endif
#ifndef FALSE
    #define FALSE ((UCHAR)0)
#endif
int e2Read(int myconst, BYTE* data, int num)
{
    int i;
    for(i=0;i<num;i++)
        *(data++) = 0; // You can change thisone to test different results.
    return 1;
}
BYTE e2CheckPINoverride(BYTE *p)
{
#define E2_ENABLECODE 3
    BYTE data[5];
    if(e2Read(E2_ENABLECODE, data, 5)) {
    if(data[0] != p[0]) return FALSE;
    if(data[1] != p[1]) return FALSE;
    if(data[2] != p[2]) return FALSE;
    if(data[3] != p[3]) return FALSE;
    if(data[4] != p[4]) return FALSE;
    }
    return TRUE;
}
int main(void)
{
    BYTE b[5] = {0,0,0,0,0};// You can change thisone to test different results.
    BYTE* pinEntry = b;
    if (e2CheckPINoverride(pinEntry) == 1){
        printf("Returned true\n");
    }
    else{
        printf("Returned false\n");
    }
    return 0;
}

Answer 7

您的数据数组从未初始化，因此它内部包含随机值。

BYTE data[5];

因此，您将数组 p 中的元素与数组数据中的随机值进行比较。它几乎总是返回 FALSE。

结论： 用有意义的数据填充数据数组。