Chyba při odalokování pomocí delete – C / C++ – Fórum – Programujte.com

Potřebuji pole pro třídu Fronty(queue), to jsem alokoval takhle:

queue *p = new queue[10];

To funguje, dokonce dokážu pracovat s více třídami tak jak potřebuji, ale problém je v tom, že při odalkokaci mi hází VS chybu a já nevím kde chyba je. Odalokovávám takhle:

delete [] p;

Chyba je takováto: 

Celkový kód vypadá nějak takhle:

#include <iostream>
#include <string.h>
#include "queue.h"

using namespace std; void main(){ queue *p = new queue 1]; delete [] p; system("pause"); }

A ještě přidávám queue.cpp:

#include "queue.h"

queue::queue(){

}

void queue::AllocSize(int capacity){
	mCapacity = capacity;
	mTail = 0;
	mHead = 0;
	mCanBeEmpty = true;
	mCanBeFull = capacity == 0;
	mPole = new int[mCapacity];
}

bool queue::Put(int arg){
	if (Full())
		return false;

	mCanBeFull = true;
	mCanBeEmpty = false;
	mPole[mTail] = arg;
	mTail = ++mTail % mCapacity;
}

int queue::Get(){
	if (Empty())
		return -1;

	mCanBeFull = false;
	mCanBeEmpty = true;
	int temp = mHead;
	mHead = ++mHead % mCapacity;
	return mPole[temp];
}

bool queue::Full(){
	return (mTail == mHead) && mCanBeFull;
}

bool queue::Empty(){
	return (mTail == mHead) && mCanBeEmpty;
}

queue::~queue(void){
	delete [] mPole;
}

Vůbec nevím kde je chyba, přitom když queue vyměním za int, nebo double, projede program bez problémů, bez chyby.

Hele ted nejak nechapu, co ma ta trida spolecneho s tim co mas v main...  ta chyba, kterou to hlasi je taky nejaka vymluvna...

#2 KIIV
Pro tu třídu alokuji pole, tak jsme si říkal, jestli chyba nebude třeba tam, co ja vím. Minule, když jsme sem dával příspěvek, mě tu dupali za to, že jsme nedal všechny zdrojáky a jestli si je mají cucat z prstu. Co se týče chyby, můžu přidat jen, že se mi otevře nějaký soubor dbgdel.cpp na řádku 52.

/***
*dbgnew.cpp - defines C++ scalar delete routine, debug version
*
*       Copyright (c) Microsoft Corporation.  All rights reserved.
*
*Purpose:
*       Defines C++ scalar delete() routine.
*
*******************************************************************************/

#ifdef _DEBUG

#include <cruntime.h>
#include <malloc.h>
#include <mtdll.h>
#include <dbgint.h>
#include <rtcsup.h>

/***
*void operator delete() - delete a block in the debug heap
*
*Purpose:
*       Deletes any type of block.
*
*Entry:
*       void *pUserData - pointer to a (user portion) of memory block in the
*                         debug heap
*
*Return:
*       <void>
*
*******************************************************************************/

void operator delete(
        void *pUserData
        )
{
        _CrtMemBlockHeader * pHead;

        RTCCALLBACK(_RTC_Free_hook, (pUserData, 0));

        if (pUserData == NULL)
            return;

        _mlock(_HEAP_LOCK);  /* block other threads */
        __TRY

            /* get a pointer to memory block header */
            pHead = pHdr(pUserData);

             /* verify block type */
ZDE SE JE ASI PROBLEM --> _ASSERTE(_BLOCK_TYPE_IS_VALID(pHead->nBlockUse));

            _free_dbg( pUserData, pHead->nBlockUse );

        __FINALLY
            _munlock(_HEAP_LOCK);  /* release other threads */
        __END_TRY_FINALLY

        return;
}

#endif  /* _DEBUG */

Co to píše za chybu?

no me slo hlavne o:

double *p = new double[1]; delete [] p;

to nema vubec nic spolecnyho s tou tridou queue... pak uz akorat ze queue je ciste pro inty.. neni to udelany jako sablona tak musis proste pri kazdym jinym typu udelat jinou tridu

#4 crazy
Screen chyby je v 1. příspěvku, pokud tě zajímá kde v k´du toho souboru, tak jsem to označil "ZDE SE JE ASI PROBLEM -->", stačí, když očima pojedeš odspodu nahoru.

#5 KIIV
jej sorry, moje chyba. misto double tam davam prave queue. Zkopíroval jsem omylem testovací skript. Omlouvam se melo tam byt: 

	queue *p = new queue[1];
	delete [] p;

potrebujes vicero tech front?

kazdopadne pak volas konstruktor a destruktor, nicmene v destruktoru nekontrolujes, jestli mas neco alokovanyho ....

dokonce ani neinicializujes zadny atribut tridy v konstruktoru, ale az v ty metode AllocSize

#8 KIIV
Prave, ze potrebuji vice front a navic nevim kolik jich budu potrebovat. AllocSize() původně byl konstruktor, ale queue(10)[10] dávalo chybu, proto jsem to začal volat jinou funkcí. Chceš mi teda říct, že chybu mi dělá destruktor? Pokud vím, tak new vezme určitou paměť a delete ji odevzdává a je jedno, jestli v ní něco je nebo tam není vůůbec nic, nepletu se?

#10 crazy
mel by se zavolat defaultni konstruktor pro kazdou polozku

Jde o to, že ty hodnoty inicializuješ jen v AllocSize.

Takže když použiješ jen new/delete, bez AllocSize mezi nimi, tak ty hodnoty zůstanou neinicializované a ten destruktor vyvolaný při delete pak pracuje s neplatným mPole.

Zajisti aby ten objekt byl správně inicializovaný (nejlépe konstruktorem).

 Treba neco takoveho funguje: 

#include <iostream>

using namespace std;

class test_constructor {
  public:
    test_constructor():pocet(0) { cout << "Constructor without parameter, default value is: " << pocet << endl; }
    test_constructor(int x):pocet(x) { cout << "Constructor with parameter: "<< pocet << endl; }
    ~test_constructor() { cout << "Destructor\n"; }
  private:
    int pocet;
};

int main(int argc, char* argv[])
{
  test_constructor arr[10] = {1,2,3,4,5,6,7,8,9,10};
}

ale s new se to neda .. to zavola defaultni konstruktory