On Mon, Oct 2, 2017 at 8:27 AM, Daniel Bastos <dbas...@toledo.com> wrote: > def make_sequence_non_recursive(N, x0 = 2, c = -1): > "What's wrong with this function? It's very slow." > last = x0 > def sequence(): > nonlocal last > next = last > last = last**2 + c > return next % N > return sequence > > It crawls pretty soon. Please advise? Thank you.
For a start, it should probably be implemented as a generator. I'm not sure what this is even trying to do. For seconds, the variable names could do with some improvement. Ditto. ChrisA -- https://mail.python.org/mailman/listinfo/python-list