Simplify the code.

Question

My assignment was following:
You are to create a lottery game program that allows a user to simulate playing the lottery. In
this lottery game, the winning number combination comprises four unique numbers, each in
the range of 1 to 42 (no duplicates).
A winning ticket will match from one to four numbers and pay-out based on the following
schedule: 1 number matched = $2.00, 2 numbers matched = $20.00, 3 numbers matched
$100.00, and 4 numbers matched = $1,000.
Your lottery game program should allow the user to:
1. purchase and store from 1 to 5 lottery tickets. The numbers on each ticket are
determined randomly. A single ticket cannot have duplicate numbers.
2. print the purchased tickets in the following format (this example shows five tickets):
Ticket 1: 21-17-22-30
Ticket 2: 16-5-42-25
Ticket 3: 40-11-28-7
Ticket 4: 34-8-16-1
Ticket 5: 26-37-15-40
3. have the program choose and print the winning lottery number in the following format:
Winning Numbers: 16-42-8-1
4. check the tickets purchased against the winning number and print out the results as
follows:
Ticket 1: 2-17-22-30
1 match: 17
Ticket total $2.00
Ticket 2: 16-5-42-25
2 matches: 16, 42
Ticket total: $20.00
Ticket 3: 40-11-28-72
no matches
Ticket total: $0.00
Ticket 4: 34-8-16-1
1 match: 16
Ticket total: $2.00
Ticket 5: 26-37-15-40
no matches
Ticket total: $0.00
Total Winnings: $24.00
Here is my code:
import java.util.Random;
* @author
* @version 0.9
public class Lottery
// instance variables
private Random number;
private int[] ticket1;
private int[] ticket2;
private int[] ticket3;
private int[] ticket4;
private int[] ticket5;
private int[] winningTicket;
private int totalMoney;
public Lottery()
number = new Random();
ticket1 = new int[4];
ticket2 = new int[4];
ticket3 = new int[4];
ticket4 = new int[4];
ticket5 = new int[4];
winningTicket = new int[4];
totalMoney = 0;
public void printTickets(int number)
if (number > 0 && number < 6)
for (int tickets = 0; tickets <= number; tickets ++)
if (tickets == 1)
ticket1 = this.tickets();
if (tickets == 2)
ticket2 = this.tickets();
if (tickets == 3)
ticket3 = this.tickets();
if (tickets == 4)
ticket4 = this.tickets();
if (tickets == 5)
ticket5 = this.tickets();
System.out.println("Ticket 1: " + this.numbers(ticket1));
System.out.println("Ticket 2: " + this.numbers(ticket2));
System.out.println("Ticket 3: " + this.numbers(ticket3));
System.out.println("Ticket 4: " + this.numbers(ticket4));
System.out.println("Ticket 5: " + this.numbers(ticket5));
else
System.out.println("You only can buy 1 to 5 tickets.");
* print out the winning ticket.
public void printWinningTicket()
winningTicket = this.tickets();
System.out.println("Winning Numbers: " + this.numbers(winningTicket));
* put the numbers of one ticket into string.
public String numbers(int[] ticket)
String numbers = ticket[0] + "-" + ticket[1] + "-" + ticket[2] + "-" + ticket[3];
return numbers;
* purchased the ticket with random numbers.
public int[] tickets()
int[] ticket = new int[5];
int index = 0;
while (index <4)
int ticketNumber = number.nextInt(42) + 1;
if ((ticketNumber != ticket[0]) && (ticketNumber != ticket[1])&&
(ticketNumber != ticket[2]) && (ticketNumber != ticket[3]))
ticket[index] = ticketNumber;
index++;
return ticket;
* print out the result of the Lottery game'
public void printResult()
System.out.println("Ticket 1: " + this.numbers(ticket1));
this.machingNumbers(winningTicket, ticket1);
System.out.println("Ticket 2: " + this.numbers(ticket2));
this.machingNumbers(winningTicket, ticket2);
System.out.println("Ticket 3: " + this.numbers(ticket3));
this.machingNumbers(winningTicket, ticket3);
System.out.println("Ticket 4: " + this.numbers(ticket4));
this.machingNumbers(winningTicket, ticket4);
System.out.println("Ticket 5: " + this.numbers(ticket5));
this.machingNumbers(winningTicket, ticket5);
System.out.println("Total Winnings: $" + totalMoney + ".00");
* compare the winning ticket with the tickets that user bought.
* print out the maching numbers and winning money.
* if there is no ticket, print out "no ticket."
public void machingNumbers(int[] winning, int[] ticket)
int maching = 0;
String machingNumbers = "";
int money = 0;
for(int winningIndex = 0; winningIndex < 4; winningIndex ++)
for(int index = 0; index < 4; index ++)
if(winning[winningIndex] == ticket[index])
maching ++;
machingNumbers = machingNumbers + " " + ticket[index];
if (ticket[0] == 0)
System.out.println(" no ticket.");
else
if (maching == 0)
System.out.println(" no maches");
else
System.out.println(" " + maching + " mach: " + machingNumbers);
if (maching == 1)
money = 2;
if (maching == 2)
money = 20;
if (maching == 3)
money = 100;
if (maching == 4)
money = 1000;
System.out.println(" Ticket total: $" + money + ".00");
System.out.println("");
totalMoney = totalMoney + money;
[code/]
How can I simplify this code? Thank you very much!

Answer 1

I'm not convinced that code will even compile.
Why so many arrays of ticket?
You should try harder to use code tags. What you have is unreadable.