dayo92/SoltMachine

Bad naming

Closed this issue ยท 3 comments

public static void GameTittle()

(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

public static void text(char row, char column, char Diagonal)

_<

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