Java-Taschenrechner Funktioniert nicht

[Roos]

Hallo Leute,

erstmal zu mir...Ich habe mich heute bei euch im Forum angemeldet.
Ich bin momentan SPS-Programmierer und interessiere mich für Hochsprachenebenen aller Art.
Ich wollte jetzt mal aus Spass mir einen Taschenrechner Programmieren.
Er soll eigentlich den Kreisflächeninhalt, den Kreisdurchmesser, den Kreisumfang und das Volumen eines Ellipsoiden berechnen können.
Leider habe ich irgendwo einen Fehler drin. Finde ihn aber nicht.

Ich Poste euch den Quellcode als Anhang und hoffe, dass ihr mir da helfen könnt.

Gruss und schönes Restwochenende Johannes Roos

 

[Marco13]

"Irgendwo einen kleinen Fehler" ist gut :noe:

    public static double Volumen(double a, double b, double c, double Ellipsoid){
      double Volumen = 1;
      double erg;
      do{
       4/3*a*b*c*Math.PI ;
      }
      return Volumen;
    }

(WTF?!)

-->

    public static double Volumen(double a, double b, double c, double Ellipsoid){
      return 4.0/3.0*a*b*c*Math.PI ;
    }

 

[Roos]

ok...sorry ist mein erstes java projekt....
erstmal vielen Dank.

Also wenn ich jetzt meinen quellcode durch deinen ersteze sollte es schon gehen?
Oder haste noch nen anderen Fehler gefunden?

Nochmals vielen Dank Johannes

 

[madboy]

Ich vermute mal, dein Taschenrechner kompiliert nicht (zumindest bei mir tut er das nicht. 47 Errors... ) ;-)

Übrigens würde ich vorschlagen, dass du mal ohne GUI anfängst und dir eine Methode nach der anderen vornimmst und sie testest:

 public static double Durchmesser(double r){
     double Durchmesser = 1;
     double erg;
     do{
      2*sqrt((Math.PI*r^2)/Math.PI)  ;
     }
     return Durchmesser;
    }

public static void main(String[] args) {
Durchmesser(2.0);
}

Übrigens:
- Klassen werden groß geschrieben ("Taschenrechner")
- Methoden werden klein geschrieben
- Keine Umlaute in Klassen-/Methodennamen

 

[Marco13]

Ja ne, kein Grund für ein "sorry" - es wundert mich nur, wie man ein fertiges (compilierbares? Hab's nicht getestet) Programm zusammenstellen kann, wenn da dann solche ... "Fehler" drin sind ... also - wie kommt man (ganz allgemein) auf die Idee, einfach mal ein "do {...}" irgendwo drumrumzuschreiben... das muss doch einen Grund haben!? ???:L

Wie auch immer: Die ganzen anderen Methoden waren eben "nach dem gleichen Muster" falsch. Also überall in die Methoden müßte im wesentlichen nur
return das was vorher in dem do{}-Block stand;

Wenn dann noch was nicht stimmt, sag' bescheid. Du kannst den Code (ist ja noch nicht so viel) auch hier posten, in
[ JAVA]
// Code
[ /JAVA]
Tags eingeschlossen (ohne Leerzeichen zwischen dem [ und dem J).

 

[Roos]

alles klar nochmals danke.

hier dann mal der neue code...
Kompelierbar war es nicht :-(
Das mit dem Programm aufsetzten ist für mich weniger das Problem...da ich c c++ pascal und strukturierten text programmiere (SPS-Programmierer) Java ist da mehr seit dem ich robocode gesehen habe mein Hobby!

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;



public class taschenrechner extends JFrame {
  // Anfang Variablen
  private JTextField jTextField1 = new JTextField();
  private JTextField jTextField2 = new JTextField();
  private JTextField jTextField3 = new JTextField();
  private JButton jButton1 = new JButton();
  private JButton jButton2 = new JButton();
  private JButton jButton3 = new JButton();
  private JButton jButton4 = new JButton();
  private JTextField jTextField4 = new JTextField();
  private JTextField jTextField5 = new JTextField();
  // Ende Variablen

  public taschenrechner(String title) {
    // Frame-Initialisierung
    super(title);
    addWindowListener(new WindowAdapter() {
      public void windowClosing(WindowEvent evt) { System.exit(0); }
    });
    int frameWidth = 445;
    int frameHeight = 493;
    setSize(frameWidth, frameHeight);
    Container cp = getContentPane();
    cp.setLayout(null);

    // Anfang Komponenten

    jTextField1.setBounds(136, 8, 73, 21);
    jTextField1.setText("Eingabe B");
    cp.add(jTextField1);
    jTextField2.setBounds(32, 8, 73, 21);
    jTextField2.setText("Eingabe A");
    cp.add(jTextField2);
    jTextField3.setBounds(240, 8, 73, 21);
    jTextField3.setText("Eingabe C");
    cp.add(jTextField3);
    jButton1.setBounds(104, 48, 209, 33);
    
    jButton1.setText("Umfang");
    cp.add(jButton1);
    jButton1.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jButton1ActionPerformed(evt);
      }
    });

    jButton2.setBounds(104, 104, 209, 33);
    jButton2.setText("Durchmesser");
    cp.add(jButton2);
    jButton2.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jButton2ActionPerformed(evt);
      }
    });

    jButton3.setBounds(104, 160, 209, 33);
    jButton3.setText("Flächeninhalt");
    cp.add(jButton3);
    jButton3.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jButton3ActionPerformed(evt);
      }
    });

    jButton4.setBounds(104, 216, 209, 33);
    jButton4.setText("Ellipsoid Volumen");
    cp.add(jButton4);
    jButton4.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jButton4ActionPerformed(evt);
      }
    });

    jTextField4.setBounds(56, 272, 305, 21);
    jTextField4.setEditable(false);
    jTextField4.setText("");
    cp.add(jTextField4);
    jTextField5.setBounds(344, 8, 73, 21);
    jTextField5.setText("Radius");
    cp.add(jTextField5);
    // Ende Komponenten

    setVisible(true);
  }

  // Anfang Ereignisprozeduren
  public void jButton1ActionPerformed(ActionEvent evt) {
  
  String Ausgabe ="";
  double Eingabea = Integer.parseDouble(Eingabea.getText());
  double Eingabeb = Integer.parseDouble(Eingabeb.getText());
  double Eingabec = Integer.parseDouble(Eingabec.getText());
  double Radiusr  =  Integer.parseDouble(Radiusr.getText());
  







  Object obj = e.getSource();
  


    if (obj == Eingabea ) {
      Ausgabe = ""+EllipsoidVolumen (Eingabe A, Eingabe B, Eingabe C);
      
      
    }
    if (obj == Eingabeb ) {
      Ausgabe = ""+Umfang (Radius);
    }
    if (obj == Eingabec ) {
      Ausgabe = ""+Flächeninhalt (Radius) ;
      
    }
    if (obj == Radius ) {
       Ausgabe = ""+Durchmesser (Radius) ;
    }
    Ausgabe.setText(AusgabeText) ;

    public static double Volumen(double a, double b, double c, double Ellipsoid){
     return 4.0/3.0*a*b*c*Math.PI ;
    }
    public static double Umfang(double r){
     return 2*Math.PI*r  ;
    }
    public static double Flächeninhalt(double r){
     return Math.PI*r^2  ;
    }
     
    public static double Durchmesser(double r){
     return 2*sqrt((Math.PI*r^2)/Math.PI)  ;
    }




       ;
  
  




  }
  public void jButton2ActionPerformed(ActionEvent evt) {

  }
  public void jButton3ActionPerformed(ActionEvent evt) {

  }
  public void jButton4ActionPerformed(ActionEvent evt) {

  }
  // Ende Ereignisprozeduren

  public static void main(String[] args) {
    new taschenrechner("taschenrechner");












  }
  
  
}

 

[madboy]

Du programmierst c, c++ und Pascal? Sorry, aber das kann ich nicht so recht glauben.
Die Syntax von c und c++ sind der von Java recht ähnlich. Unter anderem solltest du Variablennamen immer gleich schreiben, Groß- und Kleinschreibung beachten und keine Leerzeichen verwenden ;-)
Pascal kenne ich nicht.

Nochmal der Vorschlag: fange klein an. Schreibe erst mal eine Methode und teste sie. Dann die nächste usw.
Wenn du so vorgehst, wirst du recht schnell bemerken, wenn eine Klammer falsch gesetzt ist oder Variablennamen falsch geschrieben sind.

 

[frapo]

Mal eine Zeile aus deinem Code:

double Eingabea = Integer.parseDouble(Eingabea.getText());

double ist ein primitiver Datentyp, kein Objekttyp. Primitive Datentypen können von daher über keine Methode à la getText() verfügen.

 

[Roos]

also c und c++ programmiere ich in dem sinne nicht selbst da bin ich nur anwender...
Strukturierter Text ist eine Ableitung von Pascal und wird in der Automatisierungstechnik bei CoDeSys Programmiersystemen angewendet.
Daher habe ich ja auch die "Probleme" bei Java.
Denn die Hochsprachenebene für Strukturierten Text ist keine objektorientierte Programmiersprache.

Gruss Johannes

 

[Roos]

Mal eine Zeile aus deinem Code:

double Eingabea = Integer.parseDouble(Eingabea.getText());

double ist ein primitiver Datentyp, kein Objekttyp. Primitive Datentypen können von daher über keine Methode à la getText() verfügen.

Danke für den Tipp...aber was sollte ich da dann anders machen....wie gesagt ich habe mit java noch nichts gemacht und habe das als versuchs objekt gemacht!

 

[frapo]

also c und c++ programmiere ich in dem sinne nicht selbst da bin ich nur anwender...
Strukturierter Text ist eine Ableitung von Pascal und wird in der Automatisierungstechnik bei CoDeSys Programmiersystemen angewendet.
Daher habe ich ja auch die "Probleme" bei Java.
Denn die Hochsprachenebene für Strukturierten Text ist keine objektorientierte Programmiersprache.

Gruss Johannes

Ich kann dir auch nur den gut gemeinten Tipp geben dich einfach nur mal um die Basics zu kümmern. Vergiss erstmal alles was mit GUI zu tun hat, schau dir die Basics an.

Hier im Forum gibt es eine gute FAQ für ganz viele Einstiegsfragen.

Darüber hinaus gibt es im Netz auch gute Bücher die dir als Referenz dienen werden:
Galileo Computing :: Java ist auch eine Insel (8. Auflage)
www.javabuch.de - Das Handbuch der Java-Programmierung

Java lernt man nicht in drei Wochen.. nimm dir die Zeit die dazu nötig ist.

Gruß
frapo

 

[Roos]

Danke für die Links. Ich werde sie mir zu Herzen nehmen

Danke

 

[frapo]

Danke für die Links. Ich werde sie mir zu Herzen nehmen

Danke

Nochmal auf dieses eine Beispielzeile bezogen:

Du möchtest einen double-Wert einlesen. Für die Konsole kannst du dir z.b. mal die Klasse Scanner ansehen. Die befindet sich im package java.util und hat u.a. eine Methode namens nextDouble().

In dem Zusammenhang noch ein link auf die ganz elementare API-Doku von Java.
Java Platform SE 6

 

[Marco13]

Ich weiß ich nicht genau, warum....:bahnhof:

// Von http://www.java-forum.org/java-basics-anfaenger-themen/84011-java-taschenrechner-funktioniert-nicht.html
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;



public class taschenrechner extends JFrame {
  // Anfang Variablen
  private JTextField jTextField1 = new JTextField();
  private JTextField jTextField2 = new JTextField();
  private JTextField jTextField3 = new JTextField();
  private JButton jButton1 = new JButton();
  private JButton jButton2 = new JButton();
  private JButton jButton3 = new JButton();
  private JButton jButton4 = new JButton();
  private JTextField jTextField4 = new JTextField();
  private JTextField jTextField5 = new JTextField();
  // Ende Variablen

  public taschenrechner(String title) {
    // Frame-Initialisierung
    super(title);
    addWindowListener(new WindowAdapter() {
      public void windowClosing(WindowEvent evt) { System.exit(0); }
    });
    int frameWidth = 445;
    int frameHeight = 493;
    setSize(frameWidth, frameHeight);
    Container cp = getContentPane();
    cp.setLayout(null);

    // Anfang Komponenten

    jTextField1.setBounds(136, 8, 73, 21);
    jTextField1.setText("Eingabe B");
    cp.add(jTextField1);
    jTextField2.setBounds(32, 8, 73, 21);
    jTextField2.setText("Eingabe A");
    cp.add(jTextField2);
    jTextField3.setBounds(240, 8, 73, 21);
    jTextField3.setText("Eingabe C");
    cp.add(jTextField3);
    jButton1.setBounds(104, 48, 209, 33);

    jButton1.setText("Umfang");
    cp.add(jButton1);
    jButton1.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jButton1ActionPerformed(evt);
      }
    });

    jButton2.setBounds(104, 104, 209, 33);
    jButton2.setText("Durchmesser");
    cp.add(jButton2);
    jButton2.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jButton2ActionPerformed(evt);
      }
    });

    jButton3.setBounds(104, 160, 209, 33);
    jButton3.setText("Flächeninhalt");
    cp.add(jButton3);
    jButton3.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jButton3ActionPerformed(evt);
      }
    });

    jButton4.setBounds(104, 216, 209, 33);
    jButton4.setText("Ellipsoid Volumen");
    cp.add(jButton4);
    jButton4.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent evt) {
        jButton4ActionPerformed(evt);
      }
    });

    jTextField4.setBounds(56, 272, 305, 21);
    jTextField4.setEditable(false);
    jTextField4.setText("");
    cp.add(jTextField4);
    jTextField5.setBounds(344, 8, 73, 21);
    jTextField5.setText("Radius");
    cp.add(jTextField5);
    // Ende Komponenten

    setVisible(true);
  }

  // Anfang Ereignisprozeduren
  public void jButton1ActionPerformed(ActionEvent evt) {

  String Ausgabe ="";
  double Eingabea = Double.parseDouble(jTextField2.getText());
  double Eingabeb = Double.parseDouble(jTextField1.getText());
  double Eingabec = Double.parseDouble(jTextField3.getText());
  double Radiusr  =  Double.parseDouble(jTextField5.getText());








  Object obj = evt.getSource();



    if (obj == jButton4 )
    {
      Ausgabe = ""+Volumen (Eingabea, Eingabea, Eingabec);


    }
    if (obj == jButton1 ) {
      Ausgabe = ""+Umfang (Radiusr);
    }
    if (obj == jButton3 ) {
      Ausgabe = ""+Flächeninhalt (Radiusr) ;

    }
    if (obj == jButton2 ) {
       Ausgabe = ""+Durchmesser (Radiusr) ;
    }
    jTextField4.setText(Ausgabe) ;
}

    public static double Volumen(double a, double b, double c){
     return 4.0/3.0*a*b*c*Math.PI ;
    }
    public static double Umfang(double r){
     return 2*Math.PI*r  ;
    }
    public static double Flächeninhalt(double r){
     return Math.PI*r*r;
    }

    public static double Durchmesser(double r){
     return 2*Math.sqrt((Math.PI*r*r)/Math.PI)  ;
    }

 public void jButton2ActionPerformed(ActionEvent evt) {

  }
  public void jButton3ActionPerformed(ActionEvent evt) {

  }
  public void jButton4ActionPerformed(ActionEvent evt) {

  }
  // Ende Ereignisprozeduren

  public static void main(String[] args) {
    new taschenrechner("taschenrechner");












  }


}

 

[Roos]

@Marco13

Was ist mit deinem Quellcode? Du hast ihn geändert...
Und was weist du nicht genau?

Gruss Roos

 

[Marco13]

Warum ich das ganze compilier- und startbar gemacht habe.

 

[Roos]

Supper sehr nett vielen dank!gruss roos