get year, month, day

[lutfisan]

how to get year, month, and day?

[bvanelli]

I added a function to do exactly that, the only dependency being #include <time.h>.

String NTPClient::getFullFormattedTime() {
   time_t rawtime = this->getEpochTime();
   struct tm * ti;
   ti = localtime (&rawtime);

   uint16_t year = ti->tm_year + 1900;
   String yearStr = String(year);

   uint8_t month = ti->tm_mon + 1;
   String monthStr = month < 10 ? "0" + String(month) : String(month);

   uint8_t day = ti->tm_mday;
   String dayStr = day < 10 ? "0" + String(day) : String(day);

   uint8_t hours = ti->tm_hour;
   String hoursStr = hours < 10 ? "0" + String(hours) : String(hours);

   uint8_t minutes = ti->tm_min;
   String minuteStr = minutes < 10 ? "0" + String(minutes) : String(minutes);

   uint8_t seconds = ti->tm_sec;
   String secondStr = seconds < 10 ? "0" + String(seconds) : String(seconds);

   return yearStr + "-" + monthStr + "-" + dayStr + " " +
          hoursStr + ":" + minuteStr + ":" + secondStr;
}

The output seems right:

2017-07-22 19:20:25

What you guys think? This should be added to the main code?

Best Regards.

EDIT: editted the function to use less memory and use the same Epoch time for every calculation.

[lutfisan]

it's work perfect, i think you should add it to your code

you might add this

int NTPClient::getYear() {
  time_t rawtime = this->getEpochTime();
  struct tm * ti;
  ti = localtime (&rawtime);
  int year = ti->tm_year + 1900;
  
  return year;
}

int NTPClient::getMonth() {
  time_t rawtime = this->getEpochTime();
  struct tm * ti;
  ti = localtime (&rawtime);
  int month = (ti->tm_mon + 1) < 10 ? 0 + (ti->tm_mon + 1) : (ti->tm_mon + 1);
  
  return month;
}

int NTPClient::getDate() {
  time_t rawtime = this->getEpochTime();
  struct tm * ti;
  ti = localtime (&rawtime);
  int month = (ti->tm_mday) < 10 ? 0 + (ti->tm_mday) : (ti->tm_mday);
  
  return month;
}

[Testato]

#3

[bvanelli]

I agree with @Testato that this is not the purpose of the library, however this conversion is pretty useful for time-stamping and it's pretty handy to have it working out of the box, without the need for other conversions.

I also ran the benchmarks: the getFormattedTime() runs in about 100 microseconds, whereas getFullFormattedTime() is about 450 microseconds. Not very good, but not bad either.

[pwrsoft]

@sandeepmistry Could you please add the suggested changed to the library?

[Gund77]

Hi Everyone, i had a problem while adding the new public method to the NTPclient.cpp and NTPClient.h
once i added the code, the compiler gets error when processing this file.

I added the functions to the .cpp file and the header into the public section of .h file

How can i fix it?

thanks

NTPClient.zip

[bvanelli]

@Gund77 , as I stated, my workaround makes use of the C standard time library.

You can fix it by adding #include <time.h> in your header file. Your NTPClient.h's first 6 lines should look like this:

#pragma once

#include "Arduino.h"
#include <time.h>

#include <Udp.h>

[bvanelli]

For anyone that still comes to this topic seeking for help, please use the fork implementation on Github taranais/NTPClient. He implements the function getFormattedDate() that returns the time formatted in ISO 8601.

[Gund77]

Sorry, My mistate. I included Time.h library as last in the list so the compiler do not find the declaration while parsing the NTP library.
Anyway, switching the inclusions position the problem persist. i got error again. It might be a sintax error, but i cant find where... can you please send me your NTP Client.h and .cpp files?

thank you so much.

[bvanelli]

@Gund77 , it is a syntax error... You don't use the namespace when declaring a function inside the class declaration (header file). Instead of using:

     /**
     * @return time formatted like `DD:MM:YYYY`
     */
    String NTPClient::getFormattedDate() ;

     /**
     * @return time formatted like `DD:MM:YYYY hh:mm:ss`
     */
     String NTPClient::getFullFormattedTime(); 

   int NTPClient::getYear(); 
   int NTPClient::getMonth(); 
   int NTPClient::getDate();

You should declare them as:

    /**
    * @return time formatted like `hh:mm:ss`
    */
    String getFormattedTime();

    /**
    * @return time formatted like `DD:MM:YYYY`
    */
    String getFormattedDate();

     /**
     * @return time formatted like `DD:MM:YYYY hh:mm:ss`
     */
    String getFullFormattedTime(); 

    int getYear(); 
    int getMonth(); 
    int getDate();

Working version of NTPClient.zip

[thiagoarreguy]

the function getFullFormattedTime ir returnning time formatted like YYYY:MM:DD hh:mm:ss

and my output is in epoctime 1970-01-01 00:00:49

i dont understand why dont return the real date and time...

i was debuging the code and the function forceUpdate return false by timeout.

[icandura]

@thiagoarreguy I consider that replacing an NTP server is a good choice.

[Moerbius]

Hi,

Does it take into account the leap years?

[paysan18]

Thanks! I was looking for a feature like this!
Please add this to the main file. When will it be standard available?
In the mean time I got it working.....

[kaefert]

Since I didn't want to modify the library, I adapted the code snippet to work inside main.cpp:

// https://github.com/arduino-libraries/NTPClient/issues/36
String getTimeStampString() {
   time_t rawtime = timeClient.getEpochTime();
   struct tm * ti;
   ti = localtime (&rawtime);

   uint16_t year = ti->tm_year + 1900;
   String yearStr = String(year);

   uint8_t month = ti->tm_mon + 1;
   String monthStr = month < 10 ? "0" + String(month) : String(month);

   uint8_t day = ti->tm_mday;
   String dayStr = day < 10 ? "0" + String(day) : String(day);

   uint8_t hours = ti->tm_hour;
   String hoursStr = hours < 10 ? "0" + String(hours) : String(hours);

   uint8_t minutes = ti->tm_min;
   String minuteStr = minutes < 10 ? "0" + String(minutes) : String(minutes);

   uint8_t seconds = ti->tm_sec;
   String secondStr = seconds < 10 ? "0" + String(seconds) : String(seconds);

   return yearStr + "-" + monthStr + "-" + dayStr + " " +
          hoursStr + ":" + minuteStr + ":" + secondStr;
}