[c++] list probleem

[Lord_T]

In de onderstaande code probeer ik naar het goede element te gaan van een lijst, dan heeft dat element als private data element ook een lijst en die wil ik gaan vullen..... alleen dat gaat fout

Nou gebeurt dit allemaal tijdens het inlezen van een file en afhankelijk van de eerste letter moet het een verschillend aantal dingen inlezen.

C++:

    h = fopen(filename2,"r");
    ListIterator<CombiPtr> iter(mCombinationPtr->mCombiList);
    while(!feof(h))
    {
        fscanf(h,"%s",test);
        if(!strcmp(test,"C" ))
        {
            fscanf(h,"%s %s %s %s %s %s %s %s %s", Temp1,Temp15,Temp16,Temp2,Temp3,Temp4,Temp5,Temp6,Temp7,Temp8);  
            for(iter.GotoBegin();!iter.AtEnd();iter.GotoNext())
            {
                if(!strcmp(Temp1,iter.GetElement()->mTractorPtr->mTractorName))
                {
                    if(!strcmp(iter.GetElement()->mTrailerPtr->mTrailerName,Temp15))
                    {
                        if(!strcmp(iter.GetElement()->mDriverPtr->mDriverName, Temp16))
                        {
                            iter.GetElement()->mStart = atoi(Temp5);
                            iter.GetElement()->mEnd = atoi(Temp6); // goede iter.GetElement te pakken hier
                        }
                    }
                }
            }
        }
        if(!strcmp(test,"R"))
        {
            fscanf(h,"%s %s %s %s %s %s %s %s %s %s %s %s %s %s %s %s %s",Temp2,Temp3,Temp1,Temp15,Temp16,Temp15,Temp4,Temp5,Temp6,Temp7,Temp8,Temp9,Temp10,Temp11,Temp12,Temp13,Temp14);           
        }
        if(!strcmp(test,"S"))
        {
            fscanf(h,"%s %s %s %s %s %s %s %s %s %s %s %s %s", Temp1,Temp15,Temp16,Temp2,Temp17,Temp4,Temp5,Temp6,Temp7,Temp8,Temp9,Temp10,Temp11);     
            string Test = Temp16;
            iter.GetElement()->mOrderList.GetSize();
            //iter.GetElement()->mOrderList.AddAtEnd(Test);
        }
    }
    fclose(h);

Hij crashed nu altijd als ik iets wil doen met de correcte iter.GetElement()....
Terwijl hij dus wel netjes eerst door stap C en R gaat.....

Iemand enig idee?

[MSalters]

Ten eerste: feof() is niet predicitef, het vertelt je niet of je volgende read gaat lukken. Die test moet dus na elke fscanf.
Ten tweede: je gebruikt een onoverzichtelijke hoeveelheid variabelen TempXX, die allemaal
blijkbaar char buffers zijn. Eén foute buffergrootte en je hebt een slecht te debuggen probleem. Gebruik gewoon std::string oid
Ten derde, je vertelt niet eens wat voor list je gebruikt. De standaard C++ list is std::list, en die heeft een heel andere syntax.

[whoami]

Welke fout krijg je?
Ben je zeker dat je geen unallocated memory uitleest of ernaar schrijft, etc....

Debugging is the cornerstone of being a programmer. The first meaning of the verb to debug is to
remove errors, but the meaning that really matters is to see into the execution of a program by examining
it. A programmer that cannot debug effectively is blind.

_{Uit 'How to be a programmer' van Robert L. Read}

[Lord_T]

tis een geheugen fout.. das het probleem... zijn overigens allemaal char* en nou ja de bedoeling is dat ik die convert naar string in dan de string toevoeg aan die lijst.

Als ik niet regel 32 of 33 uitvoer... geeft het geen enkel probleem...
De waarde van Test is ook gewoon prima... dus ik snap het niet.....

[Lord_T]

uhm never mind... de fout zit al in de list... de mCombiList
Die kent die niet... en geeft daar geen fout op maar als je hem probeert te gebruiken wel


edit: probleem opgelost: Hij onthield niet wat iter.GetElement() was verder op. Dus even een temp variabele gemaakt en dat werkte.

[ Voor 32% gewijzigd door Lord_T op 26-08-2003 19:50 ]

[Glimi]

Maar waarom voldoet std::list niet eigenlijk?
Waarom zou je perse voor een andere lib kiezen als de meest portable en geteste library van C++ bijgesloten zit

[TaXaN]

^^^ wat hij zegt

En maak ook best van die if's met strcmp eens if - else if - else if, dat worden al de andere if'n niet uitgevoerd als er één if lukt.