Форум на Исходниках.RU https://forum.sources.ru Форум на Исходниках.RU Форум на Исходниках.RU https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3489631 Wed, 11 Jun 2014 04:13:23 +0000 Нелокальные статические объекты и статические библиотеки https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3489631 Цитата Urich @
пс. У линковщика есть параметр кажется /OPT:NOREF который запрещает выкидывать не используемый код.

Полезно.

Правда, я уже выделил регистрации в отдельный файл.

Всем спасибо.]]> WhyNot C/C++: Общие вопросы https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3489438 Tue, 10 Jun 2014 11:46:08 +0000 Нелокальные статические объекты и статические библиотеки https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3489438 Цитата WhyNot @
А какие проблемы могут возникнуть со статическими переменными?

Имелось ввиду возможность их двойной инициализации.

Цитата WhyNot @
Что-то вроде *.cpp-файла, в котором перечисляются все функции которые регистрируются ,а потом функцию, которая их всех регистрирует; и потом вызывать эту функцию где-нибудь в main?

Ну как то так. Делать регистрацию явно, когда уже приложение инициализировано.

пс. У линковщика есть параметр кажется /OPT:NOREF который запрещает выкидывать не используемый код.]]> Urich C/C++: Общие вопросы https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3489135 Mon, 09 Jun 2014 15:08:30 +0000 Нелокальные статические объекты и статические библиотеки https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3489135 Цитата Urich @
Код фабрики покажи.


ActorFactory.h
    #ifndef ACTOR_FACTORY
    #define ACTOR_FACTORY
     
    #include <map>
    #include "../Core/World.h"
     
    /**
     * Factory that allows to create any type of registered actor by their name identifier.
     *
     * Based on pattern Factory in book Andrei Alexandrescu "Modern C++ Design: Generic Programming and Design Patterns Applied"
     *
     * ActorFactory is a singleton.
     */
    class ActorFactory
    {
    public:
        typedef IActor* (*CreateActorCallback)(World *world, const Vector2D location, const Vector2D scale, const Rotator rotation);
    private:
        typedef std::map<std::string, CreateActorCallback> CallbackMap;
    public:
        /**
         * @return the single instance of the ActorFactory
         */
        static ActorFactory& Factory();
     
        /**
         * Register new actor class by an identifier and a "create" function.
         * If identifier already exists, factory will not be registered.
         *
         * @return true if class wasn't registered earlier
         */
        bool registerActor(std::string actorId, CreateActorCallback createFn);
     
        /**
         * Unregister already registred class.
         *
         * @return true if class was unregistered
         */
        bool unregisterActor(std::string actorId);
     
        /**
         * Create actor by identifier.
         *
         * @throws runtime_error when actorId is unknown
         */
        IActor* placeActor(std::string actorId, World *world, const Vector2D location, const Vector2D size, const Rotator rotation);
    private:
        CallbackMap callbacks;
     
        static ActorFactory* singleInstance;
     
        /*
         * Turn off unusable operations
         */
        ActorFactory();
        ~ActorFactory();
        ActorFactory(const ActorFactory&);
        void operator=(const ActorFactory&);
    };
     
    #endif


ActorFactory.cpp
    #include "ActorFactory.h"
     
    ActorFactory* ActorFactory::singleInstance = nullptr;
     
    ActorFactory::ActorFactory() {}
    ActorFactory::~ActorFactory() {}
     
    ActorFactory& ActorFactory::Factory()
    {
        if (ActorFactory::singleInstance == nullptr)
        {
            ActorFactory::singleInstance = new ActorFactory();
        }
     
        return *ActorFactory::singleInstance;
    }
     
    bool ActorFactory::registerActor(std::string actorId, CreateActorCallback createFn)
    {
        return this->callbacks.insert(CallbackMap::value_type(actorId, createFn)).second;
    }
     
    bool ActorFactory::unregisterActor(std::string actorId)
    {
        return this->callbacks.erase(actorId) == 1;
    }
     
    IActor* ActorFactory::placeActor(std::string actorId, World *world, const Vector2D location, const Vector2D scale, const Rotator rotation)
    {
        CallbackMap::const_iterator it = this->callbacks.find(actorId);
        
        if (it == this->callbacks.end())
        {
            throw std::runtime_error("Unknown actor identefier");
        }
        
        return (it->second)(world, location, scale, rotation);
    }


Цитата Urich @
А вообще такая регистрация плохая идея. Также как и статические переменные в либах.

А какие проблемы могут возникнуть со статическими переменными?


Как тогда лучше оформить регистрации в фабрике?
Что-то вроде *.cpp-файла, в котором перечисляются все функции которые регистрируются ,а потом функцию, которая их всех регистрирует; и потом вызывать эту функцию где-нибудь в main?

Цитата MyNameIsIgor @
Что такое "одно место линковки"?

Я думаю, имеется ввиду только один exe или dll, использующий библиотеку.]]> WhyNot C/C++: Общие вопросы https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3489009 Mon, 09 Jun 2014 11:06:33 +0000 Нелокальные статические объекты и статические библиотеки https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3489009 Цитата Urich @
Пока у тебя в проекте одно место линковки

Что такое "одно место линковки"?
Цитата Urich @
пока не используешь менеджеры памяти отличные от рантаймоских

Не вижу проблем.
Цитата Urich @
Пока это только твой проект, у тебя все хорошо.

Как раз IoC всегда помогало делать проект "не только моим".]]> MyNameIsIgor C/C++: Общие вопросы https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3489005 Mon, 09 Jun 2014 11:02:06 +0000 Нелокальные статические объекты и статические библиотеки https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3489005 Цитата MyNameIsIgor @
Идея хорошая, способствующая слабой связности. Но компоновщик её вечно портит.

Я с этой хорошей идей намучился. Пока у тебя в проекте одно место линковки, пока не используешь менеджеры памяти отличные от рантаймоских. Пока это только твой проект, у тебя все хорошо.]]> Urich C/C++: Общие вопросы https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3488976 Mon, 09 Jun 2014 10:13:49 +0000 Нелокальные статические объекты и статические библиотеки https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3488976 Цитата Urich @
Код фабрики покажи

Код фабрики здесь не при чём. Компоновщик выбрасывает "неиспользуемый" код, вот и всё - проблема известная.
Цитата Urich @
А вообще такая регистрация плохая идея.

Идея хорошая, способствующая слабой связности. Но компоновщик её вечно портит. Меня тоже волнует вопрос топикстартера.
Цитата Urich @
Также как и статические переменные в либах.

:blink: С чего бы?]]> MyNameIsIgor C/C++: Общие вопросы https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3488965 Mon, 09 Jun 2014 09:59:11 +0000 Нелокальные статические объекты и статические библиотеки https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3488965 WhyNot
Код фабрики покажи. А вообще такая регистрация плохая идея. Также как и статические переменные в либах.]]> Urich C/C++: Общие вопросы https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3488720 Sun, 08 Jun 2014 07:37:47 +0000 Нелокальные статические объекты и статические библиотеки https://forum.sources.ru/index.php?showtopic=390835&view=findpost&p=3488720 ОС: Windows


Нашёл для себя интересную особенность, связанную с компоновщиком.
Если в статической библиотеке есть объектные файлы на которые нет ссылок, то эти объектные файлы не компонуются с программой.
Соответственно, не инициализируются любые статические объекты в этом файле.

С одной стороны, всё логично.

Но вот например, у меня есть код регистрирующий конструкторы классов в фабрике:
    #include <Engine/Modules/ActorFactory.h>
    // unnamed namespase to hide from another places
    namespace
    {
        // specific factory
        IActor* CreateWall(World *world, const Vector2D location, const Vector2D scale, const Rotator rotation)
        {
            return new Wall(world, location, scale, rotation);
        }
     
        const std::string CLASS_ID = "Wall";
     
        // register specific factory in actor factory
        const bool registered = ActorFactory::Factory().registerActor(CLASS_ID, CreateWall);
    }


Такой вот код включён в исходные коды разных объектов, дабы можно было изменять/удалять регистрации классов вместе с самими объектами.

И всё было хорошо, но после вынесения всех таких классов в статическую библиотеку, часть из них перестала регистрироваться в фабрике.

Однако, если, например, создавать хотя бы один объект класса Wall вручную, то объектный файл подключается, происходит регистрация, и остальные объекты этого класса нормально создаются.

Пока не вижу хорошего решения этой проблемы:
  • Если вынести регистрации классов в основную программу, то это внесёт сложности при добавлении/удалении/модификации этих классов
  • Если вручную ссылаться на что-нибудь в этих единицах трансляции, то это будет очень неявно и когда-нибудь всё опять сломается


Что можете посоветовать в данной ситуации?]]> WhyNot C/C++: Общие вопросы