Question

#include <iostream>
#include <fstream>
#include <cmath>
#include <string>
#include <sstream>

using namespace std;

class CFile {
public:
    CFile(string filename); 
    ~CFile();               
    void ReadFile();         
    void WriteFile(string outputFilename);   
    void Calculate();   
    string  m_filename;
    int     m_numberInput;
    double* m_xData;         
    double* m_yData;
    int     m_numberOutput;
    double* m_xDataOut;     
    double* m_yDataOut;
};

CFile::CFile(string filename)
{
    m_filename = filename;
    string line;             
    ifstream myfile(m_filename.c_str());

    if (myfile.is_open())     
    {
        getline(myfile,line);    
        myfile.close();          
        stringstream Str;        
        Str << line; 
        Str >> m_numberInput;
        m_xData = new double[m_numberInput];  
        m_yData = new double[m_numberInput];
        cout << sizeof(m_xData) / sizeof(m_xData[0]) << endl;
    }
    else cout << "Unable to open file.\n";  
}

CFile::~CFile()
{ 
    delete[] m_xData; 
    delete[] m_yData;                                  
    m_xData = 0;                                 
    m_yData = 0;
    delete[] m_xDataOut; 
    delete[] m_yDataOut;                                  
    m_xDataOut = 0;                                 
    m_yDataOut = 0;
}

void CFile::ReadFile()
{
    ifstream infile(m_filename.c_str());
    string line;

    if (infile.is_open())
    {
        int x, y, i = 0;

        while (getline(infile,line))
        {
            infile >> x >> y;
            m_xData[i] = x;
            m_yData[i] = y;
            i++;
        }
        infile.close();
    }
    else cout << "Unable to open file.\n"; 
}

void CFile::WriteFile(string outputFilename)
{
    ofstream outfile(outputFilename.c_str());

    if (outfile.is_open())
    {
        for(int i=0; i < m_numberInput; i++)
            outfile << m_xDataOut[i] << " " << m_yDataOut[i] << endl;

        outfile.close();
    }
    else cout << "Unable to open file.\n"; 
}

void CFile::Calculate()
{
    m_xDataOut = new double[m_numberInput]; 
    m_yDataOut = new double[m_numberInput];

    for(int i=0; i < m_numberInput; i++)
    {
        m_xDataOut[i] = m_xData[i];
        m_yDataOut[i] = sqrt(m_yData[i]);
    }
}


int main()
{
    CFile file("Input.dat");
    file.ReadFile(); 
    file.Calculate(); 
    file.WriteFile("Output.dat");
    file.~CFile();
}

错误消息如下：

main(11915,0x7fff77d3d310) malloc: *** error for object 0x7f8a99403940: pointer 
being freed was not allocated
*** set a breakpoint in malloc_error_break to debug
Abort trap: 6

这是几周前具有讽刺意味的代码。我改变了OS和编译器，现在它不再了。我在其他线程中读到了这个错误，但无法真正理解如何在我的代码中使用解决方案。代码读入数据文件，确实如此。使用它然后将更改的值写入另一个文件。分配错误在哪里？非常感谢您的帮助！： - ）

Answer 1

你的析构函数被调用两次。您可以显式调用它，并在退出main（）时调用它。您应该删除显式的destrcutor调用。

Answer 2

两个问题：

您不必初始化所有四个指针，在这种情况下，将delete应用于它们是不安全的。您可以在执行任何操作之前将它们初始化为null来解决此问题;或者，更好的是，用std::vector<double>替换它们，这样你根本不必弄乱delete。这也将修复类的无效复制语义（它打破Rule of Three）以及如果构造失败可能的内存泄漏。
您正在file.~CFile()的末尾手动调用析构函数（main）。不要那样做：当程序离开变量的范围时会自动调用析构函数，并且调用它两次是错误的。

Answer 3

正如@claptrap指出你的记忆管理也有错误。

您应该迁移到std::vector<double>而不是原始double数组和指针。您可以将vec.resize(N)视为new double[N]的变体，从不需要明确的delete[]。
如果您使用的是Linux，最好在valgrind下运行程序，该程序会自动跟踪memomry分配/解除分配并指出任何无效的内存操作。我打赌你的Linux发行版有一个预编译的包。
- http://valgrind.org/docs/manual/quick-start.html

Answer 4

这引起了问题

  delete[] m_xDataOut; 
  delete[] m_yDataOut;

您尚未初始化构造函数中的指针

＆＃34;被释放的指针未被分配＆＃34;

4 个答案: