Если инструкция, кажется, никогда не срабатывает

Question

У меня есть цикл, который проходит через изображение, хранящееся в буфере, занимает 1024 байта за раз и отправляет этот килобайт в виде UDP-пакета. Ниже приведен соответствующий раздел кода.

Почему мой второй оператор if, if (buf.bytesused - curr*1024 > 1024) не работает, когда curr*1024 получает больше buf.bytesused? Я застрял в бесконечном цикле, который не заканчивается, пока memcpy, наконец, не вызовет ошибку seg. Мой buf.bytesused составляет около 900 000, но вместо того, чтобы зацикливать около 900 раз, он просто продолжает двигаться, даже когда buf.bytesused - curr*1024 дает отрицательный результат, который, конечно, меньше 1024.

В случае, если это было непонятно: мало, buf.bytesused - curr * 1024> 1024 выполняется, как я ожидал, и этот второй if-блок запускается. Однако, когда curr становится большим (даже настолько большим, что buf.bytesused - curr * 1024 отрицательный), второй if-блок все еще выполняется, а не третий блок, после else.

buf.bytesused - неподписанный 32-битный int.

#define FRAME_START 0x0 
    #define FRAME_END 0xF 
    #define FRAME_MID 0xA 

    uint_32 buf.bytesused = 921600; 
    char frameInfo = FRAME_START; 
    char frameNum = i; 
    int curr = 0; 

    while (frameInfo != FRAME_END) { 
    if (curr == 0) { 
     frameInfo = FRAME_START; 
     memcpy(&frameInfo, &udpBuf[0], 1); 
     memcpy(&frameNum, &udpBuf[1], 1); 
     memcpy(buffers[buf.index].start + curr*1024, udpBuf + 2, 1024); 
    }else if (buf.bytesused - curr*1024 > 1024) { 
     frameInfo = FRAME_MID; 
     memcpy(&frameInfo, &udpBuf[0], 1); 
     memcpy(&frameNum, &udpBuf[1], 1); 
     memcpy(buffers[buf.index].start + curr*1024, udpBuf + 2, 1024); 
    } else { 
     frameInfo = FRAME_END; 
     memcpy(&frameInfo, &udpBuf[0], 1); 
     memcpy(&frameNum, &udpBuf[1], 1); 
     memcpy(buffers[buf.index].start + curr*1024, udpBuf + 2, (buf.bytesused % 1024)); 
    } 

    if (sendto(s, udpBuf, BUFLEN, 0, &si_other, slen)==-1) 
     diep("sendto()"); 
    curr++; 
    } 

Как в стороне, может ли кто-нибудь сказать мне, что я делаю неправильно с memcpy? Он не копирует значения моих двух символов в буфер.

Answer 1

В случае люди задаются вопросом, реальная проблема здесь:

frameInfo = FRAME_END; 
memcpy(&frameInfo, &udpBuf[0], 1); 

frameInfo переписывается после того, как он установлен в FRAME_END, так while (frameInfo != FRAME_END) никогда не будет оценивать ложь. Тот факт, что это заставило программу войти в бесконечный цикл, заставляет OP ошибочно полагать, что его утверждение if было неправильным. Всегда хорошая практика использовать отладчик вместо того, чтобы предположить, что вы знаете, в чем проблема!

Answer 2

if (buf.bytesused - curr*1024 > 1024) fail when curr*1024 gets larger than buf.bytesused

Это может быть проблемой, когда buf.bytesused является unsigned.

Если это отрицательный и подписанный, то приведенное выше сравнение не должно быть выполнено.

Чтобы решить, вы можете использовать простую математику тоже.

if (buf.bytesused > curr*1024 + 1024) 

или

if (buf.bytesused > (curr + 1)*1024) 

Эти условия не будут вызывать проблемы из-за знак, если curr+1 не является отрицательным.

Я думаю, что это должно решить вашу проблему.