Sortiermethode bei Bubblesort

[bodo1]

Habe ein kleines Problem bei der Implementierung des Quelltextes.
Vll kann ja einer mal kurz draufgucken und mir den fehler sagen.
Dankeschön

Klasse für Prüfkörper:

package objekte;

public class Pruefkoerper {

	private double breite;
	private double hoehe;
	private double kraft;
	
	public Pruefkoerper(double breite, double hoehe, double kraft){
		this.breite=breite;
		this.hoehe=hoehe;
		this.kraft=kraft;
	}
	
}

Klasse für Sortieralgorithmus:
Fehlermeldungen in Zeilen: 20,21,23,27

package anwendung;

import objekte.Pruefkoerper;

public class auswertung {

	public  Pruefkoerper [] feld;

	private double breite;
	private double hoehe;
	private double kraft;

	public double berechneBruchfestigkeit(){
		return kraft/(breite*hoehe);
	}

	public double sortierePruefkoerper(Pruefkoerper[]feld){
		for(int j=1; j<feld.length; j++){
			for(int i = feld.length-1; i <=j; i++){
				if(feld[i].berechneBruchfestigkeit>=feld[i+1].berechneBruchfestigkeit){
					double zwischenvariable = feld[i];
					feld[i]= feld[i+1];
					feld[i+1]= zwischenvariable;
				}	
			}
		}
		return Pruefkoerper[]feld;
	}
}

 

[Asgar13]

Z.20
Wenn berechneBruchfestigkeit eine Methode ist.

feld[i].berechneBruchfestigkeit()

z. 21 du kannst nicht einfach ein Objekt in eine double variable speichern

Pruefkoerper[] zwischenvariable = feld[i];

z.23 dasselbe wie bei zeile 21 nur genau andresherum.

 

[bodo1]

hab ich in zeile 13-15 nicht genau diese methode geschrieben?

und was nehm ich anstatt double? :rtfm:

 

[Camill]

Du hast beim Aufruf der Methode jedoch die

()

vergessen, desweiteren gehört die Methode in Klasse

Pruefkoerper

.

Beim Sortieren würdest du außerdem auch noch eine Exception erhalten da du beim ersten Durchlauf mit der If-Anweisung auf einen nicht vorhandenen Index zu zugreifst:

feld[i+[COLOR="Red"]1[/COLOR]].berechneBruchfestigkeit

(

i

ist zu diesem Zeitpunkt bereits das letzte Element)

 

[bodo1]

Danke für den Hinweis (wie oft ich den fehler schon gemacht habe).

Das Problem mit dem double-Werten hab ich versucht zu lösen, indem ich sie zu Prüfkörpern gemacht habe.
Ist das ein richtiger Ansatz?
beim return jedoch kommt immer noch ein fehler. "Syntax error on token(s), misplaced construct(s)"

package objekte;

public class Pruefkoerper {

	private double breite;
	private double hoehe;
	private double kraft;

	public Pruefkoerper(double breite, double hoehe, double kraft){
		this.breite=breite;
		this.hoehe=hoehe;
		this.kraft=kraft;
	}

	public double berechneBruchfestigkeit(){
		return kraft/(breite*hoehe);
	}

	public  Pruefkoerper [] feld;

	public Pruefkoerper[] sortierePruefkoerper(Pruefkoerper[]feld){
		for(int j=1; j<feld.length; j++){
			for(int i = feld.length-1; i <=j; i++){
				if(feld[i].berechneBruchfestigkeit()>=feld[i+1].berechneBruchfestigkeit()){
					Pruefkoerper zwischenvariable = feld[i];
					feld[i]= feld[i+1];
					feld[i+1]= zwischenvariable;
				}	
			}
		}
		return Pruefkoerper[]feld;
	}
}

 

[bodo1]

reicht es nicht aus wenn ich " return feld; " schreibe?

 

[Camill]

So wäre das zurückgeben des Arrays richtig:

return feld

 

[bodo1]

Fehlermeldung in der Zeile 25
Dabei sieht doch alles so schön aus:bahnhof:
Hat jemand ne Ahnung was da schiefgegangen sein könnte?

package anwendung;

import objekte.Pruefkoerper;

public class test {

	public static void main (String[]args){

		Pruefkoerper pk1 = new Pruefkoerper(120, 200, 509000);
		Pruefkoerper pk2 = new Pruefkoerper(150, 150, 817000);
		Pruefkoerper pk3 = new Pruefkoerper(150, 150, 617000);
		Pruefkoerper pk4 = new Pruefkoerper(130, 170, 320000);
		Pruefkoerper pk5 = new Pruefkoerper(120, 180, 743000);
		Pruefkoerper pk6 = new Pruefkoerper(200, 200, 920000);

		Pruefkoerper[] m = new Pruefkoerper[6];

		m[0]=pk1;
		m[1]=pk2;
		m[2]=pk3;
		m[3]=pk4;
		m[4]=pk5;
		m[5]=pk6;

		System.out.println(m.sortiereBruchfestigkeit);
	}
}

 

[Camill]

Du kannst auf die einzelnen Methoden der Klasse

Pruefkoerper

nur mit einem Pruefkoerper Objekt drauf zugreifen, in der besagten Zeile versucht du es jedoch mit einem Array voller Pruefkoerper Objekte.

 

[pro2]

Methoden beendet man außerdem mit Klammern beim Aufruf (); ;-)

 

[bodo1]

ganz ehrlich? versteh ich nicht:autsch:
will ich nicht mein neues feld sortiert ausgegeben haben?
wenn ich dich richtig verstehe müsste ich jetzt irgendwas mit Prüfkörper schreiben oder?

 

[bodo1]

Methoden beendet man außerdem mit Klammern beim Aufruf (); ;-)

Den Fehler hab ich heute schon mal gemacht, aber daran lags leider auch nicht...danke

 

[Camill]

Habe mal eine lauffähige Version erstellt, so könnte das ganze Aussehen:

public class Pruefkoerper {
	
	private double breite;
	private double hoehe;
	private double kraft;
	
	public Pruefkoerper(double breite, double hoehe, double kraft) {
		
		this.breite = breite;
		this.hoehe = hoehe;
		this.kraft = kraft;
	}
	
	public double berechneBruchfestigkeit() {
		return kraft / (breite * hoehe);
	}
}

public class Auswertung {
	
	public static void main(String[] args) {
		
		Pruefkoerper[] pruefkoerpers = {
				new Pruefkoerper(1, 1, 50), new Pruefkoerper(10, 2, 1),
				new Pruefkoerper(7, 11, 3), new Pruefkoerper(4, 19, 1)};
		
		sortierePruefkoerper(pruefkoerpers);
		
		for(int i = 0; i < pruefkoerpers.length; i++) {
			System.out.println(pruefkoerpers[i].berechneBruchfestigkeit());
		}
	}
	
	static Pruefkoerper[] sortierePruefkoerper(Pruefkoerper[] feld) {
		
		Pruefkoerper pruefkoerper;
		for(int i = 0; i < feld.length - 1; i++) {
			
			for(int j = i + 1; j < feld.length ; j++) {
				if(feld[i].berechneBruchfestigkeit() > feld[j].berechneBruchfestigkeit()) {
					pruefkoerper = feld[i];
					feld[i] = feld[j];
					feld[j] = pruefkoerper;
				}
			}
		}
		return feld;
	}
}

Hoffe das es verständlich ist.

 

[bodo1]

irgendwie versteh ichs nicht nein.
dachte vll kann kannst du mir sagen wo ich einen fehler gemacht habe
aber nett dass du das ganze programm mal eben so neu geschrieben hast *neidig-in-den-himmel-guck*

 

[Camill]

Wie ich weiter oben bereits schrieb war bei dir Zeile 25 das Problem:
[JAVA=25]System.out.println(m.sortiereBruchfestigkeit);[/code]
Du hast versucht mit einem Array auf die Methode

sortiereBruchfestigkeit()

der Klasse

Pruefkoerper

zuzugreifen, dies ist jedoch nur mit einem einzelnen Objekt der Klasse möglich, bsp.:

m[0].sortiereBruchfestigkeit()

Somit könntest du die Sortier-Methode direkt in der Applikationsklasse schreiben(wie in der von mir geposteten Lösung). Habe die for-Schleifen noch entsprechend abgeändert sodass nun auch richtig sortiert wird.

 

[bodo1]

hatte gehofft, dass ich schon fast fertig war und nur nen denkfehler hatte, aber anscheinend war doch wieder mehr falsch...
ich versuch mal deine methode zu implementieren.

danke dir