adding chars to a string

[Rick]

Hi,

This is probably simple byt when you never did pointers and being used to
luxery strings like in Delphi or Visual Basic, C can get though. What I'm
trying to do is to add chars to a string. I looked in the string.h file but
I didn't find that kind of function (that string library is more than 7
years old) so I decided to write that function on myself. It seems to work
although when I'm trying to do printf's between those actions I get strange
messages. Anyway, now I'm trying to add chars to 2 strings. The first string
is ok but the second one seems to get overwrited. Here's my code :

/* the function to add 1 char to a string */

char *strAddChar(cha r *s1, char c){
char *s;
s = s1;
s = s + strlen(s1); // the position to write
*s = c; // this should be on the place where the null char
first
// was.
s++;
*s = 0; // add a new null string
return(s1);
} // strAddChar

/* this is how I use the function */
main(){
char *res1 = ""; // empty string 1
char *res2 = ""; // empty string 2
strAddChar( res1, 'a' );
strAddChar( res1, 'b' );
strAddChar( res1, 'c' );
strAddChar( res2, '1' );
strAddChar( res2, '2' );
strAddChar( res2, '3' );
printf( "%s\n",res1 ); // test
printf( "%s\n",res2 );

Ok, the output should be res1="abc" and res2="123" but this is my result :
res1="abc" res2="bc123"
How does "bc" come in res2?? How to fix the function?

Greetings,
Rick

[Andreas Kahari]

In article <3f************ ***********@new s.xs4all.nl>, Rick wrote:

Hi,

This is probably simple byt when you never did pointers and being used to
luxery strings like in Delphi or Visual Basic, C can get though. What I'm
trying to do is to add chars to a string.

Did you investigate strcat() and strncat()?
--
Andreas Kähäri

[Rick]

To add, I think result2 get's overwritten by result1 somehow. When those 2
got initialized they get a memory adress automatically and it seems that
when I'm adding chars to result1, result2 won't move further. Result1
probably only get 1 byte of memory so when adding the second char it's "out
of bounds" and contineus on result2.

If this is true, that would be quiete dangerous because I could overwrite a
big part of the program. Normally you can use arrays and malloc them but in
my case, I really don't know how big the result becomes. While building the
result string, I need at least 1 sub string for other stuff, that string has
the same problem, I can't calculate how big it becomes, it has to grow
dynamically. How to handle this?

Greetings,
Rick

[Rick]

You answered faster than the light! I just tried it but the same problem
(see that other post where I explained the problem a little bit more). Or
maybe I am using the code wrong, this is what I did with strncat:

char *res1 = "";
char *res2 = "";
strncat( res1, "a",1 );
strncat( res1, "b",1 );
strncat( res1, "c",1 );
strncat( res2, "1",1 );
strncat( res2, "2",1 );
strncat( res2, "3",1 );
printf( "result1 %s\n",res1 );
printf( "result2 %s\n",res2 );

result : res1 = "abc" res2="bc123". When using strcat the same.

Greetings,
Rick

[Mark A. Odell]

"Rick" <as******@hotma il.com> wrote in
news:3f******** *************** @news.xs4all.nl :

You answered faster than the light! I just tried it but the same problem
(see that other post where I explained the problem a little bit more).
Or maybe I am using the code wrong, this is what I did with strncat:

char *res1 = "";
char *res2 = "";
HEY! You have to allocate writable memory for these pointers.
strncat( res1, "a",1 );

NO! You cannot write to res1 or res2. Look up malloc() and use it, then
come back. The res1 pointer is pointing to non-writable memory at this
point. Some implemtations will allow this, others will crash. What if the
compiler put the null string that res1 points to in FLASH or PROM? How
could you write to that?

--
- Mark ->
--

[Andreas Kahari]

In article <3f************ ***********@new s.xs4all.nl>, Rick wrote:

You answered faster than the light! I just tried it but the same problem
(see that other post where I explained the problem a little bit more). Or
maybe I am using the code wrong, this is what I did with strncat:

char *res1 = "";
char *res2 = "";
strncat( res1, "a",1 );

You're not allowed to modify the contents of the string pointed
to by res1 or res2, the way you defined them. Both pointers
must point to a sufficiently large portion of allocated memory.

See here:

#include <stdio.h>
#include <string.h>

int
main(void)
{
char res1[10] = ""; /* Must be long enough */
char res2[10] = ""; /* Must be long enough */

/* strncat(res1, "abc", 3), but more verbose: */
strncat(res1, "a", 1);
strncat(res1, "b", 1);
strncat(res1, "c", 1);

/* strncat(res2, "123", 3), but more verbose: */
strncat(res2, "1", 1);
strncat(res2, "2", 1);
strncat(res2, "3", 1);

printf("res1 = '%s'\nres2 = '%s'\n", res1, res2);

return 0;
}

Result:
res1 = 'abc'
res2 = '123'

--
Andreas Kähäri

[Bjoern Pedersen]

"Rick" <as******@hotma il.com> writes:

Hi,
char *strAddChar(cha r *s1, char c){
char *s;
s = s1;
s = s + strlen(s1); // the position to write
*s = c; // this should be on the place where the null char
first
// was.
s++;
*s = 0; // add a new null string
return(s1);
} // strAddChar

/* this is how I use the function */
main(){
char *res1 = ""; // empty string 1 ^^^ ^^^^ res1 points to an (possibly)
non-writable array of 2 chars,
char *res2 = ""; // empty string 2
the same here...
strAddChar( res1, 'a' );

and here you invoke undefinde behaviour by writting to res1 ( and
writing past the end.

C-strings do not grow automagically, you would have to do this by
mallocing enough memory somewhere.

Björn

--
Bjoern Pedersen Lichtenbergstr. 1
Technische Universitaet Muenchen D-85747 Garching
ZWE Instrumentierun g FRM-II
Tel. + 49 89 289-14707 Fax -14666

[L.J. Buitinck]

Rick wrote:

If this is true, that would be quiete dangerous because I could overwrite a
big part of the program. Normally you can use arrays and malloc them but in
my case, I really don't know how big the result becomes. While building the
result string, I need at least 1 sub string for other stuff, that string has
the same problem, I can't calculate how big it becomes, it has to grow
dynamically. How to handle this?

use realloc(), which resizes malloc()'d buffers. e.g.,
char *s;

s = malloc(4);
if (s == NULL) {
/* ... */
}
strcpy(s, "foo");
s = realloc(s, 4+3);
if (s == NULL) {
free(s);
/* ... */
}
strcat(s, "bar");
if your system has them, consider using strlcpy()/strlcat().
http://www.openbsd.org/papers/strlcpy-paper.ps

--
Segui il tuo corso, e lascia dir le genti.

Lars

[Mark A. Odell]

Andreas Kahari <ak*******@free shell.org> wrote in
news:sl******** **************@ mx.freeshell.or g:

Andreas, let's be sure Rick sees the important change...

int
main(void)
{
char res1[10] = ""; /* Must be long enough */
char res2[10] = ""; /* Must be long enough */

^^^^

Rick, note that these are arrays of chars, not pointers to chars. That's
the key.

--
- Mark ->
--

[Rick]

Everybody thanks for helping I understand the problem. It's now working with
fixed sized arrays.

Greetings,
Rick