last day (14 days later) » 

8:44 AM
Hi
 
Salut ! Tu parles français ?
 
Yes
Enfin, oui
 
Ah c'est cool, ça va être plus facile comme ça
 
Comme tu as pu le comprendre, j'arrive sur un (gros) projet, codé avec les pieds
Et effectivement, la lib est maison
 
9:00 AM
Oui, je crois que j'ai pas trop compris si le Sakura.UI.Graphing, c'était le treeview ou un autre truc pour faire des graphiques ?
 
C'est un truc pour faire des graphiques
Et ces graphiques peuvent etre enregistrés sous forme de templates qui sont affichés dans la treeview
 
Ah donc le treeview est inclus dedans
 
Nop
En fait je maitrise pas encore a fond l'architecture du truc
 
oups :)
 
Donc dis moi si je suis pas clair
 
9:02 AM
et tu as le code de cette librarie ?
 
Oui
Mais je pense pas que ça vienne de la (ou alors j'ai besoin d'un autre café)
 
Haha :)
 
Je t'explique l'archi en gros :
Les différents types de graphs sont affichés dans la treeview qui appartient a la classe ViewDataSubControl
Quand tu cliques sur un type de graph, ça déplie le noeud et ça affiche les templates existant
Si tu ouvres un template tu peux le modifier et le sauver (ça en crée un nouveau)
La le graphControl dis au viewdatasubcontrol d'enregistrer le nouveau template et de l'ajouter à l'arbre
(et inversement quand tu supprimes)
 
Et le noeud est présent dans la collection Nodes, et n'est pas affiché à l'écran ?
 
Ouep
 
9:06 AM
C'est un truc de dingue
 
Bah en fait, plus aucun paint n'est effectué
si je change le nom des noeuds existant ça s'affiche pas non plus
 
Aucun paint, oui ça sent la magouille avec un Redraw ou bien un EndUpdate
 
Atta, il y a un truc que je t'ai pas encore dit
Qui cause le problème
(mais aucune idée de comment le regler)
 
Oh-ho :)
 
le viewdatasubcontrol c'est un type d'onglet qui est chargé grace à ce code :
SakuraControl uCtrl = (SakuraControl)Activator.CreateInstance(typ);
uCtrl.Dock = System.Windows.Forms.DockStyle.Fill;
uCtrl.Location = new System.Drawing.Point(0, 0);
uCtrl.Name = "ctrl";
uCtrl.Framework = (IFramework)this.Framework;
this.splitContainer1.Panel1.Controls.Clear();
this.splitContainer1.Panel1.Controls.Add(uCtrl);
Le bug apparait quand je passe de viewdatasubcontrol à un autre onglet et que je reviens
Tu vois ce que je veux dire ?
 
9:10 AM
Mais ça, ça n'est exécuté qu'une seule fois, ou chaque fois que tu passes d'un onglet à l'autre `?
 
a chaque changement d'onglet
 
Argh
 
C'est sale ?
 
Mhhhh...
Il y'a pas d'event du tout attaché au uCtrl alors ?
je veux dire, des NodeClick, tout ça
ou alors via le Concepteur ?
 
Nop, tout ça c'est dans le viewdatasubcontroler
 
9:14 AM
Donc c'est le même code là qui est utilisé la premiere fois, et les autres fois lorsque tu change d'onglet ?
 
Ouais
Et quand je change d'onglet, l'ajout de node s'affiche pas mais la suppression si
genre je change d'onglet , j'ajoute A, B, C, D, rien ne s'affiche, j'enleve le neoud Z (qui existait déjà) l'affichage se raffraichit, affiche A, B, C, D et enlève Z
 
Je suppose que parce que le controle est lourd, tu préferes le recréer quand tu changes d'onglet plutôt que de charger une instance par onglets ?
 
C'est les anciens qui ont fait ça...
Donc je sais pas
J'pourrais essayer
Mais le problème c'est que ça pèse pas mal...
 
Ouaip. Je me demande si le contrôle libere correctement ses ressources lorsque tu fais un .Clear() dans Panel1.Controls.
 
Je crois pas, puisque l'affichage est le même quand je reviens (il garde les graphs ouverts par exemple)
Mais ce qui me rends fou c'est que l'ajout n'affiche rien et la suppression si
 
9:21 AM
Si il y'a moyen, essaie voir de mettre un point d'arrêt sur le Dispose() du controle (il est dans InitializeComponents() si personne ne l'a amélioré), peut être faut il libérer des ressources supplémentaires dans ce Dispose
 
Ok je vais essayer
 
parce qu'il pourrait arriver que des évenements restent "accrochés" à des instances de controles qui ont été retirées d'une collection. Ils ne sont plus visibles, mais il ne sont pas "libérés" pour autant... Et il arrive que des évenements accrochés à de telles instances continuent de s'executer... sur un controle invisible, parce qu'il n'a plus de parent :)
 
Ah ça s'pourrait
 
Alors je propose de verifier si le Dispose() de uCtrl est appelé quand on fait un Controls.panel1.Clear()
 
Jamais appelé
 
9:27 AM
Tiens, regarde ça : c'est ce qu'il te faut pour nettoyer correctement le controle : stackoverflow.com/questions/1969705/…
C'est une extension method,c'est un peu spécial si tu connais pas je peux t'expliquer
 
Inconnu au bataillon
(j'débute en c#, j'suis un enfant du C++)
 
purée alors le C# ça va être du gâteau
C++ = total respect
Alors une extension method
C'est genre tu peux écrire une "fausse" fonction sur un objet.
Style monObject.MonExtensionMethod(mesParametres)
quand il n'y a pas de MonExtensionMethod définie dans monObject,
mais qu'en revanche il y'a une classe static comme celle là
public static class Extensions
{
public static void monExtensionMethod(this typeDeMonObject o, string unparametre)
{
o.FaisUnTruc();
}
}

Note le mot clé "this" dans la décla des paramètres
Faut essayer le Clear(bool dispose) du lien que je t'ai filé, j'ai bon espoir que ça marche
 
Ok j'vais essayer
Le problème que j'ai, (j'ai essayé un appel explicite à dispose() quand on change d'onglet)
C'est que quand je le rouvre, il plante en disant qu'il peut pas bosser sur des objets disposed
Genre il me le recrée pas
 
Ouille. Ca veut dire qu'autre chose référence l'objet en question. Je sais pas ce que fait IFramework, mais il doit certainement accrocher des evenements lui aussi.
Ce serait pas un truc genre MVC, ou MVVM ou quelque chose du genre ?
 
Aucune idée
=D
 
9:40 AM
:D
C'est quel objet qui couine qu'il peut pas bosser sur un object Disposed ?
Je parie que c'est un IFramework ^^
 
J'vais voir
System.ObjectDisposedException was unhandled
HResult=-2146232798
Message=Cannot access a disposed object.
Object name: 'TreeView'.
Source=System.Windows.Forms
ObjectName=TreeView
StackTrace:
at System.Windows.Forms.Control.CreateHandle()
at System.Windows.Forms.TreeView.CreateHandle()
at System.Windows.Forms.Control.get_Handle()
at System.Windows.Forms.TreeNode.get_State()
at System.Windows.Forms.TreeNode.get_IsExpanded()
at System.Windows.Forms.TreeNode.Remove(Boolean notify)
at System.Windows.Forms.TreeNode.Clear()
 
Ca me laisse rêveur ça uCtrl.Framework = (IFramework)this.Framework;
 
Dans quel sens ?
"Oh la cata"
Ou plutot "Kessecé?"
 
Je me demande si y'a pas plein d'évenements qui s'accrochent à IFramework dans le Set de la property Framework de uCtrl. Tu veux bien me coller le code de ce truc ?
 
Ouais
Si tu me dis ou c'est :p
 
9:48 AM
:D

sur la ligne, uCtrl.Framework = (IFramework)this.Framework; bouton de droite, go to definition sur le premier "Framework"
 
ça me parait leger...
namespace Sakura.Framework
{
public partial class SakuraControl : UserControl
{
public SakuraControl()
{
InitializeComponent();
}


protected IFramework m_Framework;

public virtual IFramework Framework
{
get { return m_Framework; }
set { m_Framework = value; }
}




}
}
 
en effet... il fait quoi avec Framework le controle ?
 
ça ?
namespace Sakura.Framework
{
public partial class SakuraControl : UserControl
{
public SakuraControl()
{
InitializeComponent();
}


protected IFramework m_Framework;

public virtual IFramework Framework
{
get { return m_Framework; }
set { m_Framework = value; }
}




}
}
merde
partial class SakuraControl
{
/// <summary>
/// Required designer variable.
/// </summary>
private System.ComponentModel.IContainer components = null;

/// <summary>
/// Clean up any resources being used.
/// </summary>
/// <param name="disposing">true if managed resources should be disposed; otherwise, false.</param>
protected override void Dispose(bool disposing)
{
if (disposing && (components != null))
{
components.Dispose();
}
base.Dispose(disposing);
}

#region Component Designer generated code
 
argh, dans le dispose il y'a que les trucs par defaut
 
Ce qui explique pourquoi il me supprime pas on interface
 
9:54 AM
la piste, c'est tous les evenements accrochés a IFramework. Tu veux bien rechercher les "+=" dans le controle ?
 
a chaque += doit correspondre un -=, c'est ça ?
 
absolument ! en particuler les evenements attachés vers l'exterieur (comme iframework)
les autres evenements attachés sur des objects à l'interieur du controle, pas besoin de dispose
 
Donc je cherche ça dans le InitializeComp ?
 
nan partout
 
Et juste ceux de la treeview ou partout partout ?
 
9:57 AM
juste ceux du treeview
 
this.viewDataTreeView.NodeMouseDoubleClick += new System.Windows.Forms.TreeNodeMouseClickEventHandler(this.viewDataTreeView_NodeMouseDoubleClick);
this.viewDataTreeView.DoubleClick += new System.EventHandler(this.viewDataTreeView_DoubleClick);
this.viewDataTreeView.AfterSelect += new System.Windows.Forms.TreeViewEventHandler(this.viewDataTreeView_AfterSelect);
this.viewDataTreeView.ItemDrag += new System.Windows.Forms.ItemDragEventHandler(this.viewDataTreeView_ItemDrag);
Pis c'est tout
Y'en avait que dans le init
 
plus d'autres +=
dans uCtrl
 
uCtrl connait pas la treeview
 
grrr
ça craint si tu m'envoies les sources ? là je sèche un peu, je croyais que le treeview était dans uCtrl
 
J'osais pas te proposer :p
 
10:03 AM
mazette, c'est un monstre !
 
La joie des grosses boites
Et encore, t'as pas vu la quantité de data qu'on garde pour utiliser avec le soft
Je crois qu'on est à 6To
 
il me dit 10 min de download, le temps de me faire couler un caf' ^^
 
Ok
 
wow 6T ça fait déja un paquet
 
Ce qui est drole c'est qu'il n'ont pas notion de taille
Un responsable à dit l'autre jour "ce dossier ? Il doit faite 2-3Go"
BIM 2To
 
10:05 AM
la vache
meme pas sur une clé usb ça tient
 
ça c'est un xml pour que tu puisse lance un graph
Ah
Timeout
 
tkt je vais juste regarder les sources
 
raaah c'est enorme
bon ça m'a l'air important, après je le benne, promis
 
Ok
Mais c'est tellement spécifique que je suis même pas sur que ça soit un problème :p
En plus on a rien inventé donc bon
 
10:13 AM
on dirait un truc de diag
 
Et en plus c'est mal codé \o/
Yes c'est ça
Je viens de me rendre compte d'un truc
 
Ca utilise ZedGraph. Joie !! J'adore ZedGraph
 
Trop puissant ce truc
Mais un beau bordel la premiere fois
Il veut pas repaint quand l'action n'est pas demandé par lui même
 
Roh la vache je m'attendais pas à une usine à gaz pareille :D
 
Tu comprends mon refus à coller du code ?
 
10:21 AM
totalement :)
 
Et mon incapacité à refaire un exemple qui reproduit le problème
 
ah je pige
sakuracontrol, une classe de base,
puis y'a databrowser, viewdatasub,. reportsub...
et viewdatasub est vérolé par ce bug de repaint
 
Ouep
Et en fait, quand le recreatetree est appelé au sein de la classe ça passe
Si c'est appelé depuis l'exterieur, ça passe pas
Comme si les appels externes n'étaient pas fait sur l'objet affiché
mais sur un autre
 
ils etaient combien sur ce truc ? o_O
 
Trop, avec trop de niveaux différents
parfois je crois des abérations
C'est rigolo
genre ça
if (item != sender && item is ToolStripButton)
{
((ToolStripButton)item).Checked = false;
}
else
{
((ToolStripButton)item).Checked = true;
}
 
10:29 AM
wow
hehehe
 
T'as trouvé un truc ?
 
je creuse le viewdatasubcontrol
 
En C# tu fais comment pour savoir si un obj est le même qu'un autre (genre comparaison d'adresse )
 
C'est GetHashCode, un truc comme ça
 
Ok
Vous ne devez pas supposer que des codes de hachage égaux impliquent l'égalité d'objet.
Saleté.
 
10:40 AM
oups
ouais mais des fois ça marche :D$
 
><
Pas faux
 
alors AddNewUserTemplate, ça sauve un template, mais ça devrait créer un noeud dans le treeview ?
 
J'vais manger et je reviens
 
ok ^^ bon ap'
 
C'est grace au recreate qu'il l'ajoute
 
10:44 AM
ok
je zieute
un truc me titille
dans GraphUINode
le constructeur
this.Text = GraphUIControlFactory.GetLabel(graphUIName);
if (this.Text == null)
{
return;
}
oops c'est l'heure de manger
 
10:57 AM
Tiens tiens
Dans public void DisplayNewGraphControl(GraphNode graphNode)
il y'a un newControl.CloseTab += OnCloseTab;
suis tenté d'ajouter un

{newControl.CloseTab -= OnCloseTab;}
pardon, un

newControl.CloseTab -= OnCloseTab;
juste après

private void OnCloseTab(object sender, EventArgs args)
sinon, le node restera accroché à cet event, même quand l'onglet est refermé
yo je vais manger pour de vrai - @+ tard
ou plutôt
((GraphUIControl)sender).CloseTab -= OnCloseTab;
sorry
 
11:33 AM
I'm back
J'vais essayer ça
 
12:03 PM
Bon ton truc a pas résolu mon problème (quoi qu'il ne fasse pas de mal non plus :p )
En fait quand on change d'onglet il en recrée un
et les graphs en cours restent attachés à l'ancien
 
12:19 PM
re
argh, bad bad
ca change d'onglet a quel endroit dans le code ?
 
C'est dans stackview
C'EST BON
CA PUTAIN DE MARCHE
T'as gagné 75rep.
 
HO c'était quoi ? c'était quoi ?
 
Quand je load the viewdatasubcontrol
Je récupère tout les graphs ouverts
et je réassigne le modèle
bon j'ai un objet dans le vent en attendant (mon ancienne viewdatasubcontrol)
Mais ça fonctionne
héhé
C'est que maintenant que tu me vote up ?
Saligo :p
Pour le bounty il va falloir attendre une heure
 
hahahahaahaha lol oui je m'en suis rappelé la tout d'un coup :D "merde j'ai pas upvoté"
 
Si j'te croise j'te paye une biere !
 
12:31 PM
Ouais on l'a bien mérité :D
 
(Sur les frais de la boite, c'est de leur faute.)
 
C'est comme meme drolement compliqué leur truc, moi je trouve qu'ils se sont bien cassés le trognon sans vraiment ajouter de la valeur, y'a moyen de faire bien bien plus simple tout de même
mais c'est beau
ben suis content que t'ai trouvé, bon j'y suis pas pour grand chose, mais on a viré un leak
 
Le problème c'est que c'est comme windows
c'est une couche dev par un mec, sur une autre couche dev par un autre gars... x50
Bah tu as élargis mon pt de vue
Et j'aurais pas trouvé tout seul
 
tu bosse ou ? moi je suis à luxembourg je fais un soft pour le pilotage des hauts fourneaux
c'est assez grisant ma foi
 
Sur Lyon chez volvo
Avant j'avais la belle vie, j'étais dans une start-up
 
12:37 PM
yeah
 
Mais bon, maintenant j'ai du temps libre
Et de l'argent
On peut pas tout avoir :p
 
Boh ici je suis a 1h15 de chez moi mais je suis comme un poisson dans l'eau
 
Trajets en RER tout les jours ?
 
oui voila, en TER, mais c'est bien je peux roupiller :)
 
atta, Luxembourg la ville ou la station de RER à Paris ?
 
12:41 PM
Ah non, la ville, je suis Lorrain
 
Aaaaaaaaah
Ok
 
Vi, ça va faire 14 que j'y travaille, avant j'étais en France pendant 6 ans, franchement j'aime bien
Yo ^^ Je vais retourner à mon projet. Content d'avoir pu t'aider, et merci tout plein pour l' "accepted answer", ça fait du bien à l'égo :)))
 
Si j'avais pu t'offrir le double, je l'aurais fait ;)
D'ailleurs si tu montes un peu sans raison les prochains jours, t'étonne pas ;)
Merci mec !
 
Hahaha, quand même :D LOL. De rien ^^ A une prochaine man !
bye ^^
 

last day (14 days later) »