Når jeg står over for et spørgsmål som dette, tænker jeg generelt på tre grundlæggende spørgsmål.
- Hvad er mest læsbar?
- Hvad tillader bedre genbrug af kode?
- Hvad er mere effektivt?
Ikke nødvendigvis i denne rækkefølge, da det afhænger af scenariet, men jeg tror i dette tilfælde, at alle tre af disse spørgsmål er til fordel for mulighed B. Hvis du nogensinde har haft brug for at ændre muligheder for createClient, skal du redigere dem i hver fil som bruger det. Som i mulighed A er hver fil, der bruger redis, og mulighed B er blot redis.js. Også hvis et nyere eller andet produkt udkommer, og du ønsker at erstatte redis. Det ville være muligt at gøre redis.js til en indpakning for en anden pakke eller endda en nyere redis-klient, hvilket reducerer konverteringstiden væsentligt.
Globaler er generelt en dårlig ting, men i dette eksempel burde redis.js ikke lagre mutable state, så der er ikke noget problem at have en global/singleton i denne sammenhæng.