Skip to content

Latest commit

 

History

History
103 lines (56 loc) · 3.75 KB

File metadata and controls

103 lines (56 loc) · 3.75 KB

Code Smell 77 - Timestamps

Code Smell 77 - Timestamps

Timestamps are widely used. They have a central issuing authority and do not go back, do they?

TL;DR: Don't use timestamps for sequence. Centralize and lock your issuer.

Problems

  • Bijection Fault.

  • Timestamp Collisions.

  • Timestamp precision.

  • Packet Disorders.

  • Bad Accidental Implementation (Timestamp) for an Essential Problem (Sequencing).

Solutions

  1. Use a centralizing sequential stamper. (NO, not a Singleton).

  2. If you need to model a sequence, model a sequence.

Sample Code

Wrong

import time
  
# ts stores the time in seconds
ts1 = time.time()
ts2 = time.time() # might be the same!!

Right

numbers = range(1, 100000)
# create a sequence of numbers and use them with a hotspot

# or
sequence = nextNumber()

Detection

Timestamps are very popular in many languages and are ubiquitous.

We need to use them just to model... timestamps.

Tags

  • Bijection

Conclusion

This smell was inspired by recent Ingenuity software fault.

If we don't follow our MAPPER rules and model sequences with time, we will face trouble.

Luckily, Ingenuity is a sophisticated Autonomous vehicle and has a robust fail-safe landing software.

This video describes the glitch

Watch the video

Relations

Code Smell 39 - new Date()

Code Smell 32 - Singletons

Code Smell 71 - Magic Floats Disguised as Decimals

More Info


The most beautiful code, the most beautiful functions, and the most beautiful programs are sometimes not there at all.

Jon Bentley

Software Engineering Great Quotes


This article is part of the CodeSmell Series.

How to Find the Stinky Parts of your Code