Bad naming
Closed this issue ยท 3 comments
SoltMachine/SoltMachine/UIMethods.cs
Line 7 in aa6f8cd
(besides the spelling error) your functions have bad naming ๐
All functions should start with a verb that indicates what is happening.
Just nouns are usually used for classes
-> "GameTitle" contain a string that holds the title of the game
-> whereas "PrintGameTitle" shows the text on the console
It is a common and good way to come up with a few common verbs in your program that you use for methods that do similar things like
"Print", "Show", "Display", ... for methods that print stuff to the console,
or "Get", "AskFor", "Accept", "Request", ... for methods that retrieve input from the user
This will make your code very consistent and easy to read, for other developers and also you in the future :D
This issue is relevant for all your files/methods :D
For example you still have functions called "AvailableBalance", "BettingQuestion", ... what should be "Print..."
Even worse is this one
SoltMachine/SoltMachine/UIMethods.cs
Line 62 in e8e6e6c
_<
As @incredibleLeitman has mentioned, this issue applies to all files and methods. UIMethods.cs has not been touched :P
You should apply method naming convensions to every single method in this file
changed naming conventions with last commit