util.Scanner problem

[Johnny919]

ich will mir die Laufende prozesse hollen und die in einer list abspeichern
das hollen mache ich so :

scanner = new Scanner(new InputStreamReader(Runtime.getRuntime().exec("tasklist.exe").getInputStream()));

aber immer wenn ich das dan in die List abspeichern will bekomm ich an der makierten stelle eine Nullpointer weis aber nicht wieso hab auch schon gedebugt und eigentlich ist das nicht lehr der scanner
hier noch mal der ganze code :

import java.io.IOException;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;


import de.grab.taskmanager.domain.Prozess;

public class ProzessService {
	private List<Prozess> prozesses = new ArrayList<Prozess>();
	private Prozess pro;
	private int i = 0;
	private Scanner scanner;

	public List<Prozess> getProzessList() throws IOException{
		scanner = new Scanner(new InputStreamReader(Runtime.getRuntime().exec("tasklist.exe").getInputStream()));
		prozesses.clear();
		while(scanner.hasNext())
		{
			if(i<5)
			{
				scanner.next();
			}
			if(i>5 && i<=10)
			{
				scanner.next();
			}
			if(i>10)
			{
				if(i==11){
					pro.setName(scanner.next()); <-----hier ist die Nullpointer wo er das erstemal den scanner .next() setzen will!!!
					pro.setName(pro.getName()+" "+scanner.next());
					pro.setName(pro.getName()+" "+scanner.next());
					pro.setPid(Integer.parseInt(scanner.next()));
					pro.setDiscription(scanner.next());
					pro.setMeetingnumber(Integer.parseInt(scanner.next()));
					pro.setAtticuse(scanner.next()+" "+scanner.next());
				}
				else{
					pro.setName(scanner.next());
					pro.setPid(Integer.parseInt(scanner.next()));
					pro.setDiscription(scanner.next());
					pro.setMeetingnumber(Integer.parseInt(scanner.next()));
					pro.setAtticuse(scanner.next()+" "+scanner.next());

				}
			}
			i++;
			prozesses.add(pro);
		}
		return prozesses;
	}
}

hoffe ihr könnt mir helfen und schonmal im vorraus ein großes dankeschön
Gruß John

 

[SlaterB]

> pro.setName(scanner.next());

vielleicht siehst du so klarer:

String st = scanner.next();
pro.setName(st);

in welcher Zeile die Exception?
war das ein schwerer Trick?

 

[Johnny919]

naja nicht ganz ^^ aber trotzdem danke weil ich durch dich auf meinen fehler gekommen bin
hatte oben ja :

 private Prozess pro;

das muss aber so sein ^^:

 private Prozess pro = new Prozess();

 

[SlaterB]

so war es genau mein Ziel

 

[Johnny919]

achsooooo na dann wars PERFEKT
aber jetzt geht es ich bekomme jetzt wenn ich das ausführe aber iwie nur den selben process 20 mal :
[tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K
, tasklist.exe 4592 Console 1 4.632 K

irgendwie stimmt da iwas nocht nicht ganze

hab nochmal nachgeguckt tasklist ist der letzte process der angezeigt wird

 

[SlaterB]

du fügst ein und dasselbe Objekt mehrfach in die Liste und änderst es ständig,
am Ende hast du 20x dasselbe Objekt in der Liste, was in der Ausgabe garantiert immer gleich ist
und zwar mit den Daten der letzten Eingabe

> private Prozess pro = new Prozess();
sollte dann doch besser ganz weg und nur eine lokale Variable sein, an der richtigen Stelle

 

[Johnny919]

eht klasse^^ naja ich weis was du dir denkst aber ich weis die einfachsten sachen mache ich falsch naja nochmal danke
ich lass das thema nochml offen das wenn ich noch iwas habe mich hier einfach wieder melde oder ??