[C][чайник][не понимаю]Преобразование типов

может crc должен иметь большую разрядную сетку, не? unsigned int например

unsigned char temp_buffer[2];
temp_buffer[0] = in_buffer[2 * i]; 
temp_buffer[1] = in_buffer[2 * i + 1]; 
scan_result = sscanf(temp_buffer, "%2X", &temp_crc); 

Не знаю в чем у тебя там проблема. Но если функция принимает строку, заканчивающуюся символом '\0', то надо ей передавать именно такую строку. БЕЗ ИСКЛЮЧЕНИЙ.

А вообще выкинь sscanf и сделай перекодирование вручную.

- unsigned char temp_buffer[2];
+ unsigned char temp_buffer[3];
+ temp_buffer[2] = '\0';

Нет, там вроде бы должно быть именно так, потому что контрольная сумма представляет собой один байт, а все переполнения игнорируются.

- unsigned char temp_buffer[2];

+ unsigned char temp_buffer[3];

+ temp_buffer[2] = '\0';

Пробовал и так. Та же фигня.

unsigned char temp_crc = 0; scan_result = sscanf(temp_buffer, "%2X", &temp_crc);

scanf-ом никогда не пользовался. Напомните, какой размер temp_crc ожидает scanf 8 бит или 32?

Заглянул в man

x соответствует беззнаковому шестнадцатеричному числу; следующий указатель должен являться указателем на unsigned int.

Но если функция принимает строку, заканчивающуюся символом '\0', то надо ей передавать именно такую строку. БЕЗ ИСКЛЮЧЕНИЙ.

Пробовал и ноль дописывать — ничего.

А вообще выкинь sscanf и сделай перекодирование вручную.

Хотелось бы «малой кровью» решить задачу, без написания своих велосипедов.

> внутри цикла переменная crc почему-то обнуляется! Что я делаю не так?

Ничего удивительного. unsigned char - это числа от 0 до 255, т.е., unsigned char crc = 256 == 0.

[quote]

x соответствует беззнаковому шестнадцатеричному числу; следующий указатель должен являться указателем на unsigned int.

[/quote]

Хм, этого я не учел.

 

- unsigned char temp_crc = 0; 
+ unsigned int temp_crc = 0;

- crc += temp_crc;
+ crc += (char)temp_crc;

Вот так вроде бы заработало. Спасибо за наводку. А есть ли в таких преобразованиях какие-нибудь подводные камни, о которых я не знаю?

crc += strtol(tmp, NULL, 16);

Спасибо, тоже интересный вариант, на эту функцию я не обратил внимания.

у меня на gcc 4.4.3 работает только если передавать в sscanf строку с '\0' на конце, т.е. с

unsigned char temp_buffer[3];

crc += (char)temp_crc; 

у меня на gcc 4.4.3 работает только если передавать в sscanf строку с '\0' на конце

Да, я тоже ноль дописываю.

и зачем crc += (char)temp_crc;

Потому как контрольная сумма — один байт. Чтобы потом вручную обрезание не делать.

и зачем crc += (char)temp_crc;

Потому как контрольная сумма — один байт. Чтобы потом вручную обрезание не делать.

Хотя сейчас проверил без преобразования типов — тоже работает.

+ man atoi

Спасибо компилятору) Но лишним преобразование не будет. Хотя в С преобразование типов в явном виде писать не всегда обязательно в отличии от С++, но читабельность кода это улучшает на мой взгляд.

> + man atoi

Зачем читать про эту гадость?

Согласен что в данном конкретном случае atoi бесполезна, но почему это гадость?

> atoi() does not detect errors.

Ну вот и накойчёрт такую гадость использовать? Причём всё равно в glibc это обёртка вокруг strtol(str, NULL, 10), так что даже никакого выигрыша в скорости в общем случае не будет. Т.е. это даже не strcat/strcpy, которые ещё имеют смысл, если чётко понимать, что делаешь...

Как все усложнили...

1) Никакие scanf или atoi использовать не надо. 2) Никакие нули в конце «строки» тоже не надо. Это же не строки, а просто буферы. 3) Делать вроде:

int decodeControllerReply(unsigned char *dst, unsigned char *src, int dst_len)
{
.....
    if((src[x] >= 'A') && (src[x] <= 'F'))
    {
        one_hex = src[x] - 'A' + 0xA;
    }
........
    if(even_byte)
    {
        dst[x] = one_hex;
    }
    else
    {
        dst[x] += (one_hex << 4);
    }
.....
}

Буду иметь в виду :)