Help wit h code

[Sydney Naidoo]

Hi
I am a student learning java
This is what i am required to do
A company sells five products Product 1, $2.98; product 2, $4.50;
product 3, $9.98; product 4, $4.49 and product 5, $6.87. Write an application that reads a series of pairs of numbers as follows:
a) product number
b) quantity sold
use a switch statement to determine the retail price for each product. It should calculate and display the total retail value of all products sold. Use a sentinel-controlled loop to determine when the program should stop looping and display the final results

Here is my code
am i on the right track

import java.util.Scanner;
 
public class Sales
{
  
   public void calculateSales()
   {
      Scanner input = new Scanner( System.in );
      
      double product1 = 0; 
      double product2 = 0; 
      double product3 = 0; 
      double product4 = 0; 
      double product5 = 0; 
      int products
    
      System.out.print("Enter  a product number between 1 and 5 or -1 to quit.   ");
    
      
      while ( products != -1  ) 
 
      {
      System.out.print("Enter the quantity of products.  ");
 

       switch ( products )
 
        {
 
            case 1:
            price = 2.98;
            product1 = (double) quantity * price;
            break;
            
            case 2:
            price = 4.50;
            product2 = (double) quantity * price;
            break;
            
            case 3:
            price = 9.98;
            product2 = (double) quantity * price;
            break;
            
            case 4:
            price = 4.49;
            product2 = (double) quantity * price
            break;
            
            case 5:
            price = 6.87;
            product5 = (double) quantity * price
            break;
            
 
              }
       
 
                if (product = = 0 && product > 5)
                    System.out.print("Enter a number between 1 and 5.");
 

 
 System.out.print("Enter the a product number between 1 and 5 or -1 to quit.   ");
          
}
      
     
      System.out.println();
      System.out.printf( "Product 1: $%.2f\n", product1 );
      System.out.printf( "Product 2: $%.2f\n", product2 ); 
      System.out.printf( "Product 3: $%.2f\n", product3 );
      System.out.printf( "Product 4: $%.2f\n", product4 );
      System.out.printf( "Product 5: $%.2f\n", product5 );
      
   } 
}

[edit: PMc added code tags and fiddled with spacing]
[ March 27, 2006: Message edited by: Pauline McNamara ]

[Pauline McNamara]

Hi Sydney, welcome to Javaranch!

This forum is for Cattle Drive questions, so I'm moving your question over to Java In General - Beginner. I also took the liberty of adding code tags to your post so the code part looks more formatted.

Good luck with your course!

Pauline

[Martin Simons]

On the right track. If this is all of your code, then I have to point
out that "price" is not declared, only defined. Change the "price" line
to "double price = ..." and then you can also lose the "(double)" on the
following lines.

I assume you will make quantity a double when you actually receive that
answer from the user (that code seems not to have been written yet).
[ March 27, 2006: Message edited by: Martin Simons ]

[fred Joly]

in the line :

 if (product = = 0 && product > 5)    

I guess you mean "OR" ("||") and not "AND" ("&&"). What about -2 ?
Maybe you could put a "default case" at the end of your switch statement that would do the same job.

regards

[Sydney Naidoo]

Thank you for directing to the correct forumn.
Can someone help with correcting or enhancing the code for me.

[ak pillai]

You need a main method to run it also would be nice to have some basic input validation to check if the user is keying in the right value.
[ March 28, 2006: Message edited by: ak pillai ]