class Actor;
class ActorManager
{
public:
    void update()
    {
        for (actors_t::const_iterator itr = m_actors.begin(); itr != m_actors.end(); ++itr)
        {
            Actir* actor = itr->second;
            actor->update();
        }
    }

    void add(Actor* actor)
    {
        m_actors[actor->get_id()] = actor;
    }

    void remove(Actor* actor)
    {
        m_actors.erase(actor->get_id());
    }

private:
    typedef std::map<int, Actor*> actors_t;
    actors_t m_actors;
};

而Actor类的实现是这样：

class Actor
{
public:
    void update()
    {
        // ...

    }

有一天，在给Actor添加逻辑的时候，update函数变成了这样

void update()
{
    // ...

    update_buff_effect();

    // ...
}

再往下

class Actor
{
       // ...

private:
    void update_buff_effect()
    {
        // ...

        apply_hp(-100);
        if (get_hp() <= 0)
        {
            die();
            return;
        }

        // ...
    }

然后……

private:
    void die()
    {
        // ...

        ActorManager::getInstance().remove(this);

        // ...
    }

在写下ActorManager的时候并没有想到会在update循环里删除对象，而实际上却有几次遇到类似的问题。

有些问题没有这么明显，但也都是出在遍历容器对象的过程中，某个执行函数删除了窗口里的对象，从而导致迭代器失效。

修改的方法很简单，给ActorManager添加一个待删除对象列表

在remove方法中并不真正删除对象，而是等到update中循环结束后再删除对象。

代码看起来会是这样：

class Actor;
class ActorManager
{
public:
    void update()
    {
        m_is_looping = true;
        for (actors_t::const_iterator itr = m_actors.begin(); itr != m_actors.end(); ++itr)
        {
            Actir* actor = itr->second;
            actor->update();
        }
        m_is_looping = false;

        if (!m_removed_actors.empty())
        {
            for (removed_actors_t::const_iterator itr = m_removed_actors.begin();
                itr != m_removed_actors.end(); ++itr)
            {
                Actor* actor = *itr;
                m_actors.erase(actor->get_id());
            }
            m_removed_actors.clear();
        }
    }

    void add(Actor* actor)
    {
        m_actors[actor->get_id()] = actor;
    }

    void remove(Actor* actor)
    {
        if (!m_is_looping)
            m_actors.erase(actor->get_id());
        else
            m_removed_actors.push_back(actor);
    }

private:
    typedef std::map<int, Actor*> actors_t;
    actors_t m_actors;

    typedef std::vector<Actor*> removed_actors_t;
    removed_actors_t m_removed_actors;
    bool m_is_looping;
};

没有给add也加保护的原因是，不会在update函数内向ActorManager添加新对象。

当然，有可能在其他地方会有这样的需求，同样也做类似的保护即可。

问题虽然不大，但是几次碰到类似的错误了。记录之，并强制要求自己，

在遇到会对容器内的对象做for…处理时，一定要谨慎的检查一下remove接口。

命名规范：

这些只有在用于公开暴露给外界的API时才是必需的

标识符大小写规则：

1.要把PascalCasing用于由多个单词构成的名字空间，类型以及成员的名字

2.要把camelCasing用于参数的名字

3.不要把闭合形式的复合词中每个单词的首字母大写，比如 callback, endpoint 等等，可以查阅英语词典来确定复合词是不是闭合的

4.不要使用匈牙利命名法。原因有几点，一是发明它的ms公司都已经明确要求在新的库在不要使用这种命名法，二是变量名前加类型标识符是个很不好的习惯，在开发过程中有可能会随时修改这些变量的类型定义，三是新的编辑器中不需要用m_前缀来确定其类型，不过对于内部实现的变量来说，用一个前缀也许会让变量的查找更方便，比如用一个_前缀

5.不要使用未被广泛接受的首字母缩写词，如何确定某个缩写词是否众所周知有个好方法，到google上搜索一下，如果前几条都是你所期望的内容，那么它就是众所周知的了

关于命名：

1.要用名词或名词短语来给类和结构体命名，使用PascalCasing的大小写风格，类名字不要加 C，但是接口前需要加 I，这是个特例

2.用形容词短语来给接口命名，在少数情况下也可以使用名词或名词短语

3.考虑在派生类的末尾使用基类的名字，比如 class FileStream : public Stream

4.用动词或动词短语来命名方法，比如 int CompareTo();

5.要用肯定性的短语(CanSeek而不是CantSeek)来命名布尔属性，可以加Is,Can,Has等前缀，要确保使用时的测试语句读起来通顺，比如

if (collection.Contains(item)) 就比 if (collection.IsContained(item)) 要通顺得多

此外，要优先选择主动语态而不是被动语态，比如

if (stream.CanSeek()) 就比 if (steam.IsSeekable()) 要强得多

6.要用现在时和过去时来赋予事件名以之前和之后的概念，不要用Before或After这样的前后缀，比如 Closing, Closed而不是AfterClose

使用规范：

1.优先使用集合，避免使用数组

2.考虑使用不规则数组，而不要使用多维数组，也就是优先使用int [][] jagedArray这样的数组，避免使用 int [,] multiDimArray这样的类型

3.要用最泛的类型来作为参数类型，大多数以集合为参数的成员都使用IEnumerable<T> 接口