Very Simple Calculator Program(New Programmer)

[JonathanH]

Hey all this community has always been friendly in the past so I decided to share my first full working program made from Java. If you wanna test it out (its a simple calculator) the download link is below.

 

Download link: https://drive.google.com/open?id=0B5ZLrP4AjBgkY2M3c0ZVaWE3cGM&authuser=0

If you think it's a virus I can tell you that it is not! Feedback is appreciated since this is my first program! (I'm only 16 ) I also had a background image which didn't seem to transfer over to the actual compressing of the program. If anyone can help me with that that will be nice!

 

Thanks,

Jonathan

 

P.S what can I add?

[,,,,,,,,,,,,,,,,,,,,,,,,,,]

Post the source code?

 

EDIT: I'm retarded. Should have checked using sandboxie before posting it was malware.

[JonathanH]

Post the source code? EDIT: This is malware, java programs are NOT .EXE FILES

its not? i converted the .jar to a .exe. Is that not what im supposed to do?

[JonathanH]

Post the source code? EDIT: This is malware, java programs are NOT .EXE FILES

package game;import java.awt.event.*;import javax.swing.*; /* *  creates a calculator that can add,subtract,multiply and divide 2 given numbers. */public class BasicCalculator extends JFrame implements ActionListener{JLabel author = new JLabel("By; Jonathan Hyun"); JButton addition = new JButton("Addition");JButton subtract = new JButton("Subtraction");JButton multiply = new JButton("Multiply");JButton divide = new JButton("Divide");JButton clear = new JButton("Clear");JButton quit = new JButton("Quit"); JTextField num1 = new JTextField(10);JTextField num2 = new JTextField(10);JTextField ans = new JTextField(10); String getTextField1;String getTextField2;String getAns; double textFieldNum1;double textFieldNum2;double textFieldAns; BasicCalculator(String title){super(title);setLayout(null);setContentPane(new JLabel(new ImageIcon("C:\\Users\\Jonathan\\Desktop\\New folder\\background.png"))); add(addition);add(subtract);add(divide);add(multiply);add(num1);add(num2);add(ans);add(clear);add(quit); addition.setBounds(100,10,100,30);subtract.setBounds(300,10,100,30);multiply.setBounds(100,125,100,30);divide.setBounds(300,125,100,30);num1.setBounds(100,70,100,30);num2.setBounds(300,70,100,30);ans.setBounds(200,200,100,50);clear.setBounds(350,300,75,30);quit.setBounds(50,300,75,30); quit.addActionListener(this);addition.addActionListener(this);subtract.addActionListener(this);multiply.addActionListener(this);divide.addActionListener(this);clear.addActionListener(this); quit.setActionCommand("quit");addition.setActionCommand("add");subtract.setActionCommand("subtract");multiply.setActionCommand("multiply");divide.setActionCommand("divide");clear.setActionCommand("clear"); ans.setEditable(false); setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);}  public void actionPerformed(ActionEvent event) {if(event.getActionCommand().equals("add")){getTextField1 = num1.getText();getTextField2 = num2.getText();textFieldNum1 = Integer.parseInt(getTextField1);textFieldNum2 = Integer.parseInt(getTextField2);textFieldAns = textFieldNum1 + textFieldNum2;getAns = Double.toString(textFieldAns);ans.setText(getAns); }if(event.getActionCommand().equals("subtract")){getTextField1 = num1.getText();getTextField2 = num2.getText();textFieldNum1 = Integer.parseInt(getTextField1);textFieldNum2 = Integer.parseInt(getTextField2);textFieldAns = textFieldNum1 - textFieldNum2;getAns = Double.toString(textFieldAns);ans.setText(getAns);}if(event.getActionCommand().equals("multiply")){getTextField1 = num1.getText();getTextField2 = num2.getText();textFieldNum1 = Integer.parseInt(getTextField1);textFieldNum2 = Integer.parseInt(getTextField2);textFieldAns = textFieldNum1 * textFieldNum2;getAns = Double.toString(textFieldAns);ans.setText(getAns);}if(event.getActionCommand().equals("divide")){getTextField1 = num1.getText();getTextField2 = num2.getText();textFieldNum1 = Integer.parseInt(getTextField1);textFieldNum2 = Integer.parseInt(getTextField2);textFieldAns = textFieldNum1 / textFieldNum2;getAns = Double.toString(textFieldAns);ans.setText(getAns);}if(event.getActionCommand().equals("clear")){num1.setText(null);num2.setText(null);ans.setText(null);}if(event.getActionCommand().equals("quit")){System.exit(0);} } public static void main(String [] args){BasicCalculator calculator = new BasicCalculator("Calculator v0.1");calculator.setVisible(true);calculator.setSize(500,400);calculator.setResizable(false); } }

Edited June 11, 2015 by colonel_mortis
Code tags and spoiler

[,,,,,,,,,,,,,,,,,,,,,,,,,,]

My bad, made a mistake. I'll blame the lack of sleep since I haven't slept for over 24 hours.

[JonathanH]

My bad, made a mistake. I'll blame the lack of sleep since I haven't slept for over 24 hours.

hehe, yeah I had a feeling people might mark it as malware. I'm still trying to figure out how to put the background image with the compressing. I see the problem since the image comes from a folder location in my computer

[Guest]

I'll review it.

[MeshFile]

Quick one i see: don't parse it as int nce users may usw float /double

[fizzlesticks]

Dividing by 0 should give an error or Nan, not infinity.

[JonathanH]

Quick one i see: don't parse it as int nce users may usw float /double

 

 

Dividing by 0 should give an error or Nan, not infinity.

Yeah I friend of mine just pointed out that he couldn't use decimals. It is easy as parsing it as a double?

 

Thanks for pointing it out ill try to fix that.

[MeshFile]

Yeah I friend of mine just pointed out that he couldn't use decimals. It is easy as parsing it as a double?

Thanks for pointing it out ill try to fix that.

Depends on how long of a digit you want for input. Check or divide by zero and users also want faster process so use other events when needed.

Experiment and that's a good time drain, r practice mean.

[JonathanH]

Depends on how long of a digit you want for input. Check or divide by zero and users also want faster process so use other events when needed.

Experiment and that's a good time drain, r practice mean.

alright thanks i'll take that into consideration

[topsu]

getTextField1 = num1.getText();getTextField2 = num2.getText();textFieldNum1 = Integer.parseInt(getTextField1);textFieldNum2 = Integer.parseInt(getTextField2);

It is unnecessary to type this into all if statements, you could just make one that regardless of which of the buttons was pressed.

[Dobbsjr]

Hey all this community has always been friendly in the past so I decided to share my first full working program made from Java. If you wanna test it out (its a simple calculator) the download link is below.

 

Download link: https://drive.google.com/open?id=0B5ZLrP4AjBgkY2M3c0ZVaWE3cGM&authuser=0

If you think it's a virus I can tell you that it is not! Feedback is appreciated since this is my first program! (I'm only 16 ) I also had a background image which didn't seem to transfer over to the actual compressing of the program. If anyone can help me with that that will be nice!

 

Thanks,

Jonathan

 

P.S what can I add?

Nice GUI. I also made a calculator recently, but in C#. 

 

https://github.com/GenesisOfTheVoid/C-.NET-Projects/tree/master/WindowsFormsApplication2

[Nuluvius]

-code snip-

 

You really should use code tags for that

[JonathanH]

You really should use code tags for that

oh sorry lol, how do you that?

[flibberdipper]

Hey, I did a really basic calculator in VSE 2012!

 

https://drive.google.com/file/d/0By5LOV3t5pwWamhFV3FfYVcwZ0U/view?usp=sharing