Klassen problem

[flashdog]

Hallo
Ich habe folgenden Code geschrieben:

import java.util.ArrayList;
import java.util.Date;

class Nachricht {
  public Date uhrzeit;
  public String nachricht;
};

class NachrichtService {
  Nachricht nachricht = new Nachricht();

  public void addNachricht(String msg) {
    nachricht.nachricht = msg;
    Date now = new Date();
    nachricht.uhrzeit = now.getTime();
  }

  public Nachricht getNaechsteNachricht() {
    return nachricht;
  }
};

public class NachrichtenTest {

  public static void main(String[] args) {
    NachrichtService nachricht = new NachrichtService();
    ArrayList<NachrichtService> nachrichten = new ArrayList<NachrichtService>();
    nachrichten.add(nachricht.addNachricht("Hallo"));
  }
}

Leider bekomme ich diese Fehlermeldungen:

NachrichtenTest.java:15: incompatible types
found   : long
required: java.util.Date
    nachricht.uhrzeit = now.getTime();
NachrichtenTest.java:28: 'void' type not allowed here
    nachrichten.add(nachricht.addNachricht("Hallo"));
2 errors

Wie kann man diesen Code zum laufen bringen?

Viele Grüße

 

[SlaterB]

andersrum gefragt: was soll der Code deiner Meinung nach leisten?
was hattest du dir dabei gedacht, als du
> nachrichten.add(nachricht.addNachricht("Hallo"));
schriebst?
was soll da in die Liste rein, die Nachricht? (= das NachrichtService-Objekt oder ein anders?)

wieso kombinierst du das Einfügen in die Liste mit dem separaten Methodenaufruf addNachricht(),
oder besteht da eine Verbindung?

 

[flashdog]

Mit addNachricht Methode würde ich ganz gerne eine Nachricht und die aktuelle Uhrzeit speichern. z.B. N1.

Wenn ich die getNaechsteNachricht() Methode aufrufe würde ich gerne die erste Nachricht erhalten, z.B. 9:30, N1. Wenn ich diese Methode nochmal aufrufe würde ich ganz gerne die zweite Nachricht erhalten, z.B. 9.35, N2 usw..

Ich bin nicht wie ich alle Nachrichten mit addNachricht() abspeichern kann und in welche Datenstruktur, so dass ich sie später mit getNaechsteNachricht() Methode diese abrufen kann und ob ich richtig die beiden Klassen (Nachricht und NachrichtService) miteinander verbunden habe.

 

[SlaterB]

hmm, leider gehts du kaum auf meine Frage ein, den Codeteil nachrichten.add() ignorierst du völlig,

also ohne lehrende Hinweise:

ArrayList<NachrichtService> nachrichten = new ArrayList<NachrichtService>();
NachrichtService nservice = new NachrichtService();
nservice.addNachricht("Hallo");
Nachricht n = nservice.getNaechsteNachricht();
nachrichten.add(n);

-------

oder

Nachricht n = nservice.addNachricht("Hallo");
nachrichten.add(n);

bzw. in deinem Code

nachrichten.add(nservice.addNachricht("Hallo"));

wenn nur addNachricht() den richtigen Rückgabetyp hätte und die neue Nachricht auch zurückgibt statt void

 

[flashdog]

Sorry, jetzt verstehe ich was du meinst und habe versucht deine Tipps umzusetzen:

import java.util.ArrayList;
import java.util.Date;

class Nachricht {
  public Date uhrzeit;
  public String nachricht;
};

class NachrichtService {
  Nachricht nachricht = new Nachricht();
  ArrayList nachrichten = new ArrayList();
  int neachstePos = 0;

  public void addNachricht(String msg) {
    nachricht.nachricht = msg;
    Date now = new Date();
    nachricht.uhrzeit = now.getTime();
    nachrichten.add(nachricht);
  }

  public Nachricht getNaechsteNachricht() {
    int pos = neachstePos;
    neachstePos++;
    return nachrichten.get(pos);
  }
};

public class NachrichtenTest {

  public static void main(String[] args) {
    NachrichtService nservice = new NachrichtService();
    nservice.addNachricht("Hallo");
    
    Nachricht n = nservice.getNaechsteNachricht();
    System.out.println(n.nachricht + " " + n.uhrzeit);
  }
}

Leider bekomme ich folgende Fehler:

NachrichtenTest.java:18: incompatible types
found   : long
required: java.util.Date
    nachricht.uhrzeit = now.getTime();
NachrichtenTest:25: incompatible types
found   : java.lang.Object
required: Nachricht
    return nachrichten.get(pos);
NachrichtenTest.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
2 errors

Was mache ich falsch?

 

[SlaterB]

die Liste nachrichten in den Service zu verschieben ist zwar nicht genau was ich meinte,
macht aber Sinn, ja, so ist es wohl besser, soll der Service die Liste vorhalten als irgendwo in der main-Methode eine freie Liste

--------

now ist ein Date, now.getTime() ist ein long-Wert,
nachricht.uhrzeit ist vom Typ Date

nun überlege, ob du da besser now speicherst oder now.getTime()

 

[flashdog]

Habe den Code weiter verbessert:

import java.util.ArrayList;
import java.util.Date;

class Nachricht {
  public Date uhrzeit;
  public String nachricht;
};

class NachrichtService {
  ArrayList nachrichten = new ArrayList();
  int neachstePos = 0;

  public void addNachricht(String msg) {
    Nachricht nachricht = new Nachricht();
    nachricht.nachricht = msg;
    nachricht.uhrzeit = new Date();
    nachrichten.add(nachricht);
  }

  public Nachricht getNaechsteNachricht() {
    int pos = neachstePos;
    neachstePos++;
    return nachrichten.get(pos);
  }
};

public class NachrichtenTest {

  public static void main(String[] args) {
    NachrichtService nservice = new NachrichtService();
    nservice.addNachricht("Hallo");
    
    Nachricht n = nservice.getNaechsteNachricht();
    System.out.println(n.nachricht + " " + n.uhrzeit.getHours() + ":" + n.uhrzeit.getMinutes());
  }
}

aber leider habe bekommen ich den letzten Fehler noch nicht weg:

NachrichtenTest.java:24: incompatible types
found   : java.lang.Object
required: Nachricht
    return nachrichten.get(pos);
Note: NachrichtenTest.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: NachrichtenTest.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
1 error

Wie bekommt den letzten Fehler weg?

 

[nrg]

du hast deine ArrayList nicht generisch deklariert.
entweder den get zu Nachricht casten

return (Nachricht)nachrichten.get(pos);

oder besser:
Arraylist generisch deklarieren:
ArrayList<Nachricht> nachrichten = new ArrayList<Nachricht>();

grüße
nrg

 

[Murray]

Entweder casten, oder gleich mit Generics arbeiten:

List<Nachricht> nachrichten = new ArrayList<Nachricht>();

 

[SlaterB]

eine ArrayList liefert Object und nicht Nachricht, wenn nicht generisch?
die angemerkten deprecated-Methoden sind übrigens uhrzeit.getHours() und ähnliche

 

[flashdog]

Danke hat funktioniert.

 

[flashdog]

Ich wollte nun noch zu jeder Nachricht eine Priorität setzen (1 = Hoch; 2 = Mittel; 3 = Niedrig), und wenn ich getNaechsteNachricht aufrufe mehrmals aufrufe wollte ich z.B. diese Ausgabe erhalten:

	1,	"N 4", 9:42,
	2,	"N 2", 9:35
	3,	"N 1", 9:30
	3,	"N 3", 9:39

Den Code habe ich schon ein wenig modifiziert:

import java.util.ArrayList;
import java.util.Date;

class Nachricht {
  public Date uhrzeit;
  public String nachricht;
  public int prioritaet;
};

class NachrichtService {
  ArrayList<Nachricht> nachrichten = new ArrayList<Nachricht>();
  int neachstePos = 0;

  public void addNachricht(int prioritaet, String msg) {
    Nachricht nachricht = new Nachricht();
    nachricht.nachricht = msg;
    nachricht.uhrzeit = new Date();
    nachricht.prioritaet = prioritaet;
    nachrichten.add(nachricht);
  }

  public Nachricht getNaechsteNachricht() {
    int pos = neachstePos;
    neachstePos++;
    return nachrichten.get(pos);
  }
};

public class NachrichtenTest {

  public static void main(String[] args) {
    NachrichtService nservice = new NachrichtService();
    nservice.addNachricht(3,	"N 1");
    nservice.addNachricht(2,	"N 2");
    nservice.addNachricht(3,	"N 3");
    nservice.addNachricht(1,	"N 4");

    Nachricht n = nservice.getNaechsteNachricht();
    System.out.println(n.prioritaet + " "+ n.nachricht + ", " + n.uhrzeit.getHours() + ":" + n.uhrzeit.getMinutes());
  }
}

Leider weiss ich nicht wie man am beste die Nachrichten speichern könnte, da ArrayList keine sortier Möglichkeit bietet. Vielleicht gibt es etwas besseres?

 

[SlaterB]

ArrayList kann man gut mit Collections.sort() sortieren,
wenn denn die Nachricht Comparable wäre

Galileo Computing :: Java ist auch eine Insel (8. Auflage) – 12 Datenstrukturen und Algorithmen

 

[flashdog]

Wie kann man die Nachricht Comparable machen, denn wenn ich nur Collection.sort(nachrichten) mache funktioniert es nicht. Macht aber auch Sinn da ich nach nachricht.prioritaet sortieren möchte, aber leider weiss ich nicht wie man es sortiert bekommt.

import java.util.ArrayList;
import java.util.Collections;
import java.util.Date;

class Nachricht {
  public Date uhrzeit;
  public String nachricht;
  public int prioritaet;
};

class NachrichtService {
  ArrayList<Nachricht> nachrichten = new ArrayList<Nachricht>();
  int neachstePos = 0;

  public void addNachricht(int prioritaet, String msg) {
    Nachricht nachricht = new Nachricht();
    nachricht.nachricht = msg;
    nachricht.uhrzeit = new Date();
    nachricht.prioritaet = prioritaet;
    nachrichten.add(nachricht);
  }

  public Nachricht getNaechsteNachricht() {
    int pos = neachstePos;
    Collections.sort(nachrichten.);
    neachstePos++;
    return nachrichten.get(pos);
  }
};

public class NachrichtenTest {

  public static void main(String[] args) {
    NachrichtService nservice = new NachrichtService();
    nservice.addNachricht(3,	"N 1");
    nservice.addNachricht(2,	"N 2");
    nservice.addNachricht(3,	"N 3");
    nservice.addNachricht(1,	"N 4");

    Nachricht n = nservice.getNaechsteNachricht();
    System.out.println(n.nachricht + " " + n.uhrzeit.getHours() + ":" + n.uhrzeit.getMinutes());
  }
}

Oder gibt es eine andere Datenstruktur um nachriten nach nachrichten.prioritaet zu sortieren?

 

[nrg]

du musst in die Klasse Nachricht das Interface Comparable implementieren und dann die Methode compareTo überschreiben.

 

[SlaterB]

du brauchst ein Sortierverfahren, dass entweder direkt die richtigen Felder abfragt oder sich auf das allgemeine Interface Comparable verlässt,
für letzteres gibt es schon Collections.sort, dann musst du nur noch Comparable implementieren

ich sehe keinen Sinn darin, das dir zu erklären, das ist ein sehr großes Grundthema in Java, lies dir dazu die 20 Seiten im Lehrbuch durch,
oder schaue bei google nach Beispielen

einen eigenen Sortieralgorithmus findet man bei google sicher auch, im Grunde muss man da das andere aber auch mit einbauen,
ist nicht leichter, kann man dennoch irgendwann mal machen, um zu verstehen, was Collections.sort leistet,
ein einfacher Algorithmus ist z.B. BubbleSort
Google
schnell gefunden, aber den dann auf deine Nachrichten anzupassen ist wie gesagt auch nicht leichter

 

[nrg]

z.b. so

Solltest dich auch bemühen etwas mehr OO zu programmieren. Kapseln/Konstruktoren/Getter/Setter (getter/setter hab ich mal dir überlassen, weil man sie bis jetzt noch nicht braucht)

import java.util.ArrayList;
import java.util.Collections;
import java.util.Date;
import java.util.List;
 
class Nachricht implements Comparable<Nachricht>{
	
	private int prioritaet;
	private String nachricht;
	private Date uhrzeit;
  
	public Nachricht(int prioritaet, String nachricht, Date uhrzeit){
		this.prioritaet = prioritaet;
		this.nachricht = nachricht;
		this.uhrzeit = uhrzeit;
	}
	@Override
	public int compareTo(Nachricht o) {
		if (this == o)
			return 0;
		else (this.prioritaet > o.prioritaet)
			return 1;
		else if (this.prioritaet < o.prioritaet)
			return -1;
		else
			return this.uhrzeit.compareTo(o.uhrzeit);
	}
	@SuppressWarnings("deprecation")
	@Override
	public String toString(){
		return "Prio:" + prioritaet + " - " + nachricht + " - " + uhrzeit.getHours() + ":" + uhrzeit.getMinutes();
	}
}
 
class NachrichtService {
	List<Nachricht> nachrichten;;
	private int neachstePos = 0;
 
	public NachrichtService(){
		nachrichten = new ArrayList<Nachricht>();
	}
	public void addNachricht(int prioritaet, String msg) {
		nachrichten.add(new Nachricht(prioritaet, msg, new Date()));
	}
 
	public Nachricht getNaechsteNachricht() {
		int pos = neachstePos;
		neachstePos++;
		return nachrichten.get(pos);
	}
  
	public void sortAndPrintNews(){
		Collections.sort(nachrichten);
		for (Nachricht n : nachrichten)
			System.out.println(n);
  }
}
 
public class Tester {
	public static void main(String[] args) {
		NachrichtService nservice = new NachrichtService();
		nservice.addNachricht(3,    "N 1");
		nservice.addNachricht(2,    "N 2");
		nservice.addNachricht(3,    "N 3");
		nservice.addNachricht(4,    "N 5");
		nservice.addNachricht(8,    "N 6");
		nservice.addNachricht(5,    "N 7");
		nservice.addNachricht(10,    "N 8");
		nservice.addNachricht(9,    "N 9");
 
		nservice.sortAndPrintNews();
	}
}

sortAndPrintNews() gibt alle Nachrichten primär nach Prio sekundär nach Uhrzeit sortiert aus. Die toString() evtl noch nach Wunsch anpassen.

btw: Date ist deprecated. Vllt dafür lieber GregorianCalendar nehmen.

edit: in der toString() sollte es natürlich Prio: heissen

 

[flashdog]

Danke es funktioniert. Wozu dient diese Bedingung in compareTo(Nachricht o):

        if (this == o)
            return 0;

 

[SlaterB]

ist dir der Sinn des Rückgabewerts nicht klar? ob 0, -1 oder +1 zurückgegeben wird ist alles extrem wichtig, siehe Lehrbuch

und es kann nunmal vorkommen, dass ein Objekt mit sich selber verglichen wird, dann ist klar was am Ende rauskommt,
statt nun alle Vergleiche durchexerzieren kann man auch gleich das Endergebnis zurückgeben,

beim Überschreiben einer equals()-Methode bietet sich das übrigens genauso an, falls du die kennst

 

[flashdog]

Danke. Ich habe versucht den Code mit Enums zu erweitern damit der User "Niedrig", "Mittel" und "Hoch" eingeben kann anstatt 0, 1, 2, aber leider weiss ich nicht ob ich enum an der richtigen Stelle definiert habe. Ich dachte das Enums wie Integer Werte behandelt werden und deshalb die compareTo Methode richtig sein sollte, aber leider trifft das nicht zu oder ich habe etwas falsch gemacht.

import java.util.ArrayList;
import java.util.Collections;
import java.util.Date;
import java.util.List;

class Nachricht implements Comparable<Nachricht>{
    enum prioritaetTyp {Niedrig, Mittel, Hoch};

    private prioritaetTyp prioritaet;
    private String nachricht;
    private Date uhrzeit;

    public Nachricht(prioritaetTyp prioritaet, String nachricht, Date uhrzeit){
        this.prioritaet = prioritaet;
        this.nachricht = nachricht;
        this.uhrzeit = uhrzeit;
    }
    @Override
    public int compareTo(Nachricht o) {
        if (this == o)
            return 0;
        else if (this.prioritaet. > o.prioritaet)
            return 1;
        else if (this.prioritaet < o.prioritaet)
            return -1;
        else
            return this.uhrzeit.compareTo(o.uhrzeit);
    }
    @SuppressWarnings("deprecation")
    @Override
    public String toString(){
        return "Prio:" + prioritaet + " - " + nachricht + " - " + uhrzeit.getHours() + ":" + uhrzeit.getMinutes();
    }
}

class NachrichtService {
    List<Nachricht> nachrichten;
    private int neachstePos = 0;

    public NachrichtService(){
        nachrichten = new ArrayList<Nachricht>();
    }
    public void addNachricht(String prioritaet, String msg) {
        nachrichten.add(new Nachricht(prioritaet, msg, new Date()));
    }

    public Nachricht getNaechsteNachricht() {
        int pos = neachstePos;
        neachstePos++;
        return nachrichten.get(pos);
    }

    public void sortAndPrintNews(){
        Collections.sort(nachrichten);
        for (Nachricht n : nachrichten)
            System.out.println(n);
  }
}

public class Tester {
    public static void main(String[] args) {
        NachrichtService nservice = new NachrichtService();
        nservice.addNachricht("Hoch",    "N 1");
        nservice.addNachricht("Hoch",    "N 2");
        nservice.addNachricht("Niedrig",    "N 3");
        nservice.addNachricht("Mittel",    "N 5");
        nservice.addNachricht("Mittel",    "N 6");
        nservice.addNachricht("Niedrig",    "N 7");
        nservice.addNachricht("Niedrig",    "N 8");
        nservice.addNachricht("Mittel",    "N 9");

        nservice.sortAndPrintNews();
    }
}

Dieser Code verursacht diese Fehler:

Tester.java:22: <identifier> expected
        else if (this.prioritaet. > o.prioritaet)
1 error

Wie bekommt man den gefixt?

 

[AmunRa]

du hast einen Punkt zu viel in der Zeile 22

statt [c] else if (this.prioritaet. > o.prioritaet)[/c] muss es [c] else if (this.prioritaet > o.prioritaet)[/c]

heißen

 

[Empire Phoenix]

Enumerations sid tatsächlih versteckte klassen (oder so ähnlich) die haben eine Methode womit man den int wert bekommen kann.

Ich empfehle mal die JavaDoc zu benutzen oder wenn du eine IDE nimmst (zb eclipse) die so einzustellen, das sie dir alle methoden anzeigt die die jeweiligen objecte können. (Ist schon ne tolle sache, ausserdem spart ma sich ne menge Zeit wenn man nicht genau weiß wie die methode geschreiben wird oder welche argumente sie braucht)

 

[SlaterB]

mal beide ähnlichen Zeilen direkt hintereinandergeschrieben:

else if (this.prioritaet. > o.prioritaet)
else if (this.prioritaet < o.prioritaet)

dir fällt kein Unterschied auf? beim Programmieren hat jedes Zeichen, oft sogar die Leerzeichen, zumindest jeweils das erste, eine Bedeutung

(edit: auf letztes Posting erste Seite bezogen, Rest nicht gesehen )

 

[Landei]

- zu spät -

 

[flashdog]

Danke, aber leider macht der Code folgende Problem:

Tester.java:22: operator > cannot be applied to Nachricht.prioritaetTyp,Nachricht.prioritaetTyp
    } else if (this.prioritaet > o.prioritaet) {
Tester.java:24: operator < cannot be applied to Nachricht.prioritaetTyp,Nachricht.prioritaetTyp
    } else if (this.prioritaet < o.prioritaet) {
Tester.java:48: cannot find symbol
symbol  : constructor Nachricht(java.lang.String,java.lang.String,java.util.Date)
location: class Nachricht
    nachrichten.add(new Nachricht(prioritaet, msg, new Date()));
3 errors

import java.util.ArrayList;
import java.util.Collections;
import java.util.Date;
import java.util.List;

class Nachricht implements Comparable<Nachricht>{
    enum prioritaetTyp {Niedrig, Mittel, Hoch};

    private prioritaetTyp prioritaet;
    private String nachricht;
    private Date uhrzeit;

    public Nachricht(prioritaetTyp prioritaet, String nachricht, Date uhrzeit){
        this.prioritaet = prioritaet;
        this.nachricht = nachricht;
        this.uhrzeit = uhrzeit;
    }
    @Override
    public int compareTo(Nachricht o) {
        if (this == o){
            return 0;
    } else if (this.prioritaet. > o.prioritaet) {
      return 1;
    } else if (this.prioritaet < o.prioritaet) {
      return -1;
    } else {
      return this.uhrzeit.compareTo(o.uhrzeit);
    }
  }

  @SuppressWarnings("deprecation")
  @Override
  public String toString() {
    return "Prio:" + prioritaet + " - " + nachricht + " - " + uhrzeit.getHours() + ":" + uhrzeit.getMinutes();
  }
}

class NachrichtService {

  List<Nachricht> nachrichten;
  private int neachstePos = 0;

  public NachrichtService() {
    nachrichten = new ArrayList<Nachricht>();
  }

  public void addNachricht(String prioritaet, String msg) {
    nachrichten.add(new Nachricht(prioritaet, msg, new Date()));
  }

  public Nachricht getNaechsteNachricht() {
    int pos = neachstePos;
    neachstePos++;
    return nachrichten.get(pos);
  }

  public void sortAndPrintNews() {
    Collections.sort(nachrichten);
    for (Nachricht n : nachrichten) {
      System.out.println(n);
    }
  }
}

public class Tester {

  public static void main(String[] args) {
    NachrichtService nservice = new NachrichtService();
    nservice.addNachricht("Hoch", "N 1");
    nservice.addNachricht("Hoch", "N 2");
    nservice.addNachricht("Niedrig", "N 3");
    nservice.addNachricht("Mittel", "N 5");
    nservice.addNachricht("Mittel", "N 6");
    nservice.addNachricht("Niedrig", "N 7");
    nservice.addNachricht("Niedrig", "N 8");
    nservice.addNachricht("Mittel", "N 9");

    nservice.sortAndPrintNews();
  }
}

Wie kann man den Kode zum laufen bekommen?

 

[eRaaaa]

Es haben dir doch jetzt schon mind. 2 Personen gesagt dass in der Zeile 22

    } else if (this.prioritaet. > o.prioritaet) {

ein Punkt enthalten ist der da nichts zu suchen hat. Ausserden kannst du > < nicht auf Enumeration anwenden....sie haben allerdings selbst eine compareTo Methode die du benutzen kannst in dem Fall !

Ausserdem (wenn ich das richtig sehe) will dein Konstruktor als ersten Parameter einen prioritaetsTypen(enum) und keinen String so wie du es aber in addNachricht tust

  public void addNachricht(String prioritaet, String msg) {
    nachrichten.add(new Nachricht(prioritaet, msg, new Date()));  //blub ?!?!?
  }

 

[flashdog]

Danke, den Punkt habe ich nach dem kompilieren eingefügt um mir die Liste der möglichen Methoden anzuzeigen zu lassen und habe es dann vergessen zu löschen.

Tester.java:47: cannot find symbol
symbol  : class prioritaetTyp
location: class NachrichtService
  public void addNachricht(prioritaetTyp prioritaet, String msg) {
1 error

import java.util.ArrayList;
import java.util.Collections;
import java.util.Date;
import java.util.List;

class Nachricht implements Comparable<Nachricht>{
    enum prioritaetTyp {Niedrig, Mittel, Hoch};

    private prioritaetTyp prioritaet;
    private String nachricht;
    private Date uhrzeit;

    public Nachricht(prioritaetTyp prioritaet, String nachricht, Date uhrzeit){
        this.prioritaet = prioritaet;
        this.nachricht = nachricht;
        this.uhrzeit = uhrzeit;
    }
    @Override
    public int compareTo(Nachricht o) {
        if (this == o){
            return 0;
    } else if (this.prioritaet.compareTo(o.prioritaet) == 1) {
      return 1;
    } else if (this.prioritaet.compareTo(o.prioritaet) == -1) {
      return -1;
    } else {
      return this.uhrzeit.compareTo(o.uhrzeit);
    }
  }

  @SuppressWarnings("deprecation")
  @Override
  public String toString() {
    return "Prio:" + prioritaet + " - " + nachricht + " - " + uhrzeit.getHours() + ":" + uhrzeit.getMinutes();
  }
}

class NachrichtService {

  List<Nachricht> nachrichten;
  private int neachstePos = 0;

  public NachrichtService() {
    nachrichten = new ArrayList<Nachricht>();
  }

  public void addNachricht(prioritaetTyp prioritaet, String msg) {
    nachrichten.add(new Nachricht(prioritaet, msg, new Date()));
  }

  public Nachricht getNaechsteNachricht() {
    int pos = neachstePos;
    neachstePos++;
    return nachrichten.get(pos);
  }

  public void sortAndPrintNews() {
    Collections.sort(nachrichten);
    for (Nachricht n : nachrichten) {
      System.out.println(n);
    }
  }
}

public class Tester {

  public static void main(String[] args) {
    NachrichtService nservice = new NachrichtService();
    nservice.addNachricht("Hoch", "N 1");
    nservice.addNachricht("Hoch", "N 2");
    nservice.addNachricht("Niedrig", "N 3");
    nservice.addNachricht("Mittel", "N 5");
    nservice.addNachricht("Mittel", "N 6");
    nservice.addNachricht("Niedrig", "N 7");
    nservice.addNachricht("Niedrig", "N 8");
    nservice.addNachricht("Mittel", "N 9");

    nservice.sortAndPrintNews();
  }
}

Wie bekommen ich nur den letzten Bug gefixt?

 

[eRaaaa]

Zeile 47 -->

 public void addNachricht(Nachricht.prioritaetTyp prioritaet, String msg) {

Zeilen 69-76...willst du da nun einen String oder die enum-Konstante übergeben ??
Da du das ja jetzt zu enum geändert hast müsste es beispielsweise

	nservice.addNachricht(Nachricht.prioritaetTyp.Niedrig, "N 1");

heißen.

Oder du belässt es bei String, dann müsstest du aber wieder die Methode addNachricht anpassen

    public void addNachricht(String prioritaet, String msg) {
	nachrichten.add(new Nachricht(Nachricht.prioritaetTyp.valueOf(prioritaet), msg, new Date()));
    }

Was du nun willst, musst du entscheiden

 

[flashdog]

Danke, der Code funktioniert, aber das sortieren klappt noch nicht ganz:

Prio: Niedrig - N 1 - 19:35
Prio: Mittel - N 3 - 19:35
Prio: Hoch - N 2 - 19:35
Prio: Niedrig - N 5 - 19:35
Prio: Mittel - N 7 - 19:35
Prio: Hoch - N 6 - 19:35

d.h. wie könnte man die Prioritäten sortieren?

Der aktuelle code sieht wie folgt aus:

import java.util.ArrayList;
import java.util.Collections;
import java.util.Date;
import java.util.List;

class Nachricht implements Comparable<Nachricht>{
    enum prioritaetTyp {Niedrig, Mittel, Hoch};

    private prioritaetTyp prioritaet;
    private String nachricht;
    private Date uhrzeit;

    public Nachricht(prioritaetTyp prioritaet, String nachricht, Date uhrzeit){
        this.prioritaet = prioritaet;
        this.nachricht = nachricht;
        this.uhrzeit = uhrzeit;
    }

    @Override
    public int compareTo(Nachricht o) {
        if (this == o){
            return 0;
    } else if (this.prioritaet.compareTo(o.prioritaet) == 1) {
      return 1;
    } else if (this.prioritaet.compareTo(o.prioritaet) == -1) {
      return -1;
    } else {
      return this.uhrzeit.compareTo(o.uhrzeit);
    }
  }

  @SuppressWarnings("deprecation")
  @Override
  public String toString() {
    return "Prio: " + prioritaet + " - " + nachricht + " - " + uhrzeit.getHours() + ":" + uhrzeit.getMinutes();
  }
}

class NachrichtService {

  List<Nachricht> nachrichten;
  private int neachstePos = 0;

  public NachrichtService() {
    nachrichten = new ArrayList<Nachricht>();
  }

  public void addNachricht(Nachricht.prioritaetTyp prioritaet, String msg) {
    nachrichten.add(new Nachricht(prioritaet, msg, new Date()));
  }

  public void sortAndPrintNews() {
    Collections.sort(nachrichten);
    for (Nachricht n : nachrichten) {
      System.out.println(n);
    }
  }
}

public class Tester {

  public static void main(String[] args) {
    NachrichtService nservice = new NachrichtService();
    nservice.addNachricht(Nachricht.prioritaetTyp.Niedrig, "N 1");
    nservice.addNachricht(Nachricht.prioritaetTyp.Hoch, "N 2");
    nservice.addNachricht(Nachricht.prioritaetTyp.Mittel, "N 3");
    nservice.addNachricht(Nachricht.prioritaetTyp.Niedrig, "N 5");
    nservice.addNachricht(Nachricht.prioritaetTyp.Hoch, "N 6");
    nservice.addNachricht(Nachricht.prioritaetTyp.Mittel, "N 7");

    nservice.sortAndPrintNews();
  }
}

 

[nrg]

Die compareTo() anpassen (wegen deinem enum)
z.b.:

	@Override
	public int compareTo(Nachricht o) {
		if (this == o){
			return 0;
		} else if (this.prioritaet.ordinal() > o.prioritaet.ordinal()) {
			return 1;
		} else if (this.prioritaet.ordinal() < o.prioritaet.ordinal()) {
			return -1;
		} else {
			return this.uhrzeit.compareTo(o.uhrzeit);
		}
	}

das sortiert dir erst nach Priorität dann nach Uhrzeit (Sortierreihenfolge ist Enumreihenfolge von links nach rechts)

 

[flashdog]

Danke, Ich glaube es hat funktioniert denn ich bekomme diese Ausgabe:

Prio: Niedrig - N 1 - 10:12:13:14
Prio: Niedrig - N 5 - 10:12:13:14
Prio: Mittel - N 3 - 10:12:13:14
Prio: Mittel - N 7 - 10:12:13:14
Prio: Hoch - N 2 - 10:12:13:14
Prio: Hoch - N 6 - 10:12:13:14

Ich habe GregorianCalendar aktualisiert, aber ich bekomme falsche Uhrzeit angezeigt. Was mache ich falsch?

Warum ist else { return this.uhrzeit.compareTo(o.uhrzeit);} nicht in der ersten if Bedingung drin?

import java.util.ArrayList;
import java.util.Collections;
import java.util.GregorianCalendar;
import java.util.List;

class Nachricht implements Comparable<Nachricht>{
    enum prioritaetTyp {Niedrig, Mittel, Hoch};

    private prioritaetTyp prioritaet;
    private String nachricht;
    private GregorianCalendar  uhrzeit;

    public Nachricht(prioritaetTyp prioritaet, String nachricht, GregorianCalendar uhrzeit){
        this.prioritaet = prioritaet;
        this.nachricht = nachricht;
        this.uhrzeit = uhrzeit;
    }

    @Override
    public int compareTo(Nachricht o) {
      /*
       -1 represent o that is less than the one we’re checking against.
        0 represent o of equal values
        1 represent o that is more than the one we’re checking.
       */
        if (this == o){
            return 0;
        } else if (this.prioritaet.ordinal() > o.prioritaet.ordinal()) {
            return 1;
        } else if (this.prioritaet.ordinal() < o.prioritaet.ordinal()) {
            return -1;
        } else {
            return this.uhrzeit.compareTo(o.uhrzeit);
        }
  }

  @Override
  public String toString() {
    return "Prio: " + prioritaet + " - " + nachricht + " - " + uhrzeit.HOUR
            + ":" + uhrzeit.MINUTE + ":" + uhrzeit.SECOND + ":" + uhrzeit.MILLISECOND;
  }
}

class NachrichtService {

  List<Nachricht> nachrichten;
  private int neachstePos = 0;

  public NachrichtService() {
    nachrichten = new ArrayList<Nachricht>();
  }

  public void addNachricht(Nachricht.prioritaetTyp prioritaet, String msg) {
    nachrichten.add(new Nachricht(prioritaet, msg, new GregorianCalendar()));
  }

  public void sortAndPrintNews() {
    Collections.sort(nachrichten);
    for (Nachricht n : nachrichten) {
      System.out.println(n);
    }
  }
}

public class Tester {

  public static void main(String[] args) {
    NachrichtService nservice = new NachrichtService();
    nservice.addNachricht(Nachricht.prioritaetTyp.Niedrig, "N 1");
    nservice.addNachricht(Nachricht.prioritaetTyp.Hoch, "N 2");
    nservice.addNachricht(Nachricht.prioritaetTyp.Mittel, "N 3");
    nservice.addNachricht(Nachricht.prioritaetTyp.Niedrig, "N 5");
    nservice.addNachricht(Nachricht.prioritaetTyp.Hoch, "N 6");
    nservice.addNachricht(Nachricht.prioritaetTyp.Mittel, "N 7");

    nservice.sortAndPrintNews();
  }
}

 

[SlaterB]

HOUR usw. sind statische Konstanten, rufe cal.get(Calendar.HOUR) auf usw.,
bzw. hole dir mit getTime() ein Date-Objekt und gib das direkt aus oder per SimpleDateFormat

> Warum ist else { return this.uhrzeit.compareTo(o.uhrzeit);} nicht in der ersten if Bedingung drin?

?
im ersten if steht eine Bedingung und Code danach, wieso sollte anderer Code mit 'drin' sein?

 

[nrg]

Warum ist else { return this.uhrzeit.compareTo(o.uhrzeit);} nicht in der ersten if Bedingung drin?

die Frage ist - wie Slater schon meinte - ein wenig falsch formuliert aber durchaus berechtigt (vorausgesetzt ich verstehe sie richtig - "Warum ist return this.uhrzeit.compareTo(o.uhrzeit); nicht im ersten if-body drin?")

Weil im ersten if auf referenzielle Gleichheit geprüft wird. Wenn die beiden Objekte die gleiche Referenz haben handelt es sich um exakt das selbe Objekt. Dh this.uhrzeit.compareTo(o.uhrzeit) würde auch 0 zurückgeben, weil die Uhrzeit natürlich auch die Gleiche wäre.

 

[DaveX]

Wenn die compare Methode fetig ist würde ich vorschlagen eine PriorityQueue (Java 2 Platform SE 5.0) zu benutzen.

Da werden alle Nchrichten automatisch je nach der funktionsweise der compareTo Funktion sortiert, und du musst statt getNext die pull Funktion aufrufen.

Das wäre dann priorisierte Nachrichtenwarteschlange - ein wenig eleganter als alles per Hand zu sortieren.

@EDIT:

zu Sortierung ist Date ein wenig umständlich - so konnte man mit System.currentTimeMillis() eine long haben die man besser vergleichen kann und anschliessend zu Date für die Ausgabe parsen mit:
DateFormatUtils.formatUTC(System.currentTimeMillis(), "MM/dd/yy HH:mm")

man braucht aber diesen import:
import org.apache.commons.lang.time.DateFormatUtils

 

[flashdog]

Danke für die Erläuterung für "return this.uhrzeit.compareTo(o.uhrzeit)"

Habe die Methode toString wie folgt geändert

  @Override
  public String toString() {
    GregorianCalendar cal = new GregorianCalendar();

    return "Prio: " + prioritaet + " - " + nachricht + " - " + cal.get(uhrzeit.HOUR)
            + ":" + cal.get(uhrzeit.MINUTE) + ":" + cal.get(uhrzeit.SECOND)
            + ":" + cal.get(uhrzeit.MILLISECOND);
  }

aber Netbeans gibt mir diese Meldung "Accessing static field HOUR" und die Ausgabe vom Programm sieht wie folgt aus:

Prio: Niedrig - N 1 - 11:32:13:825
Prio: Niedrig - N 5 - 11:32:13:843
Prio: Mittel - N 3 - 11:32:13:844
Prio: Mittel - N 7 - 11:32:13:846
Prio: Hoch - N 2 - 11:32:13:847
Prio: Hoch - N 6 - 11:32:13:848

Leider weiss ich nicht wie DateFormatUtils.formatUTC(System.currentTimeMillis(), "MM/dd/yy HH:mm") benutzt.

Wie bekommt man die Meldung "Accessing static field HOUR" korrigiert?

 

[eRaaaa]

Wie bekommt man die Meldung "Accessing static field HOUR" korrigiert?

------->

HOUR usw. sind statische Konstanten, rufe cal.get(Calendar.HOUR) auf usw.,

<-------

(allerdings hat doch die Klasse ein Attribut uhrzeit was ein GregorianCalendar-Objekt hält, wieso also ein neues erstellen? (hab allerdings dne Thread nicht ganz verfolgt))

 

[flashdog]

Danke. Habe es wie folgt korrigiert

  @Override
  public String toString() {
     return "Prio: " + prioritaet + " - " + nachricht + " - " + uhrzeit.get(Calendar.HOUR)
            + ":" + uhrzeit.get(Calendar.MINUTE) + ":" + uhrzeit.get(Calendar.SECOND)
            + ":" + uhrzeit.get(Calendar.MILLISECOND);
  }

 

[Onkel Hatti]

Falsch gedacht, sorry

 

[flashdog]

Wie könnte ich es korrigieren?

 

[SlaterB]

funktioniert etwas im Moment nicht?
bitte aktuellen Stand + Problem