Copy every single character from a char into another char.

Hey guys,

I have some problems with copying a single character from one char into another. How do I do that? I'm quite the beginner, so I would appreciate your help.

Here is the function I am using

Code:

---

void
text_col (char line[])
{
    char line1[1];
    int l, ii;

    l = strlen(line);

    //looping through the word - each character
    for (ii=0;ii<l;ii++){

    //This is my attempt to copy a character
    strcpy (line1[0], line[ii]);

    }
   


}

---

I am trying to copy in a loop every single character from char line into char line1. That is not working.
Can you tell me why and how I can fix it?

Thank you very much!

Quote:

---

Originally Posted by Roscoe Araujo

Hey guys,

I have some problems with copying a single character from one char into another. How do I do that? I'm quite the beginner, so I would qppreciate your help.

Here is the function I am using

Code:

---

void
text_col (SDL_Surface *monitor, TTF_Font *font, int color, char line[], int x, int y, int time)
{
    char line1[1];
    int l, ii;
 
    //Buchstaben werden der Reihenfolge nach gemappt
    for (ii=0;ii<l;ii++){

    //Hier soll jeder einzelne Buchstabe aus line in line1 kopiert werden
    strcpy (line1[0], line[ii]);

    }
   
}

---

I am trying to copy in a loop every single character from char line into char line1. That is not working.
Can you tell me why and how I can fix it?

Thank you very much!

---

It looks like you are confusing char with arrays of char and NUL terminated array of chars (strings).

In your example, the line[] argument denotes an array of chars of unknown size . The line1[1] local variable is defined as an array having room for exactly 1 char (so it could just be defined as a single char variable).

Moreover, in your loop you are using l uninitialized, which means it can contain pretty much anything in there (thus making it totally unreliable for controlling the loop iterations).

Al in all, it seems like you should backup and study the core C language, instead of diving directly into SDL.

In any case, if line[] is a valid cstring (that means, it contains a terminating '\0' character) the you can use...

Code:

---

size_t len = strlen(line);

---

in order to determine line's current length. Then you could use len for allocating enough memory for line1 and copy line's contents into it...

Code:

---

char *line1 = malloc(len+1);
if ( NULL == line1) {
    /* malloc() failed, handle the error here */
}
else {
    /* do whatever you want to do with line1 here, and then... */
    free( line1 );    /* free memory allocated for line1 */
}

---

EDIT:

The quote I used is the original posted by OP, before he/she editing the original post (thus the SDL reference in my answer ;-) ).

Thank you for your reply. I corrected my code (I wanted to cut it down to the central information, l contains the length of line).
How do I exactly only the first character from line into line1?

This will work:

Code:

---

*(&line1[0]) = line[ii];

---

I have to ask, why does the destination need to be a single-element array rather than just a char variable?

You also need to make sure the "l" variable is initialized to the length of "line".

Because I only need a single character in line1.

Quote:

---

Originally Posted by Roscoe Araujo

Thank you for your reply. I corrected my code (I wanted to cut it down to the central information, l contains the length of line).
How do I exactly only the first character from line into line1?

---

If you want line1 to be just a char with just the 1st character of line, then you can...

Code:

---

char line1 = 0;
if ( line )
  line1 = *line;    /* or: line1 = line[0] */

---

Quote:

---

Originally Posted by Roscoe Araujo

Because I only need a single character in line1.

---

So, you want a loop traversing all characters of line[] and assign them one by one to line1, at each iteration?
If so, then perhaps something like the following will suit you...

Code:

---

void text_col( char line[] )
{
    char line1 = 0;
    int i;

    for (i=0; '\0' != line[i]; i++) {
        line1 = line[i];
        ...
    }
}

---

Thank you very much :-) I tried something similar, it did not work, that is why I am looking for helo in this forum. I will try it again tomorrow mornng,

I have found a solution (of course with some help :redface:):

Code:

---

const char *p = NULL;
char c;
char a[2] = { '\0', '\0' };

c = line[ii];      putc(c, stdout);    putc(' ', stdout);
a[0] = line[ii];    fputs(a, stdout);  putc(' ', stdout);
p = &line[ii];      fputs(p, stdout);  putc('\n', stdout);

---

If I add this in the loop of my function, it works.
This seems to have done the magic.

Thanks for your help :D