DEV Community

Cover image for Code Smell 39 - new Date()
Maxi Contieri
Maxi Contieri

Posted on • Edited on • Originally published at maximilianocontieri.com

Code Smell 39 - new Date()

70s first tutorial: getCurrentDate(). Piece of Cake. We are in the 20s Time is global no more.

Problems

  • Coupling

  • Fragile Tests

  • Timezone Problems

Solutions

  1. Use Dependency injection to decouple time source.

Sample Code

Wrong

var today = new Date();
view raw globalDate.js hosted with ❤ by GitHub

Right

var ouagadougou = new Location();
var today = myTimeSource.currentDateIn(ouagadougou);
function testGivenAYearHasPassedAccruedInterestsAre10() {
var mockTime = new MockedDate(new Date(2021, 1, 1));
var domainSystem = new TimeSystem(mockTime);
// ..
mockTime.moveDateTo(new Date(2022, 1, 1));
// … You set up the yearly interest rate
assertEquals(10, domainSystem.accruedInterests());
}
view raw notcoupled.js hosted with ❤ by GitHub

Detection

We should forbid global functions policies. We need to couple to accidental and pluggable time sources.

Conclusion

Date.today() , Time.now()

, and other global system calls are coupling smell.

Since tests must be in full environmental control. We should easily set up time, moved it back and forth etc.

Date and Time classes should only create immutable instances. It is not their responsibility to give the actual time. This violates Single Responsibility Principle.

The passage of time is always scorned by programmers. This makes objects mutable and designs poor and coupled.

Relations

More info

Evil Mutants

Tags

  • Globals

In programming, the hard part isn't solving problems, but deciding what problems to solve.

Paul Graham

Image of Timescale

🚀 pgai Vectorizer: SQLAlchemy and LiteLLM Make Vector Search Simple

We built pgai Vectorizer to simplify embedding management for AI applications—without needing a separate database or complex infrastructure. Since launch, developers have created over 3,000 vectorizers on Timescale Cloud, with many more self-hosted.

Read more →

Top comments (0)