[C++] Ma fseek è una funzione sicura?

[giorgiogio48]

Ciao a tutti,
ho cercato di usare la funzione fseek nel mio codice. Questo è il codice:

char sLine[3];
FILE *fpt;
if ((fpt = fopen("file.txt", "r")) == NULL) {
    printf("Coudln't access file.txt.\n");
    exit(0);
}
fseek(fpt, -2, SEEK_END);
    

fgets(sLine, 3, fpt);

ma gli ultimi due caratteri del file non sempre vengono trovati. Come mai?
Grazie.

P.S.
Come mai non funzionano i form per fomattare il testo?

[Hysoka]

e chi ti dice che gli ultimi 3 caratteri non siano caratteri non-visibili?
Per esempio, se usi windows e hai il carattere di ritorno a capo, che è due caratteri \r\n, più il terminatore di riga....
io consiglierei di ciclare, dall'ultimo carattere, fino a quando non trova 2 caratteri visibili, o arriva all'inzio del file

[giorgiogio48]

Ciao Hysoka,
grazie per la risposta. Sono un pò nuovo di c++ ma so cosa è un ciclazione. Ma non saprei però come farla. Io infatti non devo trovare un carattere, ma due caratteri consecutivi. Gli ultimi due.
Mi potresti dare altri lumi?
Ciao.

[kirakira93]

una cosa del genere:

//blablabla
FILE *fpt;
fpt = fopen("path del mio file");
int i = 0;
char buf [1];
char str [3];

int count = 2;

while(1)
    {
    i++;
    if(fseek(fpt, -i, SEEK_END))
        {
        cout << "Errore! file non valido o con meno di tre caratteri visibili";
      
        return 1;
        }
    fgets(buf, 1, fpt);
    if((buf[0] >= 32 && buf[0] <= 126) || buf [0] == '\n') //nel caso di ascii 128 bit, '\n' è un carattere visibile
        {
        str[count] = buf[0];
        count--;
        }
    if(count == 0)
        break;
    

    }
fclose(fpt);
cout << "ecco i tre ultimi caratteri visibili: "<<str;
//fine programma


//codice non testato ma dovrebbe andare :)

[Alex'87]

while(1)? Che orrore

[kirakira93]

Originariamente inviato da Alex'87
while(1)? Che orrore

assolutamente no! migliora la leggibilità e la portabilità è la stessa

[Alex'87]

Originariamente inviato da kirakira93
assolutamente no! migliora la leggibilità e la portabilità è la stessa

Migliora la leggibilità un corno dato che per capire quando finisce quel ciclo bisogna leggersi tutto il blocco di codice. Hai mai sentito parlare di "buona programmazione"? while(1) e l'utilizzo del break per uscire dal ciclo non lo è nel modo più assoluto!

Ad occhio quel ciclo potrebbe essere while (count > 0) {} :master:

[kirakira93]

anche...ma allora bisognava mettere anche l'eccezione dela fseek()...per questo ho messo 2 break al posto di quello che hai messo te e un break

Poi io penso che nella programmazione anche un goto o un __try __leave __finally possono benissimo essere usati....basta stare attenti

[Alex'87]

Originariamente inviato da kirakira93
anche...ma allora bisognava mettere anche l'eccezione dela fseek()...per questo ho messo 2 break al posto di quello che hai messo te e un break

Mettere il controllo del valore di fseek nella condizione del while? E perché mai? Sai già quando uscire dal ciclo: quando count diventa zero. Il controllo del valore di ritorno della fseek è corretto farlo nel corpo del ciclo e l'eventuale errore non dovrebbe far parte della condizione di uscita in quanto si tratta di una situazione eccezionale. In questo caso sarebbe più corretto (a mio parere) far restituire un codice di errore e passare la patata bollente al chiamante (stampa dell'errore compresa).

Originariamente inviato da kirakira93
Poi io penso che nella programmazione anche un goto o un __try __leave __finally possono benissimo essere usati....basta stare attenti

Il __try __leave __finally in C non lo conosco mami pare non sia un costrutto standard. Riguardo il goto...

[giorgiogio48]

Ciao a tutti,
ho aperto io questo thread. Ringrazio tutti gli intervenuti. I vostri interventi saranno comunque utili ma io gli ultimi 2 caratteri non li trovavo semplicemente perchè non venivano sempre salvati, per errori nel codice.
Ringrazio ancora tutti.