free() kald med et this->... argument.

mere kode, tak

oki oki :)

        int *tempValues;
        tempValues=(int *)malloc((sizeof(int)*this->count));
        int i;
        for(i=1;i<=this->count;i++){
                tempValues[i-1]=this->values[i-1];
        }
        free(this->values);
        this->count--;
        this->values=(int *)malloc((sizeof(int)*count));
        for(int i=1, j=0;j<=this->count;i++){
                if(i!=idx)j++;
                this->values[j-1]=tempValues[i-1];
        }
        free(tempValues);

Hmm - kigger lige, men realloc burde kunne gøre det for dig

Hvad med
  this->values=(int *)malloc((sizeof(int)*this->count));
??

Med realloc:

1) flyt værdierne ned så du overskrive idx-positionen
2) this->count--;
3) realloc(this->values,sizeof(int)*this->count);

kan jeg også bruge realloc i den funktion, hvor jeg bruger
this->count++;
?

Yeps.

1) this->count++;
2) realloc(this->values,sizeof(int)*this->count);
3) Flyt lidt rundt på dem

Nu ser den således ud, og den brokker sig endnu mere end før:

        int *tempValues, *tempXCoordinates, *tempYCoordinates;
        tempValues=(int *)malloc((sizeof(int)*(this->count-1)));
        int i;
        for(int i=1, j=0;j<=this->count-1;i++){
                if(i!=idx)j++;
                tempValues[j-1]=this->values[i-1];
        }
        this->count--;
        this->values=(int *)realloc((void *)this->count,sizeof(int));
        for(i=1;i<=this->count;i++){
                this->values[i-1]=tempValues[i-1];
        }
        free(tempValues);

:(

Du skal jo gøre som jeg siger (utestet):

for(int i=1, j=0;j<=this->count-1;i++){
  if(i!=idx)j++;
  this->values[j-1]=this->values[i-1];
}
this->count--;
this->values=(int *)realloc(this->values,sizeof(int)*this->count);

jammen bliver al data da ikke overskrevet ved kaldet til realloc? og hvis jeg bare skriver derud af - så rammer jeg vel ualkerede sektore i rammen...

realloc kopierer selv. Ellers er der ikke meget fidus. Den er så smart at lade
data ligge hvor de ligger, hvis der er plads. Og ellers flytter den det selv.

det virker ikke i den funktion hvor jeg bruger this->count++ - der får jeg en access vilation i linien:
this->values[j-1]=this->values[i-1];
og det er jo også klart, når jeg jo bevæger mig ud på ualokkeret område i rammen:

        for(int i=1;i<=this->count+1;i++){
                this->values[i-1]=this->values[i-1];
        }
        this->count++;
        this->values=(int *)realloc(this->values,sizeof(int)*this->count);

Jamen for sytten sure sokker da også. Du skal jo gøre som jeg siger i
Kommentar fra erikjacobsen 19/06 2001 14:53:23 

For du skal lave pladsen først, ikke ??

hehe - nååå - jeg var bare doven og tog en direkte c/p af det du skrev :)~

Har jeg forstået dig ret hvis det er dette du mener?:

        this->count++;
        this->values=(int *)realloc(this->values,sizeof(int)*this->count);
        this->values[this->count-1]=value;

Det ser fint nok ud, men jeg får stadigt den acces violation... :(

Hvordan alloc-er du this->values første gang? Og er det ved første indsættelse,
eller senere du får fejlen.

Ja, det er bare smukt. Findes der noget kønnere end et kompakt C-program?
Ja, et endnu kompaktere:

      this->values=(int *)realloc(this->values,sizeof(int)*(++(this->count)));
      this->values[this->count-1]=value;

Måske man endda kan
  this->values=((int *)realloc(this->values,sizeof(int)*(++(this->count)))[this->count-1]=value);

....men ok, det er ikke dit problem lige nu. Hold dig bare til de 3 linier.

jammen hvad er det så der er galt? jeg får en acces violation, når jeg prøver at aflæse værdien af this->values i en anden funktion...

Jamen, så svar på mit spørgsmål, ellers spilder vi da hinandens tid:
\"Hvordan alloc-er du this->values første gang? Og er det ved første indsættelse,
eller senere du får fejlen.\"

Oki - det må du undskylde - havde ikke set at du havde skrevet det :(

Jeg allokerer ikke this->values, da objektet fra start af jo ikke har nogle værdier (this->count=0). Den allokeres først, når der bliver tilføjet nogle værdier (i den funktion du lige har hjulpet med).

Jeg regner med at det er ved første indsættelse, at fejlen sker, da jeg ikke kan se det precist :-/

Skal jeg allokere den til nul elementer eller?

Så nu har jeg lavet mit hjemmearbejde :)
Jeg har lavet om i constructoren, så \"values\" initieres med en malloc af een int. Så har jeg fået afgrænset problemet til, at der kun kommer acess violation i funktionen til at fjerne et element:

        for(int i=1, j=0;j<=this->count-1;i++){
                if(i!=idx)j++;
                this->values[j-1]=this->values[i-1];
                this->xCoordinates[j-1]=this->xCoordinates[i-1];
                this->yCoordinates[j-1]=this->yCoordinates[i-1];
        }
        this->count--;
        this->values=(int *)realloc(this->values,sizeof(int)*this->count);
        this->xCoordinates=(int *)realloc(this->xCoordinates,sizeof(int)*this->count);
        this->yCoordinates=(int *)realloc(this->yCoordinates,sizeof(int)*this->count);

E-)mil

Du skal ikke tage dig af de to andre vaiabler - de gennemgår bare den samme procedyre som this->values :)

Jeg har testet lidt mere, og borland builder giver en \"invalid pointer opreation\" runtime fejl - før jeg får min acces violation fra windows - stadigt i overstående funktion...

emilj>> Såvidt jeg kan se så bruger du en Dynamisk array. Det gøres således i C++ :

  int n = this->count;
  int* tempValues = new int[n];
  for(int i=1; i<= n; i++)
    tempValues[i-1]=this->values[i-1];


Jens B

når så du er færdig med at bruge din tempValues kaldes delete tempValues;

Jens B

Så vidt jeg ved, skal der da bruges en konstant værdi ved initialiseringen int* tempValues = new int[...];
Det var derfor jeg ikke prøvede den måde - har dog ikke C++ kørende, men jeg tester det om 2 sek...

Der mangler jo delen hvor jeg laver om på størrelsen af this->values; hvordan gør jeg det i dit forslag?

Du der, emil, kan du ikke lige sende al din kode til mig (klik på mit navn), så
kan jeg lige hælde det igennem en oversætter her... Det er nemmere end
at konstruere det hele igen :)

Du kan godt initialisere this->value til at pege på et array med 0 elementer. Og ja,
det skal alloc-es.

Undskyld jeg blander mig lidt, men hvis formålet med koden er at fjerne et felt (idx) fra Values,
er det så ikke nemmere og hurtigere at gøre dette

this->count--;
for(int i=idx, i<this->count;i++){
    this->values[i] = this->values[i+1];
    }
this->values=(int *)realloc(this->values,sizeof(int)*this->count);

og forøvrigt er det nemmere at bruge memcpy til at kopiere indholdet istedet for at tage hver enkelt index.

Jo, ap, det var jo noget i den stil vi skulle nå frem til, men den er vist opgivet
af spørgeren.

Det har du sikkert ret i, men det kan være han dukker op lige pludselig.

Hehe - BØH!
Narhj narhj - har skam ikke glemt jer - er bare først liiige kommet hjem fra ferie i syden :)

E-)mil

Ferie i syden - det kan bestemt ikke være bedre end ferie ude på min
græsplæne :)) Men mener du \"ferie uden Internet\" ... oh, gru, oh rædsel....

*lol*
Undskyld jeg ikke har fået testet dit bud endnu, men jeg har en del havearbejde, jeg er blevet prakket på af de gamle :)~

E-)mil
http://eej.dk
http://fotx.net/emileej

Beklager meget den lange stilhed, men tid er ikke meget jeg har af for tiden, så projektet er røget nederst i bunken...

Jeg deler pointene lige imellem jer, og kigger på jeres svar, når jeg får tid :)

E-)mil