Vererbung If-Bedingung im Konstruktor

[Java-Frage-123]

Hallo,

ich habe eine Superklasse Datum mit folgendem Konstruktor erstellt:

//(...)
public Datum(int kTag, int kMonat, int kJahr){
		
		if(this.tag <= 0 && this.tag > 31){
			System.out.println("Warnung: Ungültiger Tag" + kTag);
		}
		if(this.monat <= 0 && this.monat > 12){
			System.out.println("Warnung: Ungültiger Monat" + kMonat);
		}
		if(this.jahr <= 0){
			System.out.println("Warnung: Ungültiges Jahr" + kJahr);
		}
		this.tag = kTag;
		this.monat = kMonat;
		this.jahr = kJahr;
	} 
//(...)

Dann habe ich eine Unterklasse ISODatum erstellt, die von der Klasse Datum erbt:

public class ISODatum extends Datum{

	public ISODatum(int kTag, int kMonat, int kJahr){
		super(kTag, kMonat, kJahr);
	}
// (...)

Wenn ich jedoch ein Datum in der Testklasse ausgeben will, werden die Bedingungen im Konstruktor von Datum nicht berücksichtigt.
Wo habe ich meinen Denkfehler?

 

[Tomate_Salat]

Tipp:
mit folgender Zeile:

if(this.tag <= 0 && this.tag > 31)

überprüfst du nicht die übergebenen Parameter ;-)

Tipp2: gleiches gilt für monat und jahr ;-)

 

[Final_Striker]

Wo habe ich meinen Denkfehler?

1. Du überprüfst die falschen Variablen
2. Falsche Bedingungen in den if's

 

[Sephrox]

Nehmen wir an, der Code würde so aussehen:

public class ISODatum extends Datum{

int tag = 2;
    public ISODatum(int tag, int monat, int jahr){
        if(this.tag==2) {
          System.out.println("Equals True");
        }
    }

Dein Denkfehler ist hier , dass du wie im oberen Post schon gesagt mit "this.[Variable]" immer auf die Variable der Klasse
zugreifst. Das nutzt man Normalerweise im Konstruktor , wenn man selbe Variablennamen hat. Wie in meinem Beispiel nutze ich
die im Vorfeld für die Klasse deklarierte tag Variable mit this. , da du auf die Variable dieser Klasse zugreifst. Der Konstruktur soll bei dir aber die Übergebene Variable prüfen. Mit Gettern und Settern und automatisch generierten Konstruktur würde es so aussehen und demnach auch funktionieren.

public class ISODatum extends Datum{

int tag = 0;
    public ISODatum(int tag, int monat, int jahr){
          this.tag = tag;
        }
    }

Das heißt man überträgt der Klassenvariable tag , die übergebene Variable über den Konstruktor.
Die Klassenvariable tag wäre in dem Fall

new ISODatum(2,0,0);

2 !

Um nun also direkt beim Konstruktur zu prüfen, musst du es so machen:

public class ISODatum extends Datum{
    public ISODatum(int tag, int monat, int jahr){
          if(tag==2) {
             System.out.println("Wuhu es ist der 2. :) ");
         }

     }
    }

Hoffe es ist verständlich

EDIT: Außerdem nehmen bei dir die IF Abfragen immer falsche Werte an:
if(this.tag <= 0 && this.tag > 31){


Wie kann ein Wert tag <= 0 sein und > 31 ?
Also nehmen wir an -2 ist <= 0 und du fragst jetzt mit einer UND Bedingungen ober dieser wert gleichzeitig > 31 ist?

Du müsstest es wenn dann so abfragen: if(tag<=0 || tag > 31)
Wenn der tag jetzt 5 betragen würde , würde es so aussehen:
5<=0 = false
5>31 = false

Also hast du dein Ergebnis, es liegt in dem Bereich zwischen 0 und 31.

Hast du jetzt eine tag Variable mit 32:
32<=0 = false
32>31 = true

Ergebnis : ! Achtung ! tag ist nicht erlaubt !

 

[VfL_Freak]

Moin,

von den falschen Variablen mal ganz abgesehen, sind Deine Prüfbedingungen sehr seltsam !

Danach wären bspw. auch folgenden Daten gültig :

30. Februar 2012
15. März 928
31. September 9876

:noe:

Gruß
Klaus

 

[Java-Frage-123]

Hallo,

also nochmal danke für den Tipp mit den Übergangeparametern. Ich hatte das schon vorher da stehen, aber es hatte sich auch damals nichts an der Ausgabe geändert.

@Sephrox: Stimmt, ich habe versehentlich das Zeichen für "Und" anstatt das Zeichen für "Oder" verwendet.

@VfL_Freak: Ja, das weiß ich. Jedoch ist das nicht Teil der Aufgabe. Ich werde das vielleicht später noch bearbeiten.

Es werden trotzdem weiterhin Daten wie beispielsweise 32.14.2011 ausgegeben.

Gibt es denn noch die Möglichkeit, dass bei der Vererbung von der Klasse Datum zu der Unterklasse ISODatum ein Fehler auftritt?



Hier nochmal der korrigierte Konstruktor der Superklasse Datum:

public abstract class Datum {
	
	private int tag;
	private int monat;
	private int jahr;
	
	public Datum(int kTag, int kMonat, int kJahr){
		
		if(kTag <= 0 || kTag > 31){
			System.out.println("Warnung: Ungültiger Tag" + kTag);
		}
		if(kMonat <= 0 || kMonat > 12){
			System.out.println("Warnung: Ungültiger Monat" + kMonat);
		}
		if(kJahr <= 0){
			System.out.println("Warnung: Ungültiges Jahr" + kJahr);
		}
		this.tag = kTag;
		this.monat = kMonat;
		this.jahr = kJahr;
	}

 

[VfL_Freak]

Moin,

Es werden trotzdem weiterhin Daten wie beispielsweise 32.14.2011 ausgegeben.

ja, warum auch nicht ???:L

Du gibst zwar die Warnungen aus, setzt dann aber trotzdem die fehlerhaften Werte !!
Event. sollst Du abbrechen !!??!!??

Gruß
Klaus

 

[Java-Frage-123]

Ok, also den Fehler, den ich gemacht habe kann man vielleicht mit einer Brille oder einer Tasse Kaffee beheben:

Dass es sich um ein ungültiges Datum handelt wurde Ausgegeben, jedoch hätte ich bei Eclipse hochscrollen müssen um das zu sehen.... die Ausgabe der Fehlermeldung befindet sich also lediglich an der falschen Stelle, dass sollte sich aber beheben lassen.

Trotzdem vielen Dank für die Tipps!!!!

 

[Timothy Truckle]

ich habe eine Superklasse Datum mit folgendem Konstruktor erstellt:

Du solltes eine (Runtime-)Exception werfen, wenn die Bedingungen nicht erfüllt sind. Dass stellt sicher, dass es nur gültige Objekte gibt.
Andern Falls müsstest bei jeder Verwendung deiner Objekte erneut prüfen, ob das Datum gültig ist.

bye
TT

 

[Sephrox]

ich glaube der Kollege meinte damit, dass wenn die Werte falsch sind , diese nicht gespeichert werdens sollten
ungefähr so:

public abstract class Datum {
    
    private int tag;
    private int monat;
    private int jahr;
    
    public Datum(int kTag, int kMonat, int kJahr){
        boolean falschesdatum=false;
        
        if(kTag <= 0 || kTag > 31){
            System.out.println("Warnung: Ungültiger Tag" + kTag);
            falschesdatum=true;
        }
        if(kMonat <= 0 || kMonat > 12){
            System.out.println("Warnung: Ungültiger Monat" + kMonat);
            falschesdatum=true;
        }
        if(kJahr <= 0){
            System.out.println("Warnung: Ungültiges Jahr" + kJahr);
            falschesdatum=true;      
       }
        if(falschesdatum==false) {
        this.tag = kTag;
        this.monat = kMonat;
        this.jahr = kJahr; 
          }
    }

EDIT: Oder den Kommentar über meinem Post beachten

 

[VfL_Freak]

Moin,

ich glaube der Kollege meinte damit, dass wenn die Werte falsch sind , diese nicht gespeichert werdens sollten

genau

ungefähr so:

public abstract class Datum {
    
    private int tag;
    private int monat;
    private int jahr;
    
    public Datum(int kTag, int kMonat, int kJahr){
        boolean falschesdatum=false;
        
        if(kTag <= 0 || kTag > 31){
            System.out.println("Warnung: Ungültiger Tag" + kTag);
            falschesdatum=true;
        }
        if(kMonat <= 0 || kMonat > 12){
            System.out.println("Warnung: Ungültiger Monat" + kMonat);
            falschesdatum=true;
        }
        if(kJahr <= 0){
            System.out.println("Warnung: Ungültiges Jahr" + kJahr);
            falschesdatum=true;      
       }
        if(falschesdatum==false) {
        this.tag = kTag;
        this.monat = kMonat;
        this.jahr = kJahr; 
          }
    }

am Ende besser so :

if( !falschesdatum )  // ist überrsichtlicher !!
{
        this.tag = kTag;
        this.monat = kMonat;
        this.jahr = kJahr; 
}

Gruß
Klaus

 

[Timothy Truckle]

am Ende besser so :

if( !falschesdatum )  // ist überrsichtlicher !!
{
        this.tag = kTag;
        this.monat = ;
        this.jahr = kJahr; 
}

Nein,
Aber so:

if( falschesdatum )  // ist überrsichtlicher !!
{
       throw new RuntimeException("invalid Date: " +kTag+"."+kMonat+"."+kJahr);
}

bye
TT

 

[VfL_Freak]

Moin,

na gut, ok - so denn eine Exception gewünscht ist

Dann könnte man ja auch noch ein

System.exit(-1);

anhängen :bae:

Gruß
Klaus

 

[Timothy Truckle]

System.exit(-1);

anhängen :bae:

Das ist aber (wenn überhaupt) nur im

catch

Block in

main

erlaubt. Und mit einer Exception kommt man da am schnellsten hin...ueh:

bye
TT

 

[VfL_Freak]

Moin,

Das ist aber (wenn überhaupt) nur im

catch

Block in

main

erlaubt. Und mit einer Exception kommt man da am schnellsten hin...ueh:

ok, ok, da ist meine Ironie wohl unterwegs verloren gegangen ;(

Gruß
Klaus

 

[Timothy Truckle]

ok, ok, da ist meine Ironie wohl unterwegs verloren gegangen ;(

Mist, hab das Vorzeichen übersehen...

bye
TT