[C] Allocazione memoria dinamica di un vettore di caratteri

[jacopos81]

ho necessità di avere un vettore che contenga caratteri. pero siccome la quantità di caratteri è varibiale e puo oltrepassare quella del vettore ho necessità di usare l'allocazione dinamica, tramite la funzione realloc (anche forzato dal prof in questa scelta).

è corretta una cosa del genere?

int dim_iniziale,allocati,blocco,usati;

dim_iniziale=50; // dim iniziale della stringa
allocati=dim_iniziale; // mem allocata all'inizio
usati=0; // caratteri usati
blocco=50; // blocchi di memoria

char *array;
array= (char *)malloc(sizeof(char) * dim_iniziale);


poi andando a incrementare la variabile usati a ogni carattere aggiunto, quando usati supera allocati faccio:

if (usati > allocati) {
allocati = allocati + blocco;
array = (char *)realloc(array,allocati);
} if (usati > allocati) {
allocati = allocati + blocco;
array = (char *)realloc(array,allocati);
}

e quando ho finito

free(array);

secondo voi è corretto?

i miei dubbi sono sopratutto sulla malloc iniziale...come fareste voi?

[jacopos81]

array= (char *)malloc(sizeof(char) * dim_iniziale);

ho fatto qualche prova. il programma pare che funzioni sia con

array= (char *)malloc(sizeof(char) * dim_iniziale);

che con:

array= (char *)malloc(dim_iniziale);

quale dei due è corretto? se lo è uno dei due...

[andbin]

Originariamente inviato da jacopos81
ho fatto qualche prova. il programma pare che funzioni sia con

array= (char *)malloc(sizeof(char) * dim_iniziale);

che con:

array= (char *)malloc(dim_iniziale);

quale dei due è corretto? se lo è uno dei due...

Il primo!
Specificare la dimensione del singolo elemento (moltiplicata poi per una quantità) è sempre la cosa migliore, anche nel caso in cui sizeof(char) vale appunto 1.

[shodan]

if (usati > allocati) {
allocati = allocati + blocco;
array = (char *)realloc(array,allocati);
}

/* qui è scappato un paste di troppo immagino */

if (usati > allocati) {
allocati = allocati + blocco;
array = (char *)realloc(array,allocati);
}
/* ---- */

free(array);

Per sicurezza farei:
if (usati>=allocati) {
allocati = allocati+blocco+1
....
}
per avere la sicurezza che il pointer rimanga nel campo della validità ed eventualmente mettere il NULL alla fine.
Inoltre in teoria si dovrebbe sempre fare il controllo sul pointer ritornato dalla malloc, in teoria non è detto che l'allocazione riesca.
(Anche se era più probabile tempo fa con le macchine che avevano poca memoria)

[jacopos81]

grazie per i consigli

[LeleFT]

Altro consiglio: il linguaggio specificalo anche nel titolo, come indicato nel Regolamento, altimenti rischi la chiusura della discussione.


Ciao.

[jacopos81]

Originariamente inviato da LeleFT
Altro consiglio: il linguaggio specificalo anche nel titolo, come indicato nel Regolamento, altimenti rischi la chiusura della discussione.


Ciao.

azz.. vero, stavolta me l'ero scordato..

troppo tempo a programmare