在C++中正確地重新分配內存

#include <iostream> 
#include <stdlib.h> 

class circuitTypes{ 

protected: 
    static int size; 
    circuitTypes **Matrix; 
    int input1,input2; 
    int output1, output2; 

public: 

    circuitTypes() {}; 
    static int getSize() { return size; }; 
    static void upSize() { size++; }; 
    void ItemRegistry(); 
    virtual void setTruthTable()=0; 
    void setInputAndCalculateOutput(int a, int b); 
    int *getOutput(); 
}; 

int circuitTypes::size=0; 

int *circuitTypes::getOutput(){ 
    int Output[2]; 
    Output[0]=output1; 
    Output[1]=output2; 
    return Output; 
} 
void circuitTypes::ItemRegistry(){ 

    circuitTypes::upSize(); 
    int circuitSize=circuitTypes::getSize(); 

    if(circuitSize==1) 
     Matrix=(circuitTypes **)malloc(circuitSize*sizeof(circuitTypes *)); 
    else 
     Matrix=(circuitTypes **)realloc(Matrix,circuitSize*sizeof(circuitTypes *)); 
    if(Matrix==0){ 
     std::cout <<"No available memory \n"; 
     exit(1); 
    } 
    Matrix[circuitSize-1]=this; 
} 


void circuitTypes::setInputAndCalculateOutput(int a, int b){ 
    input1=a; 
    input2=b; 
    setTruthTable(); 
} 


class TypeA : private circuitTypes{ 

public: 
    TypeA() { ItemRegistry(); }; 
    void setTruthTable(); 
}; 

void TypeA::setTruthTable(){ 
    if (input1==0){ 
     if (input2==0){ 
      output1=0; 
      output2=0; 
     } 
     else{ 
      output1=0; 
      output2=1; 
     } 
    } 
    else{ 
     if (input2==0){ 
      output1=0; 
      output2=1; 
     } 
     else{ 
      output1=1; 
      output2=1; 
     } 
    } 
} 


class TypeB : private circuitTypes{ 
public: 
    TypeB() { ItemRegistry(); }; 
    void setTruthTable(); 
}; 

void TypeB::setTruthTable(){ 
    if (input1==0){ 
     if (input2==0){ 
      output1=0; 
      output2=0; 
     } 
     else{ 
      output1=0; 
      output2=1; 
     } 
    } 
    else{ 
     if (input2==0){ 
      output1=1; 
      output2=1; 
     } 
     else{ 
      output1=0; 
      output2=1; 
     } 
    } 
} 

void circuit (circuitTypes **Example, int a, int b){ 
    std::cout << "NIKKK"; 
    Example[0]->setInputAndCalculateOutput(a,b); 
    int Size=Example[0]->getSize(); 
    for (int i=1;i<Size;i++){ 

     Example[i]->setInputAndCalculateOutput(Example[i-1]->getOutput()[0],Example[i-1]->getOutput()[1]); 

    } 
    std::cout << "For input a= " << a << " and b= " << b << " the result is c= " << Example[Size-1]->getOutput()[0] << " and d=" << Example[Size-1]->getOutput()[1] << "\n"; 
} 


int main(){ 

    circuitTypes **Example; 
    TypeA A1,A2,A3; 
    TypeB B1,B2,B3; 
    for (int i=0;i<2;i++){ 

     for (int j=0;j<2;j++){ 
      circuit (Example,i,j); 
     } 

    } 

}

我對代碼的大小感到抱歉，但因爲我不知道問題出在哪裏，所以我決定發佈整個程序。 .cpp文件，而不在Ubuntu 11.10任何問題，編譯（使用g ++），但是當我嘗試執行a.out文件我得到這個錯誤：在C++中正確地重新分配內存

* glibc的檢測 ./a.out：realloc的（）：無效指針：0x003c2ff4 * *

=======回溯：=========

/lib/i386-linux-gnu/libc.so.6（ + 0x721a2）[0x2b71a2]

/lib/i386-linux-gnu/libc.so.6(realloc+0x2a5)[0x2bb245]

./a.out[0x8048835]

./a.out[0x8048ca9]

./a.out[0x8048b2a]

/LIB/I386-Linux的GNU/libc的。 so.6（__ libc_start_main + 0xf3）[0x25e113]

./a.out[0x80486c1]

=======存儲器映射：======

00110000 -0012c000 r-xp 00000000 08： 06 1311674 /lib/i386-linux-gnu/libgcc_s.so.1

0012c000-0012d000 - [R - P 0001b000 08:06 1311674 /lib/i386-linux-gnu/libgcc_s.so.1

等等

任何人都可以幫助我嗎？

來源

2013-02-26 iiirxs

我的建議：忘記「malloc」或「realloc」存在。改用'std :: vector'。至於你的真值表，TypeA似乎只是'output1 = input1＆input2'，'output2 = input1 | input2'。你的'getoutput'也返回一個局部變量的地址，所以使用它會導致未定義的行爲（同樣，通過返回一個向量而不是一個數組可以很容易地解決這個問題）。 – 2013-02-26 02:00:12

就在上面，仔細看看你的'main（）'，然後問**，回答**，你自己的這個問題：傳遞給函數'circuit'的'Example'的值是多少）'？ – WhozCraig 2013-02-26 03:50:13

'realloc（0，size）'與'malloc（size）'相同。所以你最初可以將'Matrix'置空，然後執行'Matrix = realloc（Matrix，n）;如果（！Matrix）oom（）;' – 2013-02-27 00:43:35

設置Matrix = realloc是不好的，因爲如果realloc失敗，您永遠無法釋放它。但是，無論如何你都會立即退出，所以代碼不會泄漏，但這仍然是一種不好的做法。
你只是聲明circuitTypes **示例並在電路（例如，i，j）中使用它而不分配內存。您可能想要使用Matrix。
這是一個純粹的猜測：你的代碼結構的方式讓我覺得你想讓Matrix變成靜態的，並用它來代替Example。

來源

2013-02-26 02:17:40 user1952500

if(circuitSize==1) 
     Matrix=(circuitTypes **)malloc(circuitSize*sizeof(circuitTypes *)); 
    else 
     Matrix=(circuitTypes **)realloc(Matrix,circuitSize*sizeof(circuitTypes *));

我不明白這裏的if條件。當然應該是if (Matrix == 0)？否則，你冒着嘗試realloc的風險，你從來沒有malloc -ed首先。

但我同意@JerryCoffin你應該拋出並使用std::vector。

來源

2013-02-26 02:09:31 EJP

在C++中正確地重新分配內存

回答

相關問題