Hello,
I am experiencing trouble adding data to character arrays:
The Program crashes when I execute it...Code:int main() { char d[32]; int i, i1; for(i = 0;i<5;i++){ for(i1 = 50;i1<82;i1++){ sprintf(d[i]+strlen(d[i]), "%c", (char)i1); } return 0; }
Please Help,
Thank you very much,
Daniel
What is it supposed to do?
There's no need to cast a char to a char... and strlen() takes a "string" not a char.
Why are you going from 50?Code:for(i1 = 50;i1<82;i1++){
Last edited by zacs7; 05-20-2007 at 04:47 AM.
First of all, you need to learn how to indent properly do you can see what is going on.
After that, you're missing a matching '}', and you're using wrong types for functions, so I don't even know how this compiled let alone ran and crashed.
The important line is this:
What you are doing is totally wrong. I don't think you really know what sprintf() does. I would suggesting reading up on char arrays (ie. C strings), and read the man pages (or google) for information on sprintf().Code:sprintf(d[i]+strlen(d[i]), "%c", (char)i1);
I could go into detail about why you're crashing, but it's pretty ridiculous. d[i]+strlen(d[i]) is not a char *, let alone a char * to point to a buffer properly. sprintf() is a function to write a formatted string to a buffer. If you're just trying to add a char inside of it, then just manually do it, with something like this:
Code:d[i] = (char)i1;
I need to point to the end of the string.
So as to add data, at the end
if you assign something to index i - you KNOW that this is the end of string... you do not need sprintf for this knowledge
All problems in computer science can be solved by another level of indirection,
except for the problem of too many layers of indirection.
– David J. Wheeler
yes yes, but what ARE you tring to do?
