Thread: Help - String input header

What I want to do is to get a string from input without having to guess at the size of the array needed beforehand, and be able to concatenate a new input string on the end of it, something like this:

Code:

char * str = instring();  //get string from input allocating just enough space for it
incat(str);                    //concatenate a new string to the end of str

My header file so far:

Code:

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

char * instring(){
    char * str = (char *) malloc(sizeof(char));
    str[0] = '\0';

    int c;		//char to be put into 
    int i = 0;		//index of string to store

    while((c = getchar()) != '\n'){
	str[i] = c;
	str = (char *)realloc(str, (i+2) * sizeof(char));
	str[i+1]='\0';

	i++;

    }
    return str;
}



void incat(char * str1){
	char * instr = instring();
	int newlength = strlen(str1) + strlen(instr)

	str1 = (char *)realloc(str1, (newlength + 1) * sizeof(char));

	strcat(str1, instr);
}

instring seems to work all the time, but incat is not right. It seems to work for shorter strings, but strangness occurs on longer ones. Any help would be appreciated.

Originally Posted by MacGyver

Are you sure you've giving us the code you're working with? 2nd line in incat() is lacking a semicolon.

After I posted the code I saw that a bunch of extra space was added so I edited it out, but I must have accidentally gotten a semicolon.

zacs7- thanks for the tips. I realize the pointer returned by realloc could point to a different location than str originally pointed to, but wouldn't the new address be assigned to the str pointer in the code I had? Maybe it's a stupid question but I don't quite get it.

No function can change the value of an input parameter for good. If you want to be able to change a value (such as str), you need to pass in a pointer to that value. (Yes, str is a pointer, but it is not a pointer to the thing you want to change.)

Originally Posted by quzah

Pointers store values.
You can't change the value of something you pass to a function, unless you instead pass its address.
Thus, to change the value of a pointer (read: what it's actually pointing at, not the value of what is being pointed at), you need to pass the address of that pointer.

Consider:

To change an int's value, you pass its address. This way you can make it keep the value changes you do inside the function.
To change a pointer's value, you pass its address.

So, to make a pointer point elsewhere, you need a pointer to a pointer.


Quzah.

Ahh, thats right, I get it now. Thanks.

I changed incat() so that it returns char * so I can do something like

Code:

str = incat(str);

That works fine if str was initialized by instring(), but something like

Code:

char * str = "Hello";
str = incat(str);

won't work. Why is that?


And a couple quick questions about zacs7s' post.

Originally Posted by zacs7

Some tips:
1. sizeof(char) is always 1. So just omit it.

It wouldn't be 2 if you're working with unicode?

Originally Posted by zacs7

5. Don't cast malloc()/realloc()/calloc() in C.

The examples I've seen online and in books have used a cast, if I remember correctly the reason my c book gave was that it isn't required in c but is in c++ so you might as well put it in in case you bring it to c++ at some point. Why shouldn't you cast it in C?

If you don't include stdlib.h, wouldn't the code just refuse to compile and say something like

"undefined implicit reference to builtin function malloc()" ?