OOP Probleme bei den Veto-Eigenschaften von Listenern

[ZoOo4]

Moin Moin ,
Ich habe vor kurzem angefangen Java zu programmieren und habe mir dazu das Buch "Java ist auch eine Insel" gekauft. Ich bin jetzt bei dem Kapitel Angewandete Objektorientierung angekommen und bin bei der Aufgabe mit MeckerMax und Trinkfreund stehen geblieben.

Problem:
Die Aufgabe besagt, dass ich eine Trinkfreund-Klasse erstellen soll und eine MeckerMax-Klasse. Trinkfreund trinkt Bier und kriegt pro Bier 0.2 Promille (Leberabbau nicht berücksichtigt ) hinzu. Der MeckerMax soll bei 1.0 Promille meckern und dem Trinkfreund "verbieten" noch ein Bier zu trinken. Bei mir darf der Trinkfreund sich aber ins Nirvana saufen, ohne dass der MeckerMax einschreitet. Mit dem Debugger habe ich herausgefunden, dass der Trinkerfreund wohl "heimlich" saufen muss, da der VetoableChange gar nicht bis zu dem MeckerMax kommt. Meine Frage ist nun warum?

Das ist meine Trinkfreund-Klasse

import java.beans.*;
public class Trinkfreund
{
    private double zAlkoholSpiegel;
    private VetoableChangeSupport vetos = new VetoableChangeSupport(this);
    
    public Trinkfreund() 
    {

    }
    
    public void nochEinBier() throws PropertyVetoException
    {
        double oldValue = this.zAlkoholSpiegel;
        vetos.fireVetoableChange("Promille", oldValue, zAlkoholSpiegel);
        zAlkoholSpiegel = zAlkoholSpiegel + 0.2;
    }
    
    public double promille()
    {
        return zAlkoholSpiegel;
    }
    
    public void addVetoableChangeListener(VetoableChangeListener l)
    {
        vetos.addVetoableChangeListener(l);
    }
}

und das ist die MeckerMax-Klasse

import java.beans.*;
public class Meckermax implements VetoableChangeListener
{    
    public Meckermax()
    {
        
    }
    
    @Override public void vetoableChange(PropertyChangeEvent e) throws PropertyVetoException
    {
        if("Promille".equals(e.getPropertyName()));
        {
            if(1.0 == (Double) e.getNewValue())
                throw new PropertyVetoException("Jetzt ist Schluß", e);
        }
    }
}

Btw. ich habe das Internet schon nach der Lösung durchsucht aber bin nicht fündig geworden. Außerdem würde ich gerne erklärt bekommen warum es nicht so geht, wie ich es haben will, da ich schon Mehreres ausprobiert habe.

 

[eRaaaa]

Tausche mal Zeile 15 mit Zeile 16 in TrinkFreund !

 

[mvitz]

Und im MeckerMax Zeile 13 würde ich anstelle von "==" lieber "<=" benutzen. Durch die Ungenauigkeiten von Double könnte es dir nämlich passieren, dass Promille nie genau 1.0 ist.

 

[ZoOo4]

Es hat geklappt !
Wobei ich mir jetzt nicht sicher bin, warum er das jetzt erst macht, da ich doch die gleiche Fehlermeldung kriegen müsste, wenn ich zum 6. mal die Methode "nochEinBier()" aufrufe, oder?

 

[SlaterB]

hier lernst du was, was du für alle Zeiten in jeder Art Programm sinnvoll einsetzen kannst:
überprüfen, was das Programm tut! = Debuggen oder Loggen,
Debuggen mit einem Programm ist etwas schwieriger, Loggen kann jeder

wenn du gar nicht weißt was wo schiefgeht, füge überall Ausgaben ein a la
System.out.println("rufe gleich veto auf");

System.out.println("Ende der Veto-Methode, kein Anlass für Exception, Werte waren: "..);


ich kann schon erahnen was der Fehler ist, auf diese Stelle hättest du ohne Denken nur durch fleißiges Loggen auch kommen können,
nämlich Zeile 13 in Meckermax

dort extrem geloggt:

System.out.println("teste value: "+e.getNewValue());
if(1.0 == (Double) e.getNewValue()) {
  System.out.println("genug ist genug");
  throw new PropertyVetoException("Jetzt ist Schluß", e);
} else {
  System.out.println("einer geht noch");
}

so, dieses if funktioniert also nicht, durch das Logging erkennst du vielleicht auch wieso,
gut möglich dass da als Value 0.99999999 oder so ankommt,
Kommazahlen werden in Java ungenau berechnet, Binärdarstellung im Computer usw.,
berechne die Differenz, davon Math.abs() und das kleiner 0.0001 oder so

oder rechne besser nur in int, addiere immer 2 und prüfe ob die Zahl 10 erreicht ist

generell solltest du bei >= die Exception werfen, vielleicht ist der Sprung ja von 0.9 bzw. 9 auf 1.1 bzw 11, dann wäre der Test auch ausgehebelt

edit:
statt kompliziert Math.abs auszurechnen reicht wirklich das >= oder <= wie mvitz auch vorschlägt,
allerdings besser nicht >= 1.0, denn dann hast du mit 0.999999 insofern Pech, als das noch ein Bier dazukommt, je nachdem wie gerundet wird,
relativ unsauber,
>= 0.95 oder >= 0.99995 wäre dagegen ein guter Test, das dürfte immer wie gewünscht hinkommen

 

[ZoOo4]

Danke für die vielen Tipps.
@Slater B: Ich habe sehr wohl viel mit System.out.println gearbeitet. Das tue ich auch generell viel. Nur wollte ich nicht unbedingt alles hier rein posten. Hinzu kommt dass der "=="-Vergleich an dieser Stelle klappte und das Programm nur durch den "Zeilendreher" hingen blieb. Womit meine Frage nach dem "Warum" noch nicht ganz beantwortet wäre (jaaa ich weiß ich bin pinkelig )

 

[eRaaaa]

Nein, die Werte wären ja dann immer gleich

    public void nochEinBier() throws PropertyVetoException {
	double oldValue = this.zAlkoholSpiegel;
	System.out.println(oldValue +" "+zAlkoholSpiegel);
	vetos.fireVetoableChange("Promille", oldValue, zAlkoholSpiegel);
	zAlkoholSpiegel = zAlkoholSpiegel + 0.2;
    }

Daher springt dann auch vetoableChange nicht an...
Daher der Zeilentausch:

    public void nochEinBier() throws PropertyVetoException {
	double oldValue = this.zAlkoholSpiegel;
	zAlkoholSpiegel = zAlkoholSpiegel + 0.2;
	System.out.println(oldValue +" "+zAlkoholSpiegel);
	vetos.fireVetoableChange("Promille", oldValue, zAlkoholSpiegel);
    }

 

[ZoOo4]

Okay jetzt habe ich es auch verstanden .
Danke für die genaue Erklärung und closed. :toll:

 

[SlaterB]

ui, dann lag ich ja falsch, aber dann hättest du eben durch Logging herausgefunden, dass du nie in Meckermax ankommt

aber auch das schriebst du ja schon wie ich jetzt sehe
> Mit dem Debugger habe ich herausgefunden, dass der Trinkerfreund wohl "heimlich" saufen muss, da der VetoableChange gar nicht bis zu dem MeckerMax kommt.

sehr gute Einstellung

vetos.fireVetoableChange("Promille", oldValue, zAlkoholSpiegel);
will zwei Values, anscheinend den alten und den neuen und macht nur was wenn die nicht gleich sind,
aber hat eRaaa auch schon erklärt,

nix mehr beizutragen von mir