InputStream gibt nur jede 2te Zeile aus

[Asgar13]

    		ping.addListener(SWT.Selection, new Listener(){
			public void handleEvent(Event e) {
				switch (e.type){
				case SWT.Selection:
					
					try{
						String atftp[]={"/bin/sh", "-c", "ping -c 5 "+feld3.getText()};
						Process d = Runtime.getRuntime().exec(atftp);
						BufferedReader in = new BufferedReader( new InputStreamReader((d.getInputStream())));
						for (String pingbuffer;(pingbuffer = in.readLine()) != null;)
						{
							control.add(pingbuffer);
							if(in.readLine().equals("Erfolgreich"))
							{
								ping.setEnabled(false);
								check.add("... wurde erfolgreich angepingt");
							}
						}
						in.close();
						
						BufferedReader is = new BufferedReader( new InputStreamReader((d.getErrorStream())));
						for (String pingbuffer;(pingbuffer = is.readLine()) != null;control.add(pingbuffer));
						is.close();
						control.add("");
					}
					catch(IOException ioexp){
					control.add("Fehler bei dem IOExecption");
					}
					catch(Exception exp){
					control.add("Fehler bei dem Execption");
					}

Das ist ein Event von mir geschrieben welches eine Pc anpingt.
Wenn ich die if abfrage weg lasse macht er alles normal, sobald ich aber das if einbaue gibt er mit beim InputStream nur jede zweite Zeile auf, weiß jemand warum?

 

[AmunRa]

for (String pingbuffer;(pingbuffer = in.readLine()) != null;)
                        {
                            control.add(pingbuffer);
                            if(in.readLine().equals("Erfolgreich"))
                            {
                                ping.setEnabled(false);
                                check.add("... wurde erfolgreich angepingt");
                            }
                        }
                        in.close();

in dier For schleife machst du ein readline und gehts damit eine Zeile weiter im Stream und in der if Bedingung machst du wieder ein readline und gehst daher wieder eine Zeile weiter.

D.h. du gehst proschleifen surchlauf immer zwei Zeilen weiter.


Oder verstehe ich deine Frage falsch?

 

[Asgar13]

Nein, das tust du nicht, du liegst wahrscheinlich Goldrichtig.
Kannst du mir einen Vorschlag machen, was statt readLine(), die aktuelle Zeile liest?

 

[NattleBet]

Hast du die Zeile denn nicht schon eingelesen und in pingbuffer gespeichert?
Nimm doch direkt die Variable.

 

[Asgar13]

Danke

 

[Asgar13]

Sorry, dass ich wieder den Thread auf machen musste, aber mir ist erst beim testen aufgefallen, das er die if bedingung nicht macht.


### Bitte löschen

 

[SlaterB]

das ist ja schön und gut für dich, wenn du aber andere nicht daran beteiligen willst, wäre Markierung 'Erledigt' doch besser geblieben

unter der Annahme, dass du andere fragst erscheinen eklatante Mängel:
- kein Fragezeichen im Posting, was möchtest du?
- kein aktueller Code gepostet, du solltest das if doch ändern, niemand weiß wie dein Code aktuell aussieht
- keine Informationen zu deiner Datenlage, gib doch jede Zeile mit System.out.println() aus, dann kann man eher beurteilen ob dein if dazu passend ist

 

[Asgar13]

Sorry, war ein dummer Fehler von mir habe ihn gefunden.