rsdi - npm Package Compare versions

Comparing version 0.0.10 to 0.0.11

definitions/FactoryDefinition.d.ts

@@ -6,5 +6,4 @@ import BaseDefinition from "../definitions/BaseDefinition";
     private readonly factory;
-    private value;
     constructor(factory: Factory);
     resolve: <T>(container: IDIContainer) => T;
 }

definitions/FactoryDefinition.js

@@ -25,7 +25,3 @@ "use strict";
         _this.resolve = function (container) {
-            if (_this.value !== undefined) {
-                return _this.value;
-            }
-            _this.value = _this.factory(container);
-            return _this.value;
+            return _this.factory(container);
         };
@@ -32,0 +28,0 @@ _this.factory = factory;

DIContainer.d.ts

@@ -11,2 +11,3 @@ import { IDefinition } from "./definitions/IDefinition";
     private definitions;
+    private resolved;
     get<T>(name: string): T;
@@ -13,0 +14,0 @@ addDefinition(name: DefinitionName, definition: IDefinition): void;

DIContainer.js

@@ -21,2 +21,3 @@ "use strict";
         this.definitions = {};
+        this.resolved = {};
     }
@@ -27,4 +28,8 @@ DIContainer.prototype.get = function (name) {
         }
+        if (this.resolved[name] !== undefined) {
+            return this.resolved[name];
+        }
         var definition = this.definitions[name];
-        return definition.resolve(this);
+        this.resolved[name] = definition.resolve(this);
+        return this.resolved[name];
     };
@@ -31,0 +36,0 @@ DIContainer.prototype.addDefinition = function (name, definition) {

package.json

 {
   "name": "rsdi",
-  "version": "0.0.10",
+  "version": "0.0.11",
   "description": "dependency injection container",
@@ -5,0 +5,0 @@ "scripts": {

README.md

@@ -21,4 +21,2 @@ # RSDI - Dependency Injection Container
 function configureHistory(container: IDIContainer): History {
-    // this factory will be called only once, during first resolving 
-    // then resolved version will be returned 
     const history = createBrowserHistory();
@@ -37,7 +35,7 @@ const env = container.get("ENV");
 const history = container.get<History>("BrowserHistory");  // History singleton will be returned
+```
 
+**All definitions are resolved once and their result is kept during the life of the container.** 
 
 
-```
-
 ## Motivation 
@@ -44,0 +42,0 @@

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

23029

increased by0.48%

Lines of code

362

increased by0.28%

Worsened metrics

Number of lines in readme file

56

decreased by-3.45%

No dependency changes