En C++, les pointeurs ne sont pas initialisés par défaut. Certains compilateurs ont des warnings à propos, mais ce n'est pas le cas de g++. Initialisez vos pointeurs à nullptr pour éviter les soucis

A l'ajout d'un noeud, il le place lorsqu'il y a de l'espace suffisant.
Exemple:
2Gates cote-à-cote génère de l'espace libre et le nouveau noeud y est placé.

Or il devrait le placer après les autres noeuds fils. Joie.

Lorsqu'un noeud est invalide, ne pas bloquer le check de ses enfants si ils existent. Par exemple la porte And bloque le check de son unique fils si elle est invalide (vu qu'elle bloque si le nombre de fils est < 2)

-la probabilité de Shape pour Weibull est capée à 99,99
-la probabilité de Scale pour Weibull est remise à 99,99 après validation pour un nombre supérieur
-la probabilité de lambda pour Exponential est remise a 1 après validation pour un nombre supérieur

Exemple: si je met 12345 dans lambda pour un Exponential, elle sera remise a 1 après avoir validé

C'est un peu bizarre à expliquer mais quand on ré-éxécute le programme plusieurs fois, sans jamais sauvegarder, le message de sauvegarde apparait avant l'application .

Encore plus bizarre, si on save sur cette fenêtre,ca segfault.
Si on cancel/close without saving,l'application s'ouvre mais n'est pas un nouveau projet.

Tout modif sur cette appli entraîne un segfault.
kesskisspass?

Je veux ajouter un attribut dans Event pour y mettre le premier correspondant.c'est le façon le plus simple😂 et ajouter aussi un public methode setPremier() et un getPremier().

L'utilisation du Couper ou du Coller n'actualisera pas l'affichage malgré les suppressions engendrées.

QList ResultBoolean::getProbabilities() doit uniquement retourner la liste. L'arbre peut avoir été modifié ou supprimé entre temps. Lors de l'instanciation de la classe, tout doit être calculé.

class EditDistributionDialog;
Constructeur:
EditDistributionDialog(QWidget *parent, Editor &editor, Distribution &dist);

Le cas où il y a bien un lien mais que l'arbre est vide n'est pas traité

Se référer au cahier des spécifications

j'ai besoin des methodes accept dans les classes EvalVisitor pour pouvoir faire l'analyse quantitative

Il suffit de sauvegarder un fichier vide, puis modifier le contenu de cette façon :

<opsa-mef author="FTEdit">
    <define-fault-tree name="Tree1">
        <attributes>
            <attribute value="true" name="keep"/> <!-- Changer "true" par "false" -->
        </attributes>
    </define-fault-tree>
</opsa-mef>

Les arbres doivent être par défaut set à keep = true, pareil pour les portes

Avoir serait plus simple pour la conception de mon visiteur et algorihtme.
Il devrait de la forme :
void accept(SaveVisitor &v) { v.visit(*this)}

XmlTreeReader.hh include "FTEdit.hh", ce qui pose des problèmes dans l'interface

la direction y de l'écran augmente vers le bas et pas vers le haut, donc la position y des nœuds est inversée

Utiliser la méthode removeDuplicates sur la liste pour retirer les valeurs en double.

Attention à la conversion de la QList en QStringList !

Le clipboard ne copie pas en profondeur l'arbre, alors des réfrences sont perdues en mémoire. Il faut INTEGRALEMENT copier la structure, sinon ça ne risque pas de fonctionner, donc faire des setMachin de getMachin, si ce sont des références ou pointeurs, nope.

Renvoie false si invalide, true si valide, or ce n'est pas le cas, par exemple la porte And :

bool And::check(QList<QString>& errors)
{
	if (children.size() < 2)
	{
		errors << prop.getName() + ": There must be 2 or more inputs.";
		return (false); // ok mais bloque le check des enfants
	}
	for (int i = 0; i < children.size(); ++i)
		children.at(i)->check(errors);
	return (errors.size() > 0); // erreur, doit renvoyer true si la liste est vide
}

Result doit pas se delete lui-même c'est déjà le rôle de son destructeur. Il faut delete les ResultBoolean et/ou ResultMCS qu'il a alloués avec new, donc les pointeurs diférents de la valeur nullptr.

Je dois utiliser une liste<Node*>, je vais mettre ses fils Gates et Containers dedans, du coup quand je veux appeler des méthodes des fils, c'est le polymorphisme, il faut ajouter des méthodes virtuelles correspondant dans Node.

Parce que je dois utiliser QMap et la fonction .contains() de QList, il faut surcharger operator== et operator< de classe Event. Sinon y'aura erreurs.

Tout est dans le titre. Si vous ne comprenez pas, relisez le titre.

VisitorDistribution n'est pas utilisable car la méthode accept est manquante dans Distribution

J'ai un segfault en appelant la méthode setDistribution(nullptr) dans la classe Event, d'ailleurs le pointeur de distribution n'est pas initialisé à nullptr dans le constructeur.

Le lien vers le sous arbre ne doit pas être supprimé, uniquement détaché (si il existe).

J'en profite aussi pour dire que le compteur de ref de link n'est pas mis à jour

La boucle doit utiliser un double au lieu d'un int, sinon une approximation est faite.
Si step < 1, alors i = i + (int)step, si step est cast en int, ça vaut 0

L'attribut proba ne sert à rien. Il suffit de remplacer

double EvalVisitor::getProba() { ... }
void EvalVisitor::reset() { ... }
void EvalVisitor::setTime(double time) { ... }

par

double EvalVisitor::getProba(Gate *top, double time)
{
    this->time = time;
    return top->accept(*this);
}

Le destructeur virtuel doit être dans la classe mère Result et non la classe fille.

Lorsqu'il y a une porte + une porte de transfert qui est liée au même arbre, il y a un segfault lors de l'évaluation (étape du check qui ne vérifie pas correctement si il y a un cycle)

Configuration du test (testable sur la branche gui actuellement) :

Arielle et moi avons besoin que cette méthode soit ne serait-ce qu'implémenté, pas fonctionnelle. Sans ça on ne peut pas utiliser Xor , et du coup ça compile pas :)

La classe Editor ne libère pas la mémoire des objets qu'elle contient lorsque le destructeur est appelé.

(Attention à tout delete, y compris les portes de transferts et les containers qui ne sont pas dans les listes, avec la méthode remove sur tous les arbres en mémoire)

Elle est inutile

J'ai un pb pour éditer les Event, la classe Container fait des copies quand la référence est changée donc c'est un peu embêtant...

Merci de faire une pull request rapidement, je suis bloqué pour l'interface

C'est normal mais du coup, pour l'analyse de l'arbre, si deux portes de transfert ont la même erreur
on ne voit qu'une erreur sur l'error list(puisque c'est le même texte et qu'on enlève les doublons) et; si une seule est fausse on ne sait pas laquelle...

Faudrait-il le changer et si oui comment?(donner un nom juste pour l'analyse, donner un nom définitif, d'autres idées...)

Describe the bug
Increment step/mission time by using arrows give access to incoherent numbers.

To Reproduce

1. Go to Analysis/Evaluate after making a tree
2. Click on Use boolean algebra
3. Click 8 times on the increase arrow for Step
4. See Step as 0.799999s
5. Click 8 times on the decrease arrow for Step
6. See Step as 2.77*e-17s
7. Increase Mission time by 0.1 then click on Continue
8. See Program not responding

Expected behavior
The increase of the mission time/step should not suddenly change, as it induce unexpected behaviours
Screenshots

Desktop (please complete the following information):

• OS: [Debian]
• Version []
  Additional context