Le topic de la programmation, string, chaînes, cuir et compagnie

**Robix66** · 16/12/2015, 17h57

Yes ce sont mes classes et elles sont disposables.
Donc quand je parle de Close(), c'est une fonction à moi qui mets le périphérique en standby/ferme la connexion/pousse des trucs dans des fichiers/...

**Foudge** · 16/12/2015, 18h10

Bizarre que tu n'arrives pas à mettre tout ça dans le Dispose : vu que c'est une classe perso, ça ne fait ni plus ni moins que ce tu lui dis de faire.
Le Dispose() est bien appelé explicitement (ou via un using) ?

**Dross** · 16/12/2015, 18h11

J'pense que Foudge t'a donné la bonne marche à suivre. Implémente IDisposable dans tes classes (qui feront ton Close et nettoyage divers), et utilise les derrière des using. Tu instancie tes objets juste avant, et tu les laisses mourir (se rendrent dispo au GC) après.

**Robix66** · 16/12/2015, 18h27

Envoyé par Foudge

Bizarre que tu n'arrives pas à mettre tout ça dans le Dispose : vu que c'est une classe perso, ça ne fait ni plus ni moins que ce tu lui dis de faire.
Le Dispose() est bien appelé explicitement (ou via un using) ?

Ben il est appelé par le finalizer...
En appelant explicitement, ça semble marcher (sauf que j'ai un truc super bizarre derrière... J'essaie de Dispose une Task... Et il m'insulte qu'il ne peut le faire que si elle est dans divers états... Dont celui dans lequel elle est.

). Je regarderais en détails demain.

Envoyé par Dross

J'pense que Foudge t'a donné la bonne marche à suivre. Implémente IDisposable dans tes classes (qui feront ton Close et nettoyage divers), et utilise les derrière des using. Tu instancie tes objets juste avant, et tu les laisses mourir (se rendrent dispo au GC) après.

En fait, mon code de fin explose (mais n'est pas catchable), et arrête le Dispose() (visiblement une propagation d'exception). (Quand pas appelé pas le Finalizer donc, mais je voudrais bien que ça marche dans ce cas aussi...)

**Dross** · 16/12/2015, 19h20

Tu appelle du code C++ pour que ça ne soit pas catchable ?
A tu essayé de canceller ta Task avant de la disposer ? (puis pourquoi disposer ta Task d'ailleurs ? cf Stephen Toub)

Sinon il va ptet falloir nous montrer un bout de code, ça sera sans doutes plus simple pour t'aider.

**Robix66** · 16/12/2015, 19h37

Envoyé par Dross

Tu appelle du code C++ pour que ça ne soit pas catchable ?

Nan, dans le cas présent, c'est un tout petit programme sans rien d'exotique.

A tu essayé de canceller ta Task avant de la disposer ?

C'est justement ce que j'essaie de faire (enfin, de la terminer).

(Et là, elle était RanToCompletion quand je me suis fait insulter).
(A noter aussi que j'utilise pas de CancellationToken (ça m'avait l'air un poil compliqué à mettre en place), mais un AutoResetEvent (mais ça marche bien en temps normal)... Là je sens qu'on va me répondre que le CancellationToken permet de faire ce que je cherche à faire depuis le début... J'avoue que j'ai pas re-regardé).

(puis pourquoi disposer ta Task d'ailleurs ? cf Stephen Toub)

Euh... Ou l'analyse de code me le demandait, ou excès de zèle.

Sinon il va ptet falloir nous montrer un bout de code, ça sera sans doutes plus simple pour t'aider.

Yes, vais faire ça demain.

**Dross** · 16/12/2015, 19h44

C'est pas compliqué le CancellationToken, faut juste prendre le temps de poser les choses.

Mais oui, comme tu t'en doutais, c'est par là qu'il faut passer.

On attends ton sample alors.

**Robix66** · 17/12/2015, 11h31

Code:

public class Logger : IDisposable
{
	public bool Ready { get { return PrivateReady; } }
	private bool PrivateReady = false;

	private object LockListEventLog = new object();

	private List<EventLogContent> ListEvent;

	private Task Worker;
	public AutoResetEvent CancelWorker;

	private string RootFolderPath = "";

	public Logger()
	{
		LockListEventLog = new object();
		RootFolderPath = Manipulator.GetApplicationPath() + @"\Logs";
		ListEvent = new List<EventLogContent>();
		Run();
	}

	~Logger()
	{
		Dispose(false);
	}

	public void Dispose()
	{
		Dispose(true);
	}

	private void Dispose(bool disposing)
	{
		if (!this.Disposed)
		{
			Disposed = true;

			if (Worker != null)
			{
				if (Worker.Status == TaskStatus.Running)
				{
					try
					{
						Stop();
					}
					catch { }
				}
			}

			if (disposing)
			{
				if (Worker != null)
				{
					Worker.Dispose();
				}

				if (CancelWorker != null)
				{
					CancelWorker.Dispose();
				}
			}
		}
	}

	private void WorkingLoop()
	{
		Stopwatch PushToFile = new Stopwatch();
		bool FileIsOpen = false;
		int EventCount = 0;

		lock (LockListEventLog)
		{
			ListEvent.Add(new EventLogContent("Info", "Starting session"));
		}

		PrivateReady = true;

		try
		{
			while (!CancelWorker.WaitOne(30))
			{
				if (FileIsOpen)
				{
					if (!CheckFileDate())
					{
						FileIsOpen = false;
					}
				}

				lock (LockListEventLog)
				{
					EventCount = ListEvent.Count;
				}

				if (!FileIsOpen && (EventCount > 0))
				{
					FileIsOpen = OpenFile();
					if (FileIsOpen)
					{
						PushToFile.Restart();
					}
				}

				if (FileIsOpen)
				{
					lock (LockListEventLog)
					{
						if (ListEvent.Count > 0)
						{
							WriteEvent(ListEvent[0]);
							ListEvent.RemoveAt(0);
						}
					}

					if (FileIsOpen && (PushToFile.Elapsed.TotalMilliseconds > 4000))
					{
						CloseFile();
						FileIsOpen = false;
						PushToFile.Stop();
					}
				}
			}
		}
		catch (Exception Ex)
		{
			//catch rien...
		}

		PrivateReady = false;
		//On vite les listes
		try
		{
			ListEvent.Add(new EventLogContent("Info", "Closing session"));

			if (ListEvent.Count > 0)
			{
				if (!FileIsOpen)
				{
					FileIsOpen = OpenFile();
				}

				while (ListEvent.Count > 0)
				{
					WriteEvent(ListEvent[0]);
					ListEvent.RemoveAt(0);
				}
			}
		}
		catch { }

		//On ferme le fichier
		if (FileIsOpen)
		{
			CloseFile();
			FileIsOpen = false;
		}
	}

	private void Run()
	{
		CancelWorker = new AutoResetEvent(false);
		
		Worker = new Task(WorkingLoop);
		Worker.Start();
	}

	public void Stop()
	{
		CancelWorker.Set();

		try
		{
			Worker.Wait(5000);
		}
		catch (Exception Ex)
		{
			//Rien qui catch...
		}
	}
}

On ne se moque pas de mes conventions de nommage.

(Il y a des trucs pas forcément logiques, mais c'est parce que la structure est récupérée d'un autre programme où je log pas mal de trucs différents).

Donc en gros :
- Si j'appelle Stop() ou Set() l'event (ils sont en public juste pour les tests), pendant le fonctionnement du programme, ça marche.
- Si j'appelle Dispose explicitement, ça marche (et ça ne crash plus après, pas comme hier

)
- Si Dispose() est appelé par le finalizer, ça plante pendant Worker.Wait(5000) ("PrivateReady = false;" n'est jamais exécutée dans la tâche). Je suppose que les objets commencent à être supprimés à ce moment là puisque je vois la consommation mémoire chuter en même temps.

Chose marrante, hier j'ai laissé le Dispose() sur un point d'arrêt pendant un moment (téléphone)... Quand je l'ai relancé, c'est la seule fois où c'est passé.

**Foudge** · 18/12/2015, 01h28

C'est à la fermeture du programme, sans appel explicite au Dispose() que t'as un crash ?

**Dross** · 18/12/2015, 03h45

Vérifie si tu attrape toutes les "Common Language Runtime Exceptions" (ça doit être coché avec un "check") dans les paramètres de ton débogueur (Ctrl+ D, E), et regarde si en debug il ne te montre par où ça coince.

Puis oui, donne nous plus de détails quand ça explose : quand tu ferme l'appli, ou quand tu passe à null un de ces objets (+ le temps que le GC passe) ?

**Robix66** · 18/12/2015, 11h18

Envoyé par Foudge

C'est à la fermeture du programme, sans appel explicite au Dispose() que t'as un crash ?

Exact. Par contre rien visuellement, le programme se ferme normalement.

Envoyé par Dross

Vérifie si tu attrape toutes les "Common Language Runtime Exceptions" (ça doit être coché avec un "check") dans les paramètres de ton débogueur (Ctrl+ D, E), et regarde si en debug il ne te montre par où ça coince.

Pas mieux.

Mais du coup, vous confirmez bien que je suis censé pouvoir faire n'importe quoi pendant le Dispose(), et que le GC commence pas déjà à nettoyer le reste du programme ?

**Frypolar** · 18/12/2015, 13h41

C’est parce que tu nommes tes données membres privées et tes variables comme des méthodes, du coup il est tout perdu

**Dross** · 18/12/2015, 17h19

Envoyé par Robix66

Exact. Par contre rien visuellement, le programme se ferme normalement.

Si c'est uniquement lors de la fermeture de ton programme, tu est p-e en train de débugger un artefact de débugger. Comment est tu sûr que c'est un crash et pas juste la fermeture de l'application ? Si tout se ferme correctement (pas de processus zombie, etc) quel est ton soucis ?

**Robix66** · 18/12/2015, 17h25

Envoyé par Dross

Si c'est uniquement lors de la fermeture de ton programme, tu est p-e en train de débugger un artefact de débugger qu'autre choses. Comment est tu sûr que c'est un crash et pas juste la fermeture de l'application ? Si tout se ferme correctement (pas de processus zombie, etc) quel est ton soucis ?

La fin de la tâche ne s'exécute pas (ici l'ajout de "Closing session" à mon log).

**Foudge** · 18/12/2015, 19h32

Envoyé par Robix66

Exact. Par contre rien visuellement, le programme se ferme normalement.

J'avais justement testé ton code hier soir (avec quelques retouches) et c'est le seul cas où j'ai pu observer ta problématique. J'ai l'impression que le GC n'a pas le temps de terminer son travaille car le processus se termine. Maintenant, je ne connais pas assez la procédure de garbage collecting en .Net, c'est donc p'tet une grosse connerie ce que je raconte.
Mais j'ai trouvé une solution simple : appeler le Dispose à la fermeture du logiciel (dans le Closing ou le Closed par ex), ça te permettra de faire le travail de nettoyage jusqu'au bout, juste avant que l'application se termine. Par contre si ton instance a une durée de vie courte, utilise un using, c'est fait pour.

**Robix66** · 18/12/2015, 21h29

Ouais mais du coup pour terminer mes trucs correctement je dois penser à appeler Dispose(), or l'idée c'était de ne plus avoir à penser à appeler Close()/Stop()/...

Bon, je vais continuer comme ça du coup, merci pour le coup de main.

**Tramb** · 18/12/2015, 21h35

On répète tous en chœur : "le garbage collecting c'est pour récupérer de la mémoire, pas pour gérer des lifetimes".

**Robix66** · 18/12/2015, 22h27

Envoyé par Tramb

On répète tous en chœur : "le garbage collecting c'est pour récupérer de la mémoire, pas pour gérer des lifetimes".

Bah à la base je cherchais pas à jouer avec le GC, je cherchais à exécuter un truc lorsque l'objet allait être détruit.

**Tramb** · 18/12/2015, 22h30

Envoyé par Robix66

Bah à la base je cherchais pas à jouer avec le GC, je cherchais à exécuter un truc lorsque l'objet allait être détruit.

Et beaucoup de gens dont moi pensent que ce n'est pas une bonne approche.

**Dross** · 19/12/2015, 01h32

Envoyé par Robix66

Ouais mais du coup pour terminer mes trucs correctement je dois penser à appeler Dispose(), or l'idée c'était de ne plus avoir à penser à appeler Close()/Stop()/...

Bon, je vais continuer comme ça du coup, merci pour le coup de main.

Tramb à raison, si tu doit finir un processus avant de fermer une application, fait en sorte d'attendre la fin du processus et ne laisse l'application se fermer qu'après. Et ne laisse surtout pas le GC ou autre chose le faire pour toi, sinon tu va avoir des comportements aléatoires comme tu a pus le voir.

C'est à toi de définir clairement le comportement de ton app, donc fait un Task.WaitAll() ou autre mécanisme/choix d'architecture.

**Robix66** · 20/12/2015, 20h35

Bien reçu.
Mais du coup vous pouviez le dire dès le début.

**Tramb** · 22/12/2015, 12h01

Mais après je passe pour un fasciste de merde.
(Alors que je suis un vrai démocrate)

**Patate** · 30/12/2015, 23h11

J'ai une petite question en c++... Bon j'ai un peu honte vu que ça concerne un type de base, mais là je m'arrache les cheveux.
J'utilise la lib libtcod qui est un ensemble d'outils pour développer des roguelikes textuels. Bref, mon projet grossi et depuis le départ, je n'arrive pas à afficher de caractères accentués. La lib se base sur un fichier image contenant les caractères :

Mais lors de l'utilisation des fonctions d'écriture de la lib, je n'ai que des caractères "bizarres" à la place des "é". Ca doit être un problème d'encodage mais il est possible de remapper chaque caractère avec cette méthode :

Code:

TCODConsole::root->mapAsciiCodeToFont(130, 9, 14);

En gros j'affecte le code ASCII 130 au caractère de la 9ème ligne et 14ème colonne.

Ce qui est cool c'est que j'arrive à afficher un "é" de cette façon :

Code:

char str = (char)130;
console_->print(0, 2, "%c", str);

Le 0, 2 c'est juste la position dans la console.

J'avoue que c'est un peu lourdingue de remapper toutes les lettres accentuées mais il n'y en a pas tant que ça et ça me permettra de continuer en FR.

Le problème c'est que tout ce que j'essai d'autre ne fonctionne pas. un const char* = "é" ne m'affiche pas mon "é".

Au passage la méthode demande un const char* pas de string. J'utilise toutefois uniquement des strings avec la fonction c_str().

C'est un peu confus, mais comment utiliser ce code ascii dans un string ou un const char* qui soit bien interprété par la lib ??? Le top serait que je fasse une fonction à qui je passe mon texte "testé" et qui me la traduit pour qu'il soit comprit par la lib.

Ca commence à faire des noeuds là :/

**Møgluglu** · 31/12/2015, 18h59

Envoyé par Patate

Le problème c'est que tout ce que j'essai d'autre ne fonctionne pas. un const char* = "é" ne m'affiche pas mon "é".

J'ai pas tout compris, mais c'est pas juste l'encodage de ton fichier .c qui n'est pas le même que celui de ton mapping perso ? (genre UTF-8 vs. ISO-8859-15 ?)

**Patate** · 01/01/2016, 09h40

Envoyé par Møgluglu

J'ai pas tout compris, mais c'est pas juste l'encodage de ton fichier .c qui n'est pas le même que celui de ton mapping perso ? (genre UTF-8 vs. ISO-8859-15 ?)

Rah yes ! C'était vraiment con enfait

J'ai lu sur le forum qu'il utilise cet encodage : https://fr.wikipedia.org/wiki/Page_de_code_850

Du coup j'ai enregistré mes sources dans cet encodage et ça fonctionne.
Merci bien Møgluglu !

**vectra** · 04/01/2016, 18h09

Yop les canards, et bonne année au passage.

Je m'adresse plus spécialement aux types en noirs, mais tout le monde peut jouer: est-ce que vous avez des configs de bourrin, genre bi-CPU, gros Xeons, CPU overclockés, mémoires tournant à plus de 2 Ghz?

Je ne vais pas vous demander de me faire des calculs, mais j'aimerais savoir ce que vous avez comme gros mastodontes, et ce qu'on peut faire surtout. Je me suis mis dans l'idée qu'on ne pouvait pas trop clocker des configs Xéon, mais je n'attends que de me faire remettre les idées à l'endroit

**Longwelwind** · 04/01/2016, 19h11

Ca compte pas, mais j'ai trouvé cette vidéo hier, sur l'intra-planète: 7 PC en uns.

**ducon** · 04/01/2016, 19h46

Bonjour, Grogro !

Envoyé par Patate

Du coup j'ai enregistré mes sources dans cet encodage et ça fonctionne.

Et pourquoi ne pas dire que tes chaînes de caractères utilisent cet encodage, ce qui t’éviterait des blagues si quelqu’un d’autre que toi touche à tes fichiers ?

**Patate** · 05/01/2016, 03h52

Envoyé par ducon

Bonjour, Grogro !

Et pourquoi ne pas dire que tes chaînes de caractères utilisent cet encodage, ce qui t’éviterait des blagues si quelqu’un d’autre que toi touche à tes fichiers ?

Je dois t'avouer que je ne sais pas comment faire ça en c++ (dire à mes strings d'utiliser tel ou tel encodage).

**ducon** · 05/01/2016, 08h40

En Python, le type bytes est un format de chaînes sans encodage, la méthode decode retourne une chaîne encodée. Ça permet de manipuler un flux d’octets.
En C++, je ne sais pas, je ne sais même pas comment coder un tri à bulles en C++.

Discussion: Le topic de la programmation, string, chaînes, cuir et compagnie

Outils de la discussion

Règles de messages