Hex zu Dezimal

[Mich A. (ebet14)]

Versuche grad eine Hexzahl in Dezimal umzurechnen mit Hilfe der Ascii 
Tabelle in C++. Allerdings funktioniert es nicht so wie es soll.
Die if Schleife durchläuft er richtig und das Ergebnis stimmt auch, 
allerdings bei denn beide else if's kommt das Falsche raus kann mit 
jemand helfen?

danke schon mal

#include <iostream>
using namespace std;

int b, c, d;
char a;

int main(){
    cout<<"Ihre Hexadezimalzahl ist: ";
    cin>>a;
    cout << a <<"\n";
            if(a == 'a'||'b'||'c'||'d'||'e'||'f'){
            b = a - 87;
            cout<<"Der Dezimalwert der Hexadezimalziffer "<< a <<" ist "<< b <<"\n";
            }
            else if(a == 'A'||'B'||'C'||'D'||'E'||'F'){
            c = a - 55;
            cout<<"Der Dezimalwert der Hexadezimalziffer "<< a <<" ist "<< c <<"\n";
            }
            else if(a == '1'||'2'||'3'||'4'||'5'||'6'||'7'||'8'||'9'||'0'){
            d = a - 48;
            cout<<"Der Dezimalwert der Hexadezimalziffer "<< a <<" ist "<< d <<"\n";
            }
            else{
            cout<<"Fehler"<<"\n";
            }
return 0;
}

[Peter II (Gast)]

Michael Estner schrieb:
> if(a == 'a'||'b'||'c'||'d'||'e'||'f'){

ich kenn zwar nicht den neusten C++ Standard, aber dieser Syntax ist mir 
neu und macht bestimmt nicht was du willst.

if(a >= 'a' && a <= 'f' ){

[Noch einer (Gast)]

Die Syntax ist OK.

C rechnet zuerst a'||'b'||'c'||'d'||'e'||'f' und vergleicht dann das 
Ergebnis mit a.

[Steffen R. (steffen_rose)]

Noch einer schrieb:
> Die Syntax ist OK.
>
> C rechnet zuerst a'||'b'||'c'||'d'||'e'||'f' und vergleicht dann das
> Ergebnis mit a.

Auch wenn Du recht hast finde ich es für einen Anfänger schwer 
verständlich, dass Du auch speziell mit der letzten Teil darauf 
hinweisen willst, dass es wohl nicht das ist, was der TO bezwecken will.

[Dirk B. (dirkb2)]

Tipp: Auch ein Zeichenliteral wie 'a' ist nur ein Zahlenwert:

b = a - 'a' + 10;
c = a - 'A' + 10;
d = a - '0';

 Und schon is es egal, welchen Code die Zeichen eigentlich haben.

[Caligulaminus (Gast)]

Noch einer schrieb:
> C rechnet zuerst a'||'b'||'c'||'d'||'e'||'f' und vergleicht dann das
> Ergebnis mit a.

Nö, andersherum.

== geht vor ||

[Max H. (hartl192)]

Schau dir mal das an, für das was du erreichen willst gibt's auch 
fertige Funktionen:
http://www.cplusplus.com/reference/cctype/


Mich Ael schrieb:
> if Schleife
http://www.if-schleife.de/

[Rene H. (Gast)]

Es kommt das falsche raus, weil Deine else if nie durchlaufen werden.

Mich Ael schrieb:

> if(a == 'a'||'b'||'c'||'d'||'e'||'f'){

ist immer true. Konkret vergleicht er

a == 'a'

und wertet dann Dein

|| 'b'

aus. Das ist aber immer true, wie auch die nachfolgenden.
Also rechnet er immer

b = a - 87;

aus.

Grüsse,
René

[Heinz L. (ducttape)]

Noch einer schrieb:
> Die Syntax ist OK.
>
> C rechnet zuerst a'||'b'||'c'||'d'||'e'||'f' und vergleicht dann das
> Ergebnis mit a.

Egal ob == oder || vorgehen, da kommt Murks heraus. Wenn == vorgeht 
sowieso, aber auch wenn man's klammert oder Du 'n Compiler hast der || 
die Präferenz gibt. Warum?

97 || 98 || 99 || 100 || 101 || 102

Das Ergebnis davon ist 103. Und damit ist a == .... NIEMALS true. Weil 
103 ist der ascii-code von g.

Und warum ist es 103?

Weil die Veroderung von Zahlen einfach nur alle Bits setzt die in 
irgendeiner der Zahlen gesetzt sind.

Entsprechend ist so ein a || b || c ... Konstrukt in fast jedem Fall 
Grütze. Syntaktisch korrekt, und der Compiler frisst es auch. Rauskommen 
tut nur halt fast sicher Nonsense.

[Martin N. (Gast)]

So ist es einfacher:

#include <iostream>
using namespace std;

int b, c, d;
char a;

int main(){
  cout << "Ihre Hexadezimalzahl ist: ";
  cin >> a;
  cout << a << "\n";
  if (a >= 'a' && a <= 'f'){
    b = a - 87;
    cout << "Der Dezimalwert der Hexadezimalziffer " << a << " ist " << b << "\n";
  }
  else if (a >= 'A' && a <= 'F'){
    c = a - 55;
    cout << "Der Dezimalwert der Hexadezimalziffer " << a << " ist " << c << "\n";
  }
  else if (a >= '0' && a <= '9'){
    d = a - 48;
    cout << "Der Dezimalwert der Hexadezimalziffer " << a << " ist " << d << "\n";
  }
  else{
    cout << "Fehler" << "\n";
  }
  return 0;
}

[Martin N. (Gast)]

Und das kürzer:

#include <iostream>
using namespace std;

char a;
int b;

int main(){
  cout << "Ihre Hexadezimalzahl ist: ";
  cin >> a;
  cout << a << "\n";

  b = a;

  if (a >= 'a' && a <= 'f') b = a - 87;
  else if (a >= 'A' && a <= 'F') b = a - 55;
  else if (a >= '0' && a <= '9') b = a - 48;
    
  if (a != b) cout << "Der Dezimalwert der Hexadezimalziffer " << a << " ist " << b << "\n";
  else cout << "Fehler" << "\n";

  return 0;
}

[rm (Gast)]

Heinz L. schrieb:
> 97 || 98 || 99 || 100 || 101 || 102
>
> Das Ergebnis davon ist 103. Und damit ist a == .... NIEMALS true. Weil
> 103 ist der ascii-code von g.
>
> Und warum ist es 103?
>
> Weil die Veroderung von Zahlen einfach nur alle Bits setzt die in
> irgendeiner der Zahlen gesetzt sind.

aber nur bei | und nicht mit ||
bitweise oder / logisch oder

[Frank M. (ukw) (Moderator)]

Martin N. schrieb:
> Und das kürzer:
> [...]

>   if (a >= 'a' && a <= 'f') b = a - 87;
>   else if (a >= 'A' && a <= 'F') b = a - 55;
>   else if (a >= '0' && a <= '9') b = a - 48;
>

Und das lesbarer und verständlicher, ohne dass man eine ASCII-Tabelle in 
der Tasche mit sich rumtragen muss:

  if (a >= 'a' && a <= 'f')      b = a - 'a' + 10;
  else if (a >= 'A' && a <= 'F') b = a - 'A' + 10;
  else if (a >= '0' && a <= '9') b = a - '0';

[Rolf M. (rmagnus)]

Heinz L. schrieb:
> 97 || 98 || 99 || 100 || 101 || 102
>
> Das Ergebnis davon ist 103.

Falsch. Das Ergebnis ist 1.

> Und warum ist es 103?
>
> Weil die Veroderung von Zahlen einfach nur alle Bits setzt die in
> irgendeiner der Zahlen gesetzt sind.

Nur daß es sich hier um ein boolesches Oder handelt. Das nimmt jeden 
Wert, der 0 ist, als false und jeden anderen als true. Das Ergebnis ist 
dann 0 für false und 1 für true.

> Entsprechend ist so ein a || b || c ... Konstrukt in fast jedem Fall
> Grütze.

Nein, der ist oft nützlich. In diesem Fall macht er aber was völlig 
anderes als beabsichtigt.

[Timmo H. (masterfx)]

Wieso nicht einfach strtol/strtoul?

[Rene H. (Gast)]

Timmo H. schrieb:
> Wieso nicht einfach strtol/strtoul?

Und dann? Für Hex geht das nicht.

[Karl H. (kbuchegg)]

Rene H. schrieb:
> Timmo H. schrieb:
>> Wieso nicht einfach strtol/strtoul?
>
> Und dann? Für Hex geht das nicht.

Nicht ganz.
strtol bzw strtoul kann man die Basis des Zahlensystems schon angeben.
Aber inklusive Fehlererkennung und allem drum und drann ist die Lösung 
mittels strtol dann schon deutlich aufwändiger als eine simple 
Fallunterscheidung für ein hexadezimales Digit.

[Dirk B. (dirkb2)]

Timmo H. schrieb:
> Wieso nicht einfach strtol/strtoul?

Der Weg ist das Ziel.

[Timmo H. (masterfx)]

Wenn man bei strtol '0' als Basis nimmt ist eine Autoerkennnung für 
0x.... drin. Warum wird hier das Rad ständig neu erfunden?
Da wo es geht sollte man immer auf Standardfunktionen zurückgreifen, die 
sind bereits hochoptimiert, und wenn es um "PC-Programmierung" geht, 
sind ein paar Bytes/Clocks mehr die das ganze brauch doch eh wurst.

[Jürgen (Gast)]

Programmiere zwar ASM, aber vielleicht hilft es dir trotzdem:

In deine Variablen a Bit5 löschen, dann hast du auf jeden Fall einen 
Grossbuchstaben.
Nun von a 48 oder '0' abziehen.

Ist a > 9 noch mal 7 abziehen.

Fertig, und nur ein IF gebraucht

[Dirk B. (dirkb2)]

Jürgen schrieb:
> In deine Variablen a Bit5 löschen, dann hast du auf jeden Fall einen
> Grossbuchstaben.

Dafür gibt es in C toupper

> Fertig, und nur ein IF gebraucht

Und was ist, wenn z.B ein 'g' eingegeben wurde?

Zudem funktioniert dein Code nur für ASCII (oder Abarten davon)

[nur so (Gast)]

#include<stdio.h>

int main(){


    char num ;
    printf("gibt dein hexnummer ein bitte");
    scanf("%hx",&num);

    printf("hex =%hx und das ist dezimal =%d\n",num, num);




    return 0;
}

[Heinz L. (ducttape)]

nur so schrieb:
> #include<stdio.h>
>
> int main(){
>
> [...]
>     printf("hex =%hx und das ist dezimal =%d\n",num, num);
>
> [...]
> }

Cheater. :)

[Rolf M. (rmagnus)]

Heinz L. schrieb:
> Cheater. :)

Und falsch ist es auch noch.

nur so schrieb:
> scanf("%hx",&num);

Das müßte heißen:

scanf("%hhx",&num);

Beim printf kann man das h dafür auch einfach weglassen.