Hvorfor fejler denne kode??

metoder :

double TrustValue::getValue()
{
    return this->value;
}

TrustValue* TrustManager::getTrustValue(const nsaddr_t& nid)
{
    //remove this check after "debug"
    map<nsaddr_t,TrustValue*>::iterator it;
    if(it == tvalues.end())
    {
      cout <<"AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAARRRRRRRRRRRRRRRRRRRRRGGGGGGGGGGGGGGGGGGGG"<<endl;
    }   
    return tvalues[nid];
}

bool TrustManager::isKnown(nsaddr_t id)
{
    map<nsaddr_t,TrustValue*>::iterator it;
    it = tvalues.find(id);//if id doesn't exist it returns iterator to the end
    return(!(it == tvalues.end()));
}


og tvalues i TrustManager er map<nsaddr_t,TrustValue*> , hvor nsaddr_t er typedef int

Der er nogen ting i den kode jeg ikke forstår:

for(int i =1; i < stopat+1;i++)

route[i]

er route dimensioneret med stopat+1 elementer (0..stopat) ?

for(int i =1; i < stopat+1;i++)

    int i = route[i].getNSAddr_t();

hvorfor laver du en bariabel der skygger for løkke tælleren ?

Hvis det ikke var klart så er det :

temp =trust_manager->getTrustValue(route[i].getNSAddr_t())->getValue();

der giver problemet.

Håber på nogen kan komme med forslag til hvad der kan være galt

13:29:50: det var bare for i testen at se om alle elementer i arrayet kunne indexeres

arrrr
: vent , jeg vender tilbage når jeg har kaldt den noget andet end i

13:29:03> route indeholder nogle adresser. Adressen på den sender ligger i route[0]. En anden metode returnerer stopat, som er der destinationen ligger.
dvs route kan godt være længere end stopat, men alt efter stopat er uinterresant i denne sammenhæng

har lige kørt en succes fuld simulering hvor int i, hvar erstattet med int h. Det burde indikerere at det ikke er array adresseringen der er problemet.

Kan du se nogle problemer/fejl i den øvrige kode?

Der er ikke noget som springer i øjnene.

Jeg tror du bliver nødt til at putte nogle fprintf ind og se hvad der sker.

Eller noget:
  dbgf <<
hvis det skal være C++'sk.

hvad er dbgf?

ofstream dbgf("debug.log");

:-)

double TrustValue::getValue() {
  // this->value == "value i TrustValue" ikke?
  if (value ikke har lovlig værdi)
    springiluften(value);

    return this->value;
}

TrustValue* TrustManager::getTrustValue(const nsaddr_t& nid)
{
    //remove this check after "debug"
    map<nsaddr_t,TrustValue*>::iterator it;
---> skal it ikke tildeles en værdi før checket???
    if(it == tvalues.end())
    {
      cout <<"AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAARRRRRRRRRRRRRRRRRRRRRGGGGGGGGGGGGGGGGGGGG"<<endl;
    } 
    return tvalues[nid];
}


er route defineret som

typex route[stopat+1];
har du check for at du ikke løber ud over enden på arrayet?

segmose> du har helt ret i at if(it == tvalues.end()) er en fejl - dog ikke den der fik det til at chrashe. Jeg har flyttet koden bid for bid til en tidligere version af simulatoren, hvor det kører uden problemer. ? :)

I må gerne lægge et svar begge 2 da i har været til stor hjælp med, dels at påpege fejl, og samtidig "bekræfte" at koden var OK

Ok.

ok

svar

svar