All of the above is overcomplicated. There is a school of thought which says design patterns are showing deficiencies of actual language.
Languages with prototype-based OOP (classless) do not need a singleton pattern at all. You simply create a single(ton) object on the fly and then use it.
As for modules in node, yes, by default they are cached, but it can be tweaked for example if you want hot-loading of module changes.
But yes, if you want to use shared object all over, putting it in a module exports is fine. Just do not complicate it with "singleton pattern", no need for it in JavaScript.
Modules are cached based on their resolved filename. Since modules may resolve to a different filename based on the location of the calling module (loading from node_modules folders), it is not a guarantee that require('foo') will always return the exact same object, if it would resolve to different files.
So, depending on where you are when you're requiring a module, it's possible to get a different instance of the module.
Sounds like modules are not a simple solution to creating singletons.
Edit: Or maybe they are. Like @mkoryak, I can't come up with a case where a single file might resolve to different filenames (without using symlinks). But (as @JohnnyHK comments), multiple copies of a file in different node_modules directories will each be loaded and stored separately.
But a small modification defeats caching. On OSX, do this:
var sg = require("./singleton.js");
var sg2 = require("./SINGLETON.js");
sg.add(1, "test");
sg2.add(2, "test2");
console.log(sg.getSocketList(), sg2.getSocketList());
Or, on Linux:
% ln singleton.js singleton2.js
Then change the sg2 require line to:
var sg2 = require("./singleton2.js");
And bam, the singleton is defeated:
{ '1': 'test' } { '2': 'test2' }
I don't know of an acceptable way to get around this. If you really feel the need to make something singleton-like and are okay with polluting the global namespace (and the many problems that can result), you can change the author's getInstance() and exports lines to:
That said, I've never run into a situation on a production system where I needed to do anything like this. I've also never felt the need to use the singleton pattern in Javascript.
Singletons are fine in JS, they just don't need to be so verbose.
In node if you need a singleton, for instance to use the same ORM/DB instance across various files in your server layer, you can stuff the reference into a global variable.
Just write a module that creates the global var if it doesn't exist, then returns a reference to that.
@allen-luce had it right with his footnote code example copied here:
but it is important to note that using the new keyword is not required. Any old object, function, iife, etc. will work - there is no OOP voodoo happening here.
bonus points if you closure a some obj inside a function that returns a reference to it, and make that function a global - then even reassignment of the global variable won't clobber the instances already created from it - though this is questionably useful.
Modules are cached after the first time they are loaded. This means
(among other things) that every call to require('foo') will get
exactly the same object returned, if it would resolve to the same
file.
Multiple calls to require('foo') may not cause the module code to be
executed multiple times. This is an important feature. With it,
"partially done" objects can be returned, thus allowing transitive
dependencies to be loaded even when they would cause cycles.
If you want to have a module execute code multiple times, then export
a function, and call that function.
Module Caching Caveats
Modules are cached based on their resolved filename. Since modules may
resolve to a different filename based on the location of the calling
module (loading from node_modules folders), it is not a guarantee that
require('foo') will always return the exact same object, if it would
resolve to different files.
Additionally, on case-insensitive file systems or operating systems,
different resolved filenames can point to the same file, but the cache
will still treat them as different modules and will reload the file
multiple times. For example, require('./foo') and require('./FOO')
return two different objects, irrespective of whether or not ./foo and
./FOO are the same file.
So in simple terms.
If you want a Singleton; export an object.
If you do not want a Singleton; export a function (and do stuff/return stuff/whatever in that function).
To be VERY clear, if you do this properly it should work, look at https://stackoverflow.com/a/33746703/1137669 (Allen Luce's answer). It explains in code what happens when caching fails due to differently resolved filenames. But if you ALWAYS resolve to the same filename it should work.
This answer refers to CommonJS (Node.js's own way to import/export modules). Node.js will most likely be switching over to ECMAScript Modules: https://nodejs.org/api/esm.html(ECMAScript is the real name of JavaScript if you didn't know)