Могу ли я использовать boost :: make_shared с частным конструктором?

Question

Рассмотрим следующий пример:

class DirectoryIterator; 

namespace detail { 
    class FileDataProxy; 

    class DirectoryIteratorImpl 
    { 
     friend class DirectoryIterator; 
     friend class FileDataProxy; 

     WIN32_FIND_DATAW currentData; 
     HANDLE hFind; 
     std::wstring root; 

     DirectoryIteratorImpl(); 
     explicit DirectoryIteratorImpl(const std::wstring& pathSpec); 
     void increment(); 
     bool equal(const DirectoryIteratorImpl& other) const; 
    public: 
     ~DirectoryIteratorImpl() {}; 
    }; 

    class FileDataProxy //Serves as a proxy to the WIN32_FIND_DATA struture inside the iterator. 
    { 
     friend class DirectoryIterator; 
     boost::shared_ptr<DirectoryIteratorImpl> iteratorSource; 
     FileDataProxy(boost::shared_ptr<DirectoryIteratorImpl> parent) : iteratorSource(parent) {}; 
    public: 
     std::wstring GetFolderPath() const { 
      return iteratorSource->root; 
     } 
    }; 
} 

class DirectoryIterator : public boost::iterator_facade<DirectoryIterator, detail::FileDataProxy, std::input_iterator_tag> 
{ 
    friend class boost::iterator_core_access; 
    boost::shared_ptr<detail::DirectoryIteratorImpl> impl; 
    void increment() { 
     impl->increment(); 
    }; 
    bool equal(const DirectoryIterator& other) const { 
     return impl->equal(*other.impl); 
    }; 
    detail::FileDataProxy dereference() const { 
     return detail::FileDataProxy(impl); 
    }; 
public: 
    DirectoryIterator() { 
     impl = boost::make_shared<detail::DirectoryIteratorImpl>(); 
    }; 
}; 

Похоже, DirectoryIterator должен быть в состоянии назвать boost::make_shared<DirectoryIteratorImpl>, потому что это друг DirectoryIteratorImpl. Однако этот код не компилируется, потому что конструктор для DirectoryIteratorImpl является закрытым.

Поскольку этот класс является внутренней деталью реализации, которую клиенты DirectoryIterator никогда не должны касаться, было бы неплохо, если бы я мог сохранить конструктор закрытым.

Является ли это моим основным недоразумением в отношении make_shared или мне нужно отметить какой-то элемент повышения, как friend для того, чтобы получить возможность комментировать?

Answer 1

Вам действительно нужно сделать для этого дополнительный кусок. В основном make_shared вызывает конструктор, и тот факт, что это делается из функции друга, не имеет значения для компилятора.

Хорошая новость заключается в том, что make_shared вызывает конструктор, а не любую другую деталь. Так что просто делать make_shared друг будет работать ... Однако это означает, что любой может тогда создать shared_ptr<DirectoryIteratorImpl> ...

Answer 2

Есть ли веская причина не использовать старый добрый shared_ptr конструктор? (Если есть один, вы можете захотеть взглянуть на make_shared реализации и сделать это)

DirectoryIterator() 
    : impl(new detail::DirectoryIteratorImpl()) 
{} 

Таким образом, вызов конструктора сделан из DirectoryIterator класса, который уже друг DirectoryIteratorImpl без открытия дверь для всего остального кода.

Answer 3

Вы можете разделить свой класс на часть интерфейса и часть реализации. Интерфейсная часть обнародована, а часть реализации может иметь общедоступные конструкторы. Однако это означает, что вам нужно использовать виртуальное наследование.