[c++] char[] vullen uit een textbestand

[Verwijderd]

Ik probeer een char[] te vullen met regels uit een tekstbestand. Het werkt wel als ik handmatig de waarden invoer:

C:

char *lijst[] = {"1. Incoming         ","2. Modification     ","3. Outgoing         ","4. Options          " };

maar als ik dit doe vanuit een tekstbestand gaat dit mis, helaas kan ik niet met string werken dus wil ik het graag met een char[] proberen

C:

int main()
{
    FILE *fread;
    char regel[200];
    char *list[20];
    int i, j;

    fread = fopen("data.txt", "r");
    i= 0;
    do
    {
        memset(regel, 0, 200);
        fgets(regel, 200, fread);
        printf("%s\n", regel);
        list[i] = regel;
        i++;
    }
    while(!feof(fread));

    fclose(fread);

    for (j = 0; j < i -1; j++)
    {
        printf("%d. %s\n", j+1, list[j]);
    }

    _getch();
    return 0;
}

tekstbestand is hier:
Incoming
Modification
Outgoing
Options

en uitvoer wordt
1.
2.
3.
4.
dus helemaal niets, iemand een idee, ik weet het gaat mis met die char[]

[Larry4]

elke pointer in list wijst naar dezelfde regel (je moet wel steeds een nieuwe regel aan maken? per regel van het text bestand

[Verwijderd]

Dacht ik al, want gister had ik wel steeds de laatste regel of ik nu list[1] of list[5] deed. Maar hoe kan ik wel een list krijgen als onderstaande uit een tekstbestand?

C:

char *lijst[] = {"1. Incoming         ","2. Modification     ","3. Outgoing         ","4. Options          " };

[Biedzjee]

Je moet zorgen dat je in je lijst ruimte hebt om iedere afzonderlijke regel op te slaan. In jouw geval, is de ruimte in de lijst 20 karakters per regel.

Je moet dan in plaats van

C:

list[i] = regel; 

gebruiken

C:

strncpy(list[1], regel, 20)

Hiermee wordt de inhoud van de "actieve" regel gekopieerd in je lijst, met een maximum van 20 karakters.

[ Voor 10% gewijzigd door Biedzjee op 08-04-2003 08:30 ]

[Verwijderd]

Mijn programma knalt er hierbij uit, het is de bedoeling dat de uitvoer wordt gedisplayed en dat er dan met een escape kan worden gestopt. Maar volgens mij kom ik zo wel verder met die strncpy, maar gaat het mis met die lijst?

C:

int main()
{
    FILE *fread;

    char regel[200];
    char *list[20];
    int i, j;

    fread = fopen("data.txt", "r");
    i= 0;
    do
    {
        memset(regel, 0, 20);
        fgets(regel, 20, fread);
        strncpy(list[i++], regel, 20);
    }
    while(!feof(fread));
    fclose(fread);

    for (j = 0; j < i -1; j++)
    {
        printf("%d. %s\n", j+1, list[j]);
    }

                _getch();

    return 0;
}

[ Voor 7% gewijzigd door Verwijderd op 08-04-2003 08:56 ]

[XTerm]

Natuurlijk gaat het mis ! Jij maakt een array van char*'s aan, maar je vraagt er nergens geheugen voor aan

C:

#include <stdio.h>
#include <stdlib.h>

int main()
{
    FILE *fread;

    char regel[200];
    char *list[20];
    int i, j;

    fread = fopen("data.txt", "r");
    i= 0;
    do
    {
        memset(regel, 0, 20);
        fgets(regel, 20, fread);
        list[i]=malloc(sizeof(char)*21);
        strncpy(list[i++], regel, 20);
    }
    while(!feof(fread));
    fclose(fread);

    for (j = 0; j < i -1; j++)
    {
        printf("%d. %s\n", j+1, list[j]);
    }

                getchar();

    return 0;
}

[ Voor 9% gewijzigd door XTerm op 08-04-2003 09:13 ]

[Verwijderd]

op onderstaande manier vul je hem direct en hoef je er geen geheugen voor aan te vragen
char *lijst[] = {"1. Incoming ","2. Modification ","3. Outgoing ","4. Options " };

maar ik doe toch char *list[20], dan krijg ik toch een lijst waar maximaal 20 char's in geplaatst kunnen worden, en met strncpy reserveer ik toch de ruimte hierbinnen?

Aub hulp hiervoor, ik ben op dit gebied nogal new.

[XTerm]

Verwijderd schreef op 08 April 2003 @ 09:11:
op onderstaande manier vul je hem direct en hoef je er geen geheugen voor aan te vragen
char *lijst[] = {"1. Incoming ","2. Modification ","3. Outgoing ","4. Options " };

maar ik doe toch char *list[20], dan krijg ik toch een lijst waar maximaal 20 char's in geplaatst kunnen worden, en met strncpy reserveer ik toch de ruimte hierbinnen?

Aub hulp hiervoor, ik ben op dit gebied nogal new.

Nah

C:

 char *list[20] 

Dit is een array (van 20 elementen) van char*'s die naar Death And Destruction pointen omdat ze uninitialised zijn.
Als je een pointer naar een array van 20 groot wil (dus een pointer naar array, type char **) heb je

C:

 char (*list)[20]  

nodig. De voorrang van de operatoren is hier belangrijk.

[Verwijderd]

char **list[200];
..
strncpy((*list)[i], regel, 20);

meer op deze manier? Ik zoek me rot, heb een goed boek gekocht, maar hier kom ik toch echt niet uit.

[XTerm]

Verwijderd schreef op 08 april 2003 @ 09:19:
char **list[200];
..
strncpy((*list)[i], regel, 20);

meer op deze manier? Ik zoek me rot, heb een goed boek gekocht, maar hier kom ik toch echt niet uit.

Nou maak je een pointer naar een pointer naar een array

Kijk, van wat ik kan afleiden wil jij gewoon een array met strings.
We gaan ervanuit dat het aantal strings gekend is :
char *x[20]; Geeft je 20 char pointers.

Goed, die pointers wijzen naar niets, en mag je dus niet gebruiken.
Eerst moet je er geheugen aan toewijzen :

for(c=0;c<20;c++)x[c]=malloc(sizeof(char)*100);

Dat geeft je 20 strings van 99 karakters.
Pointers zijn enorm tricky, het duurt wel ff voor je dit soort dingen vlot kan

[Verwijderd]

Pointers duren inderdaad lang ja, ik ben weer vooraan in mijn hoofdstuk begonnen, maar kan de hulp enorm waarderen. Iets te lang met java gewerkt, daar gaat het net even wat makkelijker

[Verwijderd]

Hmm, in Java wordt elk object (array is in java ook een object) als pointer doorgegeven, dus wat dat betreft zou je het zo op moeten pakken. Je met alleen even opletten op memory leaks, that's all...

[Verwijderd]

Wat ben je nu mee bezig eigenlijk? Je doet het in c++? Dan kan je toch gewoon ifstream en ofstream gebruiken?

code:

char ** data=new char[AANTAL_ELEMENTEN];

ifstream bestand ("data.txt", ios::in);

for(char a=0; !bestand.eof(); a++){
char * bffr=new char[255];

bestand.read(bffr,'\n', 255);

data[a]=new char[strlen(bffr)];

strcpy(data[a], bffr);
}

Ik kan de code nu niet testen, zit nu op school. Vanmiddag zal ik er nog eens maar kijken.

[Verwijderd]

He perfect Jeroen_Paul, deze method zocht ik, nou en je had hem bijna helemaal goed ook nog uit je hoofd!

[.oisyn]

Jeroen_Paul: als je toch voorstelt om de iostreams te gebruiken, waarom zeg je er dan niet gelijk bij om std::string te gebruiken? Dat scheelt een hoop gezeur kwa memory management. En gebruik dan gelijk een std::list of std::vector om alle strings in op te slaan, dan heb je een mooie dynamische array

C++:

#include <fstream>
#include <string>
#include <list>

int main ()
{
    std::list<std::string> regels;
    std::string r;

    std::ifstream file ("data.txt");
    while (std::getline (r, file))
        regels.push_back (r);

    // al je regels staan nu mooi in 'regels'

    return 0;
}

[Verwijderd]

Hij wilde string niet gebruiken, toch? Misschien wilt ie oefenen met chars, doe ik zelf ook altijd. Kan heel leuk zijn. Je kan natuurlijk ook gewoon strings gebruiken ja

offtopic vaagje: hoe maak je de code zo mooi?

[_Squatt_]

Verwijderd schreef op 08 april 2003 @ 23:35:
offtopic vaagje: hoe maak je de code zo mooi?

offtopic:
[ code=c++] gebruiken in plaats van [ code] (maar dan zonder de eerste spatie natuurlijk)

[Verwijderd]

.oisyn schreef op 08 april 2003 @ 10:05:
Jeroen_Paul: als je toch voorstelt om de iostreams te gebruiken, waarom zeg je er dan niet gelijk bij om std::string te gebruiken? Dat scheelt een hoop gezeur kwa memory management. En gebruik dan gelijk een std::list of std::vector om alle strings in op te slaan, dan heb je een mooie dynamische array

C++:

#include <fstream>
#include <string>
#include <list>

int main ()
{
    std::list<std::string> regels;
    std::string r;

    std::ifstream file ("data.txt");
    while (std::getline (r, file))
        regels.push_back (r);

    // al je regels staan nu mooi in 'regels'

    return 0;
}

dit leek een leuke optie, helaas heb ik de beperkingen dat ik geen echte string kan gebruiken en geen namespace std, weer meer terug bij af

[.oisyn]

woeps, overheen gelezen
waarom kun je eigenlijk geen strings gebruiken?

en de std namespace is op zich geen probleem, dan is het gewoon een kwestie van de .h versies van de C++ headers includen

[Verwijderd]

ik werk met c++ versie 1.52 een hele oude, de toepassing waar ik voor schrijf heeft alleen lib's die daar mee kunnen werken

als ik gewoon een string probeer aan te maken lukt dit niet

[.oisyn]

Het heet trouwens Microsoft Visual C++ 1.52.
C++ is een taal op zich, en heeft geen versienummer (ja eigenlijk wel, maar niet zoals jij het bedoelt ). Het is dan ook niet zoals bijvoorbeeld Delphi, Visual Basic of PHP wat van 1 bepaalde maker is.

Er zijn dan ook talloze C en C++ compilers te vinden, met allemaal verschillende versienummers

[Verwijderd]

maar klopt het dan ook dat ik met mijn visual c++ 1.52 geen string kan gebruiken (string h, bijvoorbeeld)

[MSalters]

MSVC 1.52 heeft geen std::string, maar wel de MFC CString class. Die is niet zo goed, maar nog steeds heel veel beter dan char[]