ProgressDialog - AsyncTask + Changement d'orientation

**Aure7780** · 23/12/2010, 13h38

Bonjour,

J'ai un gros soucis avec la gestion de l'orientation et l'affichage d'une ProgressDialog pendant un long traitement.
En effet si je pivote l'orientation pendant que la progress dialog est active, ça me faut planter mon programme...

Voici mon implementation:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
 
// android:configChanges="keyboardHidden|orientation" dans le manifest
public class ProgTv extends Activity {
 
	private LoadProgTvData progData;
	/** Called when the activity is first created. */
	@Override
	public void onCreate(Bundle savedInstanceState) {
		super.onCreate(savedInstanceState);
		setContentView(R.layout.main);
 
		//loadProgTvData();
	}
 
	@Override
	public void onConfigurationChanged(Configuration newConfig) {
		super.onConfigurationChanged(newConfig);
		setContentView(R.layout.main);
	}
 
	@Override
	public Object onRetainNonConfigurationInstance() {
		progData.setActivity(null);
		return progData;
	}
 
	@Override
	protected void onResume() {
		super.onResume();
		loadProgTvData();
	}
 
	private void loadProgTvData() {
		final Object retained = getLastNonConfigurationInstance();
		if(retained instanceof LoadProgTvData) {
			progData = (LoadProgTvData) retained;
			progData.setActivity(this);
		} else {
			progData = new LoadProgTvData(this, null);
		}
	}
}

LoadProgTvData :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
public class LoadProgTvData {
	private Activity activity;
	private static List<ProgTvObj> lst_prog = new ArrayList<ProgTvObj>();
	private static ProgTvAdapter progTvAdapter = null;
        private Hours progHours;
	public static enum Hours { BEGINING_EVENING, END_EVENING };
 
	public LoadProgTvData(Activity activity, Hours h) {
		this.activity = activity;
                this.progHours = h;
 
		if(lst_prog.isEmpty())
			new ListRefresher().execute();
		else
			refreshListView();
	}
 
	public void setActivity(Activity activity) {
		this.activity = activity;
		if(!lst_prog.isEmpty())
			refreshListView();
	}
 
	/**
         * Set List of ProgTvObj from an XML stream
         */
	private void setListFromXml() {
		// SAX Parser
		try {
			ProgSaxXml px = new ProgSaxXml(activity.getAssets().open("soir.xml")); // SAX Parser		
			lst_prog = px.getProgList();
		} catch (MalformedURLException e) {
			Log.d("MalformedURLException", "XML Url is malformed", e);
			throw new RuntimeException(e);
		} catch (IOException e) {
			Log.d("IOException", e.toString());
		}
	}
 
	/**
         * Set Adapter of ListView to fill it with ProgTvObj 
         */
	public void refreshListView() {
		//Hours progHours = (Hours) getIntent().getSerializableExtra("tabHours"); // Get Hours value or null if not defined
		if(null != activity) {
			if(null == progTvAdapter)			
				progTvAdapter = new ProgTvAdapter(activity);
			if(null != progHours)
				progTvAdapter.setListProg(myUtils.filter(lst_prog, progHours));
			else
				progTvAdapter.setListProg(lst_prog);
			ListView lstview_prog = (ListView)activity.findViewById(R.id.lstview_prog);
			lstview_prog.setAdapter(progTvAdapter);
			lstview_prog.setOnItemClickListener(progTvAdapter);
		}
	}
 
	class ListRefresher extends AsyncTask<Uri, Void, Void> {
		private ProgressDialog progressDialog;
		@Override
		protected void onPreExecute() {
			if(!lst_prog.isEmpty()) {
				this.cancel(true);
				refreshListView();
			} 
			else
				progressDialog = ProgressDialog.show(activity, null, "Chargement en cours...");
		}
 
		@Override
		protected Void doInBackground(Uri... params) {
			setListFromXml();
			return null;
		}
 
		@Override
		protected void onPostExecute(Void result) {
			progressDialog.dismiss();
			refreshListView();
		}
	}
}

Sachant que si je met loadProgTvData(); dans le OnResume c'est parce que mon Activity est un onglet de mon tableau et le faut de changer d'onglet ne rapelle pas OnCreate. Or j'ai besoin que ma liste soit rafraichit à chaque passage.

Je pense que le problème viens du fait que la ProgressDialog est crée dans un context et le changement d'orientation change le context et donc au moment du dismiss il doit pas savoir quoi faire.

Si vous pouviez m'aider ça serai vraiment sympa. Car le problème sur le lequel je bloque est quand même une chose que tout le monde doit réaliser dans une appli mobile (telechargement avec message d'attente...).

**Feanorin** · 23/12/2010, 14h38

Salut , pour la gestion de l'orientation tu n'as pas besoin de passer par
onConfigurationChanged.

Regarde plutôt du côté de

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

	protected Dialog onCreateDialog(int id) {

http://developer.android.com/guide/t...i/dialogs.html

Il gère tout seul le dialogue lors d'une rotation

.

Si tu as un souci avec cette procédure fais signe .

**Aure7780** · 23/12/2010, 15h18

J'ai rajouté :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
protected Dialog onCreateDialog(int id) {
	if(id == 0){
		ProgressDialog loadingDialog = new ProgressDialog(this);
		loadingDialog.setMessage("Chargement en cours...");
		loadingDialog.setIndeterminate(true);
		loadingDialog.setCancelable(true);
		return loadingDialog;
	} 
	return super.onCreateDialog(id);
}

et dans la inner class de LoadProgTvData j'ai modifié :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
class ListRefresher extends AsyncTask<Uri, Void, Void> {
	//private ProgressDialog progressDialog;
	@Override
	protected void onPreExecute() {
		if(!lst_prog.isEmpty()) {
			this.cancel(true);
			refreshListView();
		} else
			activity.showDialog(0);
		/*else
			progressDialog = ProgressDialog.show(activity, null, "Chargement en cours...");*/
	}
 
	@Override
	protected Void doInBackground(Uri... params) {
		setListFromXml();
		return null;
	}
	@Override
	protected void onPostExecute(Void result) {
		//progressDialog.dismiss();
		activity.dismissDialog(0);
		refreshListView();
	}
}

Mais j'obtiens le message d'erreur suivant :

ERROR/AndroidRuntime(5089): java.lang.NullPointerException
ERROR/AndroidRuntime(5089): at org.prozero.util.LoadProgTvData$ListRefresher.onPostExecute(LoadProgTvData.java:191)
ERROR/AndroidRuntime(5089): at org.prozero.util.LoadProgTvData$ListRefresher.onPostExecute(LoadProgTvData.java:1)
....

La ligne 191 est la ligne avec activity.dismissDialog(0); -> toujours le problème du dismiss et crash de l'appli.
Est ce que mon implementation est bonne ?

**Feanorin** · 24/12/2010, 10h06

Salut,

Garde en mémoire ta progress pour pouvoir interagir directement dessus.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
private ProgressDialog loadingDialog = null;
protected Dialog onCreateDialog(int id) {
     super.onCreateDialog(id);
     if(id == 0){
		loadingDialog = new ProgressDialog(this);
		loadingDialog.setMessage("Chargement en cours...");
		loadingDialog.setIndeterminate(true);
		loadingDialog.setCancelable(true);
		return loadingDialog;
	} 
	return null;
}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
	@Override
	protected void onPreExecute() {
		if(!lst_prog.isEmpty()) {
			this.cancel(true);
			refreshListView();
		} else
			showDialog(0);
 
	}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
 
	@Override
	protected void onPostExecute(Void result) {
		if(loadingDialog != null)
                    loadingDialog.dismiss();
 
		refreshListView();
	}

**Jay** · 24/12/2010, 14h11

Bonjour,

Sinon tu bloques la rotation de ton Activity.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
<activity android:name="TonActivity"
android:screenOrientation="portrait"></activity>

Cordialement,

**Aure7780** · 24/12/2010, 16h04

Envoyé par Feanorin

Salut,

Garde en mémoire ta progress pour pouvoir interagir directement dessus.

Malheureusement, la AsyncTask (qui fait le showDialog) est dans une classe à part et n'as donc pas accès aux membres de mon Activity (sauf via la valeur de activity passée en param à la classe).

En revanche j'ai outre passé le problème :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
@Override
protected void onPostExecute(Void result) {
	//progressDialog.dismiss();
	try {
		activity.dismissDialog(0);
	} catch(Exception e) {	}
	refreshListView();
}

Le try catch permet de ne pas traiter la NullException (car sinon il la throw au RunTime).
Plus de message, plus de plantage et en plus ça marche...mais c'est pas forcement la meilleure solution...

@Jay : oui mais je voudrai pouvoir garder la rotation...

Ou alors il faudrait que ma classe à part (LoadProgTvData) hérite de Activity
et que Ma classe principale hérite de la classe à part (LoadProgTvData) ?
Et redefinir le OnCreate dans la classe principale...
(car plusieurs Activity implemente ma classe LoadProgTvData actuellement)

**Feanorin** · 24/12/2010, 17h25

Tu peux aussi faire un Handler sur ta ListRefresher . Ou l'activity en cours se liera avec cet handler pour récupérer les messages envoyés, En fonction du message tu sauras si tu dois afficher ou détruire la progress , de plus cet handler peut te permettre de faire autre chose , comme gérer les messages d'erreur à partir de ton activity .

Voilà cela reste qu'une idée

ProgressDialog - AsyncTask + Changement d'orientation

Android

Vue hybride

Discussions similaires

Partager

Partager