guptarohit/asciigraph

Reformat Code

CSenshi opened this issue ยท 6 comments

It would be nice if instead of having only one function that just generates a string, we had a struct named 'struct graph' for example, that had this method, configurations and other utilities inside. Subsequently, it would be easier to change implementations of other methods without changing the interface.

Besides that, we can add other methods, e.g. one could be to plot several graphs together or show statistics. However, (from my standpoint) it is essential to rewrite code so that it supports structure which will implement methods. In addition, it would encourage other users to write code for this project and making contributions much easier.

DISCLAIMER: I don't mean to delete old code, because other projects might be using it, so I will just extend it to support classes.

If this project is still up and open I would like to work on it :)

@CSenshi good idea ๐Ÿ‘ Yes, we gotta refactor Plot function, and I was also thinking to add support for multi-graph. Please feel free to create PR :)

p.s. sorry for the delayed response.

@CSenshi I would like to work on this issue, if it's free for now.

@bnMikheili Feel free to work on it!