DaniWeb Forum Index > Software Development > C++

unsigned character pointer to Hex values

Please support our C++ advertiser: Intel Parallel Studio Home

•

Join Date: Aug 2005

Posts: 10

Reputation: Sutanu is an unknown quantity at this point

Solved Threads: 0

Sutanu

Offline

Newbie Poster

unsigned character pointer to Hex values

Aug 19th, 2005

Dear all:

I've created TCPsocket in linux using C++. It receives an array of unsigned characters. I need the corresponding hex values in character format.
i.e. "ABC" in hex it is "414243". Now I want to treat each of "414243" as character.

So I have done the coding as follow ( taking into cosideration that the TCP socket is established and the data is received in an unsigned character pointer m_cpSockData sized same to Buffer Size of the socket. And the number of byte read is m_iReadLen).

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
char* m_cHexvalue = new char[2*BufSize +1];
char *m_cCurrCharVal=new char[3];
 
*(m_cCurrCharVal+2)=0X00;
memset(m_cHexvalue ,0,2);
 
for(int count=0;count<BufSize ;count++)
{
     memset(m_cCurrCharVal,0,2);
     sprintf(m_cCurrCharVal,"%x",*(m_cpSockData+count));
     strcat(m_cHexvalue ,m_cCurrCharVal);
 
}
char* m_cHexvalue = new char[2*BufSize +1];
char *m_cCurrCharVal=new char[3];

*(m_cCurrCharVal+2)=0X00;
memset(m_cHexvalue ,0,2);

for(int count=0;count<BufSize ;count++)
{
     memset(m_cCurrCharVal,0,2);
     sprintf(m_cCurrCharVal,"%x",*(m_cpSockData+count));
     strcat(m_cHexvalue ,m_cCurrCharVal);

}

<< moderator edit: added [code][/code] tags >>

I also have similar thing while writing back to the socket.

But this reduces the efficiency of the code..

Can any body help to provide a mechanism where efficiency can be increased.

•

Join Date: Apr 2004

Posts: 4,452

Reputation: Dave Sinkula has a brilliant future

Solved Threads: 251

Dave Sinkula Dave Sinkula is offline

Offline

long time no c

Re: unsigned character pointer to Hex values

Aug 19th, 2005

This is pretty much the same, but it's all I've got at the moment.

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
#include <stdio.h>
 
char *foo(char *dst, const char *src)
{
   char *start = dst;
   while ( *src )
   {
      dst += sprintf(dst, "%x", (unsigned)*src++);
   }
   return start;
}
 
char *bar(char *dst, const char *src)
{
   char *start = dst;
   while ( *src )
   {
      unsigned value;
      if ( sscanf(src, "%2x", &value) == 1 )
      {
         src += 2;
         *dst++ = value;
      }
   }
   *dst = '\0';
   return start;
}
 
int main(void)
{
   const char abc[] = "ABC";
   char over [ sizeof abc * 2  - 1 ], back [ sizeof abc ];
   printf("\"%s\" -> \"%s\"\n", abc,  foo(over, abc));
   printf("\"%s\" -> \"%s\"\n", over, bar(back, over));
   return 0;
}
 
/* my output
"ABC" -> "414243"
"414243" -> "ABC"
*/
#include <stdio.h>

char *foo(char *dst, const char *src)
{
   char *start = dst;
   while ( *src )
   {
      dst += sprintf(dst, "%x", (unsigned)*src++);
   }
   return start;
}

char *bar(char *dst, const char *src)
{
   char *start = dst;
   while ( *src )
   {
      unsigned value;
      if ( sscanf(src, "%2x", &value) == 1 )
      {
         src += 2;
         *dst++ = value;
      }
   }
   *dst = '\0';
   return start;
}

int main(void)
{
   const char abc[] = "ABC";
   char over [ sizeof abc * 2  - 1 ], back [ sizeof abc ];
   printf("\"%s\" -> \"%s\"\n", abc,  foo(over, abc));
   printf("\"%s\" -> \"%s\"\n", over, bar(back, over));
   return 0;
}

/* my output
"ABC" -> "414243"
"414243" -> "ABC"
*/

Last edited by Dave Sinkula; Aug 19th, 2005 at 7:27 pm. Reason: Added 'bar'.

"One of the methods used by statists to destroy capitalism consists in establishing controls that tie a given industry hand and foot, making it unable to solve its problems, then declaring that freedom has failed and stronger controls are necessary." --Ayn Rand

•

Join Date: Aug 2005

Posts: 10

Reputation: Sutanu is an unknown quantity at this point

Solved Threads: 0

Sutanu

Offline

Newbie Poster

Re: unsigned character pointer to Hex values

Aug 19th, 2005

Thanks for the reply.

I understand you have removed some operations. That reduces quite a big time.

I think this can further be minimized. Do you have any idea of doing this with the help of asm codes in C.

I've not used so far but I think that may help the roundtrip time.

Thanks again.
Sutanu

•

Join Date: Apr 2004

Posts: 4,452

Reputation: Dave Sinkula has a brilliant future

Solved Threads: 251

Dave Sinkula Dave Sinkula is offline

Offline

long time no c

Re: unsigned character pointer to Hex values

Aug 19th, 2005

Perhaps discard the standard routines and roll your own.

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
#include <stdio.h>
#include <ctype.h>
 
unsigned char *foo(unsigned char *dst, const unsigned char *src)
{
   static const unsigned char nibble[] = 
   {
      '0','1','2','3','4','5','6','7','8','9','A','B','C','D','E','F'
   };
   unsigned char *start = dst;
   while ( *src )
   {
      *dst++ = nibble [ *src >> 4    ];
      *dst++ = nibble [ *src++ & 0xF ];
   }
   *dst = '\0';
   return start;
}
 
unsigned char *bar(unsigned char *dst, const unsigned char *src)
{
   unsigned char *start = dst;
   while ( *src )
   {
      if ( isdigit(*src) )
      {
         *dst = *src++ - '0';
      }
      else
      {
         *dst = *src++ - 'A' + 10;
      }
      *dst <<= 4;
      if ( isdigit(*src) )
      {
         *dst++ += *src++ - '0';
      }
      else
      {
         *dst++ += *src++ - 'A' + 10;
      }
   }
   *dst = '\0';
   return start;
}
 
int main(void)
{
   const unsigned char abc[] = "ABCMNO";
   unsigned char over [ sizeof abc * 2  - 1 ], back [ sizeof abc ];
   printf("\"%s\" -> \"%s\"\n", abc,  foo(over, abc));
   printf("\"%s\" -> \"%s\"\n", over, bar(back, over));
   return 0;
}
 
/* my output
"ABCMNO" -> "4142434D4E4F"
"4142434D4E4F" -> "ABCMNO"
*/
#include <stdio.h>
#include <ctype.h>

unsigned char *foo(unsigned char *dst, const unsigned char *src)
{
   static const unsigned char nibble[] = 
   {
      '0','1','2','3','4','5','6','7','8','9','A','B','C','D','E','F'
   };
   unsigned char *start = dst;
   while ( *src )
   {
      *dst++ = nibble [ *src >> 4    ];
      *dst++ = nibble [ *src++ & 0xF ];
   }
   *dst = '\0';
   return start;
}

unsigned char *bar(unsigned char *dst, const unsigned char *src)
{
   unsigned char *start = dst;
   while ( *src )
   {
      if ( isdigit(*src) )
      {
         *dst = *src++ - '0';
      }
      else
      {
         *dst = *src++ - 'A' + 10;
      }
      *dst <<= 4;
      if ( isdigit(*src) )
      {
         *dst++ += *src++ - '0';
      }
      else
      {
         *dst++ += *src++ - 'A' + 10;
      }
   }
   *dst = '\0';
   return start;
}

int main(void)
{
   const unsigned char abc[] = "ABCMNO";
   unsigned char over [ sizeof abc * 2  - 1 ], back [ sizeof abc ];
   printf("\"%s\" -> \"%s\"\n", abc,  foo(over, abc));
   printf("\"%s\" -> \"%s\"\n", over, bar(back, over));
   return 0;
}

/* my output
"ABCMNO" -> "4142434D4E4F"
"4142434D4E4F" -> "ABCMNO"
*/

This is for ASCII, and I think it is right. But I was wrong with it a minute ago.

Last edited by Dave Sinkula; Aug 20th, 2005 at 12:23 am. Reason: Got rid of temporary variables.

•

Join Date: Aug 2005

Posts: 10

Reputation: Sutanu is an unknown quantity at this point

Solved Threads: 0

Sutanu

Offline

Newbie Poster

Re: unsigned character pointer to Hex values

Aug 20th, 2005

Thanks again.

I performed a dry run. I think it would be better to use

if (*src <= '9')

rather than

if (isdigit(*src))

What do you think...

Sutanu

•

Join Date: Apr 2004

Posts: 4,452

Reputation: Dave Sinkula has a brilliant future

Solved Threads: 251

Dave Sinkula Dave Sinkula is offline

Offline

long time no c

Re: unsigned character pointer to Hex values

Aug 20th, 2005

•

Originally Posted by Sutanu

I performed a dry run. I think it would be better to use

if (*src <= '9')

rather than

if (isdigit(*src))

What do you think...

Sutanu

If that works better, go ahead. Many times the standard functions are implemented faster than writing an equivalent, but not always.

•

Join Date: Aug 2005

Posts: 15,625

Reputation: Ancient Dragon has a reputation beyond repute

Solved Threads: 1495

Ancient Dragon Ancient Dragon is offline

Offline

Still Learning

Re: unsigned character pointer to Hex values

Aug 20th, 2005

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
if( *src <= '9' )
if( *src <= '9' )

what if *src < '0' ? isdigit() will catch that, but your if statement won't.

•

Join Date: Aug 2005

Posts: 10

Reputation: Sutanu is an unknown quantity at this point

Solved Threads: 0

Sutanu

Offline

Newbie Poster

Re: unsigned character pointer to Hex values

Aug 20th, 2005

Thanks for the concern.

The scenario I'm talking about doesn't permit you to have any values other than 0-9 and A-F.

In other scenarios or to have a generic one I think using a isdigit() is a better option. But then also you can see what happens if any character as * ( ASCII value is 42) comes . Then you probably will have a negative value (42 - 64 +10 = -12).