BlueJ

[heizz]

Hallo leute.
Ich weiß nicht ob das hier rein passt aber ich brauche Hilfe:
Ich habe noch nicht lange Informatik. Wir haben ein Struktogramm bekommen(das mittlere) und sollen das Programm jetzt (mit BlueJ).Das ist das was ich bis jetzt habe aber es klappt nicht wirklich und ich weiß auch nicht wo ich welche Fehler gemacht habe, da ich absoluter Anfänger bin.Was aus dem Quellcode brauche ich nicht,was habe ich vergesse?Rechnen kann das Applet aber durch Vererbung aus den vorherigen.

Quellcode:

import java.awt.*;

public class aqua extends Gewicht
{  Checkbox check6 =new Checkbox();
    
    public void init()
    { super.init();
     Aufschrift1.setText("Name des Kunden");
     Aufschrift2.setText("Wasserverbrauch");
     check6.setBounds(220,140,200,20);
     check6.setLabel("Wasserverbrauch");
     check6.setCheckboxGroup(cg);
     add(check6);}
        
    public void machschon()
    {super.machschon();
       if (check6.getState()==true)
        {ausrechnen();}
    }    
    
    public void ausrechnen()
    {int w =Integer.parseInt(Eingabe2.getText());
     String name=(Eingabe1.getText());
     String xyz = Eingabe2.getText();
     
       if (xyz.equals("w<=10"))
       {double rechnung1 =20*1.16;}
        else
       {double rechnung2 = (20+w*1.5)*1.6;}

     //float xyz =Math.random();
      //0<xyz<1
     int zuf=(int) (Math.random()*100)+1;
     
     if(xyz.equals("50"))
     Eingabe3.setText("Herzlichen Glückwunsch,Sie zahlen nichts");
     if(xyz.equals("1"))
     Eingabe3.setText("Name des Kunden","Pech gehabt!");("Pech gehabt!");
      (Double.toString(rechnung2));}
    
    }

 

[Ruzmanz]

1. "}" kommen immer in die nächste Zeile.
2. "xyz.equals("w<=10")" ist falsch. Du hast einen Integer w eingelesen, mit den musst du arbeiten.
3. "{double rechnung1 =20*1.16;}" und "{double rechnung2 = (20+w*1.5)*1.6;}" sind auch falsch, da der Gültigkeitsbereich sofort endet und du mit den Variablen nichts mehr anfangen kannst.
4. "(Math.random()*100)+1" ist falsch, da das Zahlen von 1 bis 101 ergibt.
5. Der komplette letzte Teil ergibt keinen Sinn:

     if(xyz.equals("50"))
     Eingabe3.setText("Herzlichen Glückwunsch,Sie zahlen nichts");
     if(xyz.equals("1"))
     Eingabe3.setText("Name des Kunden","Pech gehabt!");("Pech gehabt!");
      (Double.toString(rechnung2));}

 

[Ikaron]

4. "(Math.random()*100)+1" ist falsch, da das Zahlen von 1 bis 101 ergibt.

Eigentlich stimmt das so, Math.random() liefert eine Zahl zwischen 0 und 1 (exklusive), (int) (Math.random() * 100) ergibt als 0-99, +1 dann 1-100.

Allgemein kann man sagen: Wenn du rechnest, brauchst du fast niemals equals. Das ist nur für komplexe Objekte, wie z.B. Strings. So könnte das ganze aussehen:

import java.awt.*;

public class aqua extends Gewicht {

    Checkbox check6 =new Checkbox();
    
    public void init() {

        super.init();
        Aufschrift1.setText("Name des Kunden");
        Aufschrift2.setText("Wasserverbrauch");
        check6.setBounds(220,140,200,20);
        check6.setLabel("Wasserverbrauch");
        check6.setCheckboxGroup(cg);
        add(check6);
    }
        
    public void machschon() {

        super.machschon();
        if (check6.getState())
            ausrechnen();
    }    
    
    private double letztesErgebnis;

    public void ausrechnen() {

        int w = Integer.parseInt(Eingabe2.getText());
        String name = Eingabe1.getText();
        String xyz = Eingabe2.getText();
     
        if (w <= 10)
            letztesErgebnis = 20 * 1.16;
        else
            letztesErgebnis = (20 + w * 1.5) * 1.6;

        int zuf = (int) (Math.random() * 100) + 1;
        // Hier bin ich nicht mehr durchgestiegen. Was soll xyz darstellen? - Aufgabenstellung lesen hat geholfen :D
        if(zuf == 50)
            Eingabe3.setText(String.format("%s, %d, Herzlichen Glückwunsch, Sie zahlen nichts", kundenname, w));
        else
            Eingabe3.setText(String.format("%s, %d, Pech gehabt! %d", kundenname, w, rechnungsbetrag));
    }
}

 

[Ruzmanz]

Eigentlich stimmt das so, Math.random() liefert eine Zahl zwischen 0 und 1 (exklusive), (int) (Math.random() * 100) ergibt als 0-99, +1 dann 1-100.(Math.random()*100)+1" ist falsch, da das Zahlen von 1 bis 101 ergibt.

Mein Fehler, hast Recht. Hatte die 0-1 in Erinnerung. Kann man jetzt anscheinend auch nicht mehr durchstreichen.