> > > >>> time=1901248 > > > >>> reduce(lambda a, b: a[:-1] + [a[-1]%b, > math.floor(a[-1]/b)], [[time], > > > 60, 60, 24]) > > > [28, 7.0, 0.0, 22.0] # secs, mins, hrs, days > > > > I recommend learning how to use a good old for-loop. That example is > > as cryptic as can be. It's also inefficient due to calling a function > > for each iteration. > > I normally frown on "me too" posts, but this time I won't refrain from > a loud "hear, hear!". "Clever" code is NOT a culturally positive trait > in the Python community (differently from most language communities... > and this is in fact one reason I love Python). > > Alex > > > It wasn't only posted to be cryptic, it's one thing that's difficult to > write with a for loop without a lot of verbosity (at least I couldn't > figure out how to do it...).
In this case, I wouldn't use a loop at all: py> time=1901248 py> minutes,seconds = divmod(time, 60) py> hours,minutes = divmod(minutes, 60) py> days,hours = divmod(hours,24) py> seconds,minutes,hours,days (28, 7, 0, 22) If you absolutely want to use a loop (because you have a variable list of divisors), write py> time=1901248 py> div = time py> res = [] py> for divisor in (60,60,24): ... div, mod = divmod(div, divisor) ... res.append(mod) ... else: ... res.append(div) ... py> res [28, 7, 0, 22] If you think this shows a lot of verbosity, please reconsider. It's not verbose. If you absolutely want it in a single expression, I'd write py> time % 60, time//60%60, time//3600%24, time//(3600*24) (28, 7, 0, 22) Regards, Martin P.S. I'm not sure why you had been using floating point operations. _______________________________________________ Python-3000 mailing list Python-3000@python.org http://mail.python.org/mailman/listinfo/python-3000 Unsubscribe: http://mail.python.org/mailman/options/python-3000/archive%40mail-archive.com
