Как исправить ошибку при вычислениях?

Question

[Владимир Владимирович]

Схема:
"Чётную" температуру выводит как нужно. Но вот, к примеру, 9 грудусов - нет. Порядок работы при последовательном увеличении на градус - 8,8,10,10,12 и так далее. В чём может быть проблема? Загвоздка в том, что мне нужно работать с 2-х байтным числом.
Сам код:
проект isis+atmel studio

#include "main.h"

int speed=1;
//----------------------------------------
void port_ini(void)
{
	DDRD=0xFF;
	PORTD=0x00;

	DDRC=0xFF;
	PORTC=0x00;

	DDRB    = (1 << PB1)|(1 << PB2)|(1 << PB0);
	
	//----------
	DDRA=0xFF;
	PORTA=0xFF;
	
}

void spi_init(void){
	volatile char IOReg;
	SPCR = ((1<<SPE)|(1<<MSTR));//Включим шину, объявим ведущим
	  IOReg   = SPSR;               // очистить бит SPIF в регистре SPSR
	  IOReg   = SPDR;
}
void spi_read_array(uint8_t num, uint8_t *data)
{
	PORTA &= ~(1<<PA7);
	while(num--){
		SPDR = *data;
		while(!(SPSR & (1<<SPIF)));
		*data++ = SPDR;
	}
	PORTA |= (1<<PA7);

}
//----------------------------------------
int main(void)
{
	
	unsigned char c;
	unsigned char tempertur;
	c=0b10000000;
	int adc_value;
	float n;
	port_ini(); //Инициализируем порты
	LCD_ini();  //Инициализируем дисплей
	ADC_Init(); //Инициализируем АЦП
	clearlcd(); //Очистим дисплей
	//PORTA=c;
	spi_init();
	setpos(0,0);
	
	 signed short int buf[2] = {0,0};
	
    while(1)
    {	
		spi_read_array(2, buf);
	    setpos(0,0);
		
		buf[0]<<=8;
		buf[0]+=buf[1];
		buf[0]>>=3;
		n= (float)buf[0];
		
		n*=0.0625;
		
		
		
		sendcharlcd(((unsigned char)n%1000)/100+0x30);//Преобразуем число в код числа
		sendcharlcd(((unsigned char)n%100)/10+0x30);//Преобразуем число в код числа
		sendcharlcd(((unsigned char)n%10)+0x30);//Преобразуем число в код числа
		
		
		//speed=buf[0];
		if(buf[0]>=10){
			setpos(1,5);
			str_lcd("vorvard");
			normal();
			setpos(1,5);
			str_lcd("reverse");
			reverse();
		}
		
		buf[0]=0;
		buf[1]=0;
		_delay_ms(50);
    }
}

Answer 1

[Ocelot]

buf объявлен как signed, а затем к нему применяется битовый сдвиг, что, вообще говоря, неопределённое поведение.

Comments

[Владимир Владимирович]

даже незнаю какой тип применить здесь... Если применять тут unsigned char, то на нечётных температурах выводит "1".

[Ocelot]

Буфер под сырые данные - uint8_t[]. А смысл дальнейших преобразований от меня ускользает. Что за датчик используется и в каком формате данные он отдаёт?

[Владимир Владимирович]

Ocelot:TC77. Прикрепил проект. Передаёт он 2 байта данных (сама температура)

[Ocelot]

Попробуйте так:
int16_t tbin = 0;
tbin = (buf[1] << 8) | buf[0];
tbin /= 8; // Important to divide, not shift, since it then preserves the sign
n = (float)tbin * 0.0625;

[Владимир Владимирович]

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

Answer 2

[15432]

У вас же явно написано, если N нечетно, добавить единицу. Вот и получается 8, 8, 10, 10.
Ко всем нечетным единица приплюсовывается.

if((unsigned char)n%2!=0){
      n+=1; 
}

Comments

[Владимир Владимирович]

на самом деле если убрать этот кусок кода, ничего не меняется.

[15432]

какие-то флоаты, буферы (хотя используется только младшая его часть)... муть.

вот так что покажет?

unsinged short temp_data; //под чтение температуры с SPI
unsigned char cur_digit; //под текущую цифру для вывода на дисплей
int i; //под счётчик
int num_of_digits = 3; //три цифры хватит, если надо - увеличить
while(1)
{	
    spi_read_array(2, &temp_data); //unsigned short как раз два байта. это для little endian
    //temp_data = (temp_data >> 8) | (temp_data << 8); //раскомментировать это для big endian
    //temp_data >>= 1; //если слишком большое значение, сдвинуть на 1 или больше бит
    for (i = 0; i < num_of_digits; ++i) 
    {
        cur_digit = (temp_data % 10) + 0x30; //перевести младшую цифру в ASCII
        //set_pos(0, i); //для обратного порядка цифр
        set_pos(0, num_of_digits - 1 - i); //для порядка цифр как у вас сейчас
        sendcharlcd(cur_digit); //вывести цифру
        temp_data /= 10; //следующая цифра
    }
    //ваш код дальше
}

[Владимир Владимирович]

15432: от датчика данные приходят 2 порциями, вот и массив. Пока что мешанина цифр. Могу приложить проект.

[Владимир Владимирович]

15432: проект isis+atmel studio

[15432]

Владимир Владимирович: формат приходящих данных где посмотреть?

[Владимир Владимирович]

15432: даташит внизу во вкладке документы
А вообще - 13 бит., 3 младших - мусор. Результат ещё нужно на 0,0625 домножить чтобы градусы получить.

[15432]

Владимир Владимирович: ок, раскомментировать битовый сдвиг:
temp_data >>= 7; (3 бита мусор + 4 бита деление на 16 (или что то же самое, умножение на 0.0625).
Подозреваю, что умножение было лишним, так что попробуйте меньшие значения сдвига (3)
Дебаггер есть? Чтоб точное значение до сдвига посмотреть

[Владимир Владимирович]

15432: да, там SPI дебаггер прикреплён