Add default values about sorting-algorithms-visualizer HOT 12 CLOSED

Currently the program does not handle errors, so if you insert some non digit caracters in the Size box it will broke! If you did all right you must have found some exception, message me again. Thanks for reporting this issue.

from sorting-algorithms-visualizer.

I would appreciate your feedback about the program, and if you intend to help as well!

from sorting-algorithms-visualizer.

Yes, I'm ready to help.

I made a clone of your project. And crunched main.py. But I did not understand what parameters need to be entered for the program to work.
Or should I add an algorithm file to the project?

from sorting-algorithms-visualizer.

You managed to open the program with the size, delay, algorithm and play buttom?, similar to the gif in README.
If you did it you just have to insert valid values, e.g. 450 in size, 0 in delay, Mergesort in algorithm and press the play buttom

All you have to insert are inside the program, for inicialize it you just need to execute main.py

from sorting-algorithms-visualizer.

I'm sorry everything works!
Let me add the default parameters.
This will make it easier to understand how the program works.

from sorting-algorithms-visualizer.

Sure, I will asign this issue to you! If you manage a way to handle this errors, so when a user forgets this parameters or insert them wrong the program does not broke i would appreciate as well.

from sorting-algorithms-visualizer.

I don't understand why such a diff turns out.
It looks like this to me:

from sorting-algorithms-visualizer.

Are you up to change the code for handle possible errors?

from sorting-algorithms-visualizer.

Yes of course

from sorting-algorithms-visualizer.

Its working well, but it would be more efficient if the default values were defined just once outside the loop, right? Its not necessary to keep verifying and neither create a bool variable.

from sorting-algorithms-visualizer.

Oh sure. I'll fix it now.

from sorting-algorithms-visualizer.

I will close this issue and merge the pull request that you add default values

from sorting-algorithms-visualizer.