Threads linkedlist/multi-thread problem

[Erinas]

Ok hallo erst mal
Ich habe da mal ein kleines Problem und hoffe ihr könnt mir da evtl. weiterhelfen

Ich habe ein Server Prog. geschrieben das die Client connections mittels Multi-Threading handlet
alles soweit kein Problem
nun mussten irgendwann die Client-Threads auf serverweite Daten zugreifen, dies habe ich dann
naiv wie ich bin mit einem Core-Thread gelöst in dem die Clients ihren s***** der reihe nach in ne linkedlist stopfen.
Der Core macht nix anderes als:

public void run() {
		while (running) {
			if (!orderList.isEmpty()) {
				execute(orderList.getFirst(), orderListThreadName.getFirst());
				orderListThreadName.removeFirst();
				orderList.removeFirst();
			}
		}
	}

klappt auch eigentlich...
nur vor kurzem ist mir aufgefallen das
der Core-Thread in unregelmäßigen Abständen abs*****t
mal nach 15 Min. mal nach 6 Stunden
alle 5 Min. bekommt der Core eine Anweisung die er ausführt
aber irgendwann kommt stattdessen ne java.util.NoSuchElementException wenn er getfirst() versucht.

Meine frage nun:
hat einer ne Idee woran das liegen kann
oder geiler noch: hat einer ne bessere Lösung xD?

ach und sorry für Ausdrucksweise und evtl. andere Missetaten
bin leider nicht der Profi unter den hobbycodern xD

 

[Gucky]

Wenn kein so ein Element vorhanden ist, wird die Liste leer sein. Da es ein bisschen dauern kann, bis der Fehler auftritt, lass dir mal alle Schritte in ein Log schreiben. Einschließlich der Zustände der Objekte. Also auch die Länge der Liste. Irgendwo leerst du die Liste oder instanziierst sie neu.

 

[Erinas]

Hm ... war vll keine so dumme Idee
hab das mal getestet und bin auf den Trichter gekommen das
das ein Timing Fehler sein könnte :O

schau das ist die Methode die die Client-Threads nutzen um die Linkedlist zu befüllen

	public static synchronized void addOrder(String order,String id){
		orderList.add(order);
		orderListThreadName.add(id);
	}

die exeption sagt aber nur getfirst() und Zeile 33 aber nicht welches der getfirsts

    public void run() {
            while (running) {
                if (!orderList.isEmpty()) {
zeile 33 -->     execute(orderList.getFirst(), orderListThreadName.getFirst());
                    orderListThreadName.removeFirst();
                    orderList.removeFirst();
                }
            }
        }

dachte mir vll liegt es ja daran das zur Laufzeiter der Client-Thread grade die Order hinzugefügt hat aber noch nicht den Thread-Name somit wäre orderListThreadName.getFirst()) das Problem

hab das jetzt mal so abgeändert:

    public void run() {
            while (running) {
änderung-->     if (!orderList.isEmpty() && !orderListThreadName.isEmpty()) {
                    execute(orderList.getFirst(), orderListThreadName.getFirst());
                    orderListThreadName.removeFirst();
                    orderList.removeFirst();
                }
            }
        }

was meinst du könnte des das Problem gewesen sein ôo?
klingt irgendwie zu einfach... hab das ganze jetzt nochmal gestartet
mach schauen was daraus wird...

 

[Gucky]

Naja das Timing? Also wenn kein Client was reingetan hat, dann ist auch nichts drin.
Mit deiner Lösung hast du die Exception verhindert aber nicht das Rätsel um die leere Liste gelöst.

Und guck dir mal Stacks und Queues an. Vielleicht sind die was für dich. Ich glaube, eine LinkedList ist hier ein bisschen zu groß.