ParalizedTime
Baseband Member
- Messages
- 78
- Location
- UK
Hi guys,
I have been learning Java, and I have just coded my first program from scratch.
It is a number guessing game. I am looking for some feedback on my code, like if it could be made simpler etc. so that I don't get into any bad practices.
Thanks.
I have been learning Java, and I have just coded my first program from scratch.
It is a number guessing game. I am looking for some feedback on my code, like if it could be made simpler etc. so that I don't get into any bad practices.
Code:
//Import random and scanner
import java.util.Random;
import java.util.Scanner;
public class numberguess{
public static void main(String[] args){
//Declare the objects and variables
Scanner sc = new Scanner(System.in);
Random rdm = new Random();
int guess = 0;
int score = 100;
//Display intro
System.out.println("Welcome to Number Guess!");
System.out.println("A number will be randomly generated (1-100) and your job is to guess it.");
System.out.println("When you enter a guess you will either be told whether the number is bigger, smaller or if you guessed it.");
System.out.println("You will receive a score at the end.");
System.out.println("Good luck!");
//Generate number
int number = rdm.nextInt(100);
//Ask for input [Before while loop in case the default value of guess is the correct answer]
System.out.print("\nPlease enter your first guess: ");
guess = sc.nextInt();
score = checkGuess(guess, number, score);
while (guess != number){
//Ask for input
System.out.print("\nPlease enter your guess: ");
guess = sc.nextInt();
score = checkGuess(guess, number, score);
}
//Display score
System.out.println("Your score is: " + score);
pause();
}
private static int checkGuess(int guess, int number, int score){
//Compare guess to number and display the appropriate message. Keep track of score.
if (guess == number) {
System.out.println("\nYou have guessed the number!");
}
else if (guess < number) {
System.out.println("The number is bigger.");
score -= 3;
}
else {
System.out.println("The number is smaller.");
score -= 3;
}
return score;
}
private static void pause() {
System.out.println("\nPress enter to exit...");
Scanner sc = new Scanner(System.in);
sc.nextLine();
}
}
Thanks.