getc() segmentation fault

[targzeta]

Non ho capito se ti riferisci a me e se l'elemento in più è nel vettore v. Comunque, se è così, l'elemento che dici apparentemente inutile e in più, in realtà è utile e non è in più.
Se dichiari un array lungo v[MAX] allora l'array contiene MAX celle da v[0] a v[MAX-1]. Se dichiari un array lungo v[MAX-1], come hai fatto tu, allora hai MAX-1 celle da v[0] a v[MAX-2].
Generalmente si usa la prima forma e quindi ho modificato quanto fatto da te senza badare se lo usavi bene o male. Diciamo che l'istinto mi suggeriva che l'errore era nella gestione dell'array v.

Per quanto riguarda la funzione getc, se gestisci male i puntatori allora l'errore può avvenire ovunque nel codice. Ad esempio potresti aver sovrascitto la struttura f che puntava al file aperto, e quindi di conseguenza alla funzione getc non passavi più ciò che pensavi di passare. E' solo un ipotesi, ma è la ragione per cui mi sono concentrato sui puntatori.

Spina

[anycolouryoulike]

Non ho capito se ti riferisci a me e se l'elemento in più è nel vettore v. Comunque, se è così, l'elemento che dici apparentemente inutile e in più, in realtà è utile e non è in più.
Se dichiari un array lungo v[MAX] allora l'array contiene MAX celle da v[0] a v[MAX-1]. Se dichiari un array lungo v[MAX-1], come hai fatto tu, allora hai MAX-1 celle da v[0] a v[MAX-2].
Generalmente si usa la prima forma e quindi ho modificato quanto fatto da te senza badare se lo usavi bene o male. Diciamo che l'istinto mi suggeriva che l'errore era nella gestione dell'array v.

Per quanto riguarda la funzione getc, se gestisci male i puntatori allora l'errore può avvenire ovunque nel codice. Ad esempio potresti aver sovrascitto la struttura f che puntava al file aperto, e quindi di conseguenza alla funzione getc non passavi più ciò che pensavi di passare. E' solo un ipotesi, ma è la ragione per cui mi sono concentrato sui puntatori.

Spina

Pensa che ero convinto che v[MAX] significasse un vettore di MAX + 1 elementi!
Grazie ora è tutto chiaro.

Codice: Seleziona tutto

char *v[MAX]

Grazie, era proprio questo l'errore!
Non ci avevo neanche fatto caso...

Grazie a tutti!

[robbybby]

v[MAX]
ma dove posizioni lo \0 finale?

v[MAX + 1];

Mario, v e' usato come un array di stringhe, non come una stringa in se' (o puntatore a caratteri che dir si voglia).
v e' un puntatore a puntatori a caratteri (char **) e non un puntatore a caratteri (char*). Infatti e' dichiarato come

Codice: Seleziona tutto

char *v[MAX]

, e non come

Codice: Seleziona tutto

char v[MAX+1]

.
Lui, dentro ogni elemento, ci mette una stringa intera, con gia' il suo NULL terminator, quindi l'ultimo elemento non lo usa proprio.
E' parso strano anche a me, appeno ho letto il codice, ma lo ha fatto cosi'.

[Mario Vanoni]

v[MAX]
ma dove posizioni lo \0 finale?

v[MAX + 1];

Mario, v e' usato come un array di stringhe, non come una stringa in se' (o puntatore a caratteri che dir si voglia).
v e' un puntatore a puntatori a caratteri (char **) e non un puntatore a caratteri (char*). Infatti e' dichiarato come

Codice: Seleziona tutto

char *v[MAX]

, e non come

Codice: Seleziona tutto

char v[MAX+1]

.
Lui, dentro ogni elemento, ci mette una stringa intera, con gia' il suo NULL terminator, quindi l'ultimo elemento non lo usa proprio.
E' parso strano anche a me, appeno ho letto il codice, ma lo ha fatto cosi'.

Scusa Roberto, deformazione mentale di vecchia data.

Come detto in un post precedente,
metto sempre una sentinella,
non si sa mai ...

[robbybby]

@Mario: la sentinella la metto sempre anch'io, e inizializzo sempre tutto. E spargo il codice di assert(): mi hanno salvato un sacco di volte.

[anycolouryoulike]

Lui, dentro ogni elemento, ci mette una stringa intera, con gia' il suo NULL terminator, quindi l'ultimo elemento non lo usa proprio.
E' parso strano anche a me, appeno ho letto il codice, ma lo ha fatto cosi'.

Se c'è un metodo migliore son ben lieto di conoscerlo!
Come ho già detto, non sono per niente esperto di C, e per me ogni consiglio è prezioso.

[robbybby]

Ho riletto meglio il tuo codice.
La scelta di utilizzare un array di puntatori a stringhe e' piu' che lecita.
E' il resto della struttura del programma che, non va.
Prima di tutto evita tutti quei break e continue. Rendono il programma difficilmente leggibile, oltre che a terzi, anche a te, se lo riprendi in mano fra qualche mese. Inoltre lo rendono anche difficilemte manutenibile. Tieni presente che, nel mondo reale, un programma e' soggetto a continue modifiche, per variazioni di specifichem o ampliamenti. Se ti abitui subito a scrivere programma facilemente manutenibili, ne hai solo guadagno. Inoltre devono essere leggibili anche da altri, perche' a un certo programma ci si lavora in piu' persone.

Riorganizza il programma senza continue e break, e che la programmazione strutturata sia con te.

Ah, devi anche mettere un controllo sulla quantita' di caratteri letti, altrimenti s puo' andare in overflow, con risultati imprevedibili (o, meglio, facilmente prevedibili).