Skip to main content

I'd keep the rule of never having magic numbers.

While

seconds = num_days * 24 * 60 * 60 

Is perfectperfectly readable most of the time, after codinghaving coded for 10 hours a day for treethree or four weeks when in crunch mode

seconds = num_days * HOURS_PER_DAY * MINUTES_PER_HOUR * SECONDS_PER_HOURSECONDS_PER_MINUTE 

is much easier to read. FrustratedWithFormsDesigner

FrustratedWithFormsDesigner's suggestion is better:

seconds = num_days * DAYS_TO_SECOND_FACTOR 

or even better

seconds = CONVERT_DAYS_TO_SECONDS(num_days) 

Things stop being obvious when you're very tired. Code defensively.

I'd keep the rule of never having magic numbers.

While

seconds = num_days * 24 * 60 * 60 

Is perfect readable most of the time, after coding 10 hours a day for tree or four weeks when in crunch mode

seconds = num_days * HOURS_PER_DAY * MINUTES_PER_HOUR * SECONDS_PER_HOUR 

is much easier to read. FrustratedWithFormsDesigner suggestion is better:

seconds = num_days * DAYS_TO_SECOND_FACTOR 

or even better

seconds = CONVERT_DAYS_TO_SECONDS(num_days) 

Things stop being obvious when you're very tired. Code defensively.

I'd keep the rule of never having magic numbers.

While

seconds = num_days * 24 * 60 * 60 

Is perfectly readable most of the time, after having coded for 10 hours a day for three or four weeks in crunch mode

seconds = num_days * HOURS_PER_DAY * MINUTES_PER_HOUR * SECONDS_PER_MINUTE 

is much easier to read.

FrustratedWithFormsDesigner's suggestion is better:

seconds = num_days * DAYS_TO_SECOND_FACTOR 

or even better

seconds = CONVERT_DAYS_TO_SECONDS(num_days) 

Things stop being obvious when you're very tired. Code defensively.

Source Link
Vitor Py
  • 4.9k
  • 1
  • 31
  • 34

I'd keep the rule of never having magic numbers.

While

seconds = num_days * 24 * 60 * 60 

Is perfect readable most of the time, after coding 10 hours a day for tree or four weeks when in crunch mode

seconds = num_days * HOURS_PER_DAY * MINUTES_PER_HOUR * SECONDS_PER_HOUR 

is much easier to read. FrustratedWithFormsDesigner suggestion is better:

seconds = num_days * DAYS_TO_SECOND_FACTOR 

or even better

seconds = CONVERT_DAYS_TO_SECONDS(num_days) 

Things stop being obvious when you're very tired. Code defensively.