Changed the basic logic of cos sin tan calculation, Move the 0 button to the position under the 8 button, Changed the variable name oneDevidedBy to oneDividedBy. Done by Shieh Eik.

Author: Aloysius911

.gitignore

@@ -6,3 +6,4 @@
 # Mac OS System
 .DS_Store*
 ._*
+/bin/

src/simplejavacalculator/Calculator.java

@@ -23,7 +23,7 @@ public enum BiOperatorModes {
     }
 
     public enum MonoOperatorModes {
-        square, squareRoot, oneDevidedBy, cos, sin, tan ,log , rate, abs
+        square, squareRoot, oneDividedBy, cos, sin, tan ,log , rate, abs
     }
 
     private Double num1, num2;
@@ -90,14 +90,14 @@ public Double calculateMono(MonoOperatorModes newMode, Double num) {
         if (newMode == MonoOperatorModes.squareRoot) {
             return Math.sqrt(num);
         }
-        if (newMode == MonoOperatorModes.oneDevidedBy) {
+        if (newMode == MonoOperatorModes.oneDividedBy) {
             return 1 / num;
         }
         if (newMode == MonoOperatorModes.cos) {
-            return Math.cos(num);
+            return Math.cos(Math.toRadians(num));
         }
         if (newMode == MonoOperatorModes.sin) {
-            return Math.sin(num);
+            return Math.sin(Math.toRadians(num));
         }
         if (newMode == MonoOperatorModes.tan) {
             if (num == 0 || num % 180 == 0) {
@@ -107,7 +107,7 @@ public Double calculateMono(MonoOperatorModes newMode, Double num) {
                 return NaN;
             }
 
-            return Math.tan(num);
+            return Math.tan(Math.toRadians(num));
         }
         if (newMode == MonoOperatorModes.log) {
             return log10(num);

src/simplejavacalculator/UI.java

@@ -50,7 +50,7 @@ public class UI implements ActionListener {
 
    private final JTextArea text;
    private final JButton but[], butAdd, butMinus, butMultiply, butDivide,
-      butEqual, butCancel, butSquareRoot, butSquare, butOneDevidedBy,
+      butEqual, butCancel, butSquareRoot, butSquare, butOneDividedBy,
       butCos, butSin, butTan, butxpowerofy, butlog, butrate, butabs, butBinary;
    private final Calculator calc;
 
@@ -87,7 +87,7 @@ public UI() throws IOException {
 
       but = new JButton[10];      
       for (int i = 0; i < 10; i++) {
-         but[i] = new JButton(String.valueOf(i));
+    		 but[i] = new JButton(String.valueOf(i));
       }      
       butAdd = new JButton("+");      
       butMinus = new JButton("-");      
@@ -96,7 +96,7 @@ public UI() throws IOException {
       butEqual = new JButton("=");      
       butSquareRoot = new JButton("sqrt");      
       butSquare = new JButton("x*x");      
-      butOneDevidedBy = new JButton("1/x");      
+      butOneDividedBy = new JButton("1/x");      
       butCos = new JButton("Cos");      
       butSin = new JButton("Sin");      
       butTan = new JButton("Tan");      
@@ -133,7 +133,7 @@ public void init() {
       butEqual.setFont(font);
       butSquareRoot.setFont(font);
       butSquare.setFont(font);
-      butOneDevidedBy.setFont(font);
+      butOneDividedBy.setFont(font);
       butCos.setFont(font);
       butSin.setFont(font);
       butTan.setFont(font);
@@ -172,13 +172,14 @@ public void init() {
       panelSub4.add(butCancel);
       panel.add(panelSub4);
 
+      panelSub5.add(Box.createHorizontalStrut(92));
       panelSub5.add(but[0]);
       panelSub5.add(Box.createHorizontalStrut(210));
       panel.add(panelSub5);
 
       panelSub6.add(butSquare);
       panelSub6.add(butSquareRoot);
-      panelSub6.add(butOneDevidedBy);
+      panelSub6.add(butOneDividedBy);
       panelSub6.add(butxpowerofy);
       panel.add(panelSub6);
 
@@ -202,7 +203,7 @@ public void init() {
       butDivide.addActionListener(this);
       butSquare.addActionListener(this);
       butSquareRoot.addActionListener(this);
-      butOneDevidedBy.addActionListener(this);
+      butOneDividedBy.addActionListener(this);
       butCos.addActionListener(this);
       butSin.addActionListener(this);
       butTan.addActionListener(this);
@@ -261,9 +262,9 @@ public void actionPerformed(ActionEvent e) {
                                    reader()));
       }
 
-      if (source == butOneDevidedBy) {
+      if (source == butOneDividedBy) {
          writer(calc.calculateMono(
-                                   Calculator.MonoOperatorModes.oneDevidedBy, reader()));
+                                   Calculator.MonoOperatorModes.oneDividedBy, reader()));
       }
 
       if (source == butCos) {

src/simplejavacalculatorTest/CalculatorTest.java

@@ -76,7 +76,7 @@ void CalculateMonoSquareRootTest() {
 	@Test
 	void CalculateMonoOneDividedByTest() {
 		Calculator calculator = new Calculator();
-		Assertions.assertEquals(0.10, calculator.calculateMono(Calculator.MonoOperatorModes.oneDevidedBy, 10.0));
+		Assertions.assertEquals(0.10, calculator.calculateMono(Calculator.MonoOperatorModes.oneDividedBy, 10.0));
 	}
 
 	@Test