做一个关于兔子群的练习，撞墙了

Question

我正在做这个列表中的最后一个练习（称为毕业）：http://www.cplusplus.com/forum/articles/12974/但有一个主要问题。我写的代码运行，但它会在这个时候崩溃（在删除半个兔子之后），有时在程序第一次删除半个兔兔之后，有时在10个这样的周期之后，请注意，我还没有实现太多，因为我想在你的帮助下修复这个错误。我也知道这不是代码审查，但一些关于风格和改进的小技巧也会很好。这是我迄今为止写的代码：

Main.cpp:

include bunnyList.h
include windows.h
using namespace std;
int main(){
    srand(time(NULL));
    bunnyList Colony;
    int turns = 0;
    Colony.setUp();
    while(Colony.getColonySize() > 0){
        //New turn
        Colony.increaseAgeAndKill();
        Colony.breedBunnies();
        std::cout << "Turn: "<< turns << ". Colony size: " << Colony.getColonySize() << std::endl;
        //Get rid of these food eaters
        if(Colony.getColonySize() > 1000){
        std::cout << "500 bunnies died!" << std::endl;
        Colony.killHalfBunnies();
        }
    Sleep(100);
    turns++;;
    }
}

bunnyList.h:

#ifndef BUNNYLIST_H
#define BUNNYLIST_H
#include <stdlib.h>
#include "node.h"
#include <time.h>
#include <iostream>
#include <string>
const int numOfNames = 4;
const int numOfColors = 4;
const int bunniesIni = 5;
const std::string colors[numOfColors] = {"Black", "White", "Brown", "Spotted"};
const std::string maleNames[numOfNames] = {"Joe", "Rafael", "Buby", "Messi"};
const std::string femaleNames[numOfNames] = {"Reichel", "Agnesa", "Mr Flufy", "Flower"};
class bunnyList{
private:
    node *head;
    int noOfBunnies;
    node *current, *prev;
public:
    bunnyList();
    void newBunny(std::string);
    void killHalfBunnies();
    void increaseAgeAndKill();
    void deleteNode();
    void breedBunnies();
    void setUp();
    int getRandomNumber(int) const;
    std::string getRandomColor();
    std::string getRandomName(bool);
    bool isMaleRandom();
    int getColonySize() const;
};
#endif

bunnyList.cpp:

#include "bunnyList.h"
bunnyList::bunnyList(){
        noOfBunnies = 0;
}
void bunnyList::setUp(){
        std::string temp = "";
        head = NULL;
        for(int i = 0; i <= bunniesIni; i++){
            newBunny(temp);
        }
}
void bunnyList::killHalfBunnies(){
    prev = head;
    current = head;
    while(noOfBunnies > 500){
        if(getRandomNumber(2) == 1){
            deleteNode();
            continue;
        } else if(current == NULL){
            current = head;
            prev = head;
       } else {
            prev = current;
            current = current->next;
            continue;
        }
    }
}
void bunnyList::newBunny(std::string color){
    node *bunny = new node();
    node *temp = head;
    if(color == ""){
        bunny->color = getRandomColor();
    } else {
        bunny->color = color;
    }
    bunny->isMale = isMaleRandom();
    bunny->name = getRandomName(bunny->isMale);
    bunny->age = 0;
    bunny->next = NULL;
    bunny->isBreedable = 0;
    if(head == NULL){
        head = bunny;
        return;
    }
    while(temp->next != NULL){
        temp = temp->next;
    }
    temp->next = bunny;
    noOfBunnies++;
}
void bunnyList::increaseAgeAndKill(){
    current = head;
    prev = head;
    while(current != NULL){
        current->age++;
        //Check if bunny can breed
        if(current->age > 2){
            current->isBreedable = 1;
        }
        //Check if its time to die :/
        if(current->age > 10){
            deleteNode();
        }
        prev = current;
        current = current->next;
    }
    current = head;
    prev = head;
}
void bunnyList::breedBunnies(){
    node *temp = head;
    bool oneMale = 0;
    int femaleCount = 0;
    //Check if there is at least one breedable male
    while(temp!=NULL){
        if(temp->isMale && temp->isBreedable){
            oneMale = 1;
            break;
        }
        temp = temp->next;
    }
    //For every female bunny over 2 years old a new bunny is born
    temp = head;
    if(oneMale){
    while(temp != NULL){
            if(temp->isMale == 0 && temp->isBreedable){
                newBunny(temp->color);
            }
        temp = temp->next;
        }
    }
}
void bunnyList::deleteNode(){
    if(current==head){
        head = current->next;
        prev = head;
        delete current;
        current = head;
        noOfBunnies--;
    } else if(current->next==NULL){
        delete current;
        prev->next = NULL;
        prev = head;
        current = head;
        noOfBunnies--;
    } else {
        prev->next = current->next;
        current->next = NULL;
        delete current;
        current = prev->next;
        noOfBunnies--;
    }
}
std::string bunnyList::getRandomName(bool isMale){
    int r = getRandomNumber(numOfNames - 1);
    if(isMale)
    return maleNames[r];
    return femaleNames[r];
}
std::string bunnyList::getRandomColor(){
    int r = getRandomNumber(numOfColors - 1);
    return colors[r];
}
bool bunnyList::isMaleRandom(){
    if(getRandomNumber(2) == 1) {return true;}
    return false;
}
int bunnyList::getRandomNumber(int limit) const{
    return rand() % limit + 1;
}
int bunnyList::getColonySize() const{
    return noOfBunnies;
}

node.h:

#ifndef NODE_H_INCLUDED
#define NODE_H_INCLUDED
#include <string>
class node {
    friend class bunnyList;
private:
    std::string name;
    int age;
    std::string color;
    bool isMale;
    node *next;
    bool isBreedable;
public:
};
#endif // NODE_H_INCLUDED

谢谢你的帮助。

Answer 1

既然你要求审查。。。

切勿写入using namespace std。从不就在今天早上，SO上有一个问题被问到，手头问题的原因是那条臭名昭著的线路。我想知道是谁和为什么认为这是一个好方法——应该有一本关于这一点的书。如果我有办法的话，它的作者将永远被手动从每个文件中删除这一行。

即使不从代码中读一行，仅凭解释，我也知道这个问题很可能（100%可能，如中所示）与内存管理有关。您正在释放未正确分配的内存，您正在释放同一内存两次，或者您正在释放根本未分配的内存或者您正在访问释放后的内存。查看您的delete并检查它们。

关于风格。您的代码基本上是业务逻辑感知列表的实现。一般来说，这不是一个好的做法。实现一个泛型列表，支持添加、删除和其他泛型列表操作，比在这个泛型列表之上实现业务逻辑要好得多。

不要在列表中使用current。相反，在delete函数中传递一个要删除的节点。

最后，在调试器中运行程序，查看要删除的变量。

编辑在通信中回答问题。

这就是我所说的业务逻辑分离。有一个通用的数据结构，称为列表。它可以是任何东西的列表，兔子或太空火箭，无关紧要，但它仍然支持基本的列表操作。显然，最重要的两个操作是插入和删除，但这并不是泛型列表的唯一操作。您可以阅读列表（数据结构）上的维基百科以获得一般想法，并在实现中查看std:：list。现在，您有了列表的特定用例，一个兔子列表。对于那个特定的用例，您将在通用列表的顶部添加功能。为了进一步澄清，从列表中删除项目是泛型列表支持的。但当这个可怜的家伙10岁时"杀死一个疯子"是一种商业逻辑。它包含迭代兔子列表（由泛型列表提供），检查年龄并做出消除生物的决定（业务逻辑级别），然后删除元素（泛型列表）。如果使用std::list编写此代码，则大致如下所示：

std::list<Bunny> bunnies;
for (auto bunny = bunnies.cbegin(), end = bunnies.cend(); bunny != end; ++bunny) {
    if (bunny->age() > 10)
        bunny = bunnies.erase(bunny);
}

Answer 2

我也崩溃了。它在deleteNode中崩溃，试图在current为NULL时引用current->next。

这是从bunnyList::killHalfBunnies调用的，在这段代码中，这就是我发现的问题：

    if (getRandomNumber(2) == 1){
        deleteNode();
        continue;
    }
    else if (current == NULL){
        current = head;
        prev = head;
    }

问题是您调用deleteNode，假设current不为NULL，然后检查current实际上不为NULL。我重新排列了if，如图所示，我不再崩溃：

    if (current == NULL)
    {
        current = head;
        prev = head;
    }
    else if (getRandomNumber(2) == 1)
    {
        deleteNode();
        continue;
    }

在deleteNode中放入一个检查也是明智的，这样，如果在current为NULL时调用它，它就可以处理它。可能是通过抛出异常或以其他方式警告你。

既然你也问过风格：deleteNode（以及其他地方）中的评论会让一切变得更清楚！

我认为这是一个课堂作业，这就是为什么不使用std::list。