frage zu programmstrukturierung

[Exceptionist]

ich hab ewigkeiten nix mit java gemacht und hab mir überlegt ich könnte, um wieder reinzukommen, ein kleines programm schreiben, was werte für einfache geometrische figuren errechnet.
ich erzeuge dann für jede form, die ich berechnen können will eine eigene klasse, in welcher dann auch wieder die formaln drinstehen sollen, mit den ich die berechnung durchführen will.

beim quadrat is es ja noch rel. einfach, da brauch ich ja nur eine seite vorgeben oder errechnen und dann kann ich den rest ausrechnen lassen, beim rechteck(und fast allen anderen formen/figuren) hingegen brauch ich ja schonmal zwei werte mindestens.
der mathematische aspekt der ganzen geschichte is kein problem, sondern eher die umsetzung des ganzen.

hatte mir überlegt input-felder zu nehmen und nur einen berechnen-button drunter zu setzen.
problem dabei ist, daß ich ja fehler abfangen muss/möchte.

wie gehe ich denn am besten bei so einer sache vor?
meine erste idee war zuerst mal alle inputfelder darauf zu überprüfen, ob ALLE leer sind und wenn dem so ist eine meldung auszugeben.
doch wie läuft des mit der restlichen aktion?
ich muss ja wissen welcher wert eingegeben wurde, um dann die richtige funktion zu benutzen, was ich jetzt habe ist irgendwie redundant (wie ich finde) und erzeugt auch typische "fehler" die nur (taschen)rechner machen zB: ich geb bei der fläche des quadrats 43 an und nach klicken auf den button steht da 42.999999999 (neun neuntel sind ja auch wieder ein ganzes, nur kapieren das die meisten rechner net^^)


hier mal der quelltext vom button:

 btn_quad_calc.addActionListener(new ActionListener(){
        public void actionPerformed(ActionEvent ae){
          Quadrat quad = new Quadrat();
          NumberFormat nf = new DecimalFormat("0.000");
         
          if(seite_a.getText().equals("") && flaeche_A.getText().equals("") && umfang_U.getText().equals("") && diagonale_d.getText().equals("")){
        	{JOptionPane.showMessageDialog(null, "Gib bitte in einem der Textfelder\neine Zahl im Format #.### an", "FEHLER", JOptionPane.OK_OPTION);}
          }else{
        
		     if(!seite_a.getText().equals("") && seite_a.getText().matches("^([\\d]*[.]?[\\d]*)$")){
		    	 	seitenlaenge = Double.parseDouble(seite_a.getText());		    	 
		    	 	flaeche_A.setText(""+ quad.a_holFlaeche(seitenlaenge));
		    	 	umfang_U.setText("" + quad.a_holUmfang(seitenlaenge));
		    	 	diagonale_d.setText("" +quad.d_holSeite(seitenlaenge));
		     }
		      
          
		     
		     if(!flaeche_A.getText().equals("") && flaeche_A.getText().matches("^([\\d]*[.]?[\\d]*)$")){
		    	 	flaeche = Double.parseDouble(flaeche_A.getText());	 
		    	 	seite_a.setText(""+ quad.A_holSeite(flaeche));
		    	 	umfang_U.setText("" + quad.a_holUmfang(quad.A_holSeite(flaeche)));
		    	 	diagonale_d.setText("" +quad.d_holSeite(quad.A_holSeite(flaeche)));
		     }
		     
		     if(!umfang_U.getText().equals("") && umfang_U.getText().matches("^([\\d]*[.]?[\\d]*)$")){
		    	 umfang = Double.parseDouble(umfang_U.getText());
		    	 flaeche_A.setText(""+ quad.a_holFlaeche(quad.U_holSeite(umfang)));
		    	 seite_a.setText(""+ quad.U_holSeite(umfang));
		    	 diagonale_d.setText("" +quad.d_holSeite(quad.U_holSeite(umfang)));
		    	 
		     }
		     
		     if(!diagonale_d.getText().equals("") && diagonale_d.getText().matches("^([\\d]*[.]?[\\d]*)$")){
		    	 diagonale = Double.parseDouble(diagonale_d.getText());	
		    	 
		     }
		     
		     
		     
		      ergseitenlaenge = quad.A_holSeite(flaeche);
		      ergflaeche = quad.a_holFlaeche(seitenlaenge);
		      ergumfang = quad.a_holUmfang(seitenlaenge);
		      ergdiagonale = quad.d_holSeite(seitenlaenge);
		     
		     String erg_seite = "Seitenlänge: " + ergseitenlaenge + 
			   "\nFläche: " + ergflaeche+
			   "\nUmfang: " + ergumfang +
			   "\nDiagonale: " + ergdiagonale;
		     System.out.println(erg_seite);
           }
        
        //  double flaeche = Double.parseDouble(flaeche_A.getText());
        //  double umfang = Double.parseDouble(umfang_U.getText());
        //  double diagonale = Double.parseDouble(diagonale_d.getText());
          
       
              
          }});

und hier ist was ich bisher fabriziert habe als JAR-File: geosoft_part.jar

das mit der diagonalen hab ich noch net mit einer funktion hinterlegt, weil mir dieser fehler bei der fläche aufgefallen ist....
ich würd gern von euch wissen, wie ich am besten vorgehen soll, damit ich das ganze benutzerfreundlich gestalten kann ohne redundant zu arbeiten.

letztlich weiß ich (und auch das programm) nicht welche werte gegeben sind und welche gesucht werden!
wenn mir jemand den richtigen weg weisen könnte, wäre ich sehr erfreut

 

[Exceptionist]

hat da niemand eine idee oder nen wink mit´m zaunpfahl?
ich will doch noch nichmal code sondern nur tips,wie ich da was für die usability und die programmstruktur tun kann....!

 

[Marco13]

Nicht so ungeduldig :roll:

Sowas wie

quad.a_holFlaeche
quad.d_holFlaeche
quad.A_holFlaeche
quad.a_holSeite
quad.d_holSeite
...
könnte man schon in Frage stellen.


Vielleicht(!!!) wäre es sinnvoll, die Klasse "Quadrat" so zu machen, dass sie einfach Methoden anbietet
getFlaeche
getSeitenlaenge
getDiagonale
die einfach immer den richtigen Wert zurückliefern, und Methoden
setFlaeche
setSeitenlaenge
setDiagonale
mit denen man die Werte setzen kann.

private float seitenlaenge = 0;

void setSeitenlaenge(float s)
{
    seitenlaenge = s;
}

float getFlaeche()
{
    return s * s;
}

Dann kann später auch die Implementierung geändert werden. D.h. man könnte es ändern zu

private float flaeche = 0;

void setSeitenlaenge(float s)
{
    flaeche = s*s;
}

float getFlaeche()
{
    return flaeche;
}

ohne dass der Aufrufer dort einen Unterschied sieht.


Zur Abfrage mit den TextFeldern und so... Ja, einiges davon würde sich schon erübrigen, wenn es so eine Klasse mit diesen Methoden gäbe: Die Abfrage, ob ALLE Felder leer sind, belibt. Ansonsten wird das erste nicht-Leere Feld gesucht, und der Wert an 'quad' übergeben. Das rechnet (intern) dann alle abhängigen Werte aus. Am Ende ruft man eine Methode auf wie

private void fillTextFieldsWithDataFrom(Quadrag quad)
{
    flaeche_A.setText(""+ quad.getFlaeche()); 
    ...

die einfach die TextFields mit den Werten aus dem übergebenen Quadrat füllt.

P.S: Das 42.9999999 sond Rundungsfehler. Fornsuche hilft.

 

[Exceptionist]

thx, werd dann mal checken ob ich das alles so umgebaut bekomme, wie du vorgeschlagen hast...

und ich denek mal den rundungsfehler werd ich wohl durch eigenständiges runden der zahl wegbekommen, denn zur zeit runde ich nirgendwo die ein- bzw ausgaben der werte...

 

[Marco13]

Das Runden übernimmt Java gezwungenermaßen für dich, weil man eine Zahl wie z.B. 0.1 eben (also float oder double) einfach nicht genau darstellen kann.

 

[Exceptionist]

nochmal ein fettes danke an dieser stelle!!!
hab des so realisieren können, wie du es vorgeschlagen hast.. und siehe da: keine rundungsfehler und alles fruchtet ohne weitere probleme!
ok ich sollte evtl ne class für nen actionlistener bauen und eine für nen validator, aber soweit haut´s hin *freu*
so langsam komm ich wieder in die gänge..
hab auch des look and feel auf windoof angepasst^^
wennste magst kannste ja nochmal gucken
geosoft_part.jar