update docs with thread safe

Author: 7bitcoder

Docs/advanced-guides/configuring-service-provider.rst

@@ -11,7 +11,7 @@ read comment documentation for details:
    :language: C++
 
 Pass the custom options to the ServiceCollection buildServiceProvider method to change produced
-service provider behaviour
+service provider behavior
 
 .. literalinclude:: ../../Examples/Guides/ConfiguredServiceProvider.cpp
    :caption: Examples/Guides/ConfiguredServiceProvider

Docs/getting-started.rst

@@ -1,6 +1,16 @@
 Getting Started
 ==========================
 
+Main Features
+--------------------
+
+* Implementation separation
+* Multiple implementations support
+* Keyed services
+* Service aliases
+* Thread safe support
+* Strong destruction order support
+
 Supported Platforms
 --------------------
 

Examples/Guides/ConfiguredServiceProvider.cpp

@@ -39,6 +39,7 @@ int main()
     options.strongDestructionOrder = true;
     options.prebuildSingletons = true;
     options.checkServiceGlobalUniqueness = false;
+    options.threadSafe = true;
 
     ServiceProvider provider = ServiceCollection{}
                                    .addSingleton<IService, Service>()

Include/SevenBit/DI/ServiceProvider.hpp

@@ -23,6 +23,8 @@ namespace sb::di
 
         /**
          * @brief Constructs service provider with specified instance provider
+         * @details If service instance provider is nullptr, constructor throws exception
+         * @throws sb::di::NullPointerException
          */
         explicit ServiceProvider(IServiceInstanceProvider::Ptr instanceProvider)
             : _instanceProvider(std::move(instanceProvider))
@@ -40,15 +42,11 @@ namespace sb::di
 
         /**
          * @brief Returns inner service instance provider
-         * @details If service instance provider is nullptr, method throws exception
-         * @throws sb::di::NullPointerException
          */
         [[nodiscard]] const IServiceInstanceProvider &getInstanceProvider() const { return *_instanceProvider; }
 
         /**
          * @brief Returns inner service instance provider
-         * @details If service instance provider is nullptr, method throws exception
-         * @throws sb::di::NullPointerException
          */
         IServiceInstanceProvider &getInstanceProvider() { return *_instanceProvider; }
 

README.md

@@ -23,13 +23,14 @@
 
 <br />
 
-## Built With
+## Main Features
 
-- [Google Test](https://github.com/google/googletest)
-- [Google Benchmark](https://github.com/google/benchmark)
-- [Sphinx](https://www.sphinx-doc.org/en/master/)
-- [Breathe](https://breathe.readthedocs.io/en/latest/)
-- [Quom](https://pypi.org/project/quom/)
+- Implementation separation
+- Multiple implementations support
+- Keyed services
+- Service aliases
+- Thread safe support
+- Strong destruction order support
 
 ## Supported Platforms
 
@@ -236,4 +237,12 @@ actionA, actionB executed.
 More examples and tutorials are available on the
 [Documentation & Examples](https://7bitDI.readthedocs.io) page
 
+## Built With
+
+- [Google Test](https://github.com/google/googletest)
+- [Google Benchmark](https://github.com/google/benchmark)
+- [Sphinx](https://www.sphinx-doc.org/en/master/)
+- [Breathe](https://breathe.readthedocs.io/en/latest/)
+- [Quom](https://pypi.org/project/quom/)
+
 @7bitcoder Sylwester Dawida 2023

Tests/Integration/ThreadSafeTest.cpp

@@ -1,10 +1,7 @@
 #include <gtest/gtest.h>
 #include <thread>
 
-#include "../Helpers/Classes/Basic.hpp"
-#include "../Helpers/Classes/CirularDependency.hpp"
 #include "../Helpers/Classes/Complex.hpp"
-#include <SevenBit/DI/Exceptions.hpp>
 #include <SevenBit/DI/ServiceCollection.hpp>
 
 class ThreadSafeTest : public testing::Test
@@ -67,8 +64,9 @@ TEST_F(ThreadSafeTest, ShouldGetSafeServices)
                         .addScoped<ITestComplexClass6, TestComplexClass6>()
                         .buildServiceProvider(options);
 
-    std::vector<std::thread> threads;
     constexpr size_t maxThreads = 50;
+    std::vector<std::thread> threads;
+    threads.reserve(maxThreads);
     for (size_t i = 0; i < maxThreads; ++i)
     {
         threads.emplace_back(getSafeServices, std::ref(provider));