home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #31 / NN_1992_31.iso / spool / comp / lang / c / 19163 < prev    next >
Encoding:
Text File  |  1993-01-03  |  2.1 KB  |  63 lines
  1. Newsgroups: comp.lang.c
  2. Path: sparky!uunet!spool.mu.edu!sgiblab!news.kpc.com!kpc!hollasch
  3. From: hollasch@kpc.com (Steve Hollasch)
  4. Subject: Re: Problem with string processing.
  5. Message-ID: <1993Jan4.022737.19792@kpc.com>
  6. Sender: usenet@kpc.com
  7. Organization: Kubota Pacific Computer, Inc.
  8. References: <1993Jan2.233446.20833@iitmax.iit.edu>
  9. Distribution: usa
  10. Date: Mon, 4 Jan 1993 02:27:37 GMT
  11. Lines: 50
  12.  
  13. matujos@elof.iit.edu (Joe Matusiewicz) writes:
  14. | 
  15. |    I need a function to append a character to the end of a string.
  16. | I can't find an ANSI function to do so, so I tried to write my own.
  17. | Here's what I have:
  18. | 
  19. |          void add_char_to_str(char ch, char *str)
  20. |          {
  21. |              char *tmp;
  22. |              tmp = (char *) calloc (2*sizeof(char));
  23. |              *tmp = ch;
  24. |              *(tmp+1) = '\0';
  25. |              strcat(str, tmp);
  26. |              free (tmp);
  27. |          }
  28.  
  29.     calloc() takes two parameters:  number of elements, and element size.
  30. You should be using malloc here.  Also, you have a lot of overhead for such
  31. a simple operation.  Here's how I'd do it.
  32.  
  33.         void add_char_to_str (
  34.             register char  ch,     /* Character to Append */
  35.             register char *str)    /* Old String */
  36.         {
  37.             while (*str) ++str;
  38.             str[0] = ch;
  39.             str[1] = 0;
  40.         }
  41.  
  42.     This assumes that the space pointed to by 'str' has room for another
  43. character.  A more forgiving routine would be:
  44.  
  45.         char *add_char_to_str (
  46.             char  ch,     /* Character to Append */
  47.             char *str)    /* Old String */
  48.         {
  49.             register char *ptr;         /* New String */
  50.  
  51.             if (!(ptr = malloc (strlen(str) + 2))) return 0;
  52.  
  53.             while (*ptr++ = *str++);    /* Note assign op and null body. */
  54.             ptr[-1] = ch;
  55.             ptr[ 0] = 0;
  56.  
  57.             return ptr;
  58.         }
  59.  
  60. ______________________________________________________________________________
  61. Steve Hollasch                                   Kubota Pacific Computer, Inc.
  62. hollasch@kpc.com                                 Santa Clara, California
  63.