Refactored isObject.js

package.json

@@ -1,6 +1,6 @@
 {
   "name": "is-it-object",
-  "version": "1.1.6",
+  "version": "1.1.7",
   "license": "MIT",
   "author": "Vladimir Kovpak",
   "homepage": "https://github.com/cn007b/is-it-object#readme",

src/isObject.js

@@ -1,28 +1,37 @@
+/**
+ * Check that value is an object using strict strategy.
+ *
+ * @param o Value for inspection is it an object.
+ * @returns {boolean} Is object.
+ */
+function checkByStrictStrategy(o) {
+  return (o instanceof Object || typeof o === 'object')
+    && (o.constructor === undefined || o.constructor === Object);
+}
+
+/**
+ * Check that value is an object using weak (non-strict) strategy.
+ *
+ * @param o Value for inspection is it an object.
+ * @returns {boolean} Is object.
+ */
+function checkByWeakStrategy(o) {
+  return o.constructor === undefined || typeof o.constructor === 'function';
+}
+
 /**
  * Check that value is an object.
  *
  * @param o Value for inspection is it an object.
  * @param {boolean} strict Strict comparison strategy.
  * @returns {boolean} Is object.
  */
-module.exports = function isObject(o, strict = true) {
+function isObject(o, strict = true) {
   if (o === null || o === undefined) {
     return false;
   }
 
-  const instanceOfObject = o instanceof Object;
-  const typeOfObject = typeof o === 'object';
-  const constructorUndefined = o.constructor === undefined;
-  const constructorObject = o.constructor === Object;
-  const typeOfConstructorObject = typeof o.constructor === 'function';
-
-  let r;
-
-  if (strict === true) {
-    r = (instanceOfObject || typeOfObject) && (constructorUndefined || constructorObject);
-  } else {
-    r = (constructorUndefined || typeOfConstructorObject);
-  }
+  return strict === true ? checkByStrictStrategy(o) : checkByWeakStrategy(o);
+}
 
-  return r;
-};
+module.exports = isObject;