Can someone review my working program and..

Question

Let me know if I wrote this out clearly and if it seems to be written alright for someone with a few days of Java. Just looking for advice or comments. Do not need anything fixed or anything like that its working how I need it to. Thanks you guys are the best.
import javax.swing.JOptionPane;
public class Carpet2
     public static void main(String[] args)
          String lengthString, widthString, carpetString;
          int length;
          int width;
          double carpet;
          lengthString = JOptionPane.showInputDialog(null,"Enter the length of the room", "Length of room", JOptionPane.INFORMATION_MESSAGE);
          widthString = JOptionPane.showInputDialog(null,"Enter the width of the room", "Width of room", JOptionPane.INFORMATION_MESSAGE);
          carpetString = JOptionPane.showInputDialog(null,"Enter the price per square foot for carpet", "Carpet price per sq. ft.", JOptionPane.INFORMATION_MESSAGE);
          length = Integer.parseInt(lengthString);
          width = Integer.parseInt(widthString);
          carpet = Double.parseDouble(carpetString);
          JOptionPane.showMessageDialog(null,"Total cost for the room is $" + String.format( "%.2f", (length * width * carpet)) );
}Edited by: vwyodapink on May 13, 2009 9:53 AM

Answer 1

vwyodapink wrote:
Let me know if I wrote this out clearly and if it seems to be written alright for someone with a few days of Java. Just looking for advice or comments. Do not need anything fixed or anything like that its working how I need it to. Thanks you guys are the best.
import javax.swing.JOptionPane;
public class Carpet2
     public static void main(String[] args)
          String lengthString, widthString, carpetString;
          int length;
          int width;
          double carpet;
          lengthString = JOptionPane.showInputDialog(null,"Enter the length of the room", "Length of room", JOptionPane.INFORMATION_MESSAGE);
          widthString = JOptionPane.showInputDialog(null,"Enter the width of the room", "Width of room", JOptionPane.INFORMATION_MESSAGE);
          carpetString = JOptionPane.showInputDialog(null,"Enter the price per square foot for carpet", "Carpet price per sq. ft.", JOptionPane.INFORMATION_MESSAGE);
          length = Integer.parseInt(lengthString);
          width = Integer.parseInt(widthString);
          carpet = Double.parseDouble(carpetString);
          JOptionPane.showMessageDialog(null,"Total cost for the room is $" + String.format( "%.2f", (length * width * carpet)) );
}Edited by: vwyodapink on May 13, 2009 9:53 AMLooks fine, you could condense it if you want to but it would mean losing those variable declarations if you were planning to use them later or were to somewhat extend to the program later.