From 7c711026ca98b118d470155dd233bc9a3a2c6766 Mon Sep 17 00:00:00 2001
From: Leon Poon <szeleung.poon@gmail.com>
Date: Wed, 19 May 2021 12:10:04 +0800
Subject: [PATCH] Eliminate need for multiple calls to nondeterministic
 millis() when getting individual day/time components

---
 NTPClient.cpp | 31 +++++++++++++++++++++++--------
 NTPClient.h   |  5 +++++
 2 files changed, 28 insertions(+), 8 deletions(-)

diff --git a/NTPClient.cpp b/NTPClient.cpp
index 2f30602..86823f0 100755
--- a/NTPClient.cpp
+++ b/NTPClient.cpp
@@ -133,27 +133,42 @@ unsigned long NTPClient::getEpochTime() const {
 }
 
 int NTPClient::getDay() const {
-  return (((this->getEpochTime()  / 86400L) + 4 ) % 7); //0 is Sunday
+  return getDay(this->getEpochTime());
+}
+unsigned long NTPClient::getDay(unsigned long epochTime) const {
+  return (((epochTime  / 86400L) + 4 ) % 7); //0 is Sunday
 }
 int NTPClient::getHours() const {
-  return ((this->getEpochTime()  % 86400L) / 3600);
+  return getHours(this->getEpochTime());
+}
+unsigned long NTPClient::getHours(unsigned long epochTime) const {
+  return ((epochTime  % 86400L) / 3600);
 }
 int NTPClient::getMinutes() const {
-  return ((this->getEpochTime() % 3600) / 60);
+  return getMinutes(this->getEpochTime());
+}
+unsigned long NTPClient::getMinutes(unsigned long epochTime) const {
+  return ((epochTime % 3600) / 60);
 }
 int NTPClient::getSeconds() const {
-  return (this->getEpochTime() % 60);
+  return getSeconds(this->getEpochTime());
+}
+unsigned long NTPClient::getSeconds(unsigned long epochTime) const {
+  return (epochTime % 60);
 }
 
 String NTPClient::getFormattedTime() const {
-  unsigned long rawTime = this->getEpochTime();
-  unsigned long hours = (rawTime % 86400L) / 3600;
+  return getFormattedTime(this->getEpochTime());
+}
+
+String NTPClient::getFormattedTime(unsigned long rawTime) const {
+  unsigned long hours = getHours(rawTime);
   String hoursStr = hours < 10 ? "0" + String(hours) : String(hours);
 
-  unsigned long minutes = (rawTime % 3600) / 60;
+  unsigned long minutes = getMinutes(rawTime);
   String minuteStr = minutes < 10 ? "0" + String(minutes) : String(minutes);
 
-  unsigned long seconds = rawTime % 60;
+  unsigned long seconds = getSeconds(rawTime);
   String secondStr = seconds < 10 ? "0" + String(seconds) : String(seconds);
 
   return hoursStr + ":" + minuteStr + ":" + secondStr;
diff --git a/NTPClient.h b/NTPClient.h
index b518c28..a7d71a8 100755
--- a/NTPClient.h
+++ b/NTPClient.h
@@ -75,9 +75,13 @@ class NTPClient {
     bool forceUpdate();
 
     int getDay() const;
+    unsigned long getDay(unsigned long epochTime) const;
     int getHours() const;
+    unsigned long getHours(unsigned long epochTime) const;
     int getMinutes() const;
+    unsigned long getMinutes(unsigned long epochTime) const;
     int getSeconds() const;
+    unsigned long getSeconds(unsigned long epochTime) const;
 
     /**
      * Changes the time offset. Useful for changing timezones dynamically
@@ -94,6 +98,7 @@ class NTPClient {
      * @return time formatted like `hh:mm:ss`
      */
     String getFormattedTime() const;
+    String getFormattedTime(unsigned long epochTime) const;
 
     /**
      * @return time in seconds since Jan. 1, 1970