Probleme mit Boolean

CdWechsler · 1. April 2013

Ich habe probleme mit boolean werten. Und zwar überprüfe ich in meinem android projekt über ein thread den onlinestatus zweier url's. der thread wird über die MainActvity.onCreate()-methode ausgeführt. im thread wird ein boolean attribut auf false gesetzt, wenn der url's nicht erreichbar sind, sonst true. diesen wert gebe ich dann an eine klasse weiter, die die boolean-werte speichert (get-set methoden). ich habe es mit log-einträgen überprüft, welches mir stets die richtigen werte nach der übergabe angezeigt hat. wenn die server aber nicht online sind, dann soll es eine nachricht ausgeben. dazu habe ich in der MainActvity.onCreate()-methode (nach den try-catch-blöcken) eine if-bedingung eingeführt, die folgendermaßen aussieht:

Code

if(!ServerOnlineCheck.isServerAisOnline() && !ServerOnlineCheck.isServerBisOnline()) {
			showServerUnreachableMessage();
		}

und obwohl die server online sind und auch über den log-eintrag die gespeicherten boolean-werte true sind, wird die showServerUnreachableMessage(); ausgeführt. habe dann davor auch log-einträge gemacht um zu sehen was für boolean-werte gespeichert sind bzw. von der if-bedingung aufgerufen werden und es sind false werte. wieso sind die false? Die true false-werte werden nur über den thread übergeben. über logeinträge sehe ich dass diese ture sind und später dann false. (kopfkratz)

kann mir das einer erklären?? Ich raffe das nicht

danke im voraus

Marco Feltmann · 1. April 2013

Spontan würde ich prüfen, ob das Objekt im Thread, dem du die Bool-Werte zuweist, dasselbe Objekt ist wie in der MainActivity bei der Abfrage.
Ich vermute mal das nicht.

schymura · 1. April 2013

Hallo CdWechsler,

Code

if(!ServerOnlineCheck.isServerAisOnline() && !ServerOnlineCheck.isServerBisOnline()) {
			showServerUnreachableMessage();
		}

eine Frage hätte ich dazu:
Was passiert, wenn du

Code

if((!ServerOnlineCheck.isServerAisOnline()) && (!ServerOnlineCheck.isServerBisOnline())) {
			showServerUnreachableMessage();
		}

schreibst?

Schöne Grüße, schymura

block_ · 1. April 2013

Ich meine in einem anderen Thread von dir gesehen zu haben, dass das bei dir wie folgt abläuft (Pseudocode)

Code

Thread erstellen;
 Thread starten;


 Abfrage (URL1.online() && URL2.online()) { tu was;}

Falls es wirklich noch so aufgebaut ist (und das ist auch das, was ich aus deiner Beschreibung des Problems vermute), vergisst du das Konzept, welches hinter Threads steht. Ein Thread zu erstellen und abzuarbeiten kostet nunmal Zeit. D.h. während der Thread erstellt und abgearbeitet wird, läuft der UI-Thread immer noch fröhlich weiter (die onCreate()-Methode). Daher ist bei der Abfrage das boolean immer noch false, weil der Thread eben nicht schnell genug war diesen zu aktualisieren.

block_

CdWechsler · 1. April 2013

@lucas de vil

ne ne, ich nutze keine weiteren bool-werte. zuweisung ist schon richtig. aber bekomme nicht du gespeicherten werte, hmm... !?

schymura

ne das bringt nichts. habe doch vorher geschrieben, dass ich mit log-einträgen gecheckt habe, was für werte aufgerufen werden. es sind false werte

Also ich weiß nicht ob ich zu kompliziert denke, aber was ich will ist nur bool-werte die über einen thread festgelegt werden in einer anderen klasse speichern. Auf diese klasse bzw. werte, möchte ich dann aus der Mainactvity zugreifen können.

Ich bin bis jetzt folgendermaßen vorgegangen:

Die onCreate()-Methode der MainActvity (rest ist egal):

Code

@Override
	protected void onCreate(Bundle savedInstanceState) {
		super.onCreate(savedInstanceState);
		setContentView(R.layout.main);
				
		setSectionPagerAdapter();
		playList = (TextView)findViewById(R.id.textView_playlist);	
		
		
	String testUrl1 = "http://google.de";  //testURL
	String testUrl2 = "http://facebook.com";  //testURL
 	
		try {
			URL url1 = new URL(testUrl1); 
			ccThread = new ConnectionControlThread(url1);
			Thread thread1 = new Thread(ccThread);
			thread1.start();
		} 
		catch (Exception e) {
			Log.e("Service", "" + e);
		}
		
		try {			
			URL url2 = new URL(testUrl2);
			ccThread = new ConnectionControlThread(url2);
			Thread thread2 = new Thread(ccThread);
			thread2.start();
		} 
		catch (Exception e) {
			Log.e("Service", "" + e);
		}
		
		Log.d("TEST3", "" + ServerOnlineCheck.isServerAisOnline() + ServerOnlineCheck.isServerBisOnline());


		if(ServerOnlineCheck.isServerAisOnline() == false && ServerOnlineCheck.isServerBisOnline() == false) {
			showServerUnreachableMessage();
		}
	}

Alles anzeigen

Thread

Code

public class ConnectionControlThread implements Runnable {


	URL website;
	Handler handler = new Handler();
	ServerOnlineCheck onlineCheck = new ServerOnlineCheck();
	boolean serviceAvailable;
	
	public ConnectionControlThread(URL url) {
		this.website = url;
	}
	
	public int setIntForURL() {
		
		int i = 0;
		
		String websiteAsString = website.toString();
				
		if(websiteAsString.contains("google")) {
			i = 1;
		}
		
		if(websiteAsString.contains("face")) {
			i = 2;
		}
		
		return i;
	}
	
	@Override
	public void run() {
			
		try {
		HttpURLConnection httpURLConnection = null;
		httpURLConnection = (HttpURLConnection) website.openConnection();
		httpURLConnection.setReadTimeout(30000);	//30 seconds timeout in milliseconds
		httpURLConnection.setConnectTimeout(5000);  //5 seconds timeout in milliseconds


		// Start connection
		httpURLConnection.connect();
		InputStream response =httpURLConnection.getInputStream();
    	Log.d("Response:", response.toString());
		
			if (httpURLConnection.getResponseCode() == 200) { // HTTP-Statuscode: 200 ist Ok! (siehe wikipedia)
		   		serviceAvailable = true;
			} 
			else { 
		   		serviceAvailable = false;
			}
		} 
		catch (IOException e) {
        	Log.e("TAG", e.getMessage());
    	}
		  
			handler.post(new Runnable() {
				
				@Override
				public void run() {
					if(setIntForURL() == 1) {
						onlineCheck.setServerAisOnline(serviceAvailable);
						Log.d("TEST1", "" + onlineCheck.isServerAisOnline());
					}
					else {
						onlineCheck.setServerBisOnline(serviceAvailable);
						Log.d("TEST2", "" + onlineCheck.isServerBisOnline());
					}
				}
			});
	}
}

Alles anzeigen

die klasse wo die bool-werte gespeichert werden:

Code

public class ServerOnlineCheck {
	
	static boolean serverAisOnline;
	static boolean serverBisOnline;
	
	public static boolean isServerAisOnline() {
		return serverAisOnline;
	}
	
	public static boolean isServerBisOnline() {
		return serverBisOnline;
	}
	
	public void setServerAisOnline(boolean serverAisOnline) {
		this.serverAisOnline = serverAisOnline;
	}
	
	public void setServerBisOnline(boolean serverBisOnline) {
		this.serverBisOnline = serverBisOnline;
	}
}

Alles anzeigen

Könnte jmd. eine simples beispiel eventuell posten, wo man über der oncreate()-methode der mainActivity einen thread ausführt, der eine url nach dem status ceckt und das ergebnis als bool-wert in einer separaten klasse speichert und letztendlich an eine textview übergibt? man könnte dafür die meisten codes von oben verwenden. ich danke im voraus für die hilfen und ratschläge

block_
habe dein post gerade gelesen. Ähm.. ja genau so sieht es aus
Aber wie können die werte false sein? diese werte gebe ich doch über den thread? bei der deklaration habe ich zumindestens keine werte zugewiesen (siehe code). wie sollte/könnte man das dann lösen?

Marco Feltmann · 1. April 2013

Zitat von CdWechsler

block_
habe dein post gerade gelesen. Ähm.. ja genau so sieht es aus
Aber wie können die werte false sein? diese werte gebe ich doch über den thread? bei der deklaration habe ich zumindestens keine werte zugewiesen (siehe code). wie sollte/könnte man das dann lösen?

Der Programmablauf ist doch der:
onCreate wird ausgeführt
Thread wird erstellt
Thread wird gestartet
onCreate wird weiter verarbeitet.

Was im Thread passiert muss da noch gar nicht existieren.
Entweder du machst statt eines Threads eine Runnable und vergleichst dann die Zustände am Ende der Runnable.
Oder du machst es richtig und stellst für die Threads noch Callback-Funktionen ein, die aufgerufen werden, wenn die Status ordentlich geprüft wurden.

block_ · 2. April 2013

Zitat von CdWechsler

Aber wie können die werte false sein? diese werte gebe ich doch über den thread? bei der deklaration habe ich zumindestens keine werte zugewiesen (siehe code).

Du hast es nicht explizit zugewiesen, aber primitive Datentypen haben immer einen Standartwert. Bei boolean ist es false, bei int ist es 0, etc.. (Nicht-primitive Datentypen, bspw. String, haben als Standartwert immer null.)

block_

CdWechsler · 2. April 2013

@Lucas:
Stimmt, du hast recht. Habe ich völlig vergessen. Die Threads werden ja parallel im Hintergrund ausgeführt, während der UI-Thread weiter macht. Wenn ich den blockieren würde, bis die Threads abgearbeitet und geschlossen wurden, wäre das keine gute Lösung. ok... dann mache ich das mal anders. Eine andere Frage: ich dachte mit dem handler ein callback-mechanismus implementiert zu haben!? (siehe code: handler.post(new Runnable() { ... }); ). Könntest du mir einen Beispiel geben wie du das gemeint hast? (allgemein gefragt)

@block
ah ok. Gut zu wissen. danke

Marco Feltmann · 3. April 2013

Zitat von CdWechsler

Stimmt, du hast recht. Habe ich völlig vergessen. Die Threads werden ja parallel im Hintergrund ausgeführt, während der UI-Thread weiter macht. Wenn ich den blockieren würde, bis die Threads abgearbeitet und geschlossen wurden, wäre das keine gute Lösung. ok... dann mache ich das mal anders.

Das ist etwas, dass ich an Androids Zwang zum Nebenthread für Netzwerkkommunikation hasse.
Am Einfachsten wäre es wohl, wenn du nur eine 'Konnektivität wird geprüft'-Activity vorschaltest und nach erfolgreichem Prüfen der Server die wahre MainActivity startest.

Zitat von CdWechsler

Eine andere Frage: ich dachte mit dem handler ein callback-mechanismus implementiert zu haben!? (siehe code: handler.post(new Runnable() { ... }); ). Könntest du mir einen Beispiel geben wie du das gemeint hast? (allgemein gefragt)

Nun, dein Callback-Mechanismus ist kein Callback-Mechanismus.
Du führst deine Aktion lediglich auf dem Mainthread aus. Das bedeutet aber eben nicht, dass deine Activity ihre Objekte manipuliert – sie ändert nur die Objekte aus deinem Thread.

Ein 'richtiges' Callback besteht im Groben aus zwei Komponenten.
1) Einem Java Interface
2) der Zuweisung eines Objekts dieses Interfaces

Beispiel:

Java

public interface StatusChecker
{
  public void serverIsOnline(ServerOnlineCheck theServer);
  public void serverIsOffline(ServerOnlineCheck theServer);
}

Deine Activity muss nun dieses Interface implementieren und damit auch die Methoden bereitstellen.
(Finde ich ein bisschen dämlich, weil es keine optionalen Methoden gibt, man also ALLES aus dem Interface implementieren muss. Java ist nun einmal so starr.)

Deinen Thread gibst du jetzt noch ein Objekt mit, dass eben dieses Interface implementieren muss.

Java

public class ConnectionControlThread implements Runnable 
{
  private StatusChecker callbackObject;
//...

In deiner Activity setzt du entsprechend diese Instanzvariable:

Java

//...
			Thread thread1 = new Thread(ccThread);
			thread1.setCallbackObject(this);
			thread1.start();
//...

Und in deinem Thread hast du nicht mehr allzu viel zu tun:

Java

//...
	catch (IOException e) {
        	Log.e("TAG", e.getMessage());
    	}
    	
    	// Statt der Abfrage solltest du lieber zwei unterschiedliche Serverobjekte haben, das würde das Ganze übersichtlicher machen.	


    	if(serviceAvailable) {
    	    	callbackObject.serverIsOnline(ServerA);
    	} else {
    	    	callbackObject.serverIsOffline(ServerA)
    	}
//...

Alles anzeigen

Es wäre übrigens noch um Längen geiler, wenn du statt eines blöden Threads lieber einen ASyncTask nützest.

Java

class CheckServer extends ASyncTask&lt;Server, void, void&gt;
{
  // Reicht uns privat.
  private StatusChecker callbackObject;
  // Von Außen braucht das Ding ja nur gesetzt zu werden.
  public void setCallbackObject(StatusCecker co)
  {
    this.callbackObject = co;
  }


  public void doInBackground(Server... params)
  {
    for(Server currentServer : params)
    {
      if(isServerOnline(currentServer))
      {
        callbackObject.serverIsOnline(currentServer);
      }
      else
      {
        callbackObject.serverIsOffline(currentServer);
      }
    }
  }
  
  private boolean isServerOnline(Server currentServer)
  {
    try 
    {
      HttpURLConnection httpURLConnection = null;
      httpURLConnection = (HttpURLConnection) currentServer.getURL().openConnection();
      httpURLConnection.setReadTimeout(30000);	//30 seconds timeout in milliseconds
      httpURLConnection.setConnectTimeout(5000);  //5 seconds timeout in milliseconds
      // Start connection
      httpURLConnection.connect();
      InputStream response =httpURLConnection.getInputStream();
      Log.d("Response:", response.toString());
      if (httpURLConnection.getResponseCode() == 200) { // HTTP-Statuscode: 200 ist Ok! (siehe wikipedia)
        return true;
      } 
      else { 
        return false;
      }
      catch(Exception e) {}
    }
  }	
}

Alles anzeigen

CdWechsler · 4. April 2013

Mit dem AsyncTask hat es wunderbar geklappt. Danke dir

Probleme mit Boolean

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema