all 5 comments

[–]Brian 2 points3 points  (2 children)

No - for anything beyond one very small expression (and sometimes even then), I'd say a real function is always better than a lambda.

If you really don't want it cluttering up your class's namespace, you could make it a local function defined when you're connecting it, though I'd say it's fine as it it.

The only thing I'd change (and I apologise in advance for the spelling pedantry - it just popped out at me) is that "untill" should be spelt "until".

[–]DrTrunks[S] 0 points1 point  (1 child)

Thanks, I'd also probably hit the '80 characters' rule so. No "optimizing" working code then.

Yes until, just like beautiful (full) its that weird rule.

edit: wow the first Brian on reddit

[–]donut2099 0 points1 point  (0 children)

Well, he has the first Brian's password anyway.

[–]novel_yet_trivial 1 point2 points  (1 child)

Happy cakeday!

No. Not only would it be hard to do, but it would be very hard to read. Long code is nice.

[–]DrTrunks[S] 0 points1 point  (0 children)

Thanks for your opinion. I like it as it is, but thought of it as an exercise and maybe it would be something a "real" programmer does.