Problem bei einer Formel (Bin Java Neuling)

[Dinceprenmark]

Huhu zusammen,

ich würde gern eine Gui Anwendung in Java coden.
In dieser Anwendung wirde diese Formel verwendet:

Das ist mein verwendeter Code:

package ArenaKalkulator;
/*
 * ArenaKalkulator.java
 *
 * Created on 4. November 2007, 22:26
 */

/**
 *
 * @author  Ich 
 */
import java.math.*;
public class ArenaKalkulator extends javax.swing.JFrame {
    
    /** Creates new form ArenaKalkulator */
    public ArenaKalkulator() {
        initComponents();
    }
    
    /** This method is called from within the constructor to
     * initialize the form.
     * WARNING: Do NOT modify this code. The content of this method is
     * always regenerated by the Form Editor.
     */
    // <editor-fold defaultstate="collapsed" desc=" Erzeugter Quelltext ">                          
    private void initComponents() {
        jPanel1 = new javax.swing.JPanel();
        wertung = new javax.swing.JTextField();
        ergebnis2 = new javax.swing.JTextField();
        ergebnis3 = new javax.swing.JTextField();
        ergebnis5 = new javax.swing.JTextField();
        jLabel1 = new javax.swing.JLabel();
        jLabel2 = new javax.swing.JLabel();
        jLabel3 = new javax.swing.JLabel();
        jLabel4 = new javax.swing.JLabel();
        jSeparator1 = new javax.swing.JSeparator();
        berechnen = new javax.swing.JButton();
        loeschen = new javax.swing.JButton();
        verlassen = new javax.swing.JButton();

        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);
        setTitle("Witchlords Arena Kalkulator by Erekoese from Aman'Thul");
        setAlwaysOnTop(true);
        setCursor(new java.awt.Cursor(java.awt.Cursor.HAND_CURSOR));
        setResizable(false);
        jPanel1.setBorder(javax.swing.BorderFactory.createTitledBorder(null, "Erekoese @  www.witchlords.de", javax.swing.border.TitledBorder.RIGHT, javax.swing.border.TitledBorder.DEFAULT_POSITION));

        jLabel1.setText("Aktuelle Wertung");

        jLabel2.setText("Punkte 2 vs 2");

        jLabel3.setText("Punkte 3 vs 3");

        jLabel4.setText("Punkte 5 vs 5");

        org.jdesktop.layout.GroupLayout jPanel1Layout = new org.jdesktop.layout.GroupLayout(jPanel1);
        jPanel1.setLayout(jPanel1Layout);
        jPanel1Layout.setHorizontalGroup(
            jPanel1Layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
            .add(jPanel1Layout.createSequentialGroup()
                .addContainerGap()
                .add(jPanel1Layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
                    .add(jLabel1, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, 119, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)
                    .add(wertung, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, 104, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)
                    .add(jPanel1Layout.createSequentialGroup()
                        .add(jPanel1Layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING, false)
                            .add(ergebnis2)
                            .add(jLabel2, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE))
                        .add(35, 35, 35)
                        .add(jPanel1Layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
                            .add(jLabel3)
                            .add(ergebnis3, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, 85, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE))
                        .add(jPanel1Layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
                            .add(jPanel1Layout.createSequentialGroup()
                                .add(31, 31, 31)
                                .add(jLabel4))
                            .add(org.jdesktop.layout.GroupLayout.TRAILING, jPanel1Layout.createSequentialGroup()
                                .addPreferredGap(org.jdesktop.layout.LayoutStyle.RELATED)
                                .add(ergebnis5, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, 85, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)))))
                .addContainerGap(73, Short.MAX_VALUE))
            .add(jSeparator1, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, 414, Short.MAX_VALUE)
        );
        jPanel1Layout.setVerticalGroup(
            jPanel1Layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
            .add(jPanel1Layout.createSequentialGroup()
                .addContainerGap()
                .add(jLabel1)
                .addPreferredGap(org.jdesktop.layout.LayoutStyle.RELATED)
                .add(wertung, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(org.jdesktop.layout.LayoutStyle.RELATED)
                .add(jSeparator1, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, 10, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)
                .add(2, 2, 2)
                .add(jPanel1Layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
                    .add(jPanel1Layout.createSequentialGroup()
                        .add(jPanel1Layout.createParallelGroup(org.jdesktop.layout.GroupLayout.BASELINE)
                            .add(jLabel4)
                            .add(jLabel3))
                        .addPreferredGap(org.jdesktop.layout.LayoutStyle.RELATED)
                        .add(jPanel1Layout.createParallelGroup(org.jdesktop.layout.GroupLayout.BASELINE)
                            .add(ergebnis3, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)
                            .add(ergebnis5, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)))
                    .add(jPanel1Layout.createSequentialGroup()
                        .add(jLabel2)
                        .addPreferredGap(org.jdesktop.layout.LayoutStyle.RELATED)
                        .add(ergebnis2, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)))
                .addContainerGap(28, Short.MAX_VALUE))
        );

        berechnen.setText("Berechnen");
        berechnen.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                berechnenActionPerformed(evt);
            }
        });

        loeschen.setText("L\u00f6schen");
        loeschen.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                loeschenActionPerformed(evt);
            }
        });

        verlassen.setText("Verlassen");
        verlassen.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                verlassenActionPerformed(evt);
            }
        });

        org.jdesktop.layout.GroupLayout layout = new org.jdesktop.layout.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
            .add(layout.createSequentialGroup()
                .addContainerGap()
                .add(layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
                    .add(jPanel1, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)
                    .add(layout.createSequentialGroup()
                        .add(berechnen)
                        .add(78, 78, 78)
                        .add(loeschen)
                        .addPreferredGap(org.jdesktop.layout.LayoutStyle.RELATED, 91, Short.MAX_VALUE)
                        .add(verlassen)))
                .addContainerGap())
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(org.jdesktop.layout.GroupLayout.LEADING)
            .add(org.jdesktop.layout.GroupLayout.TRAILING, layout.createSequentialGroup()
                .addContainerGap()
                .add(jPanel1, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE, org.jdesktop.layout.GroupLayout.DEFAULT_SIZE, org.jdesktop.layout.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(org.jdesktop.layout.LayoutStyle.RELATED, 24, Short.MAX_VALUE)
                .add(layout.createParallelGroup(org.jdesktop.layout.GroupLayout.BASELINE)
                    .add(berechnen)
                    .add(verlassen)
                    .add(loeschen)))
        );
        pack();
    }// </editor-fold>                        
    
    private void berechnenActionPerformed(java.awt.event.ActionEvent evt) {                                          
// Berechnung der Eingabe mit Hilfe der Arena Formel von Blizzard
        // Als erstes ein paar fließkomma variablen deklarieren ^^
        
        
        double num1=0, punkte = 0, result2=0, result3=0,result5=0;
        // Parsen der Eingabe in ein Float
        num1 = Float.parseFloat(wertung.getText());
        
        
      // FORMEL ZUSAMMEN BAUEN !!!!!!
        if ( num1 <= 1500 ) {
        punkte = (0.22 * num1 + 14);    
         } 
        
        else {
         punkte = ((1511.26) / (1+ 1639.28 * (Math.pow 2.71828,-0.00412) * num1));   
       
        }
        // Umwandeln in eine ganze Zahl 
        punkte = (int) punkte; 
        
    // Zuweisen der Ergebnisse in die Felde
        result2 = punkte;
        result3 = punkte;
        result5 = punkte;
        
        // Nun ins Ergebnisfeld setzen.
        
        
        ergebnis2.setText(String.valueOf(result2));
        ergebnis3.setText(String.valueOf(result3));
        ergebnis5.setText(String.valueOf(result5));
        
        
        
    }                                         

    private void loeschenActionPerformed(java.awt.event.ActionEvent evt) {                                         
        
        /** Löschen der Eingabefelder
         * Die Datenfelde wertung, ergebnis 2 - 5 leeren.
         **/
        
        wertung.setText("");
        ergebnis2.setText("");
        ergebnis3.setText("");
        ergebnis5.setText("");
        
        
    }                                        

    private void verlassenActionPerformed(java.awt.event.ActionEvent evt) {                                          
// TODO Ihre Ereignisbehandlung hier einfügen:
        System.exit(0);
    }                                         
    
    /**
     * @param args the command line arguments
     */
    public static void main(String args[]) {
        java.awt.EventQueue.invokeLater(new Runnable() {
            public void run() {
                new ArenaKalkulator().setVisible(true);
            }
        });
    }
    
    // Variablendeklaration - nicht modifizieren                     
    private javax.swing.JButton berechnen;
    private javax.swing.JTextField ergebnis2;
    private javax.swing.JTextField ergebnis3;
    private javax.swing.JTextField ergebnis5;
    private javax.swing.JLabel jLabel1;
    private javax.swing.JLabel jLabel2;
    private javax.swing.JLabel jLabel3;
    private javax.swing.JLabel jLabel4;
    private javax.swing.JPanel jPanel1;
    private javax.swing.JSeparator jSeparator1;
    private javax.swing.JButton loeschen;
    private javax.swing.JButton verlassen;
    private javax.swing.JTextField wertung;
    // Ende der Variablendeklaration                   
    
}

Aber ich bekomme von Netbeans nur eine Fehlermeldung rot unterstrichen.
Die Formel die ich da gebaut habe läuft scheinbar nicht.

BTW: Ist Netbeans für einen Anfänger die richtige Wahl ? Oder doch eher Eclipse ?

Danke für eure Hilfe.

MfG

Dince

 

[SlaterB]

Tipps: verstecke eine 10-Zeilen-Formel nicht in 250 Zeilen blöder GUI,
zumindest nicht, wenn du das Forum fragst

wenn 'wertung' gemeint ist, dann nenne die Variable auch 'wertung', nicht 'num1'

nenne genaue Fehlermeldungen + Zeilennummern,
wenn Netbeans die nicht verrät, dann wäre Eclipse in der Tat besser

weniger Klammern sind manchmal mehr (Übersicht)

wozu die 3 results, die alle das gleiche enthalten?

public class Test
{

    public static void main(String[] args)
        throws Exception
    {
        berechne(1666);
    }

    private static void berechne(double wertung)
    {
        double punkte = 0;

        if (wertung <= 1500)
        {
            punkte = 0.22 * wertung + 14;
        }
        else
        {
            punkte = 1511.26 / (1 + 1639.28 * Math.pow(2.71828, -0.00412 * wertung));
        }
        System.out.println(punkte);

        // Umwandeln in eine ganze Zahl
        punkte = (int)punkte;
        System.out.println(punkte);

    }
}

 

[Dinceprenmark]

Okay habe den ganzen GUI Code leider mit kopiert.
Sry dafür ^^

Ich teste mal ob Deine Formel geht ^^

wertung kann ich das nicht nennen weil die Wertung der Wert ist der in der GUI eingegeben wird.

Das Ergebnis sind dann die Punkte ^^

Greets und Danke für Deine schnelle Antwort

 

[Dinceprenmark]

Juhuu ^^

Es geht alles wie gewünscht nur würde ich die Kommastellen noch gern weg haben.

Das ganze soll dann aufgerundet werden.

Hoffe das ich das selber hinbekomme.
(Code erst 2 Tage Java)

Danke nochmal ein überglücklicher Dince ^^