[C++] struct

[d3v1lx]

Ciao a tutti, sono nuovo della programmazione in c++ e mi sono imbattuto in un problema che non riesco a risolvere.


#include <iostream>
using namespace std;

struct persona
{
char name[10];
int age;
};

struct impiegato
{
persona anagrafica;
float wage;
float hour;
};

void read_info(impiegato impiegati);
void write_info(impiegato impiegati);

int main ()
{
int n;
cout << "Inserire il numero di impiegati" << endl;
cin >> n;
impiegato impiegati[n];
for (int i=0; i<n; ++i)
{
read_info(impiegati[i]);
}
for (int j=0; j<n; ++j)
{
write_info(impiegati);
}
return 0;
}

void read_info(impiegato impiegati)
{
cout << "Inserire il nome" << endl;
cin >> impiegati.anagrafica.name;
cout << "Inserire l'eta'" << endl;
cin >> impiegati.anagrafica.age;
}

void write_info(impiegato impiegati)
{
cout << endl;
cout << "Nome: " << impiegati.anagrafica.name << endl;
cout << "Eta': " << impiegati.anagrafica.age << endl;
}


Creato un'array di impiegati, devo inserirne il nome e l'eta', ho creato due funzioni per fare questo, il problema e' che l'output non corrisponde con l'input.


Inserire il numero di impiegati
1
Inserire il nome //input inizio
pincopallino
Inserire l'eta'
23 //input fine

Nome: PLԑ� //output
Eta': 0


Non capisco il perche' di questo output, potete aiutarmi per favore.
Il programma viene compilato senza errorri e senza warning.
Grazie

[Pixel452]

Devi usare i puntatori, altrimenti quando passi la tua variabile ad una funzione questa viene COPIATA, quindi in realtà stai modificando una variabile che non è quella che hai nel main ma un altra che risiede in un altra zona della memoria.


void read_info(impiegato *impiegati);
void write_info(impiegato *impiegati);

int main ()
{
int n;
cout << "Inserire il numero di impiegati" << endl;
cin >> n;

impiegato *impiegati = new impiegato[n];

for (int i=0; i<n; ++i)
read_info(&impiegati[i]);

for (int j=0; j<n; ++j)
write_info(&impiegati[j]);

return 0;
}

void read_info(impiegato *impiegati)
{
cout << "Inserire il nome" << endl;
cin >> impiegati->anagrafica.name;
cout << "Inserire l'eta'" << endl;
cin >> impiegati->anagrafica.age;
}

void write_info(impiegato *impiegati)
{
cout << endl;
cout << "Nome: " << impiegati->anagrafica.name << endl;
cout << "Eta': " << impiegati->anagrafica.age << endl;
}

[WarDuck]

Oppure visto che stai usando C++ (anche se in realtà di C++ non hai usato nulla :D ) puoi usare il passaggio per riferimento:

void read_info(impiegato& impiegati)

[marco.r]

Il problema e' che usi stringhe di dimensione fissa (10 caratteri).
Se cerchi di inserire un nome piu' lungo, i caratteri in piu' "sforano" l'array e, per come e' stato dichiarata la struct, vanno a sovrascrivere le locazioni di memoria relative ai campi successivi.
O usi le string del c++, oppure limiti la lettura ai primi 10 caratteri .