Hacker News Reader

Comment by adjkant | Hacker News Reader

This code can look a lot better by replacing the compute function with this:

It looks like that reverses the arguments? You could do

    args = self.stack[-n:]
    del self.stack[-n:]
    self.push(function_requested(*args))

or maybe

    self.stack[-n:] = [function_requested(*args)]

Yeah oops coded that up quick. Either way you fix it, it’s far better than a function for calling each number of args and keeps the code readable.

For fun, an updated version that cleans up other things as well as the ordering, not relying on signatures of lambdas, etc.

Yep! Good call! ;)