3 ms·
Maybe not insanity, but I would find it a little noisy reading that code, like having to code at a desk near the receptionist. I wouldn't mind if the comment s
by adamconroy 13y ago
Maybe not insanity, but I would find it a little noisy reading that code, like having to code at a desk near the receptionist.
I wouldn't mind if the comment said "using qucksort because n is expected to be large". But only if the choice of quicksort over some other algorithm was deemed significant.
- kbenson 13y agoMy point is really to just define the block as containing quicksort when the function it's in does other things as well. Refactoring it into it's own function with a useful name would be just as (or probably more) useful. It's really that lack of either which I see as insufficient