[C] Problemi malloc()

[danieletdg]

Salve gente .. dovrei creare un vettore di stringhe .. le stringhe vengono inserite da tastiera .. credo di dover usare malloc() ma ho qualche problema.

#include <stdio.h>
#include <stdlib.h>

int main()
{
    char *s[5];
    int i;
    int j;
    
   for(i=0; i<5; i++)
   {
    s[i]= malloc(5*sizeof(char *));
    printf("inserisci una stringa: ");
    scanf("%s" , &s[i]);
    
    }
    
    for(j=0; j<5; j++){
    
    printf("%s, " , (s+j));
    }
    
    
    return 0;
}

questo è l'output da terminale .. quando stampo le stringhe che inserisco .. alcune non sono uguali .. qual è secondo voi il problema .. vi ringraizo Daniele

macbook************$ ./puntatori
inserisci una stringa: d
inserisci una stringa: da
inserisci una stringa: dan
inserisci una stringa: dani
inserisci una stringa: danie
d, da, dan, danidanif, danif,

[LeleFT]

Regolamento...

punto 2: Indicare il linguaggio di programmazione nel titolo

punto 6:

Formattazione del codice sorgente.
Quando incollate una porzione di codice all'interno di un messaggio, utilizzate l'apposito tag

[CODE]
codice
[/CODE]

che può essere inserito automaticamente premendo il pulsante CODE: questo renderà il codice leggibile e facilmente comprensibile a tutti, agevolando gli utenti che desiderano aiutarvi.

Ho corretto io.


Ciao.

[Stoicenko]

nella malloc nel sizeof toglierei il char* e metterei char

la scanf vuole un puntatore e s[i] è già un puntatore no? non credo serva l'&

[Stoicenko]

allora, il char al posto del char* è giusto l'& va bene invece..

[danieletdg]

ragazzi .. il problema è sempre lo stesso .. anche se sostituisco char al posto di char * . L'output è sempre lo stesso .. non capisco perchè .. forse non allochiamo abbastanza memoria? L'& all'interno della scanf è fondamentale .. altrimenti non risolve il riferimento. E' molto strano, eppure non arrivo manco a 20 righe di codice vi ringrazio ancora .. daniele

[YuYevon]

I suggerimenti di Stoicenko sono corretti... com'è che non funziona? Al massimo mettici un cast "char *" nella chiamata alla malloc()...

#include <stdio.h>
#include <stdlib.h>

int main()
{
    char *s[5];
    int i, j;
    
   for(i=0; i<5; i++) {

    s[i]= (char *) malloc( 5 * sizeof(char) );
    printf("inserisci una stringa: ");
    scanf("%s" , s[i]);
    
    }
    
    for(j=0; j<5; j++){
    
    printf("%s, " , (s+j));
    }
    
    
    return 0;
}

Ovviamente tieni presente che se allochi spazio per 5 caratteri, non potrai immettere stringhe che ne abbiano più di 4 (il 5° è riservato al terminatore).

EDIT: anzi no, c'è un errore.

    printf("%s, " , (s+j));

sostituisci con

    printf("%s, " , *(s+j));

o con

    printf("%s, " , s[j]);

[danieletdg]

ragazzi non mi va ..

  printf("%s, " , s[j]);

o

 printf("%s, " , *(s+j));

mi da un errore di "segmentation fault". Il cast non credo sia il vero problema .. provate a farlo girare voi .. io uso un machintosh e gcc per compilare ... Sono convinto che l'errore è nel ciclo for .. la condizione di uscita dal ciclo. Perche' ho notato che il ciclo termina quando inserisco una parola di 5 lettere .. se inserisco sempre parole di 4 lettere .. il ciclo è infinito! Se invece inserisco solo dei singoli caratteri .. anzi fino a 3 .. l'output è giusto! oddio ..
grazie ancora per l'aiuto.

[YuYevon]

Io ho compilato esattamente questo codice:

#include <stdio.h>
#include <stdlib.h>

int main()
{
    char *s[5];
    int i, j;
    
   for(i=0; i<5; i++) {

    s[i]= (char *) malloc( 5 * sizeof(char) );
    printf("inserisci una stringa: ");
    scanf("%s" , s[i]);
    
    }
    
    for(j=0; j<5; j++){
    
    printf("%s, " , s[j]);
    }
    
    
    return 0;
}

con gcc 4.2.3 su Slackware e con questa esecuzione:

bash-3.1$ ./prova
inserisci una stringa: cane
inserisci una stringa: casa
inserisci una stringa: cosa
inserisci una stringa: tre
inserisci una stringa: due
cane, casa, cosa, tre, due,

non ho alcun problema.

Ti ricordo di non immettere stringhe più lunghe di 4 caratteri... a meno che non allochi più spazio.

[danieletdg]

Risolto .. grazie ragazzi .. il problema era il 5 nella malloc() .. allocavo spazio per stringhe inferiori ai 5 caratteri, l'avevo confuso con i 5 puntatori delle 5 stringhe. Grazie ancora .. gentilissimi!