Boolean istGleich() Method

[Usarian]

Hallo, ich soll eine Klasse Uhrzeit erzeugen und irgendwelche Methode dazu addieren. Eine der Methoden soll

boolean istGleich(Uhrzeit uz)

sein und die soll 2 Uhrzeiten vergleichen und bei Gleichheit "true" ausgeben, ansonsten "false". Das ist mein Quellcode : koennte mir jemand bitte helfen (wenn ich das ausfuehre bekomme ich immer "true")

public static boolean istGleich(Uhrzeitt z1, Uhrzeitt z2){
		
		if ((z1.stunde==z2.stunde)&&(z1.minute==z2.minute)&&(z1.sekunde==z2.sekunde)){
			System.out.println("Die Zeit ist gleich");
		}
		else {
			System.out.println("Die Zeit ist verschieden");
		}
		
		return (z1.stunde==z2.stunde)&&(z1.minute==z2.minute)&&(z1.sekunde==z2.sekunde);
		
	}
	
	
	public static void main(String[] args)
	{
		Uhrzeitt zeit1= new Uhrzeitt(21,3423,34);
		Uhrzeitt zeit2=new Uhrzeitt (2,32,56);
		Uhrzeitt.istGleich(zeit2, zeit1);
		
	}
	
			
	}

 

[eRaaaa]

mal eine Frage vorweg. Wenn da steht du sollst eine methode

boolean istGleich(Uhrzeit uz)

implementieren, wie kommst du dann darauf die Methodendeklaration
zu

public static boolean istGleich(Uhrzeitt z1, Uhrzeitt z2)

zu ändern?

Lasse das static weg, und übergebe nur eine Uhrzeit, so wie gefordert. In deiner main rufst du die Methode dann nicht als Klassenmethoden auf, sondern von deiner Instanz zeit1 beispielsweise:

Uhrzeitt zeit1= new Uhrzeitt(21,3423,34);
Uhrzeitt zeit2=new Uhrzeitt (2,32,56);
zeit1.istGleich(zeit2);

Und dann sehen wir mal weiter *g*

ps.: Biste dir sicher dass die Klasse Uhrzeitt heißen soll? (Uhrzeit uz)

 

[Landei]

Sicher war Urzeit gemeint...

 

[Usarian]

Und wie soll ich dann meine Methode aendern? Weil wenn ich (Uhrzeit uz) schreibe dann bleiben z1 und z2 undefiniert...


Tja... es sollte eigentlich Uhrzeit sein aber da ist wohl was schief gegangen. Ich wollte alles neu anfangen, aber da gab mir Eclipse ein fehler, dass die klasse Uhrzeit schon existiert und ich war zu faul diese zu loeschen, deshalb hab einfach Uhrzeitt geschrieben

 

[eRaaaa]

Irgendwie so???

public  boolean istGleich(Uhrzeitt uz){
      
        return (this.stunde==uz.stunde)&&(this.minute==uz.minute)&&(this.sekunde==uz.sekunde);
        
    }

(vllt noch null-check?? )

 

[Usarian]

Aber daraus bekomm ich irgendwie keine Ausgabe???:L

 

[eRaaaa]

??
System.out.println(zeit1.istGleich(zeit2));

 

[Usarian]

Aber daraus bekomm ich immer wieder "true" auch wenn die 2 Uhrzeiten nicht gleich sind :S


// Sry fuer die noob Fragen, aber da bin ich neu in Java ^^

 

[eRaaaa]

Zeig uns doch mal deine ganze Uhrzeit-Klasse !!

 

[Usarian]

Da ist die

public class Uhrzeitt {
	
	int stunde; 
	int minute;
	int sekunde;
	
	
	
	Uhrzeitt(){
		this.stunde=00;
		this.minute=00;
		this.sekunde=00;
		
	}
	
	Uhrzeitt(int stunde, int minute, int sekunde){
		
		
		while(sekunde>59)
		{
			sekunde=sekunde-60;
			minute=minute+1;
		}
		
		while(minute>59)
		{
			minute=minute-60;
			stunde=stunde+1;
		}
		
		while(stunde>23)
		{
			stunde=stunde-24;
		}
		
		while(sekunde<0)
		{
			sekunde=60+sekunde;
			minute=minute-1;
		}
		while(minute<0)
		{
			minute=60+minute;
			stunde=stunde-1;
		}
		while(stunde<0)
		{
			stunde=24+stunde;
		}
		
		System.out.println("Uhrzeit ist "+stunde+":"+minute+":"+sekunde);
		
	}
	
	
	public  boolean istGleich(Uhrzeitt uz){
	      
        return (this.stunde==uz.stunde)&&(this.minute==uz.minute)&&(this.sekunde==uz.sekunde);
        
    }
	
	
	public static void main(String[] args)
	{
		Uhrzeitt zeit1= new Uhrzeitt(21,31,34);
		Uhrzeitt zeit2=new Uhrzeitt (2,32,56);
		System.out.println(zeit1.istGleich(zeit2)); 
		
		
	}
	
			
	}

 

[eRaaaa]

Da ich nicht weiß wo ich anfangen soll (*sorry*) gebe ich dir einen Tipp:

 System.out.println("Uhrzeit ist "+stunde+":"+minute+":"+sekunde);

ändere das mal ab zu

 System.out.println("Uhrzeit ist "+this.stunde+":"+this.minute+":"+this.sekunde);

schaue dir das Ergebnis an und frage dich wieso das so ist

 

[Usarian]

Da bekomme ich nur 0:0:0 als Ergebniss fuer die Zeit. Das ist weil da mein Default Constructor abgerufen und benutzt wird, oder? Und schon wieder True als boolean wert. Aber diesmal stimmt es

 

[eRaaaa]

Ne ne, es wird schon der Richtige aufgerufen, aber mit

sekunde=sekunde-60;

setzt du leider nicht die Instanzvariable, sondern du veränderst den Wert, den du dem Konstruktor übergibst?! Wenn deine Paramter im Konstruktor genau so heißen wie die Instanzvariablen, solltest du this. davor schreiben Oder benutze andere Paramternamen!

Aber irgendwie sieht das mit den ganzen while-Schleifen eh irgendwie merkwürdig aus ;D

ps.:: Klar, true kommt raus, weil eben beide Uhrzeit(t)en 0 0 0 sind

 

[Usarian]

Naja das ganze ding sollte wir machen weil das auch in der Aufgabenstellung steht. Und zwar:
Zeit-Konstruktor: Eswerdendrei int-Parameter ubergeben,die fur die Stunden, Minuten und Sekun-
den stehen.Der Wertebereich der Parameter ist nicht eingeschrankt, es koennen Uberlaufe/Unterlaufe
auftreten. Beispiel :0h, 90min, 0s sind aquivalent zu 1h, 30min, 0s und -1h, 0min, 0s sind aquivalent
zu 23h, 0min, 0s.

Und wie mach ich das mit dem boolean istGleich funktionieren? Das verstehe ich nicht so ganz

 

[eRaaaa]

Naja das ganze ding sollte wir machen weil das auch in der Aufgabenstellung steht. Und zwar:
Zeit-Konstruktor: Eswerdendrei int-Parameter ubergeben,die fur die Stunden, Minuten und Sekun-
den stehen.Der Wertebereich der Parameter ist nicht eingeschrankt, es koennen Uberlaufe/Unterlaufe
auftreten. Beispiel :0h, 90min, 0s sind aquivalent zu 1h, 30min, 0s und -1h, 0min, 0s sind aquivalent
zu 23h, 0min, 0s.

Und wie mach ich das mit dem boolean istGleich funktionieren? Das verstehe ich nicht so ganz

Ja, aber ich wollte dich nur darauf hinweisen, dass du nirgends

	this.minute = minute;
	this.stunde = stunde;
	this.sekunde = sekunde;

aufrufst !
setze diese Zeilen mal vor das System.out...dann sollte es klappen, wenn ich mich nicht verguckt habe!

 

[Usarian]

Achsooooo!
Alles klar :toll: Vielen vielen Dank! Da muss ich mir ueber "dot notation" noch ein bisschen ueberlegen und lesen. (so heisst es auf Englisch) Ich hab da auch einige Schwierigkeiten weil ich nicht aus Deutschland bin. Es ist nicht so leicht alles zu verstehen

Aber noch einmal - Vielen Dank!