Picturebox qui ne fonctionne plus

Question

Bien le bonjour à vous qui lissez mon message, j'espère que vous pourrez un petit peu m'aider. Donc mon problème est simple à expliquer : - Je commence depuis peu en c# -J'avais une picturebox qui afficher des courbes suivant un certain nombre de paramètre et ce mettait à jour etc. Tout allez bien dans le meilleurs des mondes. Mais sans que je comprenne pourquoi, et sans que je touche au code lier à la picturebox, maintenant elle m'affiche l'image par défaut. voici un bout de mon code private void Pbox_courbe_paint(object sender, PaintEventArgs e) { PictureBox courbeencour = (PictureBox)sender; float longueur = courbeencour.Width; float hauteur = courbeencour.Height; if (this.tc_mode_commande.SelectedIndex==0)//mode double pulse { Dessin.Pbox_courbe_paint_2pulse(longueur, hauteur, float.Parse(this.tb_db_T_Pulse1.Text), float.Parse(this.tb_db_T_Pulse2.Text), float.Parse(this.tb_db_T_Pulse3.Text), int.Parse(this.tb_niv_actif.Text), cmb_Choix_DP_Transistor.SelectedIndex, float.Parse(tb_tpsmort.Text), e); } else if (this.tc_mode_commande.SelectedIndex ==1)//mode boucle ouverte { List param = new List();//stock la valeur initial List sens = new List();//stock le sens du bras initial List bras = new List();//stock le nom du bras List couleur = new List();//stock le stylo if (this.chb_ActiveB1.Checked) { param.Add(this.tb_tri1.Text); sens.Add(this.cmb_tri1.SelectedIndex.ToString()); bras.Add("Bras 1"); if (chb_ModeDaltonien.Checked) //si mode daltonien couleur.Add(new Pen(Color.Chartreuse, 2)); else //sinon couleur.Add(new Pen(Color.Navy, 2)); } if (this.chb_ActiveB2.Checked) { param.Add(this.tb_tri2.Text); sens.Add(this.cmb_tri2.SelectedIndex.ToString()); bras.Add("Bras 2"); if (chb_ModeDaltonien.Checked) //si mode daltonien couleur.Add(new Pen(Color.Navy, 2)); else //sinon couleur.Add(new Pen(Color.Maroon, 2)); } if (this.chb_ActiveB3.Checked) { param.Add(this.tb_tri3.Text); sens.Add(this.cmb_tri3.SelectedIndex.ToString()); bras.Add("Bras 3"); if (chb_ModeDaltonien.Checked) //si mode daltonien couleur.Add(new Pen(Color.LightSkyBlue, 2)); else //sinon couleur.Add(new Pen(Color.DarkMagenta, 2)); } if (this.chb_ActiveB4.Checked) { param.Add(this.tb_tri4.Text); sens.Add(this.cmb_tri4.SelectedIndex.ToString()); bras.Add("Bras 4"); if (chb_ModeDaltonien.Checked) //si mode daltonien couleur.Add(new Pen(Color.OrangeRed, 2)); else //sinon couleur.Add(new Pen(Color.LightSeaGreen, 2)); } if (param.Count > 0) { Dessin.Pbox_courbe_paint_boucleouverte(longueur, hauteur, param, sens, couleur, "toto",bras, e); } } Donc j'ai ausculté en mode pas a pas, et des qu'il arrive à la Dessin.Pbox_courbe_paint_2pulse qui doit faire les calcul et dessiné, bah le pas a pas fait comme si il avait tout fini, il ne rentre dans Dessin.Pbox_courbe_paint_2pulse et le programme continu a fonctionné comme si tout allez bien. aussi this.refresh(); n'a plus d'effet, il n'appel même plus la fonction plus haut. Auriez vous une piste pour résoudre mon problème ? EDIT : Ajout du LANGAGE dans les balises de code (la coloration syntaxique). Explications disponibles ici : ICI Merci d'y penser dans tes prochains messages.

Whismeril · Answer

Bonjour 
J’attends qu’un modérateur ait l’obligeance de corriger la coloration syntaxique (tu n’as pas précisé que c.est du c# => pavé tout gris).

Mais j’arrive à y voir suffisamment pour une remarque.
C# travaille en 32 ou 64 bits selon l’option de compilation.
Utiliser des float ne fait pas gagner de ram, en effet, même codé en 16bit ils sont stockés dans un espace de 32 ou 64.
En plus ça force une conversion donc perte de temps d’exécution.
Conclusion travaille avec des double.

D’autre part windows form propose le controle Chart qui est tout indiqué pour les courbes.

Whismeril · Answer

En regardant de plus près ton code, maintenant qu'il est lisible, difficile de savoir ce qui ne marche plus. Il faudrait que tu exécutes pas à pas pour voir si ça rentre dans les if. Je commence depuis peu en c# peut s'interpréter je change de langage ou je débuté complètement la programmation par c#. Dans le cas 1, visiblement ça n'était pas un langage objet, et même la structure n'existait pas, "oublie" tout ce que tu connais et essaye de penser objet. Dans le cas 2, essaye de penser objet. List param = new List();//stock la valeur initial List sens = new List();//stock le sens du bras initial List bras = new List();//stock le nom du bras List couleur = new List();//stock le stylo //.... param.Add(this.tb_tri2.Text); sens.Add(this.cmb_tri2.SelectedIndex.ToString()); bras.Add("Bras 2"); if (chb_ModeDaltonien.Checked) //si mode daltonien couleur.Add(new Pen(Color.Navy, 2)); else //sinon couleur.Add(new Pen(Color.Maroon, 2)); Ça c'est tout bonnement pas possible. Donc là, dans un fichier cs à part, tu vas créer une class (pour l'exemple, comme je ne sais pas ce que cela représente je l'appelle LeTrucASeb) class LeTrucASeb { public LeTrucASeb(string Param, string Sens, string Bras, Pen Pinceau) { this.Param = Param; this.Sens = Sens; this.Bras = Bras; this.Pinceau = Pinceau; } public string Param { get; set; } public string Sens { get; set; } public string Bras { get; set; } public Pen Pinceau { get; set; } } Que tu vas initier comme ça dans ta méthode List mesTrucs = new List(); mesTrucs.Add(this.tb_tri2.Text, this.cmb_tri2.SelectedIndex.ToString(), "Bras 2", chb_ModeDaltonien.Checked ? new Pen(Color.Navy, 2) : new Pen(Color.Maroon, 2)); Une seule liste avec toutes les infos afférentes à ton truc. Un bon cours pour débuter en C# http://tahe.developpez.com/dotnet/csharp/

Whismeril · Answer

Bonjour pour les problèmes de virgules, ce code gère la majorité des cas private double ToDouble(string Valeur) { //on pourrait regarder la culture du Thread et chnager la virugle en point quand on est US et l'inverse quand on est Fr //Mais finalement forcer la conversion en US et remplacer une éventuelle virgule en point suffit dans la majorité des cas (ça ne gère pas les nombres ou le séparateur des milliers est la virgule) double resultat; if (double.TryParse(Valeur.Replace(',','.').Replace(" ", ""), NumberStyles.Number, CultureInfo.InvariantCulture, out resultat)) return resultat; //si la conversion rate, on retourne une valeur par défaut, pour l'exemple je retoune Not A Number return double.NaN; } Petit code de test List testConversion = new List(); testConversion.Add(ToDouble("1.2")); testConversion.Add(ToDouble("2,3")); testConversion.Add(ToDouble("1 234,5")); testConversion.Add(ToDouble("2 345.6")); testConversion.Add(ToDouble("3,456.7"));//seul cekui kà n'est pas converti, car une fois la virgule en point ça donne 3.456.7 Si tu penses être confronté à des nombres américains avec virgule et point, ça ce complique un peu, il faut d'abord déterminer à quoi sert la virgule, donc la culture. Pour cela je me sers d'une regex [http://regexstorm.net/tester?p=%5B%2B-%5D%3F%28%3F%3A%5Cd%7B1%2C3%7D+%3F%29%2A%5Cd%7B1%2C3%7D%28%3F%3A%2C%5Cd%7B1%2C3%7D%28%3F%3A+%3F%5Cd%7B1%2C3%7D%29%2A%29%3F&i=123+456%2C12%0D%0A123456%2C12%0D%0A%2B123+456%2C12%0D%0A-123456%2C12%0D%0A123+456+789%2C123445679%0D%0A123+456+789%2C123+456+789%0D%0A123+456+789%0D%0A123+456+789 [+-]?(?:\d{1,3} ?)*\d{1,3}(?:,\d{1,3}(?: ?\d{1,3})*)?] Et une autre pour la culture US, et une pour espace en séparateur de millier et point en séparateur décimal. private double ToDouble(string Valeur) { CultureInfo culture; //on va faire une regex pour voir si on est en FR if (Regex.IsMatch(Valeur, @"^[+-]?(?:\d{1,3} ?)*\d{1,3}(?:,\d{1,3}(?: ?\d{1,3})*)?$")) culture = CultureInfo.GetCultureInfo("FR"); else if (Regex.IsMatch(Valeur, @"^[+-]?(?:\d{1,3},?)*\d{1,3}(?:\.\d{1,3}(?:,?\d{1,3})*)?$")) culture = CultureInfo.GetCultureInfo("US"); else if (Regex.IsMatch(Valeur, @"^[+-]?(?:\d{1,3} ?)*\d{1,3}(?:\.\d{1,3}(?: ?\d{1,3})*)?$")) { Valeur = Valeur.Replace(" ", ""); culture = CultureInfo.GetCultureInfo("US"); } else return double.NaN; return double.Parse(Valeur, culture); } Petit code de test List testConversion = new List(); testConversion.Add(ToDouble("1.2")); testConversion.Add(ToDouble("2,3")); testConversion.Add(ToDouble("1 234,5")); testConversion.Add(ToDouble("2 345.6")); testConversion.Add(ToDouble("3,456.7")); testConversion.Add(ToDouble("3 456.7£"));//seul celui-ci n'est pas convertit car c'est une monnaie Et si tu veux gérer les monnaies, il faut adapter les regex, la culture et le parse en conséquence.

Whismeril · Answer

Pour DrawLine effectivement il semble que cette méthode n'accepte pas de double.

Soit

tu restes comme tu es,
soit tu castes au moment utile DrawLine(Pinceaux, (float)double1, (float)double2, (float)double3, (float)double3) 

Cependant, je te l'ai écrit plus  haut, pour les courbes il y a un contrôle dédié qui marche bien: Charts
Voici un tuto en VB.net, il suffit de transcrire le code en c# http://plasserre.developpez.com/cours/chart/

sebcsharp · Answer

Franchement merci pour ton aide. ton tuto sur les courbes me sera pas mal utile. Car je vais régulièrement avoir à en afficher dans le futur. et de tout genre.

Je pense que je vais mettre mon sujet en résolu mais avant une dernière question. Est ce que tu serais d'accord si je t’envoie mon code que tu regarde un peu, juste pour me donner quelque conseil sur comment améliorer ma manière de coder ?

Juste un survol bien sur pas que tu étudie tout ça me semblerait bien trop long.

Que tu dises oui ou non je te remercie pour toute ton aide.

Picturebox qui ne fonctionne plus

5 réponses

Votre réponse

Discussions similaires