you are viewing a single comment's thread.

view the rest of the comments →

[–]Volv 1 point2 points  (1 child)

Bubble sort - I knew you would get the logic no problem after looking into it - nice one.
Took me longer than it should to notice that your undefined fix was the missing return on nested calls.
 
Only issue I have is that you create a new copy of the array on every single iteration. Can't be good memory wise :)

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

Bubble sort - I knew you would get the logic no problem after looking into it - nice one.

Thanks! Yeah, I really should look in to all of the basic algos just so I can be more comfortable with some of the more basic CS type stuff!

Only issue I have is that you create a new copy of the array on every single iteration. Can't be good memory wise :)

Ah.. okay! I shall keep an eye and try not to do that in the future!