DS18S20 Negative Temprratur

[kurt (Gast)]

Hallo,
Ich Wandele mit der Routine den Gemessenen Wert vom DS18S20 in eine 
Temperratur um.
Bei Plusgraden geht alles einwandfrei,Nur enben nicht bei Minusgraden.
ich habe jetz eine Temperratur von -0.5Grad Angezeigt bekomme ich aber
+0.-5 Grad.
wo mache ich nur den Fehler.
vielleicht kann mir von ihnen einer weiterhelfen.
mfg

// wandelt den Messwert, um.
void DS18S20_convert_temperatur(void)
{  
    
  
    int temp;
    int temp1;
    int16_t Vorkommastelle;
    int16_t Nachkommastelle;  
  
    if( id[1] ) {                // Negativer Temperaturwert
    id[0] ^= 0xFF;               // Einer-Komplement
    id[0]++;                     // inkrementieren
    }
    id[0] >>= 1;                 // rechtes Bit für 0,5 °C rausschieben (durch 2 teilen)
    
  // Temperatur mit 0,1 °C Genauigkeit berechnen
    temp = (int)id[0]*100 - 25 + (int)(id[7] - id[6]) * 100 / id[7];
    temp1= temp;
    if( id[1])                   // Negativer Temperaturwert
     {
      temp *= -1;
   }

    else
     {
           
     }
  
    Vorkommastelle = temp / 100;
    Nachkommastelle = (temp1 % 100);
    Nachkommastelle = (temp % 100);
 

    sprintf(msg,"%+3d,%1d", Vorkommastelle, Nachkommastelle);    
    uart_puts( msg); 
    uart_putc( ' ' );
}

[Karl H. (kbuchegg)]

kurt schrieb:
> Hallo,
> Ich Wandele mit der Routine den Gemessenen Wert vom DS18S20 in eine
> Temperratur um.
> Bei Plusgraden geht alles einwandfrei,Nur enben nicht bei Minusgraden.
> ich habe jetz eine Temperratur von -0.5Grad Angezeigt bekomme ich aber
> +0.-5 Grad.

Daraus folgt erst mal: ganz offensichtlich in der Ausgabe

> wo mache ich nur den Fehler.

Machs so:
Wenn der Wert negativ wird, dann malst du erst mal ein '-' hin und 
machst den Wert positiv. Und den positiven Wert kannst du ja richtig 
ausgeben

Hintergrund.

Der Rest einer Division kann auch negativ sein!

   char Sign;



      if( temp < 0 ) {
        Sign = '-';
        temp = -temp;
      }
      else
        Sign = '+';

>     Vorkommastelle = temp / 100;
>     Nachkommastelle = (temp1 % 100);
>     Nachkommastelle = (temp % 100);
>
      sprintf(msg, "%c%3d,%1d", Sign, Vorkommastelle, Nachkommastelle);


PS: Du bist sicher, dass dein temp Wert grundsätzlich korrekt ist? Ich 
kann deine Berechnung da nicht wirklich nachvollziehen.

[kurt (Gast)]

Es wird doch hier
    if( id[1])                   // Negativer Temperaturwert
     {
      temp *= -1;
     }
Geprüft  ob der Wert Negative ist,Wenn Negative dann wird temp *= -1; 
auch Negativ.

Vor dem Prüfen habe ich mir ja den Temp Wert
    temp = (int)id[0]*100 - 25 + (int)(id[7] - id[6]) * 100 / id[7];
    temp1= temp;
kopiert in temp1 für die Nachkommastellen.

Dann sollte doch auch die Nachkomma stelle richtig angezeigt werden,
aber warum funktioniert das nicht.

ich habs jetzt mal nsch ihren Vorschlag umgesetzt aber die Teperatur 
stimmt jetzt nicht mehr

// wandelt den Messwert, um.
void DS18S20_convert_temperatur(void)
{  
    
char Sign;
    int temp;
    int temp1;
    int16_t Vorkommastelle;
    int16_t Nachkommastelle;  
  
    if( id[1] ) {                // Negativer Temperaturwert
    id[0] ^= 0xFF;               // Einer-Komplement
    id[0]++;                     // inkrementieren
    }
    id[0] >>= 1;                 // rechtes Bit für 0,5 °C rausschieben (durch 2 teilen)
    
  // Temperatur mit 0,1 °C Genauigkeit berechnen
    temp = (int)id[0]*100 - 25 + (int)(id[7] - id[6]) * 100 / id[7];
    temp1= temp;
    // if( id[1])                   // Negativer Temperaturwert
    //  {
    //   temp *= -1;
    //  }

    // else
    //  {
           
    //  }
      if( temp < 0 ) {
        Sign = '-';
        temp = -temp;
      }
      else
        Sign = '+';
  
     Vorkommastelle = temp / 100;
     Nachkommastelle = (temp1 % 100);
     Nachkommastelle = (Nachkommastelle / 10);
 

     // sprintf(msg,"%3d,%1d", Vorkommastelle, Nachkommastelle);    
   sprintf(msg,"%3d,%1d", Sign, Vorkommastelle, Nachkommastelle);
     uart_puts( msg); 
     uart_putc( ' ' );
  
 
}

[Georg G. (df2au)]

Die Lösung steht weiter oben schon: kbuchegg: Auch ein Rest kann negativ 
sein.

kurt schrieb:
> // Temperatur mit 0,1 °C Genauigkeit berechnen
>     temp = (int)id[0]*100 - 25 + (int)(id[7] - id[6]) * 100 / id[7];
>     temp1= temp;
temp1 ist hier negativ.

>       if( temp < 0 ) {
>         Sign = '-';
>         temp = -temp;
>       }
>       else
>         Sign = '+';
>
>      Vorkommastelle = temp / 100;

temp1 ist immer noch negativ.
>      Nachkommastelle = (temp1 % 100);
also ist Nachkommastelle auch negativ.

>      Nachkommastelle = (Nachkommastelle / 10);
>
>
>    sprintf(msg,"%3d,%1d", Sign, Vorkommastelle, Nachkommastelle);
Abgesehen davon, dass ich persönlich für 3 Variable auch dreimal "%" 
genommen hätte, kommt es hier zu genau der von dir gezeigten Ausgabe: 
Das Minuszeichen erscheint vor der Nachkommestelle.

[Karl H. (kbuchegg)]

kurt schrieb:

> ich habs jetzt mal nsch ihren Vorschlag umgesetzt

Nicht wirklich.
Du warst zu schlampig

     Vorkommastelle = temp / 100;
     Nachkommastelle = (temp1 % 100);

beim % temp nehmen und nicht temp1! Es muss schon immer von derselben 
Zahl ausgegangen werden!

   sprintf(msg,"%3d,%1d", Sign, Vorkommastelle, Nachkommastelle);

Zähl die auszugebenden Argumente: es sind 3  (Sign, Vorkomma, Nachkomma)
Zähl die Anzahl der % im Format String: es sind 2

Das passt nicht zusammen.

[kurt (Gast)]

Hallo.
vielleicht kann mir noch einer Weiterhelfen

Als Angezeigten Wert bekomme ich +0.5 obwohl es -0.5 grad sein müssten

// wandelt den Messwert, um.
void DS18S20_convert_temperatur(void)
{  
    
    char Sign;
    int temp;
    
    int16_t Vorkommastelle;
    int16_t Nachkommastelle;  
  
    if( id[1] ) {                // Negativer Temperaturwert
    id[0] ^= 0xFF;               // Einer-Komplement
    id[0]++;                     // inkrementieren
    }
    id[0] >>= 1;                 // rechtes Bit für 0,5 °C rausschieben (durch 2 teilen)
    
  // Temperatur mit 0,1 °C Genauigkeit berechnen
    temp = (int)id[0]*100 - 25 + (int)(id[7] - id[6]) * 100 / id[7];

      if( temp < 0 ) {
        Sign = '-';
        temp = -temp;
      }
      else
        Sign = '+';
  

     Vorkommastelle = temp / 100;
     Nachkommastelle = (temp % 100);     
   
     sprintf(msg,"%c%3d,%1d", Sign, Vorkommastelle, Nachkommastelle);
     uart_puts( msg); 
     uart_putc( ' ' );
}

Hier stimmt der wert zwar aber das minus-zeichen ist nicht an der richtigen stelle  +0.-5 als anzeige. 

Ist der Wert zb -1.3 Dann stimmt die Anzeige


void DS18S20_convert_temperatur(void)
{  
    

    int temp;
    //int temp1;
    int16_t Vorkommastelle;
    int16_t Nachkommastelle;  
  
    if( id[1] ) {                // Negativer Temperaturwert
    id[0] ^= 0xFF;               // Einer-Komplement
    id[0]++;                     // inkrementieren
    }
    id[0] >>= 1;                 // rechtes Bit für 0,5 °C rausschieben (durch 2 teilen)
    
  // Temperatur mit 0,1 °C Genauigkeit berechnen
    temp = (int)id[0]*100 - 25 + (int)(id[7] - id[6]) * 100 / id[7];
    //temp1= temp;
     if( id[1])                   // Negativer Temperaturwert
      {
       temp *= -1;
      }

     else
      {
           
      }
 
     Vorkommastelle = temp / 100;
     Nachkommastelle = (temp % 100);      
     sprintf(msg,"%+3d,%1d", Vorkommastelle, Nachkommastelle);    
 
     
     uart_puts( msg); 
     uart_putc( ' ' );
  
 
}

[Karl H. (kbuchegg)]

kurt schrieb:
> Hallo.
> vielleicht kann mir noch einer Weiterhelfen
>

Lass dir mal temp mit ausgeben. Ich denke, dass da schon was nicht 
stimmt.


      sprintf(msg,"%d  %c%3d,%02d", temp, Sign, Vorkommastelle, 
Nachkommastelle);

[Cyblord -. (cyblord)]

In C muss man da gar nichts umrechnen. Man steckt das Ergebniss direkt 
in eine int16 Variable und hat dann automatisch plus und minus korrekt. 
Wenn man es durch 16 teilt bekommt man die Temp in Grad. Macht man 
VOHRHER noch mal 10 dann bekommt man sie in zehntel Grad. Bei -0,5 Grad 
steht dann -5 in der Variablen. Ohne Umrechnen oder sonst was.

gruß cyblord

[Karl H. (kbuchegg)]

cyblord ---- schrieb:
> In C muss man da gar nichts umrechnen.

Er will vom DS1820 auch die zusätzlichen Bytes benutzen um auf die 
'erhöhte' Genauigkeit zu kommen. Dazu ist im Datenblatt eine Formel 
angegeben.

Ist zwar in den meisten Fällen sowieso Schwachsinn, weil die Temperatur 
eh nicht stimmen wird, aber des Menschen Wille ...

Ich denke allerdings, dass da beim zusammensetzen der Bytes schon was 
nicht stimmt.

[Cyblord -. (cyblord)]

Ups hab ich überlesen. Aber du dürftest Recht haben, da haperts am 
zusammenstecken der Bytes.

[kurt (Gast)]

ich habe anstatt int16_t Nachkommastelle ein uint8_t Nachkommastelle 
benutzt, so nun wird die Temperatur und das Vorzeichen auch korrekt 
Ausgegeben.


Karl Heinz Buchegger schrieb:
>
> Ich denke allerdings, dass da beim zusammensetzen der Bytes schon was
> nicht stimmt.
Wie macht mann das denn Richtig ihrer Ansicht nach.
mfg

void DS18S20_convert_temperatur(void)
{  
    

    int temp;
    int16_t Vorkommastelle;
    uint8_t Nachkommastelle;
   
    if( id[1] ) {                // Negativer Temperaturwert
    id[0] ^= 0xFF;               // Einer-Komplement
    id[0]++;                     // inkrementieren
    }
    id[0] >>= 1;                 // rechtes Bit für 0,5 °C rausschieben (durch 2 teilen)
    
  // Temperatur mit 0,1 °C Genauigkeit berechnen
    temp = (int)id[0]*100 - 25 + (int)(id[7] - id[6]) * 100 / id[7];
    
     if( id[1])                   // Negativer Temperaturwert
      {
       temp *= -1;
      }

     else
      {
          
      }
 
     Vorkommastelle = temp / 100;
     Nachkommastelle = (temp % 100);      
     Nachkommastelle = (Nachkommastelle / 10);// Zweite Nachkommastelle abtrennen
     sprintf(msg,"%+3d,%1d", Vorkommastelle, Nachkommastelle);    
 
     
     uart_puts( msg); 
     uart_putc( ' ' );
}