Dezimal zu Binär - Was mache ich falsch?

[Jan H. (janiiix3)]

GuMo,

char *decBin(unsigned char dec, char *buff)
{
  unsigned char tempCnt = 7;
  unsigned char msk = 0x80;
  
  unsigned char test[8];
  
  for(; tempCnt > 0 ; tempCnt--)
  {
    if(dec & msk)
      buff[tempCnt] = '1';
    else
      buff[tempCnt] = '0';
    
    msk>>=1;  
  }
  buff[8] = '\0';
  
  printf("Result = %s",buff);
  return buff;
}

Wenn ich das mit dem PC simuliere, bekomme ich nichts zurück außer 
"Result = ".
Benutze ich jetzt aber das "test[]" Array, funktioniert es wie 
gewünscht. Was ist verkehrt?

[Joe (Gast)]

Wie rufst Du die Funktion auf? Ganzen Code posten.

[Jan H. (janiiix3)]

Joe schrieb:
> Wie rufst Du die Funktion auf? Ganzen Code posten.

typedef char*(*calcFuncPtr)(unsigned char, char []);

char *decHex(unsigned char dec, char buff[]);
char *decBcd(unsigned char dec, char buff[]);
char *decBin(unsigned char dec, char *buff);


char *decHex(unsigned char dec, char buff[])
{  
  /* hex char set */
  char hexchars[] = "0123456789ABCDEF";
    buff[0] = '0';  
     buff[1] = 'x';   
    buff[2] = hexchars[dec >> 12 & 0xF];    
    buff[3] = hexchars[dec >>  8 & 0xF];     
    buff[4] = hexchars[dec >>  4 & 0xF];          
    buff[5] = hexchars[dec       & 0xF];
  
    return buff;
}

char *decBcd(unsigned char dec, char buff[])
{  
  buff[0] = '0';
  buff[1] = 'x';

  buff[2] = (dec / 10)+'0';
  buff[3] = (dec % 10)+'0';

  return buff;
}

char *decBin(unsigned char dec, char *buff)
{
  unsigned char tempCnt = 7;
  unsigned char msk = 0x80;
  
  unsigned char test[8];
  
  for(; tempCnt > 0 ; tempCnt--)
  {
    if(dec & msk)
      buff[tempCnt] = '1';
    else
      buff[tempCnt] = '0';
    
    msk>>=1;  
  }
  buff[8] = '\0';
  
  printf("Result = %s",buff);
  return buff;
}

typedef struct
{
  calcFuncPtr  decHex;
  calcFuncPtr  decBcd;
  calcFuncPtr  decBin;
}conv_t;

conv_t Convert ={
  .decHex = decHex,
  .decBcd = decBcd,
  .decBin = decBin,
};

int main(int argc, char *argv[]) 
{
  char buff[9] = "";
  
  printf("%s",Convert.decBin(0x80,buff));
  return 0;
}

[Peter D. (peda)]

Jan H. schrieb:
> Benutze ich jetzt aber das "test[]" Array, funktioniert es wie
> gewünscht.

Nö.

Jan H. schrieb:
> for(; tempCnt > 0 ; tempCnt--)

D.h. es läuft von 7..1. Damit ist der Inhalt von buff[0] undefiniert.

[Jan H. (janiiix3)]

Peter D. schrieb:
> Jan H. schrieb:
>> Benutze ich jetzt aber das "test[]" Array, funktioniert es wie
>> gewünscht.
>
> Nö.
>
> Jan H. schrieb:
>> for(; tempCnt > 0 ; tempCnt--)
>
> D.h. es läuft von 7..1. Damit ist der Inhalt von buff[0] undefiniert.

Das war es. Perfekt! Vielen Dank.

[Mampf unterwegs (Gast)]

buff[0] wird nicht beschrieben :)

wenn buff mit 0 initialisiert wird und das 0te element nicht beschrieben 
wird, ist der string leer, da null terminiert