[C++] Acquisizione da file testuale ed errore "segmentation fault"

[DarkRadeon]

praticamente mi sto preparando per un esame stupido, che però è assai fastidioso visto che un sacco di cose del linguaggio NON posso assolutamente usarle(break, un sacco di librerie c++ standard, classi, niente ide -.-)..

in questo caso è una stupidissima e noiosissima agenda/rubrica, su file sorgente unico (-_-). in pratica mi trovo a modificare un codice piuttosto semplice, anche se è una rottura trattarlo con emacs (gusti sono gusti ok, ma valli a capire i docenti .-. )..

in pratica devo aggiungere una serie di funzioni o procedure (almeno qua in questo caso posso scegliere!) che acquisisca da un qualsivoglia file gli elementi che altrimenti verrebbero inseriti tramite input da terminale con cin..

il codice è piuttosto lungo, pertanto posto solo le aree interessate
questa è la procedura che sto tentando di creare con i pochi elementi che posso usare (iostream, fstream e c-strings, niente strings c++)

void file_acquire(int argc, char* argv[], persona& p, gruppo& g) {
  fstream myin;
  if(argc != 2)
    cerr << "Impossibile acquisire da file. Utilizzo: ./a.out <sourcefile>\n";
  else {
    myin.open(argv[1], ios::in);
    if(myin.fail())
      cerr << "Errore, il file " << argv[1] << " non trovato.\n";
    else {
      while(!myin.eof()) {
	myin >> p.nome;
	myin >> p.cognome;
	myin >> p.indirizzo.via;
	myin >> p.indirizzo.civico;
	myin >> p.indirizzo.comune;
	myin >> p.indirizzo.cap;
	myin >> p.indirizzo.prov;
	inserisci_persona_gruppo(p, g);
      }
    }
  }
}

so che non ci sono controlli in caso il file sia sbagliato, non li ho messi perché non sono richiesti ed il file di input viene scritto ad hoc.

queste sono le strutture e le costanti definite

const int N_MAX_PERSONE = 10;
const int N_MAX_NOME = 20;
const int N_MAX_COGNOME = 30;

struct address {
  char via[N_MAX_NOME];
  int civico;
  char comune[N_MAX_NOME];
  int cap;
  char prov[3];
};

struct persona {
  char nome[N_MAX_NOME];
  char cognome[N_MAX_COGNOME];
  address indirizzo;
};

struct gruppo {
  int npers;
  persona* nomi[N_MAX_PERSONE];
  persona* cognomi[N_MAX_PERSONE];
};

per compilare compila, ma una volta che eseguo in terminale (./a.out input), se faccio partire la procedura che acquisice da file va in loop infinito e poi da segmentation fault dopo un po' di tempo. Non capisco il perché, anche perché non so come dare il comando (./a.out input) al debugger (potrò usare solo DDD), e quindi se eseguo il codice in ddd la condizione

if(argc != 2)
    cerr << "Impossibile acquisire da file. Utilizzo: ./a.out <sourcefile>\n";

diventa vera.


il file input è scritto nel seguente modo

nome
cognome
via
numero civico
comune
cap
provincia
nome2
cognome2
via2
numero civico2
comune2
cap2
provincia2
...
e via così

in modo che ogni riga abbia un solo campo testuale che termina con una newline.

sembra una cosa alquanto semplice e banale, ma con ste maledette limitazioni, diventa alquanto snervante..

[shodan]

Mostra anche il resto del codice.

[oregon]

Ovvero, il main e la funzione inserisci_persona_gruppo

[DarkRadeon]

eccole qua.. la formattazione è leggermente diversa (è il file originale) ma sono identici nel resto

int main ()
{
  persona * pos;
  char cognome[NmaxCognome];
  char nome[NmaxNome];
  persona p;
  gruppo g;
  char s[2];

  inizializza_gruppo(g);   
  do {    
    cout << endl;
    cout << " i: inserisci\n";
    cout << " f: inserisci da file\n";
    cout << " n: stampa per ordine di nome\n";
    cout << " c: stampa per ordine di cognome\n";
    cout << " o: cerca per nome\n";
    cout << " g: cerca per cognome\n";
    cout << " e: esci\n";
    cout << "\nInserisci Scelta: ";
    cin >> s;

    switch (s[0]) {
    case 'i': leggi_persona(p); 
      inserisci_persona_gruppo(p,g);
      break;
    case 'f': file_acquire(argc, argv, p, g); break;
    case 'n': stampa_gruppo_per_nome (g); break;
    case 'c': stampa_gruppo_per_cognome (g); break;
    case 'o': leggi_nome(nome);
      pos=cerca_per_nome(nome,g); 
      if (pos==NULL)
	cout << "Non trovato\n";
      else {
	cout << "Trovato: "  << endl;
	stampa_persona(*pos);
      }
      break;
    case 'g': leggi_cognome(cognome);
      pos=cerca_per_cognome(cognome,g); 
      if (pos==NULL)
	cout << "Non trovato\n";
      else {
	cout << "Trovato: "  << endl;
	stampa_persona(*pos);
      }
      break;
    case 'e': break;
    default: cout << "scelta non capita\n";
    }
  }
  while ((s[0]=='i' || s[0]=='n' || s[0]=='c' || s[0]=='o' || s[0]=='g'));
}

void inserisci_persona_gruppo (persona & p, gruppo & g) 
{
  persona * tmp;
  tmp = new(persona);
  *tmp = p;
  g.nomi[g.npers] = tmp;
  g.cognomi[g.npers] = tmp;
  g.npers++;
  ordina_per_nome(g.nomi,g.npers);
  ordina_per_cognome(g.cognomi,g.npers);
}

[oregon]

E se l'errore è nelle

ordina_per_nome
ordina_per_cognome

come lo possiamo vedere?

P.S. A proposito ... che senso ha chiamare due funzioni che ordinano per campi diversi ?

EDIT: sei fortunato ... ... il problema è in

g.nomi[g.npers]

in cui g.npers non ha un valore valido in quanto non è inizializzato in precedenza.

[DarkRadeon]

le due funzioni sono dei semplici sort(bubble, anche se volevo sostituirle una volta finito tutto con quicksort) giusto per metterli, che vanno ad agire su

  persona* nomi[N_MAX_PERSONE];
  persona* cognomi[N_MAX_PERSONE];

che altro non sono che degli arrai di puntatori, due tipi eprché poi c'è la possibilità di stampare l'intera lista della rubrica sia per ordine crescente di nomi sia per ordine crescente di cognomi.
i due sort funzioni funzionano benissimo con l'input da terminale, e non le ho toccate.

[DarkRadeon]

mmmm compilato ora con visual studio.. da determinale se lancio la funzione per acquisire da file semplciemente termina il programma

[oregon]

Hai letto quello che ti ho scritto su

g.nomi[g.npers]

?

[DarkRadeon]

sì, npers viene inizializzato in inizializza_gruppo(g);

void inizializza_gruppo(gruppo& g) {
  g.npers = 0;
}

[oregon]

Originariamente inviato da DarkRadeon
sì, npers viene inizializzato in inizializza_gruppo(g);

void inizializza_gruppo(gruppo& g) {
  g.npers = 0;
}

Beh ... se non posti TUTTO il codice, possiamo stare anni ...

A questo punto, è possibile che i nomi del file siano più di 10 ?