accediendo a un mapa estático de una función de miembro estático - falla de segmentación - C++

Question

Estoy tratando de implementar un patrón de fábrica registrando los punteros de función de la clase derivada a la fábrica en un mapa estático (miembro de la fábrica) y creando objetos por mirando el mapa Pero estoy obteniendo un error de segmentación al hacer esto.

Fragmento de código:

factory.cpp

typedef Shape* (*Funcptr)(); 

std::map<int,Funcptr> Factory::funcmap; 

int Factory::registerCreator(int ShapeID, Shape *(*CFuncptr)()) { 
    Factory::funcmap[ShapeID] = CFuncptr; 
return 1; 
} 

Shape* Factory::CreateObject(int ShapeID) { 
    std::map<int,Funcptr>::iterator iter; 
    iter = funcmap.find(ShapeID); 
    if(iter != funcmap.end()){ 
     return iter->second(); 
    } 
    return NULL; 
} 

factory.h

class Factory { 
public: 
    Factory(); 
    virtual ~Factory(); 
    static int registerCreator(int, Shape *(*CFuncptr)()); 
    Shape* CreateObject(int); 
private: 
    static std::map<int,Funcptr> funcmap; 
}; 

Square.cpp

static Shape *SquareCreator() { 
    return new Square; 
} 
static int SquareAutoRegHook = Factory::registerCreator(1,SquareCreator); 

En la creación del objeto de fábrica en el principal archivar un segmentati en falla ocurre. ¿Puede sugerir si estoy haciendo algo mal? Estoy usando CppUTest para TDD y no estoy seguro de cómo depurar esto.

Answer 1

No hay ninguna garantía sobre el orden de creación de objetos estáticos que se definen en diferentes traducciones uints * por lo que tiene un tiro 50/50 en cuanto a qué pasaría en primer lugar, la initializtion de Factory::funcmap o Factory::registerCreator(1,SquareCreator) y un comportamiento indefinido ruleta rusa no es un buen juego para jugar.

Un enfoque común para hacer frente a esto, y uno que se describe en el artículo 4 de the third edition of Scott Meyer's Effective C++ es utilizar objetos estáticos locales en lugar de objetos estáticos globales. En este caso, significa cambiar Factory a tener este aspecto:

class Factory { 
public: 
    Factory(); 
    virtual ~Factory(); 
    static int registerCreator(int, Shape *(*CFuncptr)()); 
    Shape* CreateObject(int); 
private: 
    static std::map<int,Funcptr> & GetFactoryMap() { 
     static std::map<int,Funcptr> funcmap; 
     return funcmap; 
    } 
}; 

y cambiando Factory::registerCreator a esto:

int Factory::registerCreator(int ShapeID, Shape *(*CFuncptr)()) { 
    GetFactoryMap()[ShapeID] = CFuncptr; 
    return 1; 
} 

Este funcmap manera se iniciará la primera vez registerCreator se llama y nunca será utilizado sin inicializar .

* O, en términos generales, los diferentes archivos .cpp si no está familiarizado con el término unidad de traducción

Answer 2

Se parece al static initialization order fiasco. Parece que en el momento en que se inicializó SquareAutoRegHook, el funcmap no se ha creado aún.