[C] Acquisire stringhe da tastiera e stamparle

[xglobusx]

Cosa sto sbagliando?
Vorrei prendere 4 stringhe da tastiera e ristamparle.

#include <stdio.h>

int main(void){

char *vett[4];
int i;

	for(i=0; i<4; i++){
		printf("Inserisci l'elemento numero %d:", i);
		scanf("%s", vett[i]);
	}
	for(i=0; i<4; i++){
		printf("\nElemento %d = %s",i, vett[i]);
	}
}

Compila, ma nell'eseguire mi da questo errore:Segmentation fault (core dumped)

[Vincent]

non vedo alcuna allocazione di memoria... sarà forse questo il problema?

[andbin]

Originariamente inviato da xglobusx
Cosa sto sbagliando?
Vorrei prendere 4 stringhe da tastiera e ristamparle.

char *vett[4]; definisce solamente un array che contiene 4 puntatori a char.
Non definisce alcun spazio vero e proprio per i caratteri delle stringhe!
La soluzione più semplice è quella di allocare delle memoria con malloc() per ogni stringa.

[xglobusx]

Non credo sia necessario allocare della memoria.
Se io inizializzo il vettore con

char *vett[4] = {"Ciao", "Buongiorno", "Salve", "Arrivederci"};

non da alcun errore e stampa il contenuto del vettore correttamente.

[Vincent]

innanzi tutto non lo vedo inizializzato come dici tu...

poi secondo me puoi risolvere così:

#include <stdio.h>

#define DIM 4

int main(void){

char **vett;
int i;

    for(i=0; i<DIM; i++)
       vett[i] = malloc(sizeof(char));

    for(i=0; i<DIM; i++){
        printf("Inserisci l'elemento numero %d:", i);
        scanf("%s", vett[i]);
    }
    for(i=0; i<DIM; i++){
        printf("\nElemento %d = %s",i, vett[i]);
    }
    printf("\n\n");
} 

[MItaly]

Originariamente inviato da xglobusx
Non credo sia necessario allocare della memoria.
Se io inizializzo il vettore con

char *vett[4] = {"Ciao", "Buongiorno", "Salve", "Arrivederci"};

non da alcun errore e stampa il contenuto del vettore correttamente.

Certo, perché in questo caso il compilatore aggiunge le stringhe alla tabella delle stringhe e assegna i puntatori all'area di memoria da lui automaticamente allocata ai vari elementi del vettore di puntatori.

vett[i] = malloc(sizeof(char));

Cioè, per ciascun elemento tu allochi un singolo byte?
Semmai una soluzione (la più diffusa) è di allocare staticamente la memoria in questione; al posto di

char *vett[4];

metti

char vett[4][256];

.
A scanso di buffer overflow eventualmente modifica la scanf in questo modo:

scanf("%255s", vett[i]);

.

[oregon]

Originariamente inviato da xglobusx
Non credo sia necessario allocare della memoria.

E dove li memorizzi i caratteri che fanno parte delle stringhe?

Io invece penso che *sia necessario* allocare la memoria ... e il segmentation fault te lo ricorda ...

Originariamente inviato da xglobusx
Se io inizializzo il vettore con

char *vett[4] = {"Ciao", "Buongiorno", "Salve", "Arrivederci"};

non da alcun errore e stampa il contenuto del vettore correttamente.

Questo perche', in questo caso, e' il compilatore, staticamente, ad allocare la memoria necessaria per memorizzare le stringhe.

E c'e' differenza con l'allocazione *dinamica*, per la quale non sai in partenza quanto sia la memoria richiesta ...

[Vincent]

per stare sicuro al 100% fai l'allocazione dinamica come se fosse una matrice...

#include <stdio.h>

#define DIM 4
#define NUM 1

int main(void){

char **vett;
int i;


    vett=( char ** )malloc( NUM * sizeof(char *));
       for(i=0;i<DIM;i++)    {

            vett[i]=( char * )malloc( DIM * sizeof(char));

    }

    for(i=0; i<DIM; i++)
       vett[i] = malloc(sizeof(char));

    for(i=0; i<DIM; i++){
        printf("Inserisci l'elemento numero %d:", i);
        scanf("%s", vett[i]);
    }
    for(i=0; i<DIM; i++){
        printf("\nElemento %d = %s",i, vett[i]);
    }
    printf("\n\n");
} 

e vai alla grande, infatti nel primo caso se inserisci una stringa molto grande va in seg fault!

[andbin]

Originariamente inviato da Vincent

#include <stdio.h>

#define DIM 4
#define NUM 1

int main(void){

char **vett;
int i;


    vett=( char ** )malloc( NUM * sizeof(char *));
       for(i=0;i<DIM;i++)    {

            vett[i]=( char * )malloc( DIM * sizeof(char));

    }

    for(i=0; i<DIM; i++)
       vett[i] = malloc(sizeof(char));

    for(i=0; i<DIM; i++){
        printf("Inserisci l'elemento numero %d:", i);
        scanf("%s", vett[i]);
    }
    for(i=0; i<DIM; i++){
        printf("\nElemento %d = %s",i, vett[i]);
    }
    printf("\n\n");
} 

Un po' confuso come sorgente .... hai fatto un ciclo for (il secondo) che a mio avviso non serve.
Il primo array (quello dei puntatori) l'hai allocato di NUM elementi (invece di DIM) e il secondo di DIM caratteri (invece di NUM caratteri, che dovrebbe essere comunque abbastanza grande).

[xglobusx]

è vero bisogna allocare, grazie