Error message

[Candie Duggan]

I am down from 21 errors to 1 error. Keep getting an ; expected error message but when I put it in it creates more errors. Error is on line 162. Any help would be appreciated. Here's my code

   //Music CD Inventory Part 5
//Candie Duggan
//March 27, 2009

import java.io.*;
import java.util.*; 
import java.text.*;

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

import java.net.URL;

//The Music CD class to store and display inventory information

public class MusicCDInventory {

	//Create an Inventory program to hold up to 50 Music CDs from the input that the user enters
	MusicCDInventory theMusicCDInventory = new MusicCDInventory();
	
	int index = 0;

	//Create a number formatter that turns doubles into strings that look like prices
	NumberFormat moneyFormat = NumberFormat.getCurrencyInstance();

	//GUI elements to display currently selected Music CD's
	private JLabel uPCNumberLabel;
	private JTextField uPCNumberText;
	
	private JLabel CDNameLabel;
	private JTextField CDNameText;
	
	private JLabel genreLabel;
	private JTextField genreText;
	
	private JLabel quantityOnHandLabel;
	private JTextField quantityOnHandText;
	
	private JLabel productPriceLabel;
	private JTextField productPriceText;
	
	private JLabel restockingFeeLabel;
	private JTextField restockingFeeText;
	
	private JLabel valueLabel;
	private JTextField valueText;
	
	private JLabel entireMusicCDInventoryValueLabel;
	private JLabel entireMusicCDInventoryValueText;

	//Load the logo into a component for the GUI
	private JLabel logoLabel;
	
	//Panel to hold buttons
	private JPanel buttonPanel; 

	private JFrame dvdFrame;

	//Setup buttons
	//Button to display next inventory item
	JButton nextButton;
	JButton previousButton;
	JButton firstButton;
	JButton lastButton;

	//Setup the GUI
	public MusicCDInventory()
	{
		//Tracks quantity of Music CDs the user has entered
		int counter = 0;
		
		//Create Scanner to obtain input from command window
		BufferedReader input = new BufferedReader(new InputStreamReader(System.in) );
		
		//Print out a screen title
		System.out.println("Music CD Inventory Program\n\n");
		
		//Request user input until the user enters stop for an item name
		while( counter < 50 ) {
			
			String CDName = null;
			do {
				try {
					//Prompt for user input
					System.out.print( "Enter the title of the Music CD or the word stop: " );
					CDName = input.readLine();
				} catch (IOException e) {
					System.out.println("Problem with that input. Please try again.");
				}
			} while (CDName == null);
			
			//If the user entered stop for the Music CD name, then quit the loop
			if ( CDName.equalsIgnoreCase("stop") ) {
				break;
			}
			
			//Enter UPC number
			String uPCNumber = null;
			do {
				try {
					System.out.print( "Enter the UPC number: " ); // prompt for input
					uPCNumber = input.readLine();
				} catch (IOException e) {
					System.out.println("Problem with that input. Please try again.");
				}
			} while (uPCNumber == null);
					
			//Enter Music CD genre
			String itemGenre = null;
			do {
				try {
					System.out.print( "Enter the genre of the Music CD: " );
					itemGenre = input.readLine();
				} catch (IOException e) {
					System.out.println("Problem with that input. Please try again.");
				}
			} while (itemGenre == null);					

			//Enter quantity on hand
			//Read quantity on hand
 			int quantityOnHand = -1;
			do {
				try {
					System.out.print("Enter quantity on hand: ");
					quantityOnHand = Integer.valueOf(input.readLine()).intValue();
					//While loop to repeat the entry of Quantity on Hand until a positive value is entered
					while ( quantityOnHand < 0)
					{
						System.out.print("Invalid Entry: Please enter a non-negative quantity on hand: ");
						//Read Quantity on Hand
						quantityOnHand = Integer.valueOf(input.readLine()).intValue(); 
					}
				} catch (IOException e) {
					System.out.println("Problem with that input. Please try again.");
				}
			} while (quantityOnHand < 0);
			
			
			//Enter price of the Music CD
			//Prompt for input
			double itemPrice = -1.0; 
			do {
				try {
				System.out.print( "Enter the price of the Music CD: " ); 
				//Read price of the OfficeSupply
					itemPrice = Double.valueOf(input.readLine()).doubleValue();
					//While loop to repeat the entry of price a positive value is entered
					while ( itemPrice <=0.0 )
					{
						System.out.print("Invalid Entry: Please enter a positive Music CD Price: ");
						//Read Music CD Price
						itemPrice = Double.valueOf(input.readLine()).doubleValue(); 
					}
				} catch (IOException e) {
					System.out.println("Problem with that input. Please try again.");
				}
			} while (itemPrice < 0);
			

			//Create a new MusicCD object
			Public MusicCDInventory()
			MusicCD theMusicCD = new MusicCD(CDName,  uPCNumber, itemGenre, quantityOnHand,  itemPrice);
			//Enter the new Music CD into our array at the end
			theMusicCDInventory.addMusicCD(theMusicCD);
			
			//Increment the counter
			counter++;

			//Print out an extra new line
			System.out.println();

		} // end while


		//User is done entering Music CDs,
		
		//Setup the element of the GUI
		uPCNumberLabel = new JLabel("UPC Number: ");
		CDNameLabel = new JLabel("Title: ");
		genreLabel = new JLabel("Genre: ");
		quantityOnHandLabel = new JLabel("Quantity On Hand: ");
		productPriceLabel = new JLabel("Product Price: ");
		restockingFeeLabel = new JLabel("Restocking Fee: ");
		valueLabel = new JLabel("Inventory Value: ");
		entireMusicCDInventoryValueLabel  = new JLabel("Value of Entire Music CD Inventory:");
		
		logoLabel = new JLabel();

		uPCNumberText = new JTextField("");
		CDNameText = new JTextField("");
		genreText = new JTextField("");
		quantityOnHandText = new JTextField("");
		productPriceText = new JTextField("");
		restockingFeeText = new JTextField("");
		valueText = new JTextField("");
		entireMusicCDInventoryValueText = new JLabel("");

		uPCNumberText.setEditable(false);
		CDNameText.setEditable(false);
		genreText.setEditable(false);
		quantityOnHandText.setEditable(false);
		productPriceText.setEditable(false);
		restockingFeeText.setEditable(false);
		valueText.setEditable(false);
		

		//Setup the code for the buttons
		nextButton = new JButton("Next"); 
		nextButton.addActionListener(new ActionListener() //Register event handler 
		{ 
			public void actionPerformed(ActionEvent event) //Process button event 
			{  	
				//When going to the previous item, move forward one
				index++;
				//If we were at the end (ie. index = theMusicCDInventory.getNumberOfMusicCDs() - 1), then index will be theInventory.getNumberOfMusicCDs() now
				if ( index == theMusicCDInventory.getNumberOfMusicCDs() ) {
					//The index can not exceed the size of our inventory array, so we move to the front of the inventory
					//whose index is
					index = 0;
				}
				repaint();				
			}
		});
		
		previousButton = new JButton("Previous"); 
		previousButton.addActionListener(new ActionListener() //Register event handler 
		{ 
			public void actionPerformed(ActionEvent event) //Process button event 
			{  	
				//When going to the previous item, move back one
				index--;
				//If we were at the beginning (ie. index = 0), then index will be -1
				if ( index == -1 ) {
					//We can not have index == -1, so we move to the end of the inventory
					// whose index is
					index = theMusicCDInventory.getNumberOfMusicCDs() - 1;
				}
				repaint();
			}
		});
		
		firstButton = new JButton("First"); 
		firstButton.addActionListener(new ActionListener() //Register event handler 
		{ 
			public void actionPerformed(ActionEvent event) //Process button event 
			{  	
				//The first element of the inventory is at
				index = 0;
				
				repaint();				
			}
		});
		
		lastButton = new JButton("Last"); 
		lastButton.addActionListener(new ActionListener() //Register event handler 
		{ 
			public void actionPerformed(ActionEvent event) //Process button event 
			{  	
				//The last element of the inventory is at
				index = theMusicCDInventory.getNumberOfMusicCDs() - 1;

				repaint();				
			}
		});

		dvdFrame = new JFrame(); //JFrame container

		//Title of window
		dvdFrame.setTitle("Music CD Inventory - 5");

		//End the program when the window is closed
 		dvdFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

		//Create and setup panel to hold buttons
		buttonPanel = new JPanel();
		buttonPanel.add(nextButton);
		buttonPanel.add(previousButton);
		buttonPanel.add(firstButton);
		buttonPanel.add(lastButton);
		//Add the panel to the top of the GUI's window
		dvdFrame.getContentPane().add(buttonPanel, BorderLayout.NORTH);

		//Setup a panel to collect all the components.
		//This will have 8 rows and 2 columns
		JPanel centerPanel = new JPanel(new GridLayout(8, 2, 0, 4));
		//Row 1
		centerPanel.add(uPCNumberLabel);
		centerPanel.add(uPCNumberText);

		//Row 2
		centerPanel.add(CDNameLabel);
		centerPanel.add(CDNameText);

		//Row 3
		centerPanel.add(genreLabel);
		centerPanel.add(genreText);

		//Row 4
		centerPanel.add(quantityOnHandLabel);
		centerPanel.add(quantityOnHandText);

		//Row 5
		centerPanel.add(productPriceLabel);
		centerPanel.add(productPriceText);

		//Row 6
		centerPanel.add(restockingFeeLabel);
		centerPanel.add(restockingFeeText);

		//Row 7
		centerPanel.add(valueLabel);
		centerPanel.add(valueText);
		
		//Place the panel to the center of the GUI's window
		dvdFrame.getContentPane().add(centerPanel, BorderLayout.CENTER);

		//Setup a panel to hold the information about the Music CD inventory's value
		JPanel entireMusicCDInventoryValuePanel = new JPanel();
		entireMusicCDInventoryValuePanel.add(entireMusicCDInventoryValueLabel);
		entireMusicCDInventoryValuePanel.add(entireMusicCDInventoryValueText);

		//Place this panel to the bottom of the GUI's window
		dvdFrame.getContentPane().add(entireMusicCDInventoryValuePanel, BorderLayout.SOUTH);
		
		//Load the logo.jpg image from the disk
		try {
			logoLabel = new JLabel();
			logoLabel.setIcon(new javax.swing.ImageIcon(getClass().getResource("logo.jpg")));
			dvdFrame.getContentPane().add(logoLabel, BorderLayout.EAST);
		} catch (Exception e) {
			//Display an error message
			JOptionPane.showMessageDialog (null,
				"Problem loading image logo.jpg",
				"Music CD Inventory GUI",
				JOptionPane.ERROR_MESSAGE);
		}
		
		//User is done entering Music CDs, and the GUI is setup
		//Sort the Music CDs by name
		theMusicCDInventory.sortMusicCDInventory();
		//Display the GUI on screen
		setVisible(true);
	}

	public void setVisible(boolean status) {
		//Display the window
		dvdFrame.pack();
		dvdFrame.setVisible(status);	
		repaint();
	}

	//When the program data is modified, called repaint() to get the most recent information on the screen
	public  void repaint() {

		//Display the current item on the screen's text area
		//Get the Music CD at the current index
		MusicCD theMusicCD = theMusicCDInventory.getMusicCD(index);
	
		//Print out Music CD information to the screen
		if (theMusicCD != null) {

			uPCNumberText.setText( ""+theMusicCD.getuPCNumber() );
			
			CDNameText.setText( theMusicCD.getCDName() );

			genreText.setText( theMusicCD.getGenre() );

			productPriceText.setText( moneyFormat.format(theMusicCD.getPrice()) );

			restockingFeeText.setText( moneyFormat.format(theMusicCD.restockingFee()) );

			quantityOnHandText.setText( ""+theMusicCD.getquantityOnHand() );
			
			valueText.setText( moneyFormat.format(theMusicCD.getMusicCDInventoryValue()) );
			
		}
		
		entireMusicCDInventoryValueText.setText( moneyFormat.format(theMusicCDInventory.getTotalValue()) );

	}

	public static void main(String args[]) throws IOException
	{	

		new MusicCDInventory();

	}
}
 

Thank you in advance for the help.

[Larry Frissell]

Public MusicCDInventory() try public MusicCDInventory()

[Candie Duggan]

When I change the capital p to a lowercase p it tells me it is an illegal start of expression and gives me the 2nd error as ; expected. I am becoming totally confused on this.

Thank you for your help.

[Garrett Rowe]

Why is that line in there at all? First of all it's not valid Java syntax, and second of all you're attempting to call the constructor (I think) of MusicCDInventory from within the constructor of MusicCDInventory. I think you need to take a step back from the code, and consider what it is you're trying to do, and how all the pieces fit together.

One piece of advice I would give is don't try to code up your entire application, and then try to compile and debug it. Try more frequent compile schedule like:

Add a small bit of code

Compile

Fix any compile errors (if present)

Write some code to test the code you just compiled

Fix any logic errors

Repeat

When I say a small bit of code I literally mean no more than 3 - 5 lines.