✔ java Klasse

[g.sandra.1995]

Ausgabe bei mir nur [Auto@41, Auto@42]

 

[HonniCilest]

Ausgabe bei mir nur [Auto@41, Auto@42]

Du musst deinem Objekt Auto schon beibringen, wie es auf der Console ausgegeben werden soll, indem du die Methode toString() überschreibst.
(was ich in meinen Beispielen auch mit aufgeführt hatte)

 

[vfl_freak]

Moin,

was ich in meinen Beispielen auch mit aufgeführt hatte

schon, nur dass Du es dann auch nicht benutzt hast

System.out.println(autos);

Gruß Klaus

 

[g.sandra.1995]

danke habe die toString()-Methode vergessen

 

[HonniCilest]

Moin,

schon, nur dass Du es dann auch nicht benutzt hast

Gruß Klaus

Doch klar habe ich sie benutzt. Natürlich habe ich sie nicht implizit aufgerufen, aber "autos" war in diesem Fall eine HashSet und die Ausgabe hier erfolgt wie in meiner aufgezeigten Ausgabe. HashSet.toString() ist bereits sinnvoll implemtiert, jedoch nicht von einem Array. Bei Arrays ist Arrays.toString(array) notwendig.

 

[g.sandra.1995]

Guten Tag



Vielen für Ihre Hilfe.



Ich fasse nun zusammen



die enum Klasse ist

enum Farbe {



ROT,

BLAU,

GELB,

WEISS,

GRAU,

SCHWARZ;



}

die Auto Klasse ist

import java.util.*;



public class Auto implements Comparable<Auto> {

String kennzeichen;

Farbe farbe;

double ps;



public Auto(String myKennzeichen, Farbe myFarbe, double myPs) {

this.kennzeichen = myKennzeichen;

this.farbe = myFarbe;

this.ps = myPs;

}



public String getKennzeichen() {

return kennzeichen;

}



public Farbe getFarbe() {

return farbe;

}



public double getPs() {

return ps;

}



private static boolean createAuto(ArrayList<Auto> autoCollection, Auto neuesAuto) {

boolean isCreateable = true;

for (Auto auto : autoCollection)

if (auto.getKennzeichen() == neuesAuto.getKennzeichen()) {

isCreateable = false;

break;

}

return isCreateable;

}



@Override

public String toString() {

String erg="";

erg = erg + String.format(Locale.ENGLISH, "%s Farbe=%s mit %1.3f PS\n",

kennzeichen, farbe, ps);

erg = erg + String.format(Locale.ENGLISH, "\n");

return erg;

}



@Override

public boolean equals(Object o) {

if (!(o instanceof Auto))

return false;

Auto auto = (Auto) o;

return kennzeichen.equals(auto.kennzeichen);

}



@Override

public int hashCode() {

return kennzeichen.hashCode();

}



@Override

public int compareTo(Auto ot) {

if (farbe.ordinal() < ot.farbe.ordinal())

return -1;

else if (farbe.ordinal() > ot.farbe.ordinal())

return 1;

else

return 0;

}

}

… alles testen

import java.util.*;



public class AutoTest {



public static void main(String[] args) {

HashSet<Auto> autos = new HashSet<Auto>();

autos.add(new Auto("A", Farbe.WEISS, 28));

autos.add(new Auto("A", Farbe.SCHWARZ, 37));

autos.add(new Auto("B", Farbe.ROT, 59));

autos.add(new Auto("C", Farbe.BLAU, 28));

autos.add(new Auto("D", Farbe.GRAU, 37));

autos.add(new Auto("E", Farbe.GRAU, 59));

System.out.println(autos);

System.out.println("+++++sortiert+++++");

List sortedList = new ArrayList(autos);

Collections.sort(sortedList);

System.out.println(sortedList);

}

}

bei Compilieren bekomme ich die Meldung




Note: AutoTest.java uses unchecked or unsafe operations.

Note: Recompile with -Xlint:unchecked for details.



hat jemand Verbesserungen?

 

[vfl_freak]

Moin,

eine enum ist eigentlich keine eigene Klasse ...

Wo wird enum Farbe denn für´s Testprogramm deklariert ?

Note: AutoTest.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

siehe bspw. hier: http://stackoverflow.com/questions/16617901/uses-unchecked-or-unsafe-operations
oder

Gruß Klaus

 

[Bratkartoffel]

Hi,

1) Die Warning kommt bei deinem Test, Zeile 29. Hier fehlen die Generics auf der Liste:

List sortedList = new ArrayList(autos);

2) Deine Auto.toString() ruft die format() 2x auf, dies kann man auf einen Aufruf zusammenfassen
3) Die Auto.hashCode() und equals() sind nicht null-safe (Exception wenn kennzeichen = null)
4) Attribute sind normalerweise private. Da du keine Setter hast, kannst du diese bei dir auch final machen.
5) Die Auto.createAuto() kann man als Einzeiler schreiben:

return autoCollection.stream().noneMatch(neuesAuto::equals);

6) Deine Auto.compareTo() kann man auch als Einzeiler schreiben:

return farbe.ordinal() - ot.farbe.ordinal();

Grüsse,
BK

 

[g.sandra.1995]

Generics auf der Liste? wa sist das? noch am Anfang bin ich

 

[Bratkartoffel]

Hi,

http://openbook.rheinwerk-verlag.de/javainsel/javainsel_09_001.html

Grüsse,
BK