Frage zu type cast in C

[Daniel S. (supernova01)]

Hallo,

kleine Frage, ich habe folgendes:

int8_t diff=-102;

uint8_t out=0;

nun steht ja in diff die Bitfolge 0x9A, also b10011010.

Ich möchte nun aber die 102 als positive Ganzzahl (0x66-b01100110) in 
out stehen haben.

geht das so?

out=(uint8_t)diff;

oder so:

out=(diff-1)^0xFF;

Habe gerade einen Knoten im Hirn.

Danke
DS

[Adam P. (adamap)]

Das wäre am einfachsten, oder suchst du eine Lösung mit Bit-Operation 
ohne "Hilfsfunktionen"?

out = abs(diff);

edit:
so gehts auch:

out = (diff * -1);

oder:

out = (diff ^ 0xFF) + 1;

[EAF (Gast)]

Dennis S. schrieb:
> nun steht ja in diff die Bitfolge 0x9A, also b10011010.

Nein, dieses ist  Implementation Defined, somit nicht als gesichert 
anzunehmen

[Peter D. (peda)]

Dazu gibt es die Funktion abs().

https://www.tutorialspoint.com/c_standard_library/c_function_abs.htm

[MaWin (Gast)]

out = (uint8_t)-diff;

[P. S. (namnyef)]

Selbst abs() kann zu undefiniertem Verhalten führen, weil der Betrag der 
kleinsten negativen Zahl in Zweierkomplementdarstellung größer ist als 
die größte positive Zahl. Die Verwendung dieser Funktion ist aber der 
naheliegendste erste Ansatz bevor man Murks auf Bit-Ebene fabriziert.

Ob das undefinierte Verhalten von abs() relevant ist, muss je nach 
Anwendung beurteilt werden.

[Adam P. (adamap)]

P. S. schrieb:
> Selbst abs() kann zu undefiniertem Verhalten führen

Also laut GNU C ist abs() so implementiert:

/* Return the absolute value of I.  */
int
abs (int i)
{
  return i < 0 ? -i : i;
}

P. S. schrieb:
> weil der Betrag der kleinsten negativen Zahl in Zweierkomplementdarstellung > 
größer ist als die größte positive Zahl.

Das würde stimmen, wenn du es wieder einem int8_t zuweisen möchtest,
aber nicht wenn du als Ziel uint8_t verwendest.

// Das ergibt wieder -128
int8_t i = -128;

i = abs(i);

Oder sehe ich da jetzt etwas falsch?

[Daniel S. (supernova01)]

Adam P. schrieb:
> Oder sehe ich da jetzt etwas falsch?

Der Betrag von -128 ist 128.

Der Zahlenbereich eines int8_t geht von -128 bis 127.

Ich denke das war gemeint. Also der Betrag von -128 passt nicht in eine 
int8_t.

P. S. schrieb:
> Ob das undefinierte Verhalten von abs() relevant ist, muss je nach
> Anwendung beurteilt werden.

Das wäre aber ja nur ein Problem wenn man den Rückgabewert wieder einem 
int8_t zuspielt. In meinem Fall geht er in einen uint8_t, der geht ja 
bis 255...

MaWin schrieb:
> out = (uint8_t)-diff;

tatsächlich so einfach. probiere ich aus, nachher.


EAF schrieb:
> Nein, dieses ist  Implementation Defined, somit nicht als gesichert
> anzunehmen

Was wäre denn "sicher", oder wasserdicht und üblich?

Danke bis hierher.

[Adam P. (adamap)]

Dennis S. schrieb:
> Das wäre aber ja nur ein Problem wenn man den Rückgabewert wieder einem
> int8_t zuspielt.

So habe ich es ja geschrieben.

[Hannes (Gast)]

Dennis S. schrieb:
> Habe gerade einen Knoten im Hirn.

Meine Tipp: Meide Type-Cast auf Teufel komm raus!

Notwendig erscheinende Casts sind fast immer ein Zeichen, dass mit den 
Datenstrukturen irgendwas faul ist.

Die Warnings oder Errors, die man mit wilden Casts meint unterdrücken zu 
müssen, schmeißt der Compiler aus einem ganz bestimmten Grund, 
namentlich weil er hier Probleme erkannt hat. Dies, wie Muttis lästige 
Händi-Anrufe einfach wegzudrücken ist sicher nicht der beste Weg -- 
sondern gleicht eher einer Vogelstrauß-Taktik.

just my 2 ct

[Oliver S. (oliverso)]

Hannes schrieb:
> Meine Tipp: Meide Type-Cast auf Teufel komm raus!

Damit schliesst du für dich das Rechnen mit Datentypen, die kleiner als 
ein Int sind, faktisch aus.

Oliver

[Wf88 (wf88)]

Wenn du einfach nur das Vorzeichen entfernen willst, dann musst du nur 
das höchstwertige Bit löschen.

int8_t byte = 0xFF;
int16_t word = 0xFFFF;
uint16_t result;

result = (byte & ~0x80);
result = (word & ~0x8000);

[Oliver S. (oliverso)]

Wf88 schrieb:
> Wenn du einfach nur das Vorzeichen entfernen willst, dann musst du
> nur
> das höchstwertige Bit löschen.

Das stimmt zwar, ist aber nur sinnvoll, wenn der Zahlenwert keine Rolle 
spielt. Das ist eher selten…

Oliver

[Dirk B. (dirkb2)]

Wf88 schrieb:
> Wenn du einfach nur das Vorzeichen entfernen willst, dann musst du nur
> das höchstwertige Bit löschen.
>
>

> int8_t byte = 0xFF;
> int16_t word = 0xFFFF;
> uint16_t result;
> 
> result = (byte & ~0x80);
> result = (word & ~0x8000);
>

Wegen Integer Promotion kommt beim byte nicht das raus, was du 
erwartest.

[MaWin (Gast)]

Wf88 schrieb:
> Wenn du einfach nur das Vorzeichen entfernen willst, dann musst du nur
> das höchstwertige Bit löschen.

Im Zweierkomplement ist das leider völliger Unsinn.

[Hannes (Gast)]

Oliver S. schrieb:
> Hannes schrieb:
>> Meine Tipp: Meide Type-Cast auf Teufel komm raus!
>
> Damit schliesst du für dich das Rechnen mit Datentypen, die kleiner als
> ein Int sind, faktisch aus.
>
> Oliver

Wär mir neu - aber vielleicht reden wir auch von ganz verschiedenen 
Dingen?

Jedenfalls hattest Du bei meinem AG deine liebe Not, beim 'code review' 
einen Cast durchgeboxt zu kriegen.

-H

[A. S. (Gast)]

warum nicht straight:

if(diff >= 0) {out = diff;} else {out = -diff;}

Es ist müßig, irgendwelche halbgaren Tricks oder Standard-Funktionen zu 
vergewaltigen, um ein offensichtliches if zu sparen. Und Casts brauchts 
auch nicht.

[Jörg W. (dl8dtl) (Moderator)]

Dennis S. schrieb:
> out=(uint8_t)diff;

Das ergibt übrigens 154 statt deiner gewünschten 102. Es wird nur das 
Bitmuster kopiert, ohne jede Rechnung.

out=(uint8_t)-diff;

würde gehen.

[Rolf M. (rmagnus)]

Hannes schrieb:
> Dennis S. schrieb:
>> Habe gerade einen Knoten im Hirn.
>
> Meine Tipp: Meide Type-Cast auf Teufel komm raus!

Mein Tipp wäre eher: Lerne zu verstehen, wo und warum Casts nötig sind. 
Das hilft dabei, sie zu vermeiden, wo sie nicht nötig sind und korrekt 
anzuwenden, wo sie es sind.
Anders gesagt: Immer, wenn man einen Cast hinschreibt, sollte man genau 
sagen können, was er an der Stelle bewirkt und warum er dort 
unvermeidbar ist.

> Die Warnings oder Errors, die man mit wilden Casts meint unterdrücken zu
> müssen, schmeißt der Compiler aus einem ganz bestimmten Grund,
> namentlich weil er hier Probleme erkannt hat. Dies, wie Muttis lästige
> Händi-Anrufe einfach wegzudrücken ist sicher nicht der beste Weg --
> sondern gleicht eher einer Vogelstrauß-Taktik.

Das ist richtig. Der Grund für einen Cast darf niemals sein: "Um die 
Compiler-Warnung wegzubekommen".

Oliver S. schrieb:
> Hannes schrieb:
>> Meine Tipp: Meide Type-Cast auf Teufel komm raus!
>
> Damit schliesst du für dich das Rechnen mit Datentypen, die kleiner als
> ein Int sind, faktisch aus.

Warum?

#include <stdio.h>
#include <inttypes.h>
int main(void)
{
    uint8_t a = 23:
    uint8_t b = 45;
    uint8_t c = a + b;
    printf("%" PRIu8 "\n", c);
}

Ganz ohne Casts.