爲什麼我有一個運行時錯誤動態內存分配（堆）？

Question

我每次取消註釋l70-72時，程序都會發生運行時錯誤。這不是我第一次遇到這個特殊問題。

我做一個指向對象的數組，發送指針變成一個功能，嘗試執行在主函數定義的命令，並得到一個運行時錯誤，都同時使用堆。該指針或數組中的指針會發生明顯的變化。

當我調用manage_Rectangle_arr(string strCommand, Rectangle * *arr, short *siArrayL)以下任何一種時出現錯誤：「填充數組」，「獲取區域」和「空內容」（顯然最後兩個與「填充數組」有關，因爲他們需要它首先執行以功能）

void manage_Rectangle_arr(string, Rectangle * *, short*); 

int main() 
{ 
    Rectangle * * arr; 
    short siArrayL=1; 

    manage_Rectangle_arr("make array", arr, &siArrayL); 
    manage_Rectangle_arr("fill array", arr, &siArrayL); //Problem here 
    manage_Rectangle_arr("get areas", arr, &siArrayL); //Problem here 
    manage_Rectangle_arr("empty contents", arr, &siArrayL); //Problem here 
    manage_Rectangle_arr("delete array", arr, &siArrayL); 
} 

void manage_Rectangle_arr(string strCommand, Rectangle * *arr, short *siArrayL) 
{ 
    if(strCommand=="make array") 
    arr = new Rectangle * [ *siArrayL]; 

    if(strCommand=="fill array") //Problem here 
    for(short s=0; s< *siArrayL; ++s) 
    arr[s]= new Rectangle(1, 1); 

    if(strCommand=="get areas") //Problem here 
    for(short s=0; s< *siArrayL; ++s) 
    cout << arr[s]->getArea(); 

    if(strCommand=="empty contents") //Problem here 
    for(short s=0; s< *siArrayL; ++s) 
    delete arr[s]; 

    if(strCommand=="delete array") 
    delete [] arr; 
} 

//When I call manage_Rectangle_arr(string strCommand, Rectangle * *arr, short *siArrayL) with any of the following: "fill array", "get areas" and "empty contents" (obviously the last two are related to "fill array" in nature, since they need it to be executed first in order to function) 

Full C++ Source Code

Answer 1

即使這似乎讓很多downvotes的，我會盡力提供一個有用的答案。 :)

你的陣列是Rectangle**類型的 - 也就是說，一個指向矩形的指針。你可以通過價值把它發送給一個方法。這意味着原始方法中的arr的值將被複制到manage_Rectangle_arr中的變量arr中。因此，在分配arr時，值（new內存區/數組的地址）arr將僅在被調用的函數中包含此值 - 因爲這些值完全是不同的變量。

如果改爲聲明manage_Rectangle_arr作爲

void manage_Rectangle_arr(string strCommand, Rectangle * * &arr, short *siArrayL) 

即變量/存儲區域將被引用發送，這意味着它將是另一個，自動，間接的電平。你可以通過使用一個類型爲Rectangle ***的變量和取消引用變量arr來實現這一點，但這很快就變得愚蠢。

也有許多其他的在你的代碼「不是最佳做法」，但我不會談論他們都在這裏。 :)希望這會幫助你回到正軌！

Answer 2

基本上，除了Max指出的按值調用問題外，這段代碼還有兩個錯誤。

第一個錯誤是您沒有正確使用函數，並濫用字符串以提供命令。第二個錯誤是you are using pointers。每一個C++課程都會犯這個錯誤。指針在大多數C++代碼中都沒有位置。不要陷入壞習慣。

更正和簡化，代碼應該如下所示：

int main() 
{ 
    short siArrayL=1; 
    std::vector<Rectangle> arr(siArrayL); 

    fill_array(arr); 
    print_areas(arr); 
} 

通過兩種方法：特別

void fill_array(std::vector<Rectangle>& arr) { 
    for (std::vector<Rectangle>::iterator i = arr.begin(); i != arr.end(); ++i) 
     *i = Rectangle(1, 1); 
} 

void print_areas(std::vector<Rectangle> const& arr) { 
    for (std::vector<Rectangle>::const_iterator i = arr.begin(); i != arr.end(); ++i) 
     std::cout << i->getArea(); 
} 

注意如何有沒有更多的使用了「空內容」和「刪除陣」。從不使用指針只有兩個好處。