Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

How can I create a dynamically sized array of structs?

I know how to create an array of structs but with a predefined size. However is there a way to create a dynamic array of structs such that the array could get bigger?

For example:

    typedef struct     {         char *str;     } words;      main()     {         words x[100]; // I do not want to use this, I want to dynamic increase the size of the array as data comes in.     } 

Is this possible?


I've researched this: words* array = (words*)malloc(sizeof(words) * 100);

I want to get rid of the 100 and store the data as it comes in. Thus if 76 fields of data comes in, I want to store 76 and not 100. I'm assuming that I don't know how much data is coming into my program. In the struct I defined above I could create the first "index" as:

    words* array = (words*)malloc(sizeof(words)); 

However I want to dynamically add elements to the array after. I hope I described the problem area clearly enough. The major challenge is to dynamically add a second field, at least that is the challenge for the moment.


I've made a little progress however:

    typedef struct {         char *str;     } words;      // Allocate first string.     words x = (words) malloc(sizeof(words));     x[0].str = "john";      // Allocate second string.     x=(words*) realloc(x, sizeof(words));     x[1].FirstName = "bob";      // printf second string.     printf("%s", x[1].str); --> This is working, it's printing out bob.      free(x); // Free up memory.      printf("%s", x[1].str); --> Not working since its still printing out BOB even though I freed up memory. What is wrong? 

I did some error checking and this is what I found. If after I free up memory for x I add the following:

    x=NULL; 

then if I try to print x I get an error which is what I want. So is it that the free function is not working, at least on my compiler? I'm using DevC??


Thanks, I understand now due to:

FirstName is a pointer to an array of char which is not being allocated by the malloc, only the pointer is being allocated and after you call free, it doesn't erase the memory, it just marks it as available on the heap to be over written later. – MattSmith

Update

I'm trying to modularize and put the creation of my array of structs in a function but nothing seems to work. I'm trying something very simple and I don't know what else to do. It's along the same lines as before, just another function, loaddata that is loading the data and outside the method I need to do some printing. How can I make it work? My code is as follows:

    # include <stdio.h>     # include <stdlib.h>     # include <string.h>     # include <ctype.h>      typedef struct     {         char *str1;         char *str2;     } words;      void LoadData(words *, int *);      main()     {         words *x;         int num;          LoadData(&x, &num);          printf("%s %s", x[0].str1, x[0].str2);         printf("%s %s", x[1].str1, x[1].str2);          getch();     }//      void LoadData(words *x, int * num)     {         x = (words*) malloc(sizeof(words));          x[0].str1 = "johnnie\0";         x[0].str2 = "krapson\0";          x = (words*) realloc(x, sizeof(words)*2);         x[1].str1 = "bob\0";         x[1].str2 = "marley\0";          *num=*num+1;     }// 

This simple test code is crashing and I have no idea why. Where is the bug?

like image 962
D. Rattansingh Avatar asked Nov 04 '08 04:11

D. Rattansingh


1 Answers

You've tagged this as C++ as well as C.

If you're using C++ things are a lot easier. The standard template library has a template called vector which allows you to dynamically build up a list of objects.

#include <stdio.h> #include <vector>  typedef std::vector<char*> words;  int main(int argc, char** argv) {          words myWords;          myWords.push_back("Hello");         myWords.push_back("World");          words::iterator iter;         for (iter = myWords.begin(); iter != myWords.end(); ++iter) {                 printf("%s ", *iter);         }          return 0; } 

If you're using C things are a lot harder, yes malloc, realloc and free are the tools to help you. You might want to consider using a linked list data structure instead. These are generally easier to grow but don't facilitate random access as easily.

#include <stdio.h> #include <stdlib.h>  typedef struct s_words {         char* str;         struct s_words* next; } words;  words* create_words(char* word) {         words* newWords = malloc(sizeof(words));         if (NULL != newWords){                 newWords->str = word;                 newWords->next = NULL;         }         return newWords; }  void delete_words(words* oldWords) {         if (NULL != oldWords->next) {                 delete_words(oldWords->next);         }         free(oldWords); }  words* add_word(words* wordList, char* word) {         words* newWords = create_words(word);         if (NULL != newWords) {                 newWords->next = wordList;         }         return newWords; }  int main(int argc, char** argv) {          words* myWords = create_words("Hello");         myWords = add_word(myWords, "World");          words* iter;         for (iter = myWords; NULL != iter; iter = iter->next) {                 printf("%s ", iter->str);         }         delete_words(myWords);         return 0; } 

Yikes, sorry for the worlds longest answer. So WRT to the "don't want to use a linked list comment":

#include <stdio.h>   #include <stdlib.h>  typedef struct {     char** words;     size_t nWords;     size_t size;     size_t block_size; } word_list;  word_list* create_word_list(size_t block_size) {     word_list* pWordList = malloc(sizeof(word_list));     if (NULL != pWordList) {         pWordList->nWords = 0;         pWordList->size = block_size;         pWordList->block_size = block_size;         pWordList->words = malloc(sizeof(char*)*block_size);         if (NULL == pWordList->words) {             free(pWordList);             return NULL;             }     }     return pWordList; }  void delete_word_list(word_list* pWordList) {     free(pWordList->words);     free(pWordList); }  int add_word_to_word_list(word_list* pWordList, char* word) {     size_t nWords = pWordList->nWords;     if (nWords >= pWordList->size) {         size_t newSize = pWordList->size + pWordList->block_size;         void* newWords = realloc(pWordList->words, sizeof(char*)*newSize);          if (NULL == newWords) {             return 0;         } else {                 pWordList->size = newSize;             pWordList->words = (char**)newWords;         }      }      pWordList->words[nWords] = word;     ++pWordList->nWords;       return 1; }  char** word_list_start(word_list* pWordList) {         return pWordList->words; }  char** word_list_end(word_list* pWordList) {         return &pWordList->words[pWordList->nWords]; }  int main(int argc, char** argv) {          word_list* myWords = create_word_list(2);         add_word_to_word_list(myWords, "Hello");         add_word_to_word_list(myWords, "World");         add_word_to_word_list(myWords, "Goodbye");          char** iter;         for (iter = word_list_start(myWords); iter != word_list_end(myWords); ++iter) {                 printf("%s ", *iter);         }          delete_word_list(myWords);          return 0; } 
like image 180
Tom Avatar answered Sep 16 '22 12:09

Tom