How do I add char array to another array?

Code:

---

const void *convToChar(const char *file_path, unsigned int char_length, unsigned int line_length){
    FILE *file;
    char f_text[char_length + 1];
    char text[char_length + 1];
    file = fopen(file_path, "r");


    for(unsigned int i = 0; i < line_length; i++){
        fgets(f_text, char_length + 1, file);
        text += f_text;
    }


    const char *p_text = text;
    fclose(file);


    printf("%s", text);


    return p_text;
}

---

Above code is for OpenGL shader source function. The shader file needs to be converted from shader file to a char array.



This is where i'm stuck,

Code:

---

    for(unsigned int i = 0; i < line_length; i++){
        fgets(f_text, char_length + 1, file);
        text += f_text;
    }

---

text += f_text;

It says "invalid operands to binary + (have 'char[(sizetype)(char_length + 1u)]' and 'char *')"


fgets gives me the current line of the file but I need a function that stores my whole file into a big char array. Is there a function or a way to solve my problem?

You can either read into the destination array directly, or copy from the array that stores the input into the destination array.

The former would be more efficient but the latter would be easier to handle if you want to recover from a read failure.

That said, you have two problems:

• f_text and text have the same number of chars, so they can store the same maximum amount of data. Therefore, if text already has a portion in use, and f_text is fully used, you cannot fully append the contents of f_text to the contents of text without writing out of bounds.
• At the end, you return p_text. p_text points to the start of text, but text is a local array. Therefore, this means that you are returning a pointer to a local non-static object. This is problematic because by the time control returns to the caller, this object no longer exists.

Since you "need a function that stores (your) whole file into a big char array", perhaps you should consider if using dynamic memory allocation with malloc/realloc/free would be appropriate here.

I think I almost got it but I have a small glitch in my program.

Code:

---

int main(){
    char *sentences = convToChar("tfile.txt", 96, 15);
    printf("%s", sentences);


    return 0;
}


char *convToChar(const char *file_path, unsigned int char_length, unsigned int line_length){
    FILE *file;
    char f_text[char_length + 1];
    //char text[char_length + 1];
    char *text = malloc(sizeof(char) * (char_length + 1));
    file = fopen(file_path, "r");


    for(unsigned int i = 0; i < line_length; i++){
        fgets(f_text, char_length + 1, file);
        strcat(text, f_text);
    }


    fclose(file);


    return text;
}

---

When I run this program, it prints out my text file but with the weird symbols like below.

"╪%UHello!"


But when I use printf("%s", text); in my convToChar,

Code:

---

//stuffs
//stuffs
    fclose(file);

        printf("%s", text);


    return text;
}

---

This prints out the actual text file...


So I think there's something wrong in the main,

Code:

---

char *sentences = convToChar("tfile.txt", 96, 15);
    printf("%s", sentences);

---

Am I doing something wrong?

Is the shader file in a binary format or text? At the moment you are treating your test file content as text, but if that is not the case for the actual file, then using fgets to read and printf with %s to write would be wrong.

There are other things to note:

• Remember to include the appropriate headers and to declare convToChar before calling it in main.
• Where did you get the magic numbers 96 and 15 from? What happens if a line is longer than expected, or if the entire file has more characters than expected?
• Although you used malloc, you did not use realloc to allocate more space when needed, and then you did not call free in main. You also did not check that malloc did not return a null pointer.
• You should check that fopen did not return a null pointer.
• Although strcat could work if your file format is text, when run in a loop like this it is inefficient. You should keep track of where the end of the string is and concatenate by copying directly starting from that point.
• sizeof(char) == 1 is always true.

By the way, note that this declares a variable length array:

Code:

---

char f_text[char_length + 1];

---

It is not wrong, but there's the caveat that the variable length array feature only became standard C in C99, then it became optional in C11.

As laserlight suggest your best option is to copy the line read direct onto destination array. However you may end up with a problem where the destination array wont have enough memory to hold all of your text in it. One way to resolve this would be to open the file and then calculate the size of the file using fseek until the end of the file. Having calculated the size. Allocate enough memory using malloc and use fgets to read text direct to the destination array.

~H

Also posted here.

JIm

Quote:

---

Originally Posted by yj1214

Code:

---

char *convToChar(const char *file_path, unsigned int char_length, unsigned int line_length){
    FILE *file;
    char f_text[char_length + 1];
    //char text[char_length + 1];
    char *text = malloc(sizeof(char) * (char_length + 1));
    file = fopen(file_path, "r");


    for(unsigned int i = 0; i < line_length; i++){
        fgets(f_text, char_length + 1, file);
        strcat(text, f_text);
    }


    fclose(file);


    return text;
}

---

---

The contents of the memory pointed to by text is uninitialized. You need to at least set the first element to a null character before you use it:

Code:

---

text[0] = '\0';

---

That line ensures that it's a valid string, which strcat expects.