Что означает этот код? Опции

[RW6MKA]

Здравствуйте уважаемые форумчане. Возник вопрос при оптимизации кода. Было

Код

                data[0] = w1_receive_byte();//читаем два байта с температурой
        data[1] = w1_receive_byte();
            //загоняем в двух байтную переменную
        temp = data[1];
        temp = temp<<8;
        temp |= data[0];

Подсказали что лучше использовать такое выражение

Код

        *((char *)&Temp;) = w1_receive_byte();
        *((char *)&Temp; + 1) = w1_receive_byte();

Вот никак не могу понять смысла этого кода. Я так понимаю что * и & это операции над указателями , а () приведение к типу но....
Вообщем если не трудно объясните начинающему подробно смысл сей конструкции.

[Harvester]

Ужас какой-то. Это как раз тот случай, когда достоинства языка превращают в недостатки кода
Вполне достаточно такого кода, все понятно и ничего лишнего:

Код

temp = w1_receive_byte();
temp <<= 8;
temp |= w1_receive_byte();

Хотя, если data[] - локальная переменная и больше нигде не используется, компилятор ее сам выкинет.

[SSerge]

Ужас какой-то. Это как раз тот случай, когда достоинства языка превращают в недостатки кода
Вполне достаточно такого кода, все понятно и ничего лишнего:

Код

temp = w1_receive_byte();
temp <<= 8;
temp |= w1_receive_byte();

Немного позанудствую.
С первой и второй строкой всё нормально, а вот с третьей...
Результат выполнения этого может быть разным в зависимости от типов данных переменной temp и типа, возвращаемого функцией w1_receive_byte(). Даже если он определён как char, это ещё ничего не гарантирует, в С char может быть как знаковым так и беззнаковым, отдано на откуп реализации.
Для конкретного компилятора может и прокатит, как правило сейчас char считается беззнаковым.
Но если заботиться о переносимости, то надёжнее явно привести к беззнаковому типу:

temp |= (unsigned char)w1_receive_byte();

или, ещё лучше и короче c использованием типов из stdint.h :
temp |= (uint8_t)w1_receive_byte();