java - Problem Med Lommeregner, Double Og Kvadratrod - Udvikleren.dk

Tags: java

Bruger #16945 @ 24.09.12 23:21

Hellow guys..

Jeg har lavet en low quality lommeregner, med min low quality hjerne. Jeg har bare stille og rolig lavet den, startet med plus, minus, divideret og gange, gået op til at kunne slette, clear, kvadratrod.. Så besluttede jeg mig for at lave den om til at kunne bruge decimal tal.. Det var ikke så svært, istedet for int skulle det da bare være double eller float.. Jaaa-nej, det var ikke bare lige sådan, den kan da godt regne med double, lige på nær kvadratrods funktionen.. I made something wrong, men efter at have rettet alle de fejl jeg kunne finde, er mit problem altså stadig ikke helt løst..

Nogen som kan hjælpe mig ud her?

Java kode

 import javax.swing.JFrame;
 import javax.swing.JPanel;
 import javax.swing.JButton;
 import javax.swing.JOptionPane;
 import javax.swing.JLabel;
 
 import java.awt.Color;
 import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
 
 public class Main extends JFrame
 {    
     double first, second, result;
     String number = "";
     boolean bplus, bminus, btimes, bdivided;
     
     private JPanel panel;
     private JLabel label;
     private JButton b1;
     private JButton b2;
     private JButton b3;
     private JButton b4;
     private JButton b5;
     private JButton b6;
     private JButton b7;
     private JButton b8;
     private JButton b9;
     private JButton b0;
     private JButton equals;
     private JButton plus;
     private JButton minus;
     private JButton times;
     private JButton divided;
     private JButton squareRoot;
     private JButton c;
     private JButton delete;
     private JButton period;
     
     
     public Main()
     {
         this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
         this.setSize(432, 880);
         this.setVisible(true);
         
         panel = new JPanel();
         panel.setLayout(null);
         
         label = new JLabel(getNumber(), null, JLabel.CENTER);
         label.setBounds(0, 0, 160, 40);
         panel.add(label);
         
         b1 = new JButton("1");
         b1.setBounds(0, 40, 50, 40);
         panel.add(b1);
         
         b1.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent arg0)
             {
                 number += "1";
                 label.setText(getNumber());
             }
         });
         
         b2 = new JButton("2");
         b2.setBounds(50, 40, 50, 40);
         panel.add(b2);
         
         b2.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent arg0)
             {
                 number += "2";
                 label.setText(getNumber());
             }
         });
         
         b3 = new JButton("3");
         b3.setBounds(100, 40, 50, 40);
         panel.add(b3);
         
         b3.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent arg0)
             {
                 number += "3";
                 label.setText(getNumber());
             }
         });
         
         b4 = new JButton("4");
         b4.setBounds(0, 80, 50, 40);
         panel.add(b4);
         
         b4.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent arg0)
             {
                 number += "4";
                 label.setText(getNumber());
             }
         });
         
         b5 = new JButton("5");
         b5.setBounds(50, 80, 50, 40);
         panel.add(b5);
         
         b5.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent arg0)
             {
                 number += "5";
                 label.setText(getNumber());
             }
         });
         
         b6 = new JButton("6");
         b6.setBounds(100, 80, 50, 40);
         panel.add(b6);
         
         b6.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent arg0)
             {
                 number += "6";
                 label.setText(getNumber());
             }
         });
         
         b7 = new JButton("7");
         b7.setBounds(0, 120, 50, 40);
         panel.add(b7);
         
         b7.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent arg0)
             {
                 number += "7";
                 label.setText(getNumber());
             }
         });
         
         b8 = new JButton("8");
         b8.setBounds(50, 120, 50, 40);
         panel.add(b8);
         
         b8.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent arg0)
             {
                 number += "8";
                 label.setText(getNumber());
             }
         });
         
         b9 = new JButton("9");
         b9.setBounds(100, 120, 50, 40);
         panel.add(b9);
         
         b9.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent arg0)
             {
                 number += "9";
                 label.setText(getNumber());
             }
         });
         
         b0 = new JButton("0");
         b0.setBounds(50, 160, 50, 40);
         panel.add(b0);
         
         b0.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent arg0)
             {
                 number += "0";
                 label.setText(getNumber());
             }
         });
         
         plus = new JButton("+");
         plus.setBounds(150, 160, 50, 40);
         panel.add(plus);
         
         plus.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent e)
             {
                 label.setText("+");
                 first = Double.parseDouble(number);
                 number = "";
                 bplus = true;
                 bminus = false; 
                 bdivided = false; 
                 btimes = false;
             }
         });
         
         minus = new JButton("-");
         minus.setBounds(150, 120, 50, 40);
         panel.add(minus);
         
         minus.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent e)
             {
                 label.setText("-");
                 first = Double.parseDouble(number);
                 number = "";
                 bplus = false;
                 bminus = true; 
                 bdivided = false; 
                 btimes = false;
             }
         });
         
         times = new JButton("*");
         times.setBounds(150, 80, 50, 40);
         panel.add(times);
         
         times.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent e)
             {
                 label.setText("*");
                 first = Double.parseDouble(number);
                 number = "";
                 bplus = false;
                 bminus = false; 
                 bdivided = false; 
                 btimes = true;
             }
         });
         
         divided = new JButton("/");
         divided.setBounds(150, 40, 50, 40);
         panel.add(divided);
         
         divided.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent e)
             {
                 label.setText("/");
                 first = Double.parseDouble(number);
                 number = "";
                 bplus = false;
                 bminus = false; 
                 bdivided = true; 
                 btimes = false;
             }
         });
         
         equals = new JButton("=");
         equals.setBounds(100, 160, 50, 40);
         panel.add(equals);
         
         equals.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent e)
             {
                 second = Double.parseDouble(number);
                 
                 if(bplus)
                     result = first + second;
                 
                 else if(bminus)
                     result = first - second;
                 
                 else if(bdivided)
                     result = first / second;
                 
                 else if(btimes)
                     result = first * second;
                 
                 number = Double.toString(result);                
                 label.setText(number);
             }
         });
         
         squareRoot = new JButton("v");
         squareRoot.setBounds(200, 160, 50, 40);
         panel.add(squareRoot);
         
         squareRoot.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent e) 
             {
                 double temp = Double.parseDouble(number);
                 
                 for(double i = 0.0; i < temp; i += 0.1)
                 {
                     if(i*i == temp)
                     {
                         result = i;
                         break;
                     }
                     else
                     {
                         System.out.println("forloop has run " + i + " times");
                     }
                 }
                 
                 number = Double.toString(result);    
                 label.setText(number);
             }
             
         });
         
         c = new JButton("C");
         c.setBounds(200, 40, 60, 40);
         panel.add(c);
         
         c.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent e) 
             {
                 setDefault();
                 label.setText(number);
             }
             
         });
         
         delete = new JButton("<-");
         delete.setBounds(180, 0, 80, 40);
         panel.add(delete);
         
         delete.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent e) 
             {
                 number = number.substring(0, number.length() -1);
                 label.setText(number);
             }
             
         });
         
         period = new JButton(",");
         period.setBounds(0, 160, 50, 40);
         panel.add(period);
         
         period.addActionListener(new ActionListener()
         {
             public void actionPerformed(ActionEvent e) 
             {
                 number += ".";
                 label.setText(getNumber());
             }
             
         });
         
         
         this.add(panel);
     }
     
     public static void main(String args[])
     {
         Main main = new Main();
     }
     
     public String getNumber()
     {
         return number;
     }
     
     public void setDefault()
     {
         first = 0; second = 0; result = 0;
         number = "";
         bplus = false; bminus = false; btimes = false; bdivided = false;
     }
 }

5 svar postet i denne tråd vises herunder
3 indlæg har modtaget i alt 23 karma

Sorter efter stemmer Sorter efter dato

Bruger #10576 @ 24.09.12 23:55

287

Math.sqrt(X)

Bruger #14645 @ 25.09.12 00:27

806

Der forskellige ting der er problematiske i din udregning:

Java kode

 for(double i = 0.0; i < temp; i += 0.1)
 {
 if(i*i == temp)
 {
 result = i;
 break;
 }
 else
 {
 System.out.println("forloop has run " + i + " times");
 }
 }

1. For det første er det en meget ineffektiv måde at udregne det. Hvis jeg nu skrev 10000.0 vil den potentielt bruge 100000 iterationer. Men den ville selvfølgelig finde svaret meget hurtigere (efter 1000) iterationer...

2. Men du tjekker kun hvert 0.1 så det er ikke engang sikkert at den rammer den rigtigt værdi, f.eks. kvadratroden af 2.0 er omkring 1.41 hvilket 0.1 jo ikke går op i...

3. Og så nok det vigtigste. Floating point tal (Float og Double) er IKKE præcise!. Du kan ikke altid regne med at f.eks. 4.0-1.0 == 3.0 i floating point beregninger, hvorimod hos heltal vil det altid være tilfældet at 4-1 == 3.

Så hvad har du af muligheder? Brug den indbyggede Math.sqrt som Benni foreslår - eller find en algorithme der finder kvadratroden af et tal (f.eks. wikipedia).

Som sagt kan du ikke regne med floating point sammenligninger, så hvordan sammenligner man float og doubles? En typisk måde er at bruge en epsilon værdi. Hvis du vil tjekke om to floating point tal er lig med hinanden gjorde du før x == y. I stedet før du abs (x - y) < 0.1. Hvis forskellen mellem de 2 tal er mindre en 0.1 (epsilonværdien) betragtes de som ens. abs er en funktion der altid sørger for et tal er positivt, f.eks. abs(5) = 5, abs(-5) = 5. På den måde kan du sammenligne floating point værdier. Epsilon værdien er dog stadig problematisk for hvad skal den være? Hvis arbejder med tal som x = 53431434.34 og 123123213.34 er 0.1 nok en rimelig værdi, men hvis dine tal er x = 0.14, y = -0.05 er 0.1 en alt for stor epsilon værdi.

Indlæg senest redigeret d. 25.09.2012 00:35 af Bruger #14645

Bruger #11328 @ 25.09.12 10:21

1.323

Math.sqrt() funktionen returnere svaret - Den tildeler ikke variablen svaret. Derfor skulle din kode formentlig snarere være lig:

Java kode

 double tmp = Double.parseDouble(number);
 tmp = Math.sqrt(tmp);
 number = Double.toString(tmp);
 label.setText(number);

og kunne formentlig også skrives kortere:

Java kode

 double tmp = Double.parseDouble(number);
 label.setText(Double.toString(Math.sqrt(tmp)));

Bruger #16945 @ 25.09.12 08:01

Der forskellige ting der er problematiske i din udregning:

Java kode
 for(double i = 0.0; i < temp; i += 0.1)
 {
 if(i*i == temp)
 {
 result = i;
 break;
 }
 else
 {
 System.out.println("forloop has run " + i + " times");
 }
 }
1. For det første er det en meget ineffektiv måde at udregne det. Hvis jeg nu skrev 10000.0 vil den potentielt bruge 100000 iterationer. Men den ville selvfølgelig finde svaret meget hurtigere (efter 1000) iterationer...

2. Men du tjekker kun hvert 0.1 så det er ikke engang sikkert at den rammer den rigtigt værdi, f.eks. kvadratroden af 2.0 er omkring 1.41 hvilket 0.1 jo ikke går op i...

3. Og så nok det vigtigste. Floating point tal (Float og Double) er IKKE præcise!. Du kan ikke altid regne med at f.eks. 4.0-1.0 == 3.0 i floating point beregninger, hvorimod hos heltal vil det altid være tilfældet at 4-1 == 3.

Så hvad har du af muligheder? Brug den indbyggede Math.sqrt som Benni foreslår - eller find en algorithme der finder kvadratroden af et tal (f.eks. wikipedia).

Som sagt kan du ikke regne med floating point sammenligninger, så hvordan sammenligner man float og doubles? En typisk måde er at bruge en epsilon værdi. Hvis du vil tjekke om to floating point tal er lig med hinanden gjorde du før x == y. I stedet før du abs (x - y) < 0.1. Hvis forskellen mellem de 2 tal er mindre en 0.1 (epsilonværdien) betragtes de som ens. abs er en funktion der altid sørger for et tal er positivt, f.eks. abs(5) = 5, abs(-5) = 5. På den måde kan du sammenligne floating point værdier. Epsilon værdien er dog stadig problematisk for hvad skal den være? Hvis arbejder med tal som x = 53431434.34 og 123123213.34 er 0.1 nok en rimelig værdi, men hvis dine tal er x = 0.14, y = -0.05 er 0.1 en alt for stor epsilon værdi.

Okay, har prøvet med Math.sqrt(); istedet, men hvis jeg taster 9 ind, så bliver resultatet bare 9.0, hvis jeg taster 9.0 ind, bliver resultatet stadig 9.0, det samme hvis jeg gør det med tallet 20, osv, osv, osv.

Java kode

 double temp = Double.parseDouble(number);
                 Math.sqrt(temp);
                                 number = Double.toString(temp);    
                 label.setText(number);

Bruger #16945 @ 25.09.12 10:25

Mange tak, nu virker det!

Problem Med Lommeregner, Double Og Kvadratrod

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler