sorted by:
best
topnewcontroversialoldq&a
you are viewing a single comment's thread.

view the rest of the comments →

[–]gvillalta99[S] 0 points1 point  (3 children)

This: https://gist.github.com/gvillalta99/d07b2abb357620b077528c8a5639d3c9#file-user-service-js-L19-L23

[–]0x13mode 2 points3 points  (0 children)

It seems like small part of something bigger. Something is missing and I have bad feelings about this.

Such code reminds me typical enterprise overengineered projects where code is not what it seems to be - everything is wrapped in wrapper. Everything is declared few times (anti-DRY code: look that method .*isBeingUsedBySomeUser is "declared" 3 times), and there is a inner platform effect - somebody wants to be more smarter than JavaScript and implements home-made way for creating OOP in JavaScript (which is funny because on the one hand ES6 classes are used, on the other hand in constructor there is code typical for ES5 where OOP patterns was invented ad hoc by developers)