带指针参数的C ++函数

时间:2017-07-27 23:23:35

标签: c++ xcode pointers memory-management

我正在编写一个C ++程序,将数据输出到文件,生成 python 脚本并调用pyplot进行绘图。

但是,当我根据指针传递参数时,它可以正确编译,但无法运行。它返回错误。当我使用 Xcode调试模式并逐步执行时,它会偶然提供正确的结果,但并非总是如此。有时它也会返回错误。

我怀疑它可能是由某些内存分配问题引起的,但我无法确定究竟是什么问题。

我的代码如下:

1)主要

#include <iostream>
#include <stdlib.h>
#include <cmath>
#include "PyCPlot.h"

using namespace std;

double pi = 3.1415926;

int main(int argc, const char * argv[]) {

    int nline = 100;

    double * np_list = new double(nline);
    double * pack_fraction_np = new double (nline);

    for (int i=0; i<nline; i++){
        np_list[i] = double(i)/double(nline)*2*pi;
        pack_fraction_np[i] = cos(np_list[i]);
    }

    PyCPlot_data_fout("RandomPacking", nline, np_list, pack_fraction_np);
    PyCPlot_pythonscript("RandomPacking", "Random Packing");
    PyCPlot_pyplot("RandomPacking", "Random Packing");

    return 0;
}

2)头文件

#ifndef PyCPlot_h
#define PyCPlot_h

#include <iostream>
#include <cmath>
#include <fstream>
#include <string>
#include <stdlib.h>

using namespace std;


int PyCPlot_data_fout(string datafilename, int nline, double * x, double *y){
    ofstream fout;
    fout.open(datafilename+".txt");
    fout << nline << endl;
    for (int i=0; i<nline; i++){
        fout << x[i] << "  " << y[i] << endl;
    }
    fout.close();
    return 0;
}

int PyCPlot_pythonscript(string datafilename, string plttitle){

    string strhead = "import numpy as np\nimport matplotlib.pyplot as plt\n";
    string strpltfig = "plt.figure()\n";
    string strpltplt = "plt.plot(xlist, ylist)\n";
    string strplttit = "plt.title('"+plttitle+"')\n";
    string strpltshw = "plt.show()\n";

    string strfileopen ="f = open('"+datafilename+".txt', 'r')\n";
    string strreadline ="size = map(int, f.readline().split())\n";
    string strpltlist ="xlist = np.zeros((size))\nylist = np.zeros((size))\n";
    string strfor = "for i in range(size[0]):\n    xlist[i], ylist[i] = map(float, f.readline().split())\n";

    ofstream pyout;
    pyout.open("PyCPlot_"+datafilename+".py");
    pyout << strhead << strfileopen << strreadline << strpltlist << strfor;
    pyout << strpltfig << strpltplt << strplttit << strpltshw;
    pyout.close();
    return 0;
}

int PyCPlot_pyplot(string datafilename, string plttitle){

    string strsystemsh ="source ~/.bashrc; python PyCPlot_"+datafilename+".py";
    system(strsystemsh.c_str());

    return 0;
}

#endif /* PyCPlot_h */

运行时,我收到以下错误消息

malloc: *** error for object 0x1002002e8: incorrect checksum for freed object - object was probably modified after being freed.

3 个答案:

答案 0 :(得分:4)

你想传递一个数组,所以传递一个可以在运行时调整大小的实际数组(std::vector),而不是一些希望指向数组第一个元素的随机指针(在这种情况下) ,它没有&lt; t)

您的错误是使用new double(x)而不是new double[x]。前者分配一个double,其值等于x,后者分配一个大小为double的{​​{1}}数组,并返回指向第一个元素的指针,但是正如我已经说过的那样,如果你实际使用x并且没有涉及像90年代早期风格的指针那么你根本不会遇到这个问题(更不用说,你不会这样做)如果使用std::vector),则会出现内存泄漏。

答案 1 :(得分:3)

你做了一些不正确或看似奇怪的事情。

代码new double(nline)将分配一个值为nline的双精灵,这看起来并不像你想要的那样。 看起来您打算动态分配 double 的数组。实际上,从您已经显示的代码中可以看出,由于在编译时已知大小,因此您无法做一个简单的数组。

以下代码:

double * np_list = new double(nline);
double * pack_fraction_np = new double (nline);

可以替换为:

double np_list[nline];
double pack_fraction_np[nline];

答案 2 :(得分:0)

以下几行为np_listpack_fraction_np分配了一个双精度数据:

double * np_list = new double(nline);
double * pack_fraction_np = new double (nline);

因此,当您进入循环时,您将数据写入无效的内存块。你必须使用方括号来定义一个数组(我没有得到任何错误;使用clang ++)。

考虑使用std::vector类。以下示例(您的示例)std::pair double嵌套在std::vector中:

PyCPlot.h

#ifndef PyCPlot_h
#define PyCPlot_h

#include <iostream>
#include <cmath>
#include <fstream>
#include <string>
#include <stdlib.h>
#include <vector>
#include <utility>

using namespace std;

int PyCPlot_data_fout(string datafilename, std::vector<std::pair<double, double>>& v){
    ofstream fout;
    fout.open(datafilename+".txt");
    fout << v.size() << endl;
    for (int i=0; i < v.size(); i++){
        fout << v[i].first << "  " << v[i].second << endl;
    }
    fout.close();
    return 0;
}

int PyCPlot_pythonscript(string datafilename, string plttitle){

    string strhead = "import numpy as np\nimport matplotlib.pyplot as plt\n";
    string strpltfig = "plt.figure()\n";
    string strpltplt = "plt.plot(xlist, ylist)\n";
    string strplttit = "plt.title('"+plttitle+"')\n";
    string strpltshw = "plt.show()\n";

    string strfileopen ="f = open('"+datafilename+".txt', 'r')\n";
    string strreadline ="size = map(int, f.readline().split())\n";
    string strpltlist ="xlist = np.zeros((size))\nylist = np.zeros((size))\n";
    string strfor = "for i in range(size[0]):\n    xlist[i], ylist[i] = map(float, f.readline().split())\n";

    ofstream pyout;
    pyout.open("PyCPlot_"+datafilename+".py");
    pyout << strhead << strfileopen << strreadline << strpltlist << strfor;
    pyout << strpltfig << strpltplt << strplttit << strpltshw;
    pyout.close();
    return 0;
}

int PyCPlot_pyplot(string datafilename, string plttitle){

    string strsystemsh ="source ~/.bashrc; python PyCPlot_"+datafilename+".py";
    system(strsystemsh.c_str());

    return 0;
}

#endif /* PyCPlot_h */

PyCPlot.cpp

#include <iostream>
#include <stdlib.h>
#include <cmath>
#include "PyCPlot.h"

using namespace std;

double pi = 3.1415926;

int main(int argc, const char * argv[]) {

    int nline = 100;
    std::vector<std::pair<double, double>> v;
    v.reserve(nline);

    double a;
    for (int i=0; i < nline; i++){
        a = double(i)/double(nline)*2*pi;
        v.push_back(std::make_pair(a, cos(a)));
    }

    PyCPlot_data_fout("RandomPacking", v);
    PyCPlot_pythonscript("RandomPacking", "Random Packing");
    PyCPlot_pyplot("RandomPacking", "Random Packing");

    return 0;
}