OOP Klassenmethode in andere Methode?

[Salvo]

Hello,

ich sitze gerade haareraufend vor einer Aufgabe, bei der ich nach bestimmten Kriterien eine Klasse namens Time24 erstellen soll (Uhrzeit einlesen und im 24 Std Format ausgeben). Jetzt habe ich bei folgender Methode ein Problem, da ich nicht weiß, wie ich sie genau programmieren soll:
Die Methode readTime() liest die Attributwerte eines Time24-Objektes ein. Sie verwendet dazu die Klassenmethode readInt() der Klasse TastaturEingabe.

Was ich gemacht habe:
Ich habe die Klasse TastaturEingabe mit folgender Methode erstellt (vorher habe ich das utility TastaturEingabe, welches Eingaben einliest,importiert)

	public readInt(int XX, int YY)
	        {
		XX = readInt.TastaturEingabe("Stundenanzahl eingeben");
		YY = readInt.TastaturEingabe("Minutenanzahl eingeben");
	        }

Nun sitze ich hier vor dem Bildschirm und überlege, wie ich diese readInt() Methode mit der readTime() Methode verknüpfen soll.... so ist es ja bestimmt falsch, oder habe ich schon den richtigen Ansatz ???:L

public int readTime()
		{readInt()}

Hoffentlich ist es verständlich, was mein Problem ist. Habe vorher noch nie OOP gemacht.

Gruß
Salvo

 

[javimka]

Wie sieht denn die Klasse TastaturEingabe aus?

Deine Methode readInt hat keinen Rückgabewert. Wenn die Zahl zurückgegeben werden soll, müsste der int sein, ansonsten void. Oder ist das etwa ein Konstruktor?
"readInt.TastaturEingabe" sieht komisch aus. Hier verwendest du readInt als ein Objekt. Das müsste doch eher etwas sein wie TastaturEingabe.readInt().

Die Methode readTime muss einen int zurückgeben. Wahrscheinlich musst du da schreiben: [c]return readInt();[/c]

 

[AmunRa]

Bei deinem ersten Code ist mal so ein gewaltiger Fehler

du schreibst

readInt.Tastatureingabe()

das würde heißen, dass du von der Klasse readInt die methode Tastatureingabe verwenden möchtest. Das hast du aber in deinem Text anders geschrieben.

Es muss immer heisen, zuerst Klasse dann punkt und dann Funktionsname()also

TastaturEingabe.readInt()

 

[Salvo]

Danke schonmal für die schnellen Antworten.

Okay, das mit Tastatureingabe.readInt hatte ich verdreht...

@javimka
Nein, das sollte kein Konstruktor werden. Habe nur verplant int zu schreiben Meine Klasse TastaturEingabe sieht wie folgt aus. (mit Verbesserung)

import utilities.TastaturEingabe;

public class TastaturEingabe
{
	public int readInt(int XX, int YY)
	{
		XX = TastaturEingabe.readInt("Stundenanzahl eingeben");
		YY = TastaturEingabe.readInt("Minutenanzahl eingeben");
	}
}

Ich weiß nicht, ob ich noch Attribute oder Konstruktoren einbauen soll, da dies in der Aufgabenstellung nicht genannt wird. Es wird nur die Methode readInt() der Klasse TastaturEingabe verlangt...

 

[Schumi]

TastaturEingabe und readInt gibt es doch schon, die musst Du nicht mehr erstellen.
Du sollst die Methode readTime erstellen. Die sollte wohl in einer Time24 Klasse stehen. In dieser Methode musst Du dann mit Hilfe der readInt Methode die Stunden und Minuten einlesen.

Eta: und Du solltest Dir dringend einmal das Konzept von Übergabeparametern von Methoden anschauen. Was Du in Deiner (unnötigen) readInt Methode mit XX und YY machst ist ziemlich sinnlos. So wie Du es geschrieben hast, erwartet die Methode zwei ints als Übergabeparameter, aber Du überschreibst sie dann direkt.

 

[javimka]

Ich glaube Salvo wollte XX und YY auch überschreiben, aber so funktioniert das nicht, denn XX und YY sind dann zwar überschrieben, aber die Methode, die diese XX und YY übergeben hat, merkt nichts davon. Salvo wollte wahrscheinlich XX und YY einer Methode übergeben, die dann die richtigen Werte darauf setzt. Aber das funktioniert so nicht.
Du musst mit Rückgabewerten arbeiten, die dann auf deine Variabeln speichern. Aber du kannst immer nur einen Rückgabewert haben, du wirst es also schwer haben, auf elegante Weise XX und YY in einer einzigen Methode zu berechnen. Was du notfalls machen könntest wäre, ein Array int[] zurückzugeben und die Werte daraus dann auf XX,YY zu speichern.