C++의 소멸자를 통해 동적 배열의 벡터를 소멸

Question

나는 이미지 조작 (블렌딩 및 확대/축소)과 관련된 과제 중 하나를 수행하고 있으며 어려움을 극복해야한다는 문제에 봉착했습니다. 내 응용 프로그램의

구조

• RGB 구조체

  클래스 이미지 : 오버로드 운영자 (그들은 평평 픽셀이야) 수레가 포함되어 있습니다.

• 픽셀 : 생성자를 통해 h * w가되도록 초기화되는 1 차원 픽셀 배열
• 클래스 소멸자. 공용 클래스 이미지
  :

소멸자 지금

virtual ~Image() 
{ 
    if (pixels != NULL) 
     delete[] pixels; 
} 

내가 이미지에서

클래스 필터를 상속 필터라는 다른 클래스를 사용하고 ... 조금 다음과 같습니다

• 이미지의 벡터 imgStack; 블렌드 할 이미지의 컨테이너
• std :: vector of Rgb pixelBuffer; 각 이미지 1 픽셀의 픽셀 컨테이너입니다. 이것은 동적이 아니므로 삭제를 걱정하지 않아도됩니다.

파생 클래스의 소멸자. 나는 기본적으로 벡터에 이미지를 저장하는 혼합 객체에 13 개 이미지를 실행 해요 내 MAIN.CPP의이 부분이 어떻게 생겼는지

~Blend() 
{ 
    delete &imageStack; 
} 

... 내 MAIN.CPP에서

while (userInput != 5) 
{ 
    switch(userInput) 
    case 1: 
    { 
     Blend *meanImage = new Blend(3263, 2505, 13); 
     meanImage->writePPM("Mean_" + std::to_string(count) + ".ppm");//every iteration, the file name is unique 
     meanImage->~Blend(); 
    } 
} 

내 모든 기능을 수행 할 컨테이너. 실행 시간 동안 사용 된 공간은 약 1.3GB이지만 내 객체가 루프에 있으므로 (다중 기능을위한 메뉴가 있음) 객체는 대개 범위를 벗어나지 않으므로 소멸자가 자동으로 호출되지 않으므로 호출합니다 그것은 수동으로 이것을 좋아한다; medianImage-> ~ Blend(); 이제 모든 오류는 내 응용 프로그램이 "중단 점을 트리거했습니다."라는 오류 메시지가 표시됩니다. 참고로 중단 점은 어디에도 없습니다. 동적 배열을 사용하는 것은 일반적으로 모든 유형의 메모리 문제를 일으키는 것이 일반적으로 나쁘다는 것을 알고 있습니다.하지만이 문제를 해결하기 위해 앞으로 이러한 문제를 해결하는 방법을 알고 싶습니다.

코드에 대한 질문 사항이 있으면 snippers를 게시 할 수 있습니다.

편집 : 여기 내 블렌드 수업이 있습니다.

#pragma once 
#include "stdafx.h" 
#include "Image.h" 
#include <vector> 
#include <string> 
class Blend : public Image 
{ 
private: 
    std::vector<Image> imageStack; 
    std::vector<Rgb*> pixelBuffer;//only works with pointers (no copying) 
    int m_noOfImages; 

    Rgb* getMedianFromSet(); 
    Rgb getMeanFromSet(); 
    Rgb getStdDev(Rgb meanPix); 
public: 
    Blend(int width = 0, int height = 0, int noOfImages = 0):Image(width, height), m_noOfImages(noOfImages), imageStack(noOfImages), pixelBuffer(noOfImages) 
    {} 
public: 
    void stack(bool push = true); 
    void meanBlend(); 
    void medianBlend(); 
    void sigmaClipping(float sigma = 0.5f); 

    ~Blend() 
    { 
     delete &imageStack; 
    } 
}; 

Answer 1

#pragma once 
#include "stdafx.h" 
#include "Image.h" 
#include <vector> 
#include <string> 
#include <memory> 
class Blend : public Image 
{ 
private: 
    std::vector<Image> imageStack; 
    // Changed to shared_ptr<T> could use unique_ptr<T> depending on need. 
    std::vector<std::shared_ptr<Rgb>> pixelBuffer;//only works with pointers (no copying) 
    int m_noOfImages; 

    Rgb* getMedianFromSet(); 
    Rgb getMeanFromSet(); 
    Rgb getStdDev(Rgb meanPix); 
public: 
    Blend(int width = 0, int height = 0, int noOfImages = 0):Image(width, height), m_noOfImages(noOfImages), imageStack(noOfImages), pixelBuffer(noOfImages) 
    {} 
public: 
    void stack(bool push = true); 
    void meanBlend(); 
    void medianBlend(); 
    void sigmaClipping(float sigma = 0.5f); 

    // Clear Entire Buffer 
    void cleanup() { 
     // When using the heap with smart pointers 
     for (auto item : containerVariable) { 
      item.reset(); 
      item = nullptr; 
     } 
     containerVariable.clear(); 
    } 

    // Remove Single Element 
    void remove(unsigned idx) { 
     // Write function to remove a single element from vector 
    } 

    ~Blend() 
    { 
     // This is definitely wrong here: You are trying to delete a reference 
     // to a template container that is storing `Image` objects that 
     // are on the stack. 
     // delete &imageStack; 
    } 
}; 

함수 메모리를 정리하고,이 클래스의 소멸자를 사용하는 것보다 동적 메모리를 사용하는 경우 용기에서 특정 요소를 제거하는 물품 것이 좋다.