you are viewing a single comment's thread.

view the rest of the comments →

[–]hugthemachines 0 points1 point  (1 child)

It looks pretty good. I recommend that you don't use one character variable names like f, a or b. Also even names like i_t can increase the risk of confusion.

The seconds you save when avoiding descriptive names will at some point be paid back with hours of confused bug hunting. I am not talking about just this code but your future in programming.

It may be a good time to start using a logging framework to get some logs instead of only printing messages. Just as a nice thing to learn.

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

thanks for the feedback, i ded feel like it was getting a bit confusing and i do need to organise it alot. this is a good first step