einfach verkettete Liste, intersect

  • Themenstarter Themenstarter Reap
  • Beginndatum Beginndatum
Status
Nicht offen für weitere Antworten.
R

Reap

Gast
Hallo...ich hab da ein Problem wo ich einfach nicht weiß warum das auftritt.....und das komische ist, wenn ich die gleiche Methode nur ein bißl anders verwende gehts auf einmal...ich bin total fertig...aber seht selbst:

Gegeben Sei eine Liste welche Studenten verwaltet, also Studenten als Knoten besitzt:

Code:
public class Student {

	private String forename;
	private String surname;
	private int matrNr;
	private int[] exercises;
	public Student next;
	....usw.
}

Das Problem tritt jetzt bei der Liste auf und zwar bei der Methode static public StudentSet intersect(StudentSet s1, StudentSet s2){...}

Liste:

Code:
public class StudentSet {
   
	private Student head;
	private Student tail;
	private int nElems;
	
	public StudentSet(){
		head = tail = null;
		nElems = 0;
	}
	
	public void printList(){
		Student cur = head;

		while (cur != null){
			System.out.println(cur.toString());
			System.out.println();
			System.out.println(cur.getName());
		    cur = cur.next;
		}
	}
	
	/**
          * Neuen Studenten hinzufügen (dessen MNr noch nicht in der Menge ist sonst
	 * ignorieren)
          */
	public void addStudent(Student obj){
	     		
		/*if(obj.getMatrNr() == 8456322){
			System.out.println("Hallo");
		}
		*/
		if(!contains(obj.getMatrNr())){
	    	 
			 if(isEmpty()){
				 
				 head = obj;
				 tail = head;
			 }else{
				 Student cur;
				 Student prev;
				 cur = head;
				 prev = null;
				 
				 boolean fixPosition = false;
				 
				 while(cur != null && !fixPosition){
					 if(cur.getSurname().compareTo(obj.getSurname())<0){
						 prev = cur;
						 cur = cur.next;
					 }else{
						 fixPosition = true;
					 } 
				 }
				 
				 if(cur == head){
					 obj.next = head;
					 head = obj;
				 }else{
				     if(cur == null){
				    	 tail.next = obj;
				    	 obj = tail;
				     }else{
				    	 obj.next = cur;
				    	 prev.next = obj;
				     }
				 }  
			 } 
			 nElems++;
	     }
	}

	/**
	 * Neuen Studenten hinzufügen (dessen MNr noch nicht in der Menge ist sonst
	 * ignorieren)
     */ 
	public Student addStudent(String name, int matrNr){
		Student obj = new Student(name,matrNr);
		addStudent(obj);
		return obj;
	}
	
	/** Test, ob Student mit Matrikelnummer matrNr enthalten ist */ 
	public boolean contains(int matrNr){
		Student cur;
		if(!isEmpty()){
			cur = head;
			while(cur != null){
				if(cur.getMatrNr() == matrNr){
					return true;
				}
				cur = cur.next;
			}
		}
		return false;
	}
	
	public Student getHead(){
		return head;
	}
	
	/** Schnittmenge von zwei Studentenmengen */ 
	static public StudentSet intersect(StudentSet s1, StudentSet s2){
		StudentSet s4 = new StudentSet();
		Student cur = s1.getHead();
		while(cur != null){
			if(s2.contains(cur.getMatrNr())){
				//intsctn.addStudent(cur.getName(),cur.getMatrNr());
				s4.addStudent(cur);
			}
			
			cur = cur.next;
		}
		return s4;	
	}
	
	
	//private methods --------------------------------------
	
	private boolean isEmpty(){
		if(head == null){
			return true;
		}
		return false;
	}
	
}

Wenn ich das Ganze jetzt teste geht die Methode intersect nicht korrekt. Also wenn ich folgendes Testszenario gegeben habe:

Code:
                   StudentSet s1 = new StudentSet();
		 Student a = new Student("Hullu Kanne",9456521);
		 Student b = new Student("Gusenhuber Michael",8456322);
		 Student c = new Student("Zen Susi",8256322);
		 Student h = new Student("Hammer Michael",7327242);
		 
		 s1.addStudent(a);
		 s1.addStudent(b);
		 s1.addStudent(c);
		 s1.addStudent(h);
		 
		 StudentSet s2 = new StudentSet();
		 Student d = new Student("Bullu Panne",7432314);
		 Student e = new Student("Gusenhuber Michael",8456322);
		 Student f = new Student("Zen Susi",8256322);
		 Student g = new Student("Bachmaier Christian",6439737);
		 
		 s2.addStudent(d);
		 s2.addStudent(e);
		 s2.addStudent(f);
		 s2.addStudent(g);
		 
		  StudentSet s3 = StudentSet.intersect(s1, s2);
		  s3.printList();

Dann gibt es mir für s3 folgendes aus🙁in verkürzter Form)
Gusenhuber Michael; Hammer Michael;Hullu Kanne; Zen Susi
anstatt:
Gusenhuber Michael;Zen Susi

Wenn ich jetzt aber jetzt in der Methode intersect anstatt
Code:
s4.addStudent(cur);
Code:
s4.addStudent(cur.getName(),cur.getMatrNr())
hischreibe was ja im Grunde genau dasselbe ist dann is plötzlich alles richtig...warum ist das so...was sehe ich da falsch...es muss irgendwas mit cur zu tun haben....

mfg,
Reap
 
zunächst mal, muss es nicht
Code:
}else{
                 if(cur == null){
                    tail.next = obj;
                    tail = obj;

statt

Code:
}else{
                 if(cur == null){
                    tail.next = obj;
                    obj = tail;
heißen? das scheint mir mehr Sinn zu machen und mit deiner Version schaffe ich es nicht einmal Liste 2 aufzubauen,

----------

ansonsten liegt der Fehler darin, dass du versuchst Elemente einer anderen Kette in die neue Kette einzubauen,
da fügst du das erste Element Gusenhuber ein und was hängt da hinten dran? der Rest von Kette 1,
also ist die gesamte Kette 1 mit einem Schlag in Kette 4 drin,

Abhilfe: korrekt so wie es zufällig schon machst: einfach ein neues Objekt erzeugen,
ansonsten hilft auch am Anfang von add der Befehl obj.next = null;
da kann einem dann die vorherige Zugehörigkeit nicht mehr stören,
dummerweise wird dabei die alte Kette 1 zerstört, insofern ist
addStudent(cur.getName(),cur.getMatrNr()) wohl die beste Lösung,


lerne daraus: eine verkettete Liste wie das Beispiel LinkedList gibt nie ihre Kettenglieder nach außen,
nimmt nie Kettenglieder von außen auf, sondern macht das alles intern,
da gibts Container die mit anderen Containern verlinkt ist,
Student-Objekte sind nur deren Dateninhalt und nicht selber verkettet,

wenn man das so trennt, dann kann es konzeptionell gar nicht zu einem solchen Fehler kommen!
(man kann natürlich noch viele andere Fehler einbauen 😉 )
 
Ok...danke...(dass ich da nicht selbst draufgekommen bin 🙂)..und danke für den kleinen aber blöden Fehler in der addMethode.... :wink:

Ich weiß aber nicht recht was du mit deinem Tipp am Schluss meinst:

lerne daraus: eine verkettete Liste wie das Beispiel LinkedList gibt nie ihre Kettenglieder nach außen,
nimmt nie Kettenglieder von außen auf, sondern macht das alles intern,
da gibts Container die mit anderen Containern verlinkt ist,
Student-Objekte sind nur deren Dateninhalt und nicht selber verkettet,


Meinst du dass ich eine eigenständige Node Klasse erstellen soll also z.b.:

Code:
class Node{

Student a;
Node next;

public Node(Student a){
  this.a = a;
  next = null;
}


}

was du ja hier erwähnst(Student-Objekte sind nur deren Dateninhalt und nicht selber verkettet,)...aber aud deinen anderen Sätzen werd ich noch nicht so recht schlau....denn ich will das Bsp. natürlich nicht ungeschickt gemacht abgeben.....

mfg,
Reap
 
Weiß keiner was

lerne daraus: eine verkettete Liste wie das Beispiel LinkedList gibt nie ihre Kettenglieder nach außen,
nimmt nie Kettenglieder von außen auf, sondern macht das alles intern,
da gibts Container die mit anderen Containern verlinkt ist,
Student-Objekte sind nur deren Dateninhalt und nicht selber verkettet,


konkret heißt...bzw. was ich da tun soll um mein Programm zu verbessern oder hab ich irgendwas unklar gefragt?

mfg,
Reap
 
ich war ne Weile nicht da, aber ich finde meine Sätze ziemlich deutlich 😉

schau dir die Klasse LinkedList an (wenn du die nicht kennst ist das natürlich schwerer zu verstehen),
da kann man beliebige Objekte reintun und wieder herausnehmen,

das ist toll, schrecklich wäre es wenn man für jede eigene Klasse die Listenfunktionen (wie kompliziert die sind merkst du ja selber) neu schreiben müsste..

das ist schrecklich aber auch gar nciht nötig,
wie du mit der Klasse Node richtig andeutest kann man sich Behälterklassen schaffen,
natürlich nicht mit einem Konstruktor der nach Student verlangt,
dann müsste man die -NodeKlasse ja auch für jede eigene Klasse anpassen und hätte wenig gewonnen,

Code:
class Node{

Object a;
Object next;

public Node(Object a){
  this.a = a;
  next = null;
}


}

ist besser und für beliebige Objekte verwendbar,

-------

aber mach dir nicht allzu große Gedanken, vielleicht ist die Aufgabe genau so gemeint speziell für die Studenten die Liste zu bauen,
ist nicht abwegig,
im Zweifel nachfragen
(Liste nur für Studenten oder allgemeine Liste für beliebige Objekte)
 
Status
Nicht offen für weitere Antworten.

Neue Themen


Zurück
Oben