I've just made my first small project with java, I've been learning java for 3 weeks now and i wanted to test myself with this project.
The project's concept is about trying to guess names given randomly by the computer and the more you guess the harder it becomes
When you commit to source control, you'd normally commit the file name as is, not change it to something like GuessTheName_SourceCode. I expected to see GuessTheName.java instead since it's a Java program.
Many of the names don't follow the normal Java convention for capitalization though. Method names and variable names start with lowercase letters and camelcase for any other words so RandomNameGenerator should be randomNameGenerator instead, PlayerLevel should be playerLevel and so on.
Some methods are way too long and do way too much. The random name generator method and the probability method in particular are just monsters. Consider applying the Compose Method Refactoring to make the method shorter and to a single level of abstraction. Eliminate duplication as well.
I have gone to look for myself. If I should return before I get back, keep me here with this tiny ad: