loadmodule jshint errors fixed

Author: linusjf

loadmodule/anothermodule.js

@@ -1,54 +1,49 @@
 /*jshint globalstrict: true*/
 /*jshint node: true */
+/*jshint latedef: false */
+/*jshint unused:false*/
 "use strict";
-var fs = require('fs');
-var req = require;
+
+const fs = require('fs');
+//save the original require
+let originalRequire = require;
+
 function loadModule(filename, module, require) {
-  var wrappedSrc = '( function( module, exports, require) {' +
-                   fs.readFileSync(filename, 'utf8') +
-                   '})( module, module.exports, require);';
-  eval(wrappedSrc); //jshint ignore:line
+  const wrappedSrc =
+    `(function(module, exports, require) {
+      ${fs.readFileSync(filename, 'utf8')}
+    })(module, module.exports, require);`;
+  eval(wrappedSrc); // jshint ignore:line 
 }
 
-var require = function(moduleName) {
-  console.log('Require invoked for module: ' + moduleName);
-  var id = require.resolve(moduleName);
-  //[ 1]
-  if (require.cache[id]) {
-    //[ 2]
+// We intentionally use var in the next line to avoid "SyntaxError: Identifier 'require' has already been declared"
+var require = (moduleName) => {
+  console.log(`Require invoked for module: ${moduleName}`);
+  const id = require.resolve(moduleName);      //[1]
+  if(require.cache[id]) {           //[2]
     return require.cache[id].exports;
   }
-  // module metadata
-  var module = {
-    //[ 3]
-    exports : {},
-    id : id
+
+  //module metadata
+  const module = {               //[3]
+    exports: {},
+    id: id
   };
-  // Update the cache
-  require.cache[id] = module;
-  //[ 4]
-  // load the module
-  loadModule(id, module, require);
-  //[ 5]
-  // return exported variables
-  return module.exports;
-  //[ 6]
+  //Update the cache
+  require.cache[id] = module;           //[4]
+
+  //load the module
+  loadModule(id, module, require);         //[5]
+
+  //return exported variables
+  return module.exports;             //[6]
 };
+
 require.cache = {};
-require.resolve =
-    function(moduleName) {
-  /* resolve a full module id fromthe moduleName */
-  if (moduleName.startsWith('./')) {
-    if (moduleName.indexOf('.js'))
-      return moduleName;
-    return moduleName + '.js';
-  }
-  return req.resolve(moduleName);
+require.resolve = (moduleName) => {
+  //reuse the original resolving algorithm for simplicity
+  return originalRequire.resolve(moduleName);
 };
 
-// load another dependency
-var dependency = require('./anothermodule.js');
-// a private function
-function log() { console.log('Well done ' + dependency.username); }
-// the API to be exported for public use
-module.exports.run = function() { log(); };
+//Load the entry point using our homemade 'require'
+require(process.argv[2]);

loadmodule/loadmodule.js

@@ -0,0 +1,5 @@
+"use strict";
+
+const moduleA = require('./moduleA');
+moduleA.run();
+

loadmodule/main.js

@@ -0,0 +1,10 @@
+"use strict";
+
+const moduleB = require('./moduleB');
+
+module.exports = {
+  run: () => {
+    moduleB.log();
+  }
+};
+

loadmodule/moduleA.js

@@ -0,0 +1,6 @@
+"use strict";
+
+exports.log = () => {
+  console.log("From module B: the homemade require works!");
+};
+