in reply to Re: Convert time in seconds to hh::mm::ss
in thread Convert time in seconds to hh::mm::ss

- Did you measure how fast it takes?

No I didn't, that Devel::NYTProf looks one serious module - thanks. I do know how to use Benchmark but would be good if I had some other method to compare this with. Any examples I found wanted to convert to days, hours, etc. but I need to keep to hours only.

edit - Oh, I see what you mean - should be no need to worry, that's good.
  • Comment on Re^2: Convert time in seconds to hh::mm::ss

Replies are listed 'Best First'.
Re^3: Convert time in seconds to hh::mm::ss
by stevieb (Canon) on Mar 16, 2016 at 01:41 UTC

    If you know how to benchmark, and you know that there's no translation of DST etc involved, why bother doing premature profiling? Unless you're doing this thousands of times per second, I see nothing wrong with this simple sub.

    What exactly is the issue here?

      - What exactly is the issue here?

      This is part of a web app where I call an external API and get back a few hundred results and have to do quite a few conversions. I can't speed up the API call so am looking to improve speed anywhere I can. Just thought this might be a likely candidate.
        This is part of a web app where I call an external API
        Whoa, whoa, whoa... Stop right there!

        Third rule of Optimization Club:
        If your app is running faster than the underlying transport protocol, the optimization is over.

        Networks are slow. CPUs are fast. Each API call is easily taking tens of thousands, if not millions, of times longer than your seconds-to-hh:mm:ss conversion routine. Even if you could optimize that conversion down so that a hundred calls take absolutely no time at all, it still wouldn't make a perceptible dent in the overall speed.