Les premières lignes de QElectroTech remonte à 2006, celui-ci a été crée en tant que projet d’étudiant et n'avait pas pour ambition d'être ce qu'il est aujourd'hui. De ce fait, certains choix techniques qui on été fait à l'époque ne sont plus adapté à l'heure actuel (mauvais design, manque d’évolutivité, code spaghetti, code pas assez découplé, etc…). En 2012 le créateur de QElectroTech quitta le projet et donna la mains à Joshua (la personne qui écrit ces lignes), bien que celui-ci était très motivé il était autodidacte (et l'est encore) et aggrava encore plus la situation. C'est pourquoi un jour il faudra passer par une refonte très conséquente du code afin de remettre celui-ci à niveau, ce travail est difficilement quantifiable mais il est certains qu'il prendra beaucoup de temps, ainsi il ne sera commencé que quand toutes les fonctionnalité de base seront implémenter ( à l'heure actuel on peut considéré qu'il manque : le générateur de bornier, le générateur de câble, structure de projet +=-)
Le but entre autre est de découpler le code en plusieurs parties bien distincte, revoir des parties du code afin d'enlever toutes les rustines, reprendre de grande partie de code afin de mieux corresponde au fonctionnement actuel de qet. Une fois ces buts atteint, le code de Qet sera plus solide mais aussi plus facile à lire et comprendre pour les contributeurs externe..
C'est l'une des parties la plus importante aussi bien d'un point de vue de temps de travail, que de changement profond.
Les fichiers éléments sont un dériver grossier du format svg avec beaucoup de modification plus ou moins propre. Le nouveau format (.selmt ? scalable element ?) sera donc . -La partie graphique sera du vrai svg, concrètement il faudra utiliser le namespace svg dans le xml pour toutes la partie graphique. Avantage du svg :
-Les parties non graphique (texte dynamque, information, bornes etc…) seront en dehors du namespace svg, il faudra la aussi réfléchir à amélioré tout ça.
Bien que ce format étais parfait pendant de longue année, celui-ci montre quelque limite d'un point de vue de la facilité de mise en œuvre. Le nouveau format sera : Un dossier zipper (lib zip) dans ce dossier on retrouvera plusieurs chose dont :
Le fichier .qet aura la liste des folios dans l'ordre, dans chaque folio, la liste des éléments présent dans le folio ainsi que leurs coordonné (x y z)sous forme d'un chemin de collection, la liste des images avec leurs coordonné (x y z), le cartouche utilisé avec la tailles des colonnes/lignes nombre de colonne/lignes affiché ou non. Les images, élément et cartouches ne seront plus embarqué dans le .qet, mais dans leurs dossier respectif, ce qui en plus de la simplicité d'utilisation aura une taille très petite grâce à la compression zip (la plupart des fichiers étant du texte, ceux-ci se comprime très bien). Le dossier conf quand a lui contiendra entre autre choses toutes les configuration par defaut du projet (conducteur par défaut, folio et cartouche par défaut etc…) Le fichier .sqlite quand à lui contiendra toutes les informations nécessaire au projet, il sera facile de retrouver les information entre le .qet et le .sqlite par les uuid (element, folio et autres si besoins posséderont leurs propre uuid) la bdd contiendra entre autre chose :
Cette base de donnée ressemblera beaucoup à la classe project data base
-QetDiagramEditor, il y a une quantité énorme de QAction membre de la classe, voir si il serais possible de mettre toute ces QAction dans une autre classe (ActionPool qui devrais être un singletone ou un QSharedPointer appartenant à Qetapp ? Bref il doit y avoir qu'une seul instance car la classe fera les connexion elle mêmes et les slot seront aussi dans cette classe.) QetDiagramEditor irais chercher les QAction dont il a besoins dans le pool, quand le signal triggered sera émis, le slot exécutera l'action. Souvent l'action est relatif au diagramme en cours, il suffira soit de retrouver le sender du signal (le diagram editor, puis retrouvé le projet courant) ou alors dans une méthode statique de qetapp retrouver le projet qui a le focus, une fois que l'on possède le projet en cours on connais tous le reste. L'avantage c'est qu'on enlève une grosse partie du code de qet diagram editor (instanciation de QAction, connection, slot dans le QetdiagramEditor car il faut bien qu'il soient quelque part mais pourrais tout aussi bien être ailleur.) ainsi la classe se concentrera plus sur ce qu'elle doit réellement faire. Il sera aussi plus facile de retrouver les QAction et slot, car présent dans une poignée de classe précise
-Classe element information, revoir c'elle-ci car un gros micmac au fur et à mesure des évolutions de qet. De plus elle devra s'interfacer avec la bdd, elle ne devrait plus contenir les infos elle même mais plutôt d'outils afin de récuprer une info dans la bdd ou mettre à jours une info dans la bdd ainsi que signaler à l'élément qu'une info à été changer dans la bdd (par exemple après un undo)
-Ajouter pour cette classe la possibilité d'avoir plusieurs référence (ajouté à la volé) pour un même élément (pas juste contacte aux 1 et 2). Par exemple un simple bouton poussoir est constitué de plusieurs référence (ou une seul si le fabricant propose un montage prêt à l’emploie) le bouton, le socle, le/s contacte/s, le voyant.
-Le code des cartouches, à chaque fois j'ai un mal de chien à comprendre ce code, ça part dans tout les sens mais ça marche donc à voir si c'est utile ou non d'améliorer cette partie
Conducteur -Ajouter la notion de potentiel électrique. Quand un conducteur est posé, en parallèle un potentiel électrique est crée (une classe) ainsi quand un autre conducteur est posé celui-ci prendra les propriété définie dans le potentiel, mais si besoins l'utilisateur pourra redéfinir une propriété pour le conducteur uniquement. Cela corrige l'utilisation pas toujours efficace du “appliqué a l'ensemble des conducteur du potentiel” donc pour résumé dans les propriété d'un conducteur il y aura deux panneau : propriété du potentiel et propriété du conducteur. Par défaut le conducteur utilise les propriété du potentiel sauf si l'utilisateur indique explicitement d'utilisé une propriété d'une conducteur.
-Dans le xml, le chemin des conducteurs peut être auto ou manuel. Contrairement à manuel, les points des chemins en auto ne sont pas écrit dans le code. Jusqu'à présent cela n'a jamais posé de problème car l'algo de création du chemin des conducteurs n’a pas changer depuis les début de qet. Le problème c'est que si à l’avenir l’algo devais changer, lors de l'ouverture d'un projet fait avec l'ancien algo sur une version de qet avec le nouvel algo les chemin des conducteurs automatique ne serais plus les même. Il faut donc aussi écrire les points du chemin en auto afin d'être sûre lors de l'ouverture d'un projet d'avoir exactement le même chemin.
-La creation d'un conducteur est un mélange entre les événements souris d'une borne + un affichage fait depuis le diagram (Diagram::conductorStart / stop) + les undo sont gérer dans la class terminal (création conducteur et autonum) + le chemin est effectué par le conducteur, une vrai usine à gaz. Il faut externalisé tout, pour au minimum que la borne ne gère plus rien et le diagram non plus. Le fait d'externaliser tout ça pourra rendre nettement plus simple l'automatisation de création de plusieurs conducteur en même temps (par ex cliquer sur plusieurs bornes afin de toutes les lier ensemble au lieu de faire une à une les liaison entre borne)
Revoir l'ensemble des fonctions membres to/from xml (au moins pour la sauvegarde dans le .qet) afin que la sauvegarde ne soit plus géré par les classes elles mêmes (élément, projet etc….) mais une classe spécifique uniquement destiné à cela. L'idéal serais de sauvegarder par le biais d'un plugins. Ainsi la sauvegarde (d'un .qet .elmt ou tout autre) fera appel à une classe abstraite et celle ci fera appel à une classe concrète qui sera un plugin, l'avantage sera qu'en plus des formats natif de qet, il sera très facile pour un contributeur externe de créer sont propre plugin afin de sauvegarder sous un autre format sans avoir à toucher et apprendre le code de qet.. Voir ce qui a déjà été fait avec la classe terminal strip item. Les avantages sont entre autre :
Les poignées utilisé pour modifier les formes primitive sont géré par les formes elle même, il serais mieux que les poignées soit gérer par une classe extérieure. Idéalement donner la forme en argument à la classe et le tout est géré par la classe (création des poignées, maj de la forme suite au déplacement des poignées, création de l'undo command) étant donné qu'il existe des formes dans l’éditeur d'éléments et de diagramme, il serais bien de crée un peu d'abstraction afin de mutualisé le code. Idée à la volé, une classe qui gère purement la partie mathématique des modifications, une autre qui gère la partis affichage, une autre qui gère la partie undo command, peut être envisagé un pattern decorator.
Virer la classe qet graphic shape item, et créer une classe pour chaque type de primitive. Revoir la manière dont les shapes items sont sauvegardé dans le .qet afin que chaque type de shape ai sont propre xml (actuellement tous les type de shape item sont sauvegardé dans le même element xml). Faire au plus près du standard svg.
Les informations de folio (titre, auteur, date, etc…) ne doivent pas être géré par le cartouche (un cartouche sert juste à afficher entre autre les infos) mais par une classe dédié, qui doit s'interfacer avec la bdd de la même manière que les informations d'éléments.
La variable %folio du cartouche peut être utilisé n'importe comment, car d'un coté on peut afficher plusieurs variable (%id/%total) et d'un autre coté %folio est utilisé en tant que numéro de folio dans la nomenclature / sommaire, ce qui est faux. Il faudrait pourquoi pas garder la variable %folio uniquement pour le cartouche mais ne pas l’intégrer dans la bdd du projet et voir pour ajouter une nouvelle variable dans le diagramme qui sera utilisé pour le numéro de folio (à ne pas confondre avec la position du folio).
Revoir les prefix d’éléments, actuellement c’est n’importe quoi…. Tout cela se déroule en deux parties distincte :
Revoir intégralement l'auto-numérotation qui est hyper compliqué, et géré avec la pile d'annulation.
Ajouter un nouveau type d’autonum ‘’intelligent’’ celui-ci sera basé sur une formule qui devra contenir la variable %{prefix}. Ensuite pas besoins de changer de règle d’autonum entre par exemple moteur/contacteur/disjoncteur etc.… c’elle ci basculera toutes seul en fonction du type d’élément toute en conservant indépendament l’état dans lequel elle est pour chaque type : Ex soit une formule intelligente suivante :-%{prefix}%{seqt} (prefix + chiffre sequentiel 2 digit)
Tout ça sans jamais changer de formule, c’elle ci s’adapte en fonction de l’élément.
Revoir les Xref afin que c'elle ci soit beaucoup plus souple dans leurs utilisations et ce individuellement (représentation croix/contact, afficher ou non, position auto ou manuelle, taille…)
Agrandir le champ d'utilisation des Xref (cela nécessite de revoir en profondeur les Xref afin d'être suffisament abstrait dans le code) afin qu'elles puissent être utilisé ailleurs
https://qelectrotech.org/forum/viewtopic.php?id=788
https://qelectrotech.org/forum/viewtopic.php?pid=3009#p3009
Tous les tailles des textes de schéma doivent être exprimé en pixels au lieu de points afin que ceux-ci aient la même apparence peut importe sur quel type d'écran ils sont affiché. Il est possible que cette modification soit effectué avant la refonte de qet. voir : https://qelectrotech.org/forum/viewtopic.php?pid=6267#p6267
Revoir les textes indépendant des schémas, actuellement un texte peut être simple ou html, mais c'est le bordel à gérer + bug pour des raisons historique. -Ce qu'il faudrait c'est soit avoir deux types de texte indépendant, des simple et des html ainsi tout sera plus simple à gérer ou alors mieux gérer le fait qu'un texte soit simple ou html rien de bien compliqué au final mais cassera les textes existant dans tous les cas. La solution 1 est probablement la meilleurs, un poil plus de code mais mieux découpé.
Classe projectconfigpage, la methode virtuel init() appel les methodes initwidgets et initlayout ainsi que readValuesFromProject ces méthodes semble être utilisé n'importe comment (un coup oui, un coup non en fonction de la classe dans lequel on se situe) enlever tout ça et laisser faire les classe toutes seule.