Thread: Logic error: need advice

hurm.. what is the correct way to return that value?

Normally I suggest passing the array to the function as a parameter. Otherwise using static will work, but you must be careful if you try to call the function more than once. Another way is to use dynamic memory allocation for this array (malloc/free).

and there's also problem with the value inside hexnum[j] when I convert something more than '111111111' (9 digits)

That's probably because your binNumChar has been defined with a size of 9.

What exactly are you passing into this function? Please give a couple of examples.

Jim

What compiler are you using?

Also I don't see where you are putting the end of string character into your array before you use the string functions.

Jim

Im using Dev-C++ 4.9.9.2. So where should I put the end of string character into my array? and what is the use of the end of string character?

In C a string is a zero terminated character array. Without this terminating character '\0' you just have an array of characters. The functions in the string.h header file require C-strings and will not work with non-terminated character arrays.

So where should I put the end of string character into my array?

After the last valid character. You also need to insure that your array size allows for this extra character. So a C-string of "Hello" requires space for 6 characters, the five characters and the end of string character, 'H','e','l','l','o','\0'

Jim

Code:

char* convBinToHex(long long int binNumInt){
   
   char binNumChar[16],hexNum[10],arrToStrTemp[MAX];
   static char arrToStr[5]="";
   int i=0,j=0,temp;
   
   snprintf(binNumChar, sizeof(binNumChar), "%lld", binNumInt); //convert integer to char
   
   while(binNumChar[i]){
      binNumChar[i] = binNumChar[i] -48;
      ++i;
   }

   --i;
   
   while(i-2>=0){
       temp =  binNumChar[i-3] *8 + binNumChar[i-2] *4 +  binNumChar[i-1] *2 + binNumChar[i] ;
       if(temp > 9)
           hexNum[j++] = temp + 55;
       else
            hexNum[j++] = temp + 48;
       i=i-4;
   }

   if(i ==1)
      hexNum[j] = binNumChar[i-1] *2 +binNumChar[i] + 48 ;
   else if(i==0)
      hexNum[j] =  binNumChar[i] + 48 ;
   else
      --j;
      
   strcpy(arrToStr,""); 
   while(j>=0){
               sprintf(arrToStrTemp, "%c",hexNum[j--]);
               strcat(arrToStr,arrToStrTemp); //combine all string

    }
    
    return arrToStr;

I try this one:
sprintf(arrToStrTemp, "%c",hexNum[j--]);
strcat(arrToStr,arrToStrTemp); //combine all string
and it works..can you tell me what is happening?

Well sprintf() takes your character and converts it to a C-string, strcat() then appends one C-string to another.

Jim