Calculator code

[black_orc]

Can anyone look at my calculator code... I get a trouble when compiling it.
Heres the code: p.s. im new to java so please be gentle...

import java.awt.*;
import java.awt.event.*;



public class Calculator extends Panel implements ActionListener {
  private TextField display = new TextField("0");

  private String buttonText = "789/456*123-0.=+";

  private double result = 0;

  private String operator = "=";

  private boolean calculating = true;

  public Calculator() {
    setLayout(new BorderLayout());

    display.setEditable(false);
    add(display, "North");

    Panel p = new Panel();
    p.setLayout(new GridLayout(4, 4));

    for (int i = 0; i < buttonText.length(); i++) {
      Button b = new Button(buttonText.substring(i, i + 1));
      p.add(b);
      b.addActionListener(this);

    }
    add(p, "Center");
  }

  public void actionPerformed(ActionEvent evt) {
    String cmd = evt.getActionCommand();
    if ('0' <= cmd.charAt(0) && cmd.charAt(0) <= '9' || cmd.equals(".")) {
      if (calculating)
        display.setText(cmd);
      else
        display.setText(display.getText() + cmd);
      calculating = false;
    } else {
      if (calculating) {
        if (cmd.equals("-")) {
          display.setText(cmd);
          calculating = false;
        } else
          operator = cmd;
      } else {
        double x = Double.parseDouble(display.getText());
        calculate(x);
        operator = cmd;
        calculating = true;
      }
    }
  }

  private void calculate(double n) {
    if (operator.equals("+"))
      result += n;
    else if (operator.equals("-"))
      result -= n;
    else if (operator.equals("*"))
      result *= n;
    else if (operator.equals("/"))
      result /= n;
    else if (operator.equals("="))
      result = n;
    display.setText("" + result);
  }

  public static void main(String[] args) {
     Frame frame = new Frame();
    frame.setTitle("Calculator");
    frame.setSize(200, 200);
    frame.addWindowListener(new WindowAdapter() {
      public void windowClosing(WindowEvent e) {
        System.exit(0);
      }
    });

    Container contentPane = frame.getContentPane();
    contentPane.add(new Calculator());
    frame.show();
  }
}

[Belisarius]

What's the error message you get when you compile it?

[@script]

you might have using the deprecated method. try to refine your code in the section of:

Container contentPane = frame.getContentPane();

[Rotkiv]

use JFrame and JPanel instead of just normal Frame and Panel by using

import javax.swing.*;

and get rid of container, in the end your main function should look like this.

public static void main(String[] args) {
     JFrame frame = new JFrame();
    frame.setTitle("Calculator");
    frame.setSize(200, 200);
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    frame.add(new Calculator());
    frame.setVisible(true);
}

JFrame has the .setDefaultCloseOperation() method so that takes the place of your listener and 5 lines of code.

with all those changes it works fine for me and looks great. GJ.