Inventory Program Part 1

Question

I am so lost with Part 1 of the Inventory Program for my Java class. I am receiving an error and cannot figure out where it is comming from. The assignment is to:
Choose a product that lends itself to an inventory (for example, products at your
workplace, office supplies, music CDs, DVD movies, or software).
� Create a product class that holds the item number, the name of the product, the number
of units in stock, and the price of each unit.
� Create a Java application that displays the product number, the name of the product, the
number of units in stock, the price of each unit, and the value of the inventory (the
number of units in stock multiplied by the price of each unit). Pay attention to the good
programming practices in the text to ensure your source code is readable and well
documented.
Here is the code I have so far:
import java.util.Scanner; // program uses class Scanner
public class Inventory
     private static void Quit()
System.out.println("Goodbye");
     System.exit (0);
// main method begins execution of Java application
public static Void main (String args [] )
     // create Scanner to obtain input from command window
Scanner input = new Scanner(System.in);
     Product Prod = new Product();
     System.out.printf (Prod.toString () );
     System.out.print("\nEnter Prod Name (stop to quit): "); // prompt for name
Prod.setName(input.nextLine()); // get name
     if(Prod.getName().compareToIgnoreCase("stop") == 0)
System.out.println("Stop entered, Thank you");
Quit();
} //end if
System.out.print("Enter Prod number for " + Prod.getName() + ": "); // prompt
Prod.setitemNum(input.nextLine()); // read Prod number from user
System.out.print("Enter Units on hand: "); // prompt
Prod.setunits(input.nextDouble()); // read second number from user
     if(Prod.getunits() <= 0)
          System.out.println ("Invalid amount, Units on hand worked must be positive");
          System.out.print("Please enter actual units on hand: ");
          Prod.setunits(input.nextDouble());
     } //end if
     System.out.print("Enter Unit Price: $ "); // prompt
Prod.setprice(input.nextDouble()); // read third number from user
     if(Prod.getprice() <= 0)
          System.out.println ("Invalid amount, Unit Price must be positive");
          System.out.print("Please enter actual unit price: $ ");
          Prod.setprice(input.nextDouble());
     } //end if
     String fmtStr = "\n%-7s %-10s %12s %12s %12s\n";
     System.out.printf(fmtStr, "Prod #", "Prod Name",
"Units On Hand", "Unit Cost", "Total Cost");
for(int i = 0; i<5; i++);
string Product[] prodArray = new string[5]; *******************
     //Prod[] myArray = new Prod[5]; // allocates memory for 5 strings
prodArray[0] = "Book"; // initialize first element
prodArray[1] = "Mag"; // initialize second element
prodArray[2] = "Hat"; // etc.
prodArray[3] = "Scarf";
prodArray[4] = "Rain Gauge";
System.out.println("Element at index 0: " + prodArray[0]);
System.out.println("Element at index 1: " + prodArray[1]);
System.out.println("Element at index 2: " + prodArray[2]);
System.out.println("Element at index 3: " + prodArray[3]);
System.out.println("Element at index 4: " + prodArray[4]);
     }//end for
}//end while
} //end main
}// end class Inventory
// Class Product holds Product information
class Product
private String name;
private String itemNum;
private double units;
private double price;
//default constructor
public Product()
name = "";
itemNum = "";
units = 0;
     price = 0;
}//end default constructor
//Parameterized Constructor
public Product(String name, String itemNum, double units, double price)
this.name = name;
this.itemNum = itemNum;
this.units = units;
     this.price = price;
}//end constructor
     public void setName(String name) {
this.name = name;
     String getName()
          return name;
public void setitemNum ( String itemNum )
this.itemNum = itemNum;
public String getitemNum()
     return itemNum;      }
public void setunits ( double units )
this.units = units;
public double getunits()
          return units;
public void setprice ( double price )
this.price = price;
public double getprice()
     return price;
public double getvalue()
     return (units * price);
}//end Class Product
I am receiving an error on Line 61 ( I have place a few asterisks beside it). The error says inventory.java:61: ';' expected string Product[] prodArray = new string[5]
Does anyone have an idea what I am doing wrong.

Answer 1

I reformatted the code and put it inside code tags to retain formatting, anywhere there is a comment, take a pause and compare it to your original to see the differences. Any questions don't hesitate to post them.
import java.util.Scanner;
public class Inventory implements Runnable{
    private Product[] prodArray = new Product[5];
    public Inventory() {
        int arraySize = 5;
        prodArray = new Product[arraySize];
        //the for loop did not make sense, since you were loading each object with different information.
        //This is an array of Product Objects
        //anything between two double quotes is a String literal
        //essentially an Object of type String
        //Use your setter methods in the Product class to set the properties of each "new" Product Object.
        //Or use the Parameterized version of your constructor to set the variables inside each "new" object
        //everytime you use the "new" keyword you get a brand spanking new object
        prodArray[0] = new Product();//"Book"; // initialize first element
        //Since you use an empty constructor none of the variable in your Product object have the values you want.
        //Set them after you create your object and assign it to the array.
        prodArray[0].setName("Book");
        //prodArray[0].setprice(0.00);
        //etc...
        prodArray[1] = new Product();//"Mag"; // initialize second element
        prodArray[1].setName("Mag");
        prodArray[2] = new Product();//"Hat"; // etc.
        prodArray[2].setName("Hat");
        prodArray[3] = new Product();//"Scarf";
        prodArray[3].setName("Scarf");
        prodArray[4] = new Product();//"Rain Gauge";
        prodArray[4].setName("Rain Gauge");
        //You never override the toString() method of Product to your output will be the String representation of the
        //Object's reference.
        //I have overidden the toString() method of Product for you using your format, look at it closely.
        System.out.println("Element at index 0: " + prodArray[0]);
        System.out.println("Element at index 1: " + prodArray[1]);
        System.out.println("Element at index 2: " + prodArray[2]);
        System.out.println("Element at index 3: " + prodArray[3]);
        System.out.println("Element at index 4: " + prodArray[4]);
    public void run() {
        showHeader();
        //You never set the name of the Product here and toString doesn't return anything you want to see.
        Scanner input = new Scanner(System.in);
        Product prod = new Product();
        //This show nothing we want to see, try overriding the toString() method in the Product class if you
        //want to see a specific variable of the Product class displayed with the toString() method.
//        System.out.printf (prod.toString () );
        String inputString;
        int inputInt;
        double inputDouble;
        while( true ) {
            //Don't set invalid data into your Product Object, check to see if it meets your criteria first and then
            //set it.
            System.out.print("\nEnter prod Name (stop to quit): "); // prompt for name
            inputString = input.nextLine();
            if( inputString.compareToIgnoreCase("stop") == 0 ) {
                System.out.println("Stop entered, Thank you");
                quit();
            } else {
                prod.setName(input.nextLine()); // get name
            System.out.print("Enter prod number for " + prod.getName() + ": "); // prompt
            prod.setitemNum(input.nextLine()); // read prod number from user
            System.out.print("Enter Units on hand: "); // prompt
            inputInt = input.nextInt(); // read second number from user
            //to do this check, put your input into a loop, not an if statement.
            //and get an integer from the Scanner not a double,  doubles are not precise and cannot be checked
            //consistently with a loop or if condition
            while ( inputInt <= 0) {
                System.out.println ("Invalid amount, Units on hand worked must be positive");
                System.out.print("Please enter actual units on hand: ");
                inputInt = input.nextInt();
            prod.setunits( inputInt );
            //There are better ways to store currency such as an integer, but there are a lot of conversion you need to do
            // to display them, so just use double for now.
            System.out.print("Enter Unit Price: $ "); // prompt
            inputDouble = input.nextDouble(); // read third number from user
            //while loop here as well and you don't want your products to be $0.00 do you?
            while ( inputDouble < 0.01) {
                System.out.println ("Invalid amount, Unit Price must be positive");
                System.out.print("Please enter actual unit price: $ ");
                prod.setprice(input.nextDouble());
            } //end if
            prod.setprice( inputDouble );
            System.out.println( prod.toString() );
            //You never store the input from the user consider implementing an ArrayList
    private void showHeader() {
        String fmtStr = "\n%-7s%-10s%12s%12s%12s\n";
        System.out.printf(fmtStr, "prod #", "prod Name", "Units On Hand", "Unit Cost", "Total Cost");
    private void quit() {
        System.out.println("Goodbye");
        System.exit (0);
    //Void is not the same as the reseerved word "void"
    //Be sure you watch your capitalizations, all Java reserved words (void, public, static, etc.) are lowercase
    public static void main (String args [] ) {
        //Create an object of type inventory
        Inventory i = new Inventory();
        //start the run method of that object;
        i.run();
* Class Product holds Product information
class Product {
    private String name;
    private String itemNum;
    private int units;
    private double price;
     * Constructor
    public Product() {
        //To save space and your sanity you could initialize these variables when they're declared like this.
        // private String name = "";
        name = "";
        //private String item = ""; etc.
        itemNum = "";
        units = 0;
        price = 0;
     * Constructor
     * @param name
     * @param itemNum
     * @param units
     * @param price
    public Product(String name, String itemNum, int units, double price) {
        this.name = name;
        this.itemNum = itemNum;
        this.units = units;
        this.price = price;
    public void setName(String name) {
        this.name = name;
    //This method because it does not have the public keyword is now Package private,
    // it cannot be used outside the current Package.
    String getName() {
        return name;
    public void setitemNum( String itemNum ) {
        this.itemNum = itemNum;
    public String getitemNum(){
        return itemNum;
    //Try to be consistent with your variable and method names, they should start with lower case and each subsequent
    //word should be capitalized like this
    //public void setUnits( double units ) {
    public void setunits( int units ) {
        this.units = units;
    public double getunits() {
        return units;
    public void setprice ( double price ) {
        this.price = price;
    public double getprice() {
        return price;
    public double getvalue() {
        return (units * price);
    public String toString() {
        String fmtStr = "%-7s%-10s%12d%12.2f%12.2f";
        return String.format(fmtStr, itemNum, name, units, price, getvalue() );
}