How to make a dynamic array of strings in C?

Question

I need to create a separate function that will read the string, break it up into sentences, save them into a two-dimensional array and delete the same sentences. How to make it so that I could pass a two-dimensional array created in the reedAndSort function to the main function?

#include <stdio.h> #include <stdlib.h> #include <string.h> #include <ctype.h> int reedAndSort (int *numberOfStr, char ***text); int main(){ int i = 0; char **text; int numberOfStr = 0; reedAndSort (&numberOfStr, &text) ; } int reedAndSort (int *numberOfStr, char ***text){ *text = malloc(sizeof(char**)); char *string = malloc(sizeof(char*)); char symbol; int schet = 0; // счетчик для поиска одинаковых строк int j = 0; int i = 0; int lenStr = 1; while (symbol != '\n'){ while ((symbol != '.') && (symbol != '\n')){ symbol = getchar(); lenStr++; string = (char *)realloc(string, lenStr*sizeof(string)); string[lenStr - 2] = symbol; } symbol = getchar(); string[lenStr - 1] = '\0'; for (i = 0; i < *numberOfStr; i++){ if (strlen(string) == strlen((*text)[i])){ for(j = 0; j < strlen(string); j++){ if (tolower(string[j]) == tolower((*text)[i][j])) schet++; } if (schet == strlen(string)) break; else schet = 0; } } if (schet != strlen(string)){ *numberOfStr++; *text = (char**)realloc(*text, *numberOfStr*sizeof(char*)); (*text)[*numberOfStr - 1] = (char*)realloc(*text[*numberOfStr - 1], strlen(string)*sizeof(char)+1); strcpy((*text)[*numberOfStr - 1], string); } schet = 0; free(string); string = NULL; lenStr = 1; //for(i = 0; i < *numberOfStr; i++) printf("%s\n", *text[i]); } }

PS I'm just learning to write in C, for this I will be glad to any criticism.

Criticism: 1) it is better to first read the symbol, and then check it;
2) after entering the entire line, you read the extra character (so, on the second call, you lose the first character);
3) to search for duplicates case-insensitive, it is better to call strcasecmp() ;
5) (*text)[*numberOfStr - 1] = (char*)realloc(... does not make sense, you just need to save a pointer to the sentence there and then not free this memory ( strcpy(...) after that needed); 6) there is no return in the function ???

Accepted Answer · 2018-12-02T00:15:34

You return the accumulated text (an array of pointers to sentences) to main correctly, but I do not see two-dimensional arrays here.

There are errors in your code and it is (to put it mildly) not unambiguous ... The main (fatal) error is *numberOfStr++; . Here you increment the pointer, not the number to which it points (it will be correct - (*numberOfStr)++; ).

It seems to me that the whole function may look something like this:

 int reedAndSort (int *numberOfStr, char ***text) { *text = 0; char *str = 0; // текущее предложение int ls = 0, // его длина nt = 0, // количество предложений в text[] freestr, // признак, включено ли последнее предложение в text[] c; // прочитанный из stdin символ do { c = getchar(); // конечно, realloc для каждого символа жутко не эффективно, // отложим оптимизацию на потом... -) (str = realloc(str, ++ls + 1))[ls - 1] = (c == EOF) ? 0 : c; if (c == EOF || c == '\n' || c == '.') { // конец предложения // если пустое предложение или дубликат окажется последним, то надо будет освободить память freestr = 1; if (ls > 1) { str[ls] = 0; // а теперь сделаем накопленные символы сишной строкой int i = 0; while (i < nt) // ищем дубликат if (strcasecmp(str, (*text)[i++]) == 0) break; if (i == nt) { // это не пустое предложение и не дубликат (*text = realloc(*text, ++nt * sizeof(char *)))[nt - 1] = str; str = 0; // теперь тут будет выделяться новая память freestr = 0; // даже если это последнее предложение, то эту память не надо освобождать } } // для пустого предложения будет использована уже выделенная память ls = 0; } } while (c != '\n' && c != EOF); if (freestr) free(str); *numberOfStr = nt; // вернем последний символ ввода, // по нему можно определить конец файла // (если в дальнейшем предполагается читать более одной строки) return c; }

It is clear that the name of the function reedAndSort with an error and does not quite correctly reflect its essence (the sort has nothing to do with it ...)

How to make a dynamic array of strings in C?

1 answer 1

More articles: