4 ms·
Don’t do something silly like take a parameter called current_time. Instead, name it what it’s actually used for or doing. This could be send_emails_on or the l
by jackblemming 3y ago
Don’t do something silly like take a parameter called current_time. Instead, name it what it’s actually used for or doing. This could be send_emails_on or the like. Parameters should tell the reader what not why.
- Sankozi 3y ago`send_emails_on` suggests strongly some scheduling capability. `current_time` says exactly what it is needed for.
- jackblemming 3y ago>`send_emails_on` suggests strongly some scheduling capability Yes. >`current_time` says exactly what it is needed for. Not always.