DHT22 Vorzeichen

[roger (Gast)]

Ich arbeite mit dem DHT22/AM2302 , Die Temperaturwerte stimmen soweit 
auch unter null Grad.

Mein Problem liegt darin das Vorzeichen mit auszugeben.

Bei zb. -12.8Grad erhalte ich als Ausgabe +112.8Grad.

int sensor_byte = 0;
int kl = 0;
char buffer[15];

 kl = (sensor_data[2] << 8) + sensor_data[3];
   // negative
  if ( kl & 0x8000)
  {
  
   *temp = (kl & 0x7FFF)*-1;
   
  }
 
    else
  {
     *temp = (sensor_data[2] << 8) + sensor_data[3];
  }

 sprintf(buffer, "%+d.%d ", temp /10, temp %10));

[dhhdj (Gast)]

wie ist denn temp definiert?

[roger (Gast)]

dhhdj schrieb:
> wie ist denn temp definiert?

als int temp;

[asdfasd (Gast)]

Zuweisung an "*temp" und lesen aus "temp"?!? Da müsste der Compiler aber 
meckern. Auch sonst einiges ... merkwürdig ;-)

int temp = (sensor_data[2] << 8) + sensor_data[3]; // one's-complement
int abstemp = temp & 0x7fff;
if (temp & 0x8000)
    temp = -abstemp;
sprintf(buffer, "%+d.d", temp / 10, abstemp % 10);

Beachte das "abstemp % 10", modulo von negativen Zahlen ...

[PittyJ (Gast)]

Einmal wird temp als Pointer benutzt
 *temp =
einmal als normaler int
 temp /10

Du solltest dich für eine Variante entscheiden und dann durchgängig 
benutzen.

[roger (Gast)]

//Aufruf in der Main
char buffer[15];
am2302(&humidity, &temp);


//Ausgabe
 sprintf(buffer, "%+d.%d ", temp /10, temp %10));





am2302( int*temp)
{
int sensor_byte = 0;
int kl = 0;


 kl = (sensor_data[2] << 8) + sensor_data[3];
   // negative
  if ( kl & 0x8000)
  {
  
   *temp = (kl & 0x7FFF)*-1;
   
  }
 
    else
  {
     *temp = (sensor_data[2] << 8) + sensor_data[3];
  }
}

[Jim M. (turboj)]

roger schrieb:
> // negative
>   if ( kl & 0x8000)
>   {
>
>    *temp = (kl & 0x7FFF)*-1;
>
>   }

Da ist IIRC 1-er Komplement. Heutzutage verwendet man aber 2er 
Komplement, da funktioniert das so nicht.

Beispiel, original Bytes 0xFF 0xFF:

0xFFFF = -1
(0xFFFF &0x7FFF) = 32767 = 0x7FFF

(0xFFFF &0x7FFF) *(-1) = -32767 = 0xFFFF8001

Besser wäre:

if ( kl & 0x8000) {
  *temp = 0xFFFF0000 | kl;
}

*temp ist hier ein 32-bit int.

[Joe F. (easylife)]

int temp;

temp = ((sensor_data[2] & 0x7F) << 8) | sensor_data[3];
if (sensor_data[2] & 0x80) { // highest bit indicating negative temperature
 temp = -temp;
}

[roger (Gast)]

Joe F. schrieb:
> int temp;
>
> temp = ((sensor_data[2] & 0x7F) << 8) | sensor_data[3];
> if (sensor_data[2] & 0x80) { // highest bit indicating negative
> temperature
>  temp = -temp;
> }

So erhalte ich als Ausgabe +65425 bei -9.8 Grad

sprintf(buffer, "%+d.%d ", temp /10, temp %10));

[Wolfgang (Gast)]

roger schrieb:
> So erhalte ich als Ausgabe +65425

Dann guck doch mal, wieviel Bit dein int hat. Die Bit-Puhlerei wird nur 
mit einem int16_t funktionieren, aber dann könnte in temp nicht 65425 
drin stehen

[roger (Gast)]

Wolfgang schrieb:
> roger schrieb:
> So erhalte ich als Ausgabe +65425
>
> Dann guck doch mal, wieviel Bit dein int hat. Die Bit-Puhlerei wird nur
> mit einem int16_t funktionieren, aber dann könnte in temp nicht 65425
> drin stehen

Deklariert als 16Bit int16_t

[Joe F. (easylife)]

roger schrieb:
> So erhalte ich als Ausgabe +65425 bei -9.8 Grad
>
> sprintf(buffer, "%+d.%d ", temp /10, temp %10));

Das kann ja nicht sein. Da fehlt ja dann mindestens der "." bei der 
Ausgabe.

Prüfe mal, was in sensor_data[2] und sensor_data[3] steht, bzw. gibt 
diese Werte auch mal mit aus.

int16_t temp;
uint8_t sensor_data[4];
 
sensor_data[2] = 0x80;  // negativ
sensor_data[3] = 98;    // 9.8°

temp = ((sensor_data[2] & 0x7F) << 8) | sensor_data[3];
if (sensor_data[2] & 0x80) { // highest bit indicating negative temperature
  temp = -temp;
}

printf("%+d.%d ", temp/10, abs(temp)%10);

gibt bei mir "-9.8 " aus.



Kleiner Schönheitsfehler noch:

sprintf(buffer, "%+d.%d ", temp /10, temp %10));

sollte besser

sprintf(buffer, "%+d.%d ", temp/10, abs(temp)%10));

lauten, sonst hast du negative Vorzeichen auch vor den Nachkommastellen.