i don't think so. i think the logic of your code is correct.
i'm not sure about the values in the Strings.
perhaps even a hex dump of the Strings would reveal something. you could try initializing them at startup to rule out things, including the RTC String