Блог

Я пытаюсь написать одноэлементный класс, и я следовал стандартному шаблону, как показано ниже. Но я получаю /usr/include/c /7/ext/new_allocator.h:136:4: ошибка: является закрытой в этом контексте { ::new((void *)__p) _Up(std::forward<_Args>(__args)…);}

 status_publisher.hpp
  

     namespace test {
    class StatusPublisher {
     public:
       static std::shared_ptr<StatusPublisher> getStatusPublisher();

     private:
       StatusPublisher();
       static std::shared_ptr<StatusPublisher> instance_;  
    }

  

status_publisher.cpp

  namespace test {
  std::shared_ptr<StatusPublisher> StatusPublisher::getStatusPublisher() {
     if(instance_.get()==nullptr) {
         instance_ = std::make_shared<StatusPublisher>();
     }
     
     return instance_;
    }

    StatusPublisher::StatusPublisher() {

    // Some code
    }
}

  

Поскольку instance_ изначально не является shared_ptr в вашем коде, функция .get() не будет там работать, поскольку instance_ не является объектом like shared_ptr<StatusPublisher> { nullptr } . Я предлагаю вам переместить объявление instance_ в StatusPublisher::getStatusPublisher и вернуть экземпляр туда. Поскольку он объявлен статическим, будет создан только один экземпляр.

Редактировать: я видел другой альтернативный ответ, который предлагает использовать необработанный указатель. Однако, поскольку shared_ptr имеет некоторые преимущества для использования необработанного указателя, я все равно предлагаю версию shared_ptr:

status_publisher.hpp

 namespace test {
   class StatusPublisher {
 public:
   static std::shared_ptr<StatusPublisher> getStatusPublisher();

 private:
   StatusPublisher();
   };
}
  

status_publisher.cpp

 namespace test {
     std::shared_ptr<StatusPublisher> StatusPublisher::getStatusPublisher() {
         static std::shared_ptr<StatusPublisher> instance_ = std::shared_ptr<StatusPublisher>{new StatusPublisher()};
         return instance_;
     }

    StatusPublisher::StatusPublisher() {

    // Some code
    }
}