truy cập vào bản đồ tĩnh từ một hàm thành viên tĩnh - lỗi phân đoạn - C++

Question

Tôi đang cố gắng triển khai mẫu nhà máy bằng cách đăng ký con trỏ hàm của lớp dẫn xuất đến nhà máy trong bản đồ tĩnh (thành viên của nhà máy) và tạo đối tượng nhìn lên bản đồ. Nhưng tôi nhận được một lỗi phân đoạn khi làm điều này.

Đoạn Mã:

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); 

On tạo đối tượng cho Nhà máy trong chính nộp một phân đoạn về lỗi xảy ra. Bạn có thể đề nghị nếu tôi làm sai điều gì đó. Tôi đang sử dụng CppUTest cho TDD và không chắc chắn làm thế nào để gỡ lỗi này.

Answer 1

Không đảm bảo về thứ tự tạo các đối tượng tĩnh được xác định trong các bản dịch khác nhau * do đó bạn có ảnh 50/50 như trước, phiên bản đầu tiên của Factory::funcmap hoặc Factory::registerCreator(1,SquareCreator) và Hành vi không xác định Russian Roulette không phải là một trò chơi hay để chơi.

Một cách tiếp cận chung để giải quyết vấn đề này và cách được mô tả trong mục 4 của the third edition of Scott Meyer's Effective C++ là sử dụng các đối tượng tĩnh cục bộ thay vì các đối tượng tĩnh chung. Trong trường hợp này nó có nghĩa là thay đổi Factory trông như thế này:

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; 
    } 
}; 

và thay đổi Factory::registerCreator này:

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

cách funcmap này sẽ được khởi tạo lần đầu tiên registerCreator được gọi và sẽ không bao giờ được sử dụng chưa được khởi tạo .

* Hoặc, nói chung, các tệp .cpp khác nhau nếu bạn không quen với cụm từ dịch thuật

Answer 2

Hình như static initialization order fiasco. Dường như tại thời điểm SquareAutoRegHook được khởi tạo, funcmap chưa được tạo.