Generic und Listen

[GroovyChuc]

Hey hab folgendes Problem:

Für den untenstehenden Code hab ich mir eine Testklasse geschrieben, und leider kommt bei der Methode getStudentsInStudgang nie etwas auf der Konsole und die Klasse getStudent liefert immer null zurück. Ich vermute, dass es an der contains-Methode liegt, da in der if-Abfrage wahrscheinlich immer false rauskommt. Ist das so, und wie kann ich es besser machen?

package verwaltung;

import java.io.PrintStream;
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;

public class StudentVerwaltung implements IStudentVerwaltung {

	List<IStudent> studentList = new ArrayList<IStudent>();
	List<IStudent> list2 = new ArrayList<IStudent>();

	@Override
	public void printStudents() {
		for (IStudent s1 : studentList) {
			System.out.printf("%s %s %s %s", s1.getSurname(), s1.getName(),
					s1.getSemester(), s1.getCourse());
			System.out.println();
		}
	}

	@Override
	public List<IStudent> getStudentsInStudgang(String course) {
		for (IStudent s1 : studentList) {
			if (studentList.contains(course)) {
				list2.add(s1);
				System.out.printf("%s %s %s %s", s1.getSurname(), s1.getName(),
						s1.getSemester(), s1.getCourse());
			}
		}
		return list2;
	}
	
	@Override
	public List<IStudent> getStudentsInSemester(int sem) {
		// TODO Auto-generated method stub
		return null;
	}

	@Override
	public IStudent createStudent(String name, String surname, String course)
			throws StudentAlreadyExistException {
		if (studentList.contains(name) && studentList.contains(surname)) {
			throw new StudentAlreadyExistException();
		}
		Student s = new Student(name, surname, 1, course);
		studentList.add(s);
		return s;
	}

	@Override
	public IStudent getStudent(String name, String surname) {

		for (IStudent s1 : studentList) {
			if (studentList.contains(name) && studentList.contains(surname)) {
				return s1;
			}
		}

		return null;
	}

}

 

[faetzminator]

Wie oft rufst du [c]createStudent()[/c] auf, mit welchen Parametern?

 

[GroovyChuc]

ackage verwaltung;

public class StudentTest  {
	
	public static void main(String[] args) {
		StudentVerwaltung sv = new StudentVerwaltung();
		try {
			sv.createStudent("Hans", "Wurst", "Kurs1");
			sv.createStudent("Max", "Mustermann", "Kurs2");
			sv.createStudent("Franz", "Meyer", "Kurs1");
			System.out.println(sv.getStudent("Max", "Mustermann"));
		} catch (StudentAlreadyExistException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
	sv.printStudents();
	sv.getStudentsInStudgang("KDI");
	}

}

Ausgabe auf der Konsole:
null
Wurst Hans 1 Kurs1
Mustermann Max 1 Kurs2
Meyer Franz 1 Kurs1

 

[Spacerat]

...Erst mal den Ganzen Unsinn vergessen, wo du Birnen mit Äpfeln besser gesagt Studenten mit Zeichenketten vergleichst. Also Dinge wie "if(studentList.contains(course)" z.B.

 

[Landei]

In [c]getStudentsInStudgang[/c] packst du alle "passenden" Studenten in die Member-Variable [c]list2[/c]. Das heißt, dass beim zweiten Aufruf der Methode die Werte vom ersten Aufruf noch in [c]list2[/c] gespeichert sind. Schlimmer noch: Da wo die Liste nach dem ersten Aufruf verwendet wird, ist diese Änderungen auch zu sehen - es handelt sich ja immer um das gleiche Objekt.

Du solltest also [c]list2[/c] unbedingt zu einer lokalen Variable machen.

Prinzipiell solltest du dir über den Unterschied zwischen Member-Variablen und lokalen Variablen klar werden, ebenso die Arbeitsweise von Referenzen. Ich kann aus Erfahrung sagen, dass Fehler dieser Art in einem größeren Programm wirklich sehr schlecht zu finden sind.

 

[anti-held]

wenn du in deinen Schleifen über deine Studenten iterierst, dann überprüfst du mit deinem

if (studentList.contains(course))

ob die Liste den String course enthält!
Ein vergleich von einem IStudent-Objekt mit einem String, wird immer false ergeben!
Deshalb wird deine Methode getStudentsInStudgang immer nur eine leere Liste zurückgeben.
Ebenso funktionieren deine Vergleiche in den anderen Methoden auch nicht!

in dem Beispiel von getStudentsInStudgang sollte dein Code so aussehen:

if (s1.getCourse().equals(course))

 

[GroovyChuc]

wenn du in deinen Schleifen über deine Studenten iterierst, dann überprüfst du mit deinem

if (studentList.contains(course))

ob die Liste den String course enthält!
Ein vergleich von einem IStudent-Objekt mit einem String, wird immer false ergeben!
Deshalb wird deine Methode getStudentsInStudgang immer nur eine leere Liste zurückgeben.
Ebenso funktionieren deine Vergleiche in den anderen Methoden auch nicht!

in dem Beispiel von getStudentsInStudgang sollte dein Code so aussehen:

if (s1.getCourse().equals(course))

Danke die Abfrage funktioniert jetzt

In getStudentsInStudgang packst du alle "passenden" Studenten in die Member-Variable list2 . Das heißt, dass beim zweiten Aufruf der Methode die Werte vom ersten Aufruf noch in list2 gespeichert sind. Schlimmer noch: Da wo die Liste nach dem ersten Aufruf verwendet wird, ist diese Änderungen auch zu sehen - es handelt sich ja immer um das gleiche Objekt.

Du solltest also list2 unbedingt zu einer lokalen Variable machen.

Prinzipiell solltest du dir über den Unterschied zwischen Member-Variablen und lokalen Variablen klar werden, ebenso die Arbeitsweise von Referenzen. Ich kann aus Erfahrung sagen, dass Fehler dieser Art in einem größeren Programm wirklich sehr schlecht zu finden sind.

Ich verstehe nicht ganz wie ich eine Listenvariable als lokal definieren kann.
Das Problem mit dem zweiten Aufruf würde ich so angehen:

list2 = null;

Damit kann die liste jedesmal neu genutzt werden.
Allerdings kommt bei mir folgende Exeption wenn ich diese Zeile

list2.add(s1);

ausführe:

Exception in thread "main" java.lang.NullPointerException

at verwaltung.StudentVerwaltung.getStudentsInStudgang(StudentVerwaltung.java:28)

at verwaltung.StudentTest.main(StudentTest.java:17)

Hier nochmal der Code:

	@Override
	public void printStudents() {
		for (IStudent s1 : studentList) {
			System.out.printf("%s %s %s %s", s1.getSurname(), s1.getName(),
					s1.getSemester(), s1.getCourse());   <------- Line 17
			System.out.println();
		}
	}

	@Override
	public List<IStudent> getStudentsInStudgang(String course) {
		list2 = null;
		for (IStudent s1 : studentList) {
			if (s1.getCourse().equals(course)) {

				list2.add(s1);         <------- Line 28
			}
		}
		return list2;
	}

 

[SlaterB]

> Damit kann die liste jedesmal neu genutzt werden.

mache das auch, null-Setzen kannst du dir eher sparen:
list2 = new ArrayList<IStudent>();

in eine null-Liste kann nichts eingefügt werden

> Ich verstehe nicht ganz wie ich eine Listenvariable als lokal definieren kann.

die StudentVerwaltung hast du in der main lokal definiert, statt ein statische Attribut zu benötigen,
einfach kein Attribut, normale lokale Variable

Student s hast du in createStudent() lokal definiert ohne ein Attribut zu benötigen,
scheint gar nicht so schwer, oder?

 

[GroovyChuc]

Ok Super.
Jetzt klappt alles
Danke für eure Hilfe.