Implement ChainedMap.getOrCompute

README.md

@@ -168,6 +168,15 @@ delete(key)
 get(key)
 ```
 
+```js
+// Fetch the value from a Map located at the corresponding key.
+// If the key is missing, the key is set to the result of function fn.
+// key: *
+// fn: Function () -> value
+// returns: value
+getOrCompute(key, fn)
+```
+
 ```js
 // Set a value on the Map stored at the `key` location.
 // key: *

src/ChainedMap.js

@@ -72,6 +72,13 @@ module.exports = class extends Chainable {
     return this.store.get(key);
   }
 
+  getOrCompute(key, fn) {
+    if (!this.has(key)) {
+      this.set(key, fn());
+    }
+    return this.get(key);
+  }
+
   has(key) {
     return this.store.has(key);
   }

src/Config.js

@@ -96,19 +96,11 @@ module.exports = class extends ChainedMap {
   }
 
   entry(name) {
-    if (!this.entryPoints.has(name)) {
-      this.entryPoints.set(name, new ChainedSet(this));
-    }
-
-    return this.entryPoints.get(name);
+    return this.entryPoints.getOrCompute(name, () => new ChainedSet(this));
   }
 
   plugin(name) {
-    if (!this.plugins.has(name)) {
-      this.plugins.set(name, new Plugin(this, name));
-    }
-
-    return this.plugins.get(name);
+    return this.plugins.getOrCompute(name, () => new Plugin(this, name));
   }
 
   toConfig() {

src/Module.js

@@ -10,19 +10,11 @@ module.exports = class extends ChainedMap {
   }
 
   defaultRule(name) {
-    if (!this.defaultRules.has(name)) {
-      this.defaultRules.set(name, new Rule(this, name));
-    }
-
-    return this.defaultRules.get(name);
+    return this.defaultRules.getOrCompute(name, () => new Rule(this, name));
   }
 
   rule(name) {
-    if (!this.rules.has(name)) {
-      this.rules.set(name, new Rule(this, name));
-    }
-
-    return this.rules.get(name);
+    return this.rules.getOrCompute(name, () => new Rule(this, name));
   }
 
   toConfig() {

src/Resolve.js

@@ -25,11 +25,7 @@ module.exports = class extends ChainedMap {
   }
 
   plugin(name) {
-    if (!this.plugins.has(name)) {
-      this.plugins.set(name, new Plugin(this, name));
-    }
-
-    return this.plugins.get(name);
+    return this.plugins.getOrCompute(name, () => new Plugin(this, name));
   }
 
   toConfig() {

src/Rule.js

@@ -31,19 +31,11 @@ module.exports = class Rule extends ChainedMap {
   }
 
   use(name) {
-    if (!this.uses.has(name)) {
-      this.uses.set(name, new Use(this, name));
-    }
-
-    return this.uses.get(name);
+    return this.uses.getOrCompute(name, () => new Use(this, name));
   }
 
   oneOf(name) {
-    if (!this.oneOfs.has(name)) {
-      this.oneOfs.set(name, new Rule(this, name));
-    }
-
-    return this.oneOfs.get(name);
+    return this.oneOfs.getOrCompute(name, () => new Rule(this, name));
   }
 
   pre() {

test/ChainedMap.js

@@ -28,6 +28,14 @@ test('get', t => {
   t.is(map.get('a'), 'alpha');
 });
 
+test('getOrCompute', t => {
+  const map = new ChainedMap();
+
+  t.is(map.get('a'), undefined);
+  t.is(map.getOrCompute('a', () => 'alpha'), 'alpha');
+  t.is(map.get('a'), 'alpha');
+});
+
 test('clear', t => {
   const map = new ChainedMap();