Return type required

[Nick Harms]

This public Retailitem() is giving me this error, not sure why, to the naked eye it looks just like my example, probably a simple fixed.
Says invalid method declaration return type required,
This is a class a demo uses.

   
/** 
    RetailItem Class
 */
public class RetailItem
{
   private String Description;
   private int UnitsOnHand;
   private double Price;
   
   public Retailitem()                     
   {
       Description = "";
       UnitsOnHand = 0;
       Price = 0;

    }
    
    public Retailitem(String myDescription, int myUnitsOnHand, double myPrice)
    {
       Description = myDescription
       UnitsOnHand = myUnitsOnHand
       Price = myPrice

    /**
      The setDescription method sets the description.
      @param myDescription The description.
   */

   public void setDescription(String myDescription)
   {
      Description = myDescription;
   }

   /**
      The setUnitsOnHand method returns the Units on hand.
      @param myUnitsOnHand The units on hand.
   */

   public void setUnitsOnHand(myUnitsOnHand)
   {
      UnitsOnHand = myUnitsOnHand;
   }
   
    /**
      The setUnitsOnHand method returns the Units on hand.
      @param myUnitsOnHand The units on hand.
   */

   public void setPrice(myPrice)
   {
      Price = myPrice;
   }

   /**
      The getdescription method returns a description.
      @return The description.
   */
   
   public String getDescription()
   {
      return Description;
      
   }

   /**
      The getUnitsOnHand method returns Units on Hand.
      @return the units on hand.
   */

   public double getUnitsOnHand()
   {
      return UnitsOnHand;
   }
   
   /**
      The getPrice method returns the Price.
      @return price.
   */
    
    public double getPrice()
    {
        return Price;
    }
}
 

[ahamed irshad]

it should be capital i (I)in Retailitem.plesae make change in the constructor.

[Nick Harms]

Ok now its saying cannot find symbol - class item in the demo. on the item one = new RetailItem(); line. Why is that? It matches the class name.

  

/**
    Demo Program for the Retail Item class
    Chapter 6, Programming Challenge 4
 */

public class RetailItemDemo
{
   public static void main(String[] args)
   {
      //Create the objects
      Item one = new RetailItem();
      Item two = new RetailItem();
      Item three = new RetailItem();
      
      // Set item one info.
      one.setDescription("Jacket");
      one.setUnitsOnHand(12);
      one.setPrice(59.95);
      
      // Set item two info.
      two.setDescription("Designer Jeans");
      two.setUnitsOnHand(40);
      two.setPrice(34.95);
      
      // Set item three info.
      three.setDescription("Shirt");
      three.setUnitsOnHand(20);
      three.setPrice(24.95);
      
      //Display object 1 info
      System.out.println("         Item 1 Information"         );
      System.out.println("====================================");
      System.out.println("Description: " + one.getdescription());
      System.out.println("Units on hand: "  + one.UnitsOnHand());
      System.out.println("Price: "          + one.Price);
      
      //Display object 2 info
      System.out.println("         Item 2 Information"         );
      System.out.println("====================================");
      System.out.println("Description: " + two.getdescription());
      System.out.println("Units on hand: "  + two.UnitsOnHand());
      System.out.println("Price: "          + two.Price);
      
      //Display object 3 info
      System.out.println("         Item 3 Information"         );
      System.out.println("====================================");
      System.out.println("Description: " + three.getdescription());
      System.out.println("Units on hand: "  + three.UnitsOnHand());
      System.out.println("Price: "          + three.Price);
      
      
   }
}
  

[Nick Harms]

Never mind I got it, should be RetailItem one= new RetailItem();

[Seetharaman Venkatasamy]

Nick Harms wrote:Never mind I got it, should be RetailItem one= new RetailItem();

or *may be* you forget to implements *Item* interface . and,

remember : Constructor name must be same as class name

[Janeice DelVecchio]

Nick,

Why do you have an empty constructor? Does it ever make sense to have an item with no information in this program?

You can use the overloaded constructor how it's supposed to and get rid of the setter method calls in your demo class. if it's a requirement to use the setter methods, you can use a

this.setDescription()    

inside the overloaded constructor.

[Saul Groene]

#
# public double getUnitsOnHand()
# {
# return UnitsOnHand;
# }

should have an 'int' return type not a 'double'

[Saul Groene]

# //Display object 1 info
# System.out.println(" Item 1 Information" );
# System.out.println("====================================");
# System.out.println("Description: " + one.getdescription());
# System.out.println("Units on hand: " + one.UnitsOnHand());
# System.out.println("Price: " + one.Price);
#
# //Display object 2 info
# System.out.println(" Item 2 Information" );
# System.out.println("====================================");
# System.out.println("Description: " + two.getdescription());
# System.out.println("Units on hand: " + two.UnitsOnHand());
# System.out.println("Price: " + two.Price);
#
# //Display object 3 info
# System.out.println(" Item 3 Information" );
# System.out.println("====================================");
# System.out.println("Description: " + three.getdescription());
# System.out.println("Units on hand: " + three.UnitsOnHand());
# System.out.println("Price: " + three.Price);
#


The bolded method calls above should be the 'getter' methods ie... three.getUnitsOnHand() instead of what is there..

[Janeice DelVecchio]

Saul,

Welcome to Java Ranch!

Please remember to use code tags... it helps format the code to make it easier to read

Thanks