[C]programma klapt eruit bij linked list bewerking

[Ronald Koeman]

Ik heb het volgende stukje code(fragment van een groter programma):

code:

while (lijst->next != NULL)
  {
  if ((lijst->woord)== zoekwoord)
  {
    aantal++;
    slaover=lijst->next;
    vorigelijst->next=slaover;
  }
  vorigelijst=lijst->next;
  lijst=lijst->next;
}

vooraf is een node van de lijst gedeclareerd als:

code:

typedef struct _node
{ char woord[256];
  int aantal;
  struct _node *next;
}
node, *pNode;

lijst, slaover en vorigelijst zijn pointers van het type pNode.
Op het einde van mijn lijst heb ik node->next de waarde NULL gegeven:

code:

lijst->next=NULL;

Het programma knalt er dus uit in de while-lus, met een acces violation. Ik begrijp alleen niet waarom...iemand?

[ Voor 19% gewijzigd door Ronald Koeman op 26-09-2004 21:38 ]

[Robtimus]

Accessviolation kan in 1 van de volgende statements zitten:

code:

while (lijst->next != NULL)
if ((lijst->woord)== zoekwoord)
slaover=lijst->next;
vorigelijst->next=slaover;
vorigelijst=zoeklijst->next;

Dus lijst, vorigelijst of zoeklijst is NULL.

Ik neem aan dat lijst != NULL, dan is of vorigelijst of zoeklijst NULL. Print de waarde daarvan eens uit en je ziet waar het probleem ligt.

[ Voor 4% gewijzigd door Robtimus op 26-09-2004 19:10 ]

[Soultaker]

Het zal wel aan mij liggen, maar moet je niet ergens een keer wat nieuws aan lijst toekennen?

[Ronald Koeman]

Euhm...:s

Ik zat ff behoorlijk te slapen, dank voor jullie reacties

[ Voor 72% gewijzigd door Ronald Koeman op 26-09-2004 19:22 ]

[Ronald Koeman]

OK, code geupdate, alleen nog steeds die access violation...begrijp 't niet.

Edit: lijst lijkt de waarde 0 te hebben, vandaar de foutmelding.

[ Voor 34% gewijzigd door Ronald Koeman op 26-09-2004 21:44 ]

[Zoijar]

Je vergelijkt hiermee: if ((lijst->woord)== zoekwoord), ook geen strings heh. Maar alleen het adres, dus die zal iha nooit gelijk zijn.

Als je in je eerste itteratie het woord zou vinden, dan is vorigelijst nog niet geinitialiseerd... Ik denk dat je even je volledige code moet posten.

[ Voor 11% gewijzigd door Zoijar op 26-09-2004 21:48 ]

[Ronald Koeman]

lijst->woord is het adres van woord, en niet 't woord zelf bedoel je?
Moet ik dan &lijst->woord doen oid?

vorige lijst wordt buiten de loop al geinitialiseerd (dacht ik).
Ik controleer het even op de manier de iceman suggereert, met wat printf's.

[ Voor 45% gewijzigd door Ronald Koeman op 26-09-2004 21:52 ]

[Zoijar]

Nee, je kan zou niet een string vergelijken. Je moet het byte voor byte afgaan. Of anders iets als str(n)cmp() gebruiken. Je vergelijkt nu alleen of ze op hetzelfde geheugen adres beginnen.

[curry684]

Ronald Koeman schreef op 26 september 2004 @ 21:40:
OK, code geupdate, alleen nog steeds die access violation...begrijp 't niet.

Edit: lijst lijkt de waarde 0 te hebben, vandaar de foutmelding.

Even stomme vraag, maar heb je al eens simpelweg door de code gestept en de variabelen geinspecteerd of naar een console window gedumpt?

GoT is niet bedoeld om code voor te debuggen maar om problemen op te lossen waar je niet uitkomt nadat je zelf al het debugwerk hebt gedaan wat je kunt doen. Lijkt me dat je hier met wat simpel stepwerk toch wel uit moet kunnen komen?