OOP falscher Code

[Angelika_24]

Hallo Leute, ich bin gerade dabei Java zu lernen ( in der Schule ) und bin auf ein Problem gestoßen.
Ich habe hier diesen Quelltext bei Eclipse eingegeben und dieser funktioniert nicht.
Anscheinend sind die Methoden irgend wie falsch. Das verwirrt mich, denn dieser
Quellcode ist ja nicht von mir, sondern von der Schule...
Was müsste ich denn ändern?

public class car {
	String type="";
	static int wheels=0;
	
	setType(String type) {
		this.type=type;
	}
	
	static setWheels (int wheels) {
		Auto.wheels=wheels;
	}
	
	car beispiel1=new car();
	beispiel1.setType("Beispiel1");
	
	car beispiel2=new car();
	beispiel2.setType("Beispiel2");
	
	car.setWheels(5);
	car.setWheels(3);

}

Vielen Dank jetzt schon mal für eure Hilfe!!!


Liebe Grüße

 

[Final_Striker]

Anscheinend sind die Methoden irgend wie falsch. Das verwirrt mich, denn dieser

Ja sind sie, und ich denke der Sinn dieses Quellcodes ist, dass DU die Fehler findest und verbesserst. ;-)

 

[Angelika_24]

Nein der Sinn dieses Quelltextes ist es, dass die Klassenvariable zum Schluss den Wert 3 hat.
Das ist auch keine Aufgabe, sondern ein Beispielquelltext.
Weil ich alles ohne Hindernisse verstehen möchte, habe ich diesen Quelltext in Eclipse eingegeben um
alles nachzuvollziehen. Wirklich!
Ich habe mir überlegt. Die erste Methode müsste doch so heißen, oder?

public void setType (String type) {
     this.type=type;
}

Das bedeutet, dass diese Methode nichts zurückgibt und dass sie öffentlich ist, oder?

Bei der zweiten Methode bin ich mir unsicher wegen diesem "Auto", ansonsten würde ich schreiben:

public static setWheels (int wheels) {
     Auto.wheels=wheels;
}

Wenn mein Ansatz richtig ist, warum steht dann da "static"?
Was müsste ich statt "Auto" schreiben?
Müsste ich da ein Objekt Auto anlegen?

Liebe Grüße

 

[SlaterB]

static steht da weil das ganze statisch sein soll,
das hat gewisse Vor- und Nachteile (kein Objekt),
man kann es machen oder nicht, da gibt es kein richtig oder falsch,

wenn die Methode statisch bleibt wäre es jedenfalls ziemlich komisch, darin ein Objekt anzulegen,
wieso eigentlich Auto statt car, benennst du die Klasse um?
ist zwar löblich aber dann ruhig deutlicher erwähnen,

nun hast du schon einiges korrigiert, den Rest auch noch,
Code darf nicht direkt in Klassen stehen, nur in Methoden oder Konstruktoren, soviel sei verraten

 

[Angelika_24]

Wenn meine erste Methode richtig war, warum wird dann das bei mir rot unterstrichen?

	car beispiel1=new car();
	beispiel1.setType("Beispiel1");
	
	car beispiel2=new car();
	beispiel2.setType("Beispiel2");

Also nur diese Zeilen werden rot unterstrichen:

beispiel1.setType("Beispiel1");
beispiel2.setType("Beispiel2");

Ich weiß nicht warum Auto und nicht car, ich habe den Quelltext nicht geschrieben.
"Code darf nicht direkt in Klassen stehen, nur in Methoden oder Konstruktoren, soviel sei verraten "
--> Da weiß ich nicht was du meinst.


Wegen der zweiten Methode, natürlich heißt es:

public static void setWheels (int wheels) {
		Auto.wheels=wheels;
	}

Da habe ich das void vergessen.
Aber was da jetzt anstatt des Autos hinkommt weiß ich immernoch nicht.


Auch das Aufrufen ist komisch.
Warum geht das so nicht:

car.setWheels(5);
	car.setWheels(3);

 

[SlaterB]

alle Fehler sind auf

Code darf nicht direkt in Klassen stehen, nur in Methoden oder Konstruktoren

zurückzufühen,

offensichtlich meine ich genau diese Code-Zeilen, die rot unterstrichen sind, das ist der 'Code' in meinem Satz,
was Methoden sind weißt du hoffentlich?
und der Rest ist kein klares Deutsch?

 

[Angelika_24]

Achsooooooooooooo, man bin ich blöd, die main-Methode gab es ja noch gar nicht.....
Ich habe das jetzt mal so umgeändert:

public class car {
	String type="";
	static int wheels=0;
	
	public void setType(String type) {
		this.type=type;
	}
	
	
	public static void setWheels (int wheels) {
		car.wheels=wheels;
	}
	
	public static void main (String args[]) {
		
	
	
	car beispiel1=new car();
	beispiel1.setType("Beispiel1");
	
	car beispiel2=new car();
	beispiel2.setType("Beispiel2");
	
	car.setWheels(5);
	car.setWheels(3);
	
	
	}
}

Hättet ihr das auch so gemacht?

Jetzt hab ich ja doch die Fehler gefunden, cool

 

[Final_Striker]

Hättet ihr das auch so gemacht?

Ja.

Das static bei dem wheels ist zwar ein bisschen komisch, aber ich denke in dem Fall dient es den Unterschied zwischen static und nicht zu zeigen.

Tipp: Klassen fangen mit einem großen, Variablen und Methoden mit einem kleinen Buchstaben an. ;-)

 

[Angelika_24]

Die Tipps werde ich beherzigen


Dankeschön!