Array.sort() time complexity
fedemengo opened this issue ยท 6 comments
Shouldn't the time complexity of Array.sort()
be O(NlogN) instead of O(logN)? In questions/big-o-notation.md
Yeah, I think you're right as well. My friend told me the same thing. Feel free to submit PR if you would like to ๐
Thanks for noticing this ๐
IF this hasn't been fixed yet it shouldn't be closed. Either we close it with a fix, they close it with a fix, or we put it as 'will not fix'. Otherwise it should stay open since it has not been closed
(fixed or decided on) just yet
@skatcat31 You're right. Thanks for letting me know! ๐
@skatcat31 In my local commit there are two more fixes you can integrate into your last fix. See if it's fine for you.
NlogN
is 3000ms
, no?
... I wish I had seen that before I tried to remember what base the notation was in... this really has been a long week.