[acetown]

I cannot figure out why total inventory is coming up null. It runs like it is supposed to, but it puts total inventory is null. I need it to produce the output of NumberofUnits * UnitPrice
Here is my code

import java.util.Scanner;

  class Camera
	{

	    private String ProductName;
	    private int ItemNumber;
	    private int NumberofUnits;
	    private double UnitPrice;
	 
	    public Camera(int i, String Name, int Number, int Units, double Price)
	    {
	 
                ProductName = Name;
	        ItemNumber = Number;
	        NumberofUnits = Units;
	        UnitPrice = Price;
	    }
	 
	    public void setProductName(String title, String Name)
	    {
	        ProductName = Name;
	    }
	 
	    public String getProductName()
	    {
	        return ProductName;
	    }
	 
	    public void setItemNumber(int Number)
	    {
	        ItemNumber = Number;
	    }
	 
	    public double getItemNumber()
	    {
	        return ItemNumber;
	    }
	 
	    public void setNumberofUnits(int Units)
	    {
	        NumberofUnits = Units;
	    }
	 
	    public double getNumberofUnits()
	    {
	        return NumberofUnits;
	    }
	 
	    public void setUnitPrice(double Price)
	    {
	        UnitPrice = Price;
	    }
	 
	    public double getUnitPrice()
	    {
	        return UnitPrice;
	    }

            public void setTotalInventory(double Inventory)
            {
            }
	 
	    public double getTotalInventory()
	    {
	        return UnitPrice * NumberofUnits;
	    }
          
	}


	 
	public class Inventory
	{
	 
	    private static Scanner input = new Scanner(System.in);
		private static String TotalInventory;
	 
	    public static void main(String args[] )
	    {
	        Camera[] camera = new Camera[5];
	 
	        camera[0] = new Camera(0, "Nikon", 0001, 5, 1499);
	        camera[1] = new Camera(0, "Olympus", 0002, 7, 799);
	        camera[2] = new Camera(0, "Canon", 0003, 3, 1549);
	        camera[3] = new Camera(0, "Casio", 0004, 10, 89);
	        camera[4] = new Camera(0, "Kodak", 0005, 3, 299);
	 
	        System.out.println("Camera Inventory Program");
	 
	        for (int i = 0; i < camera.length; i++)
	        {
	            System.out.println("Product Name is " + camera[i].getProductName());
	            System.out.println("Item Number is " + camera[i].getItemNumber());
	            System.out.println("Number of Units is "+ camera[i].getNumberofUnits());
	            System.out.println("Unit Price is " + camera[i].getUnitPrice());
	           
				
				System.out.println("TotalInventory is " + TotalInventory);
	        }
	 
	    }
	 
	    public static Scanner getInput()
	    {
	        return input;
	    }
	 
	    public static void setInput(Scanner input)
	    {
	        Inventory.input = input;
	    }
	}
   

[GregBrannon]

Total inventory is printed using this line (#099):

System.out.println("TotalInventory is " + TotalInventory);

Where is TotalInventory modified from its default value of null as an Inventory instance variable?

[pbl]

	public class Inventory
	{
	 
	    private static Scanner input = new Scanner(System.in);
            // TotalInventory declared here it is null
		private static String TotalInventory;
	 
	    public static void main(String args[] )
	    {

	        Camera[] camera = new Camera[5];
	 
	        for (int i = 0; i < camera.length; i++)
	        {
	            System.out.println("Product Name is " + camera[i].getProductName());
	            System.out.println("Item Number is " + camera[i].getItemNumber());
	            System.out.println("Number of Units is "+ camera[i].getNumberofUnits());
	            System.out.println("Unit Price is " + camera[i].getUnitPrice());
	           
			// TotalInventory print here. Nowhere you assigned a value to it
                        // so it is still null	
			System.out.println("TotalInventory is " + TotalInventory);
	        }

[guntario]

Your getters and setters are in the wrong class. They should be in the same class as TotalInventory.

[pbl]

guntario, on 09 May 2012 - 05:14 PM, said:

Your getters and setters are in the wrong class. They should be in the same class as TotalInventory.

???

[acetown]

I am trying to get it to print a total but cannot seem to get it. I hate being new to java.

[guntario]

pbl, on 09 May 2012 - 02:18 PM, said:

guntario, on 09 May 2012 - 05:14 PM, said:

Your getters and setters are in the wrong class. They should be in the same class as TotalInventory.

???

He's updating private String TotalInventory from the Camera class, but he declared TotalIventory in the inventory class.

[pbl]

no he probably wants to println()

camera[i].getTotalInventory()

[acetown]

it was actually something that simple. I changed it to that and it worked just like it should have. Thanks for all the help.

[pbl]

guntario, on 09 May 2012 - 05:22 PM, said:

He's updating private String TotalInventory from the Camera class,

where do you see that ?

[DimitriV]

Doesn't look like there is one.

[acetown]

Now I have to Create a method to calculate the value of the entire inventory.
Create another method to sort the array items by the name of the product.

[pbl]

for the Toatl

  double total = 0;
  for(int i = 0; i < camera.length; ++i) {
     ...
     System.out.println("Total inv: " + camera[i].getTotalInventory());
     total += camera[i].getTotalInventory();
  }
  System.out.pritnl(total);

for the sort have your class Camera:

class Camera implements Comparable<Camera> {
  
   public int compareTo(Camera other) {
     return String.compareTo(productName, other.productName);
   }

in your main simply do: Arrays.sort(camera);

[pbl]

DimitriV, on 09 May 2012 - 05:36 PM, said:

Doesn't look like there is one.

Yes I know, my +1 was a missclick

butseems that acetown gives +1 to anybody who reply to his post
Whatever they reply

[acetown]

When I implement the class Camera like you show, it gives this message The method compareTo(String) in the type String is not applicable for the arguments (String, String)
Here is the code

import java.util.Arrays;
import java.util.Scanner;

  class Camera implements Comparable<Camera>{
	  
	  public int compareTo(Camera other){
		  return String.compareTo(ProductName, other.ProductName);
	  }
  
	
       
	   
	    private String ProductName;
	    private int ItemNumber;
	    private int NumberofUnits;
	    private double UnitPrice;
	 
	    public Camera(int i, String Name, int Number, int Units, double Price)
	    {
	    	
	        
                ProductName = Name;
	        ItemNumber = Number;
	        NumberofUnits = Units;
	        UnitPrice = Price;
	    }
	 
	    public void setProductName(String title, String Name)
	    {
	        ProductName = Name;
	    }
	 
	    public String getProductName()
	    {
	        return ProductName;
	    }
	 
	    public void setItemNumber(int Number)
	    {
	        ItemNumber = Number;
	    }
	 
	    public double getItemNumber()
	    {
	        return ItemNumber;
	    }
	 
	    public void setNumberofUnits(int Units)
	    {
	        NumberofUnits = Units;
	    }
	 
	    public double getNumberofUnits()
	    {
	        return NumberofUnits;
	    }
	 
	    public void setUnitPrice(double Price)
	    {
	        UnitPrice = Price;
	    }
	 
	    public double getUnitPrice()
	    {
	        return UnitPrice;
	    }

            public void setTotalInventory(double Inventory)
            {
            	Inventory = getTotalInventory();
            }
	 
	    public double getTotalInventory()
	    {
	        return UnitPrice * NumberofUnits;
	    }

            
	}
  

	 
	public class Inventory
	{
	 
	    
		private static Scanner input = new Scanner(System.in);
		
		
		
	 
	    public static void main(String args[])
	    {
	    	
	        Camera[] camera = new Camera[5];
	        Arrays.sort(camera);
	        
	        camera[0] = new Camera(0, "Nikon", 0001, 5, 1499);
	        camera[1] = new Camera(0, "Olympus", 0002, 7, 799);
	        camera[2] = new Camera(0, "Canon", 0003, 3, 1549);
	        camera[3] = new Camera(0, "Casio", 0004, 10, 89);
	        camera[4] = new Camera(0, "Kodak", 0005, 3, 299);
	 
	        System.out.println("Camera Inventory Program");
	 
	        for (int i = 0; i < camera.length; i++)
	        {
	            System.out.println("Product Name is: " + camera[i].getProductName());
	            System.out.println("Item Number is: " + camera[i].getItemNumber());
	            System.out.println("Number of Units is: "+ camera[i].getNumberofUnits());
	            System.out.println("Unit Price is: " + camera[i].getUnitPrice());
		    System.out.println("TotalInventory is: " + camera[i].getTotalInventory());
		    System.out.println();
	        }
	        
	        double total = 0;
	        	for(int i = 0; i < camera.length; ++i) {
	        	   
	        	   System.out.println("Total inv: " + camera[i].getTotalInventory());
	        	   total += camera[i].getTotalInventory();
	        	}
	        	System.out.println(total);
	 
	    }
	 
	    public static Scanner getInput()
	    {
	        return input;
	    }
	 
	    public static void setInput(Scanner input)
	    {
	        Inventory.input = input;
	    }
	}