Help wit h code

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 ]

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

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 ]

in the line :

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

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

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 ]