Everyone is a beginner once in a while.
It is good to be a beginner as it means that you are growing and learning new things.
What is not good is being a beginner for too long.
You have to keep learning as a beginner so that you move to the level above beginner, then to the level above that and so on.
Since this blog is about beginners with Selenium test automation with Java, there are some things that spell beginner to anyone who looks at your code:
commented out code
comments
unused variables
un-unused import statements
unused methods
un-formatted code
lines that are so long that you need to do scrolling left and right
not using camel case naming convention
bad names for variables, methods, classes
long methods (over 20 lines)
long classes (over 500 lines)
duplicated code
re-inventing the wheel (creating your wait methods, methods that wait until the page is loaded, etc)
duplicated variables (variables in a method that duplicated class fields)
complex code (methods with lots of statements - for, while, if, try)
methods with more than 4 parameters
What is the solution then?
Work on not doing these, please.
This is a good start for your trip to the next level.