[C++]dividere il contenuto di un file su un array

[MrModd]

Ciao, ho bisogno di fare una funzione che mi prenda un file di testo che contiene diverse istruzioni e mi assegni ogni istruzione ad una locazione di un array.
Mi spiego un po' meglio:
Io ho un file e dentro c'è scritto:

prima riga
seconda riga
terza riga
quarta riga
...
n riga

a questo punto la funzione mi deve creare un array (char* testo[4]; ) e in ogni locazione mi deve mettere una riga. Però dalla quarta fino alla fine deve metter tutto in una singola locazione.
Come fo'?

Io pensavo ad una cosa del tipo

for (int i; i<4; i++)
while (non c'è un ritorno a capo)
{
scrivimi la riga su testo[i];
}
while (non finisce il testo)
{
scrivi su testo[3];
}


Idee?

[DanieleC88]

Non ho ben capito qual'è il problema... :stordita:

[MrModd]

Io ho un file di testo che contiene queste righe:

2435
riga 1
riga 2
riga 3
qui comincia
il testo fino
alla fine del file


Io ho pensato ad una cosa del genere:

char* file_read(int i)
{
FILE *testo;
testo=fopen("testo.txt", "r");
char num[200], riga1[800], riga2[1000], riga3[1000], testo[15000];
rewind(testo);
fgets(num,200,testo);
fgets(riga1,800,testo);
fgets(riga2,1000,testo);
fgets(riga3,1000,testo);
fgets(testo,15000,testo);
if (i==1) return num;
if (i==2) return riga1;
if (i==3) return riga2;
if (i==4) return riga3;
if (i==5) return testo;
}
char a=file_read(1);
int numa=(int)a;


Ho 3 problemi:
1. Non so perchè tutti gli array funzionano tranne il primo. fgets non riesce a leggere bene la prima riga.
2. l'ultimo array dovrebbe contenere tutto il resto del file, ma con fgets legge solo una riga.
3. Il valore di numa, convertito in int da char, non è lo stesso valore scritto nel file di testo.

[MrModd]

Ho risolto il primo punto aumentando la dimensione dell'array.
Ora restano gli altri due problemi...

[DanieleC88]

È un po' problematico come codice, non fai alcune cose nel modo corretto.
Prima di tutto, devi sempre assicurarti che la fopen() non fallisca, altrimenti non ha senso continuare.
Il problema 1) dici di averlo risolto.
Il 2) è naturale, è così che funziona fgets():
fgets() reads in at most one less than size characters from stream and
stores them into the buffer pointed to by s. Reading stops after an
EOF or a newline. If a newline is read, it is stored into the buffer.
A '\0' is stored after the last character in the buffer.

Dovresti invece fare qualcosa del genere:
FILE *fp = fopen(...);

/* ... */
while (!feof(fp))
{
fgets(CharArray, sizeof(CharArray), fp);
}
La 3) invece è normalissima, il C++ (come il C) quando effettua un cast tra tipi primitivi, non fa altro che "riconsiderare" i dati già contenuti in memoria, senza fare conversioni di tipo. Nel tuo caso, non puoi interpretare una sequenza di caratteri come fosse un intero, ma devi delegare questa operazione ad una funzione della libreria standard (nello specifico, ad atoi()) che si occuperà di fare il lavoro sporco.

ciao ;)

[MrModd]

È un po' problematico come codice, non fai alcune cose nel modo corretto.
Prima di tutto, devi sempre assicurarti che la fopen() non fallisca, altrimenti non ha senso continuare.
Il problema 1) dici di averlo risolto.
Il 2) è naturale, è così che funziona fgets():

Dovresti invece fare qualcosa del genere:
FILE *fp = fopen(...);

/* ... */
while (!feof(fp))
{
fgets(CharArray, sizeof(CharArray), fp);
}
La 3) invece è normalissima, il C++ (come il C) quando effettua un cast tra tipi primitivi, non fa altro che "riconsiderare" i dati già contenuti in memoria, senza fare conversioni di tipo. Nel tuo caso, non puoi interpretare una sequenza di caratteri come fosse un intero, ma devi delegare questa operazione ad una funzione della libreria standard (nello specifico, ad atoi()) che si occuperà di fare il lavoro sporco.

ciao ;)

Oki, ho risolto il punto 3 con atoi, come mi hai consigliato.
Il punto 2 però non funziona così: in questo modo sull'array mi ritrovo solo l'ultima riga, perchè fgets sovrascrive ad ogni passaggio.

[DanieleC88]

Il punto 2 però non funziona così: in questo modo sull'array mi ritrovo solo l'ultima riga, perchè fgets sovrascrive ad ogni passaggio.
Infatti la mia era solo una bozza, e se guardi bene non ho mai nominato le variabili del codice originale: devi leggere una riga in un buffer temporaneo (va bene anche un'array statica per i tuoi scopi), poi accodarla al buffer finale che conterrà tutto il testo, avendo cura di controllare che ci sia abbastanza spazio per l'input (e, in caso contrario, provvedere a troncarlo o riallocare il buffer). ;)

Giacché ci siamo, toglimi la curiosità: perché in C++ usi le funzioni di libreria del C per accedere ai file? Se è per necessità o per studio, ok, continua ad usarle per questo momento, ma altrimenti ti complichi la vita inutilmente. :)

ciao ;)

[MrModd]

Infatti la mia era solo una bozza, e se guardi bene non ho mai nominato le variabili del codice originale: devi leggere una riga in un buffer temporaneo (va bene anche un'array statica per i tuoi scopi), poi accodarla al buffer finale che conterrà tutto il testo, avendo cura di controllare che ci sia abbastanza spazio per l'input (e, in caso contrario, provvedere a troncarlo o riallocare il buffer). ;)

Giacché ci siamo, toglimi la curiosità: perché in C++ usi le funzioni di libreria del C per accedere ai file? Se è per necessità o per studio, ok, continua ad usarle per questo momento, ma altrimenti ti complichi la vita inutilmente. :)

ciao ;)

Uhm perchè cosa c'è in C++? Io ho usato cstdio.
Insomma come lo risolvo il problema del buffer?

[tomminno]

Uhm perchè cosa c'è in C++? Io ho usato cstdio.
Insomma come lo risolvo il problema del buffer?

cstdio come dice il nome stesso è lo standard IO del C.
In C++ hai gli stream e le stringhe:


#include <fstream>
#include <string>

using namespace std;
...

ifstream input("nomefile");
string line;
while(getline(input, line))
{
...
}

[DanieleC88]

Leggi qui: http://www.cppreference.com/wiki/io/start
Se fai una ricerca per fstream troverai molti esempi. :)

Se vuoi continuare ad usare la libreria standard del C puoi farlo, comunque. Il metodo per aggirare il problema del buffer te l'ho spiegato nell'ultimo post. ;)

[MrModd]

cstdio come dice il nome stesso è lo standard IO del C.
In C++ hai gli stream e le stringhe:


#include <fstream>
#include <string>

using namespace std;
...

ifstream input("nomefile");
string line;
while(getline(input, line))
{
...
}


Non lo conoscevo, non ho mai lavorato con i file.
Non ho capito cosa fa questo codice.

[tomminno]

Non lo conoscevo, non ho mai lavorato con i file.
Non ho capito cosa fa questo codice.

Mi sembrava di aver usato variabili quasi autoesplicative, comunque:
con "input" indico la variabile di tipo input stream (sola lettura) che accede al file "nomefile".
con "line" indico la variabile d'appoggio per la lettura di una linea del file
getline è una funzione di libreria che legge una linea da "input" e la mette in "line".
Tutto molto semplice.

[MrModd]

Ok, ma se inserisco diverse righe su line si sommano o si sovrascrivono?
E poi registra pure il return alla fine della riga? Io dovrei escluderlo in alcuni casi ed includerlo in altri.

[DanieleC88]

Ok, ma se inserisco diverse righe su line si sommano o si sovrascrivono?
Si sovrascrivono, ma puoi risolvere facilmente:
string buffer;
string line;

buffer.assign("");
while (getline(input, line))
{
buffer.append(line);
}
O eventualmente usare un ostringstream, che è forse ancor più comodo per questo caso.

ciao ;)

[MrModd]

Oki chiamatela ispirazione divina ma ho risolto il problema del ritorno a capo:
praticamente per eliminarlo dall'array creato con fgets ho creato un ciclo for che controlla per ogni locazione dell'array se è presente il return (in ascii è 10); se lo trova lo cancella ed esce dal for.

Ora c'è solo il problema di assegnare diverse righe ad un solo array, ma forse ho la soluzione...
Vi farò sapere...

[MrModd]

Ho risolto tutti i problemi ;)