Discussion :

[Coudrak]

Bonjour,

J'ai souvent lu des gens qui déconseillaient l'utilisation de la fonction Application.ProcessMessages sans que je ne comprenne jamais pourquoi.

Dans mon application pour mobile Android, j'utilise cela dans une boucle pour attendre la fin des threads (qui lancent une requête HTTP avec DataSnap).
Ca a l'avantage d'attendre (rien ne doit se passer avant la fin de la requête, donc je veux attendre la fin du thread.

Par exemple comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
  thread := TThread.CreateAnonymousThread(procedure
  begin
      json := ClientModule.ServerMethods1Client.GetUserListFromServer(params);
      Env.LastResponse := Now;
  end);
 
  thread.FreeOnTerminate := True;
  thread.Start;
  while not thread.Finished do
    Application.ProcessMessages;
 
  Result := json;

Je sais que l'autre façon de faire est d'envoyer un pointeur de procédure sur le OnTerminate du thread, mais ça me demanderait beaucoup de changements dans mon app dont le code est déjà complexe.

Cette boucle avec Application.ProcessMessages peut-elle provoquer des problèmes, lags, freeze ou bugs (surtout sur Android/iOS)... ? Est-ce vraiment à éviter autrement que pour du purisme de codeur ? Merci !

[tourlourou]

Ce qui m'étonne, c'est de tester si un thread libéré (FreeOnTerminate) est Finished, à moins qu'il n'y ait latence avant sa libération...

A signaler, une lecture intéressante ici, même si elle ne répond pas à ta problématique de ne pas trop modifier ton code.

[Paul TOTH]

Cette boucle avec Application.ProcessMessages peut-elle provoquer des problèmes, lags, freeze ou bugs (surtout sur Android/iOS)... ? Est-ce vraiment à éviter autrement que pour du purisme de codeur ? Merci !

le principal problème avec ProcessMessages c'est que cela peut provoquer un traitement en boucle, exemple dans Button1Click je fais une boucle longue qui inclut un ProcessMessages...je peux donc de nouveau cliquer sur le bouton, ce qui va relancer une nouvelle boucle qui elle même appellera ProcessMessages qui me permettra de cliquer à nouveau sur le bouton qui....

à contrario, si je supprime le ProcessMessages, je ne peux plus cliquer sur le bouton puisque les messages souris ne sont plus gérés, mais l'interface utilisateur est gelée puisque les messages concernant l'affichage ne sont pas no plus traités.

les solutions proposées par Marco sont standards:

1) faire le traitement par petits bout dans un Timer (quand c'est possible)
2) lancer le traitement en tâche de fond

[Andnotor]

FreeOnTerminate est implicite dans un thread anonyme. Pas besoin de le redéfinir

Sinon, je rejoins tourlourou : si un traitement long est en attente et que Application.ProcessMessages le démarre, il y a des chances pour que le thread soit déjà libéré à la prochaine boucle entraînant une VA sur le test. Mieux vaut un WaitForSingleObject sur son handle (à voir sur mobile).

Mais désactiver certaines fonctions (boutons) durant le traitement serait plus à mon goût.

[Coudrak]

@tourlourou: merci pour le lien, très intéressant vu que je fais une app pour mobile.

@Paul: oui effectivement, je veux que l'IHM soit désactivée, sauf l'icône animée de chargement (TAniIndicator).

@Andnotor: ouep, bon après réflexion je pense que finalement je vais opter pour faire les grosses modifs de mon code et me passer d'une boucle qui attend le thread, et réfléchir à désactiver mon IHM jusqu'au OnTerminate.

Merci à vous trois !