Vettore di variabili strutturate in C

[mic85rm]

Scrivere una funzione C che inizializza l'elemento i-esimo di un vettore di variabili strutturate di tipo

struct item {
int val;
char stringa[100];
};


con una stringa e un valore intero.

la mia soluzione è questa :

struct item *inizializza(int valore,char string[100])
{
 struct item *inizia;
 inizia=(struct item *)malloc(sizeof(struct item));
 if(inizia==null)
 {
  return 0;
 }
 inizia->val=valore;
 inizia->stringa=string;
 return inizia;
}

è corretta?o meglio ho interpretato bene quello che l'esercizio mi chiede?

[clockover]

Totalmente sbagliate un sacco di cose... la prima cosa che hai sbagliato è che non hai indentato il codice... c'è un pulsantino qui sopra oppure scrivi CODE tra parentesi quadre, codice, /CODE sempre tra quadre

poi

struct item inizializza(...)

vuol dire che devi ritornare una struct item, non un intero cioè

return 0;

e non un puntatore alla struct

return inizia;

quindi ti consiglio di cambiare la tua funzione in

struct item * inizializza(...)

se fai una qualunque operazione dopo il return, quella qualunque operazione non verrà mai eseguita

return inizia;
free(inizia);

intanto aggiusta tutto questo poi parliamo del resto

[mic85rm]

aggiornato

[clockover]

E ti funziona?

[mic85rm]

purtroppo non ho un compilatore c...

[AngeL)]

Originariamente inviato da mic85rm

purtroppo non ho un compilatore c...

http://www.google.com/search?q=mingw se sei su win, altrimenti su linux hai già il gcc.

per compilare dai

gcc nomefile

al terminale.

[Supdario]

 if(inizia==null)
 {
  return 0;
 }

NULL va scritto in maiuscolo, inoltre ti consiglierei di returnare NULL e non 0 (è la stessa cosa, ma si tratta di scrivere il codice in modo più rigoroso), quindi:

if(inizia == NULL)
 {
  return NULL;
 }

Poi, questo non lo puoi fare:

inizia->stringa=string;

Per copiare una stringa devi servirti dell'apposita funzione "strcpy", o in questo caso dato che la tua stringa è allocata nello stack ed ha una dimensione massima di 100 caratteri ti conviene usare strncpy:

strncpy(inizia->stringa, string, 100);

[mic85rm]

il compilatore c non ce l'ho qui a lavoro

grazie a tutti per le risposte comunque ora posto la soluzione corretta dal problema posto sopra...

typedef struct item item; /* così non devo scrivere tutte le volte struct */
struct item {
    int val;
    char stringa[100];
};

/* ... */

void inizializza(item arr[], int i, int valore, char string[100])
{
    /* manca la verifica degli argomenti... */
    arr[i].val = valore;
    strcpy(arr[i].stringa, string); /* non puoi fare l'assegnazione in quel modo per le stringhe e in generale gli array.. */
}