Помогите с переводом HexToBin

[fedrom]

Вот мой код по переводу из hex в bin

char *HexToBin(char Hex[])
{  char Bin[25]="";
   printf(HexToBin(Hex));
   int i = 0;
   while (i<=strlen(Hex))
    {if (i==0){
      if (Hex[i] == '1') strcat(Bin,"1");
      if (Hex[i] == '2') strcat(Bin,"10");
      if (Hex[i] == '3') strcat(Bin,"11");
      if (Hex[i] == '4') strcat(Bin,"100");
      if (Hex[i] == '5') strcat(Bin,"101");
      if (Hex[i] == '6') strcat(Bin,"110");
      if (Hex[i] == '7') strcat(Bin,"111");
      if (Hex[i] == '8') strcat(Bin,"1000");
      if (Hex[i] == '9') strcat(Bin,"1001");
      if ((Hex[i] == 'A')||(Hex[i] == 'a')) strcat(Bin,"1010");
      if ((Hex[i] == 'B')||(Hex[i] == 'b')) strcat(Bin,"1011");
      if ((Hex[i] == 'C')||(Hex[i] == 'c')) strcat(Bin,"1100");
      if ((Hex[i] == 'D')||(Hex[i] == 'd')) strcat(Bin,"1101");
      if ((Hex[i] == 'E')||(Hex[i] == 'e')) strcat(Bin,"1110");
      if ((Hex[i] == 'F')||(Hex[i] == 'f')) strcat(Bin,"1111");}
      else{
      if (Hex[i] == '0') strcat(Bin,"0000");
      if (Hex[i] == '1') strcat(Bin,"0001");
      if (Hex[i] == '2') strcat(Bin,"0010");
      if (Hex[i] == '3') strcat(Bin,"0011");
      if (Hex[i] == '4') strcat(Bin,"0100");
      if (Hex[i] == '5') strcat(Bin,"0101");
      if (Hex[i] == '6') strcat(Bin,"0110");
      if (Hex[i] == '7') strcat(Bin,"0111");
      if (Hex[i] == '8') strcat(Bin,"1000");
      if (Hex[i] == '9') strcat(Bin,"1001");
      if ((Hex[i] == 'A')||(Hex[i] == 'a')) strcat(Bin,"1010");
      if ((Hex[i] == 'B')||(Hex[i] == 'b')) strcat(Bin,"1011");
      if ((Hex[i] == 'C')||(Hex[i] == 'c')) strcat(Bin,"1100");
      if ((Hex[i] == 'D')||(Hex[i] == 'd')) strcat(Bin,"1101");
      if ((Hex[i] == 'E')||(Hex[i] == 'e')) strcat(Bin,"1110");
      if ((Hex[i] == 'F')||(Hex[i] == 'f')) strcat(Bin,"1111");}
      i++;
    }
    printf(HexToBin(Bin));
  //strcat(Bin,'\0');
return Bin;
}

в чем ошибка может кто опытным глазом посмотрит и скажет в чем проблема)

Модератор: тег CODE

[fedrom]

передатеся С-строка в hex формате например "3DА" возращается тоже С-строка "1111011010"

[Bearw08]

чтобы посмотреть "опытным" глазом, нужно опытно внести код в тег <code>

[vvviperrr]

Ошибка в говнокоде (в данном случае это именно так).
нафига столько условий??? (при том, что проверятюся ВСЕ, даже если сработает первое, else надо было ставить). нужно использовать switch.
потом. что за рекурсия? почему в HexToBin вызывается printf(HexToBin(Bin)); ???
и потом. ты выделил в функции массив на 25 элементов чар, и при выходе из функции возвращаешь указатель на этот массив! что происходит со статической памятью? правильно, освобождается. возможно конечно, какое то время этот указатель и будет валидным, но не всегда.

пс. на алгоритм я даже и не смотрел

[fedrom]

Спасибо за быстры ответ и критику буду иправлять)

[pu4koff]

strlen(Hex) еще уберите из условия цикла. Это ж каждую итерацию будет определяться длина неменяющейся С-строки.
if (i == 0) нечего делать в теле цикла. Истинным это выражение будет 1 раз в самом начале, а проверяться будет каждую итерацию.

[Fixiel]

Сообщение от pu4koff

strlen(Hex) еще уберите из условия цикла. Это ж каждую итерацию будет определяться длина неменяющейся С-строки.
if (i == 0) нечего делать в теле цикла. Истинным это выражение будет 1 раз в самом начале, а проверяться будет каждую итерацию.

Если не сложно, то напиши как будет выглядеть код представленный выше без недочётов. К примеру я ввожу HEX-код в MaskEdit и вывожу на Label Bin-код.

[Stilet]

Я не знаю что можно сделать с кодом в топе, но могу предложить свой вариант для переводов: http://www.programmersforum.ru/showp...4&postcount=36