problem mit verketteter liste

Status
Nicht offen für weitere Antworten.

ramonl

Mitglied
wir sollten ne doppelt verkettete liste schreiben, u.a. mit ner inneren Klasse, die man nutzt um ein Elemenet zu erstellen, welches auch gleichzeitig auf seinen vorgänge rund Nachfolger zeigt.


Soweit zu gut alles, das meiste bisher geht auch, nur folgende komische Situation nicht:


Wenn ich folgende Elemente in der Liste habe:

a
b
c
d

dann kann ich problem los mit meiner insertAfter Methode, nach b z.b. "bb" was einfügen, die Liste ausgeben lasen, das letzte Element löschen lassen, wieder ausgeben lassen, dann steht korrekt

a
b
bb
c

da.

(Nach dem letzten Element einfügen kann noch nicht klappen bei mir), jedoch wenn ich von meiner ausgangsliste nach c, also dem vorletzten Element etwas einfüge, und die liste ausgebe, ist es zunächst korrekt, doch wenn ich dann das letzte element löschen lassen, werden die letzten beiden, also das vona nfang an letzte, und das eingefügte element gelöscht.

Ich weiss absolut nicht wo der fehler liegt, vllt. findet ihn ja jemand.


Hier die relevanten programmierteile:
Code:
public class DoubleLinkedList implements IDoubleLinkedList{

   ListElement start;     // The start of the double linked list
   ListElement end;       // The end of the double linked list
   ListElement help;    // an element used for insertions

// innere Klasse:

 public class ListElement
   {
	  ListElement before;  // The Element of the List BEFORE this one
	  ListElement after;   // The Element of the List AFTER this one
	  Object entry;        // The entry wich is saved in each List Element
      
	  /**
	   * The constructor with a parameter
	   * @param newEntry The entry wich is to be saved in the element
	   */
	  ListElement(Object newEntry)
	  {
		 entry = newEntry;      
	  }
   }
   


 public void insertAfter(Object after, Object anObject)
   {
	  if(anObject == null || after == null)
		 throw new IllegalArgumentException("Es wird kein Object übergeben");
         
	  ListElement element = new ListElement(anObject);
	  ListElement search = start;
      
	  while(search.entry != after || search == end)   
	  {
		 search = search.after;    
	  }
	  	 
	  help = search;
	  element.after = search.after;
	  element.before = search;
	  search.after = element;
	  help.after.before = element;
   }


 public Object removeLast()
   {
	  if(start == null)
	   throw new NullPointerException("Die Liste ist leer.");
         
	
	  help = end;
	  end = end.before;
	  end.after = start;
	
	  return help;
   }


 public void output()
   {
	  if(start == null)
		 System.out.println("Die Liste ist leer.");
	  else
	  {
		 ListElement outputElement;
		 outputElement = start;
		 do
		 {
			System.out.println(outputElement.entry);
			outputElement = outputElement.after;
		 }
		 while(outputElement != end.after);
    
	  }
 
//Schönere Fassung:

Wir sollen eine doppelt verkettete Liste schreiben, u.a. mit einer inneren Klasse, welche ein Element darstellt, welches auf seinen Vorgänger und Nachfolger zeigt.

Jedoch habe ich folgendes Problem:

Wenn ich folgende Elemente in der Liste habe:

a
b
c
d

dann kann ich problemlos mit meiner "insertAfter()"-Methode, nach "b" ein neues Element ("bb") in die Liste einfügen.
Anschließend lösche ich das letzte Element und erhalte folgende - korrekte - Ausgabe:

a
b
bb
c
d //Das fehlte auch.



Das Einfügen nach dem letzten Element habe ich noch nicht implementiert, jedoch wenn ich in meiner Ausgangsliste nach dem vorletzten Element("c") etwas einfüge und die Liste ausgebe, ist die Ausgabe korrekt. Lösche ich jedoch das letzte Element, werden die letzten beiden Elemente gelöscht.


//Unkorrigiert
Ich weiss absolut nicht wo der fehler liegt, vllt. findet ihn ja jemand. //vllt.? Wo lernt man das?

Ok... ein Kommentar von mir. Auch ich habe eine leichte Rechtschreibschwäche, wie man vieleicht an dem ein oder anderen verbliebenen Fehler erkennen kann. ABER WO GEHST DU ZU SCHULE??????

Nach dem ich das jetzt geschafft habe, kann ich jetzt über deine Frage nachdenken.
 
ich ann bisher keinen fehler erkennen.

eines ist mir aufgefallen, du hast eine ringverkettung (erkenne ich aus deiner löschen methode, da du das letze element wieder mit dem aller ersten verbindest). daraus ergibt sich ein problem:
Code:
     while(search.entry != after || search == end)    
     { 
       search = search.after;    
     }

das ganze könnte in einer endlosschleife enden, wenn das element nicht vorhanden ist (korrigiere mich wenn ich das jetzt falsch gesehen habe). des weiteren bin ich mir garnicht sicher, dass du überhaupt eine ringverkettung nutzen wolltest.
Code:
//hier nochmal der teil aus dem ich schliesse, das es eine ringverkettung ist
end.after = start;
sollte sie doch gewollt sein fehlt da noch die verkettung des startes mit dem ende:
Code:
start.before=end;

wenn ich aber das output anschaue, könnte doch eine ringverbindung gewollt sein
Code:
while(outputElement != end.after);

leider kann ich sonst keinen fehler entdecken.


kleiner tip zum letzen element einfügen:
musst nur die zeiger des letzen elements auf das neue setzen und dann die zeiger richtig umbiegen. zum schluss noch das neue als end definieren und fertig. (umsetzung liegt an dir)
 
Hab es in der FH alles gelöst bekommen, waren noch viele fehler drin, trotzdem danke für die angebotene Hilfe !


Was soll die Frage wo ich zur Schule gehe ? Auf eine Fachhochschule, studiere dort angewandte Informatik.
 
ramonl hat gesagt.:
Was soll die Frage wo ich zur Schule gehe ? Auf eine Fachhochschule, studiere dort angewandte Informatik.

Grüß dich, Fast-Nachbar!
Die Frage zielte eher darauf ab, wo du Deutschunterricht hattest. Sag's ihm besser nicht.. 😉
 
derHai hat gesagt.:
Auch ich habe eine leichte Rechtschreibschwäche, wie man vieleicht an dem ein oder anderen verbliebenen Fehler erkennen kann.

dann habe ich auch eine, denn meine postings sehen meist schlimmer aus und ich konnte bei dir nix falsches finden. bildest du dir das vieleicht auch nur ein ?
 
Status
Nicht offen für weitere Antworten.

Neue Themen


Zurück
Oben