Canvas.cpp

@@ -78,7 +78,6 @@ void Canvas::changeMode(Mode _mode)
 
 Line* Canvas::findAt(QPointF& pos, qreal tolerance)
 {
-    // TODO: реализовать проверку, находится ли точка на линии
     for (Line* line : lines) {
         if (line->contains(pos, tolerance)) {
             return line;
@@ -140,33 +139,39 @@ bool Canvas::areAlreadyParallel(Line* l1, Line* l2)
 // Методы работы с ограничениями
 // ===================================================================
 
-void Canvas::remove_constraints()
+// TODO - переделать в bool для отображения статуса выполнения
+void Canvas::remove_constraint(int tag)
 {
+    auto it = C_Info.find(tag);
+    if (it == C_Info.end()) return;
+
+    ConstraintInfo info = it->second;
+
     // Удаляем ограничение из солвера
-    sys.clearByTag(constraints_count - 1);
+    sys.clearByTag(tag);
 
     // Удаляем из соответствующих контейнеров в зависимости от типа ограничения
-    switch (lastConstraint.mode) {
+    switch (info.mode) {
     case Mode::Horizontal: {
-        if (auto* pair = std::get_if<PointPair>(&lastConstraint.data)) {
+        if (auto* pair = std::get_if<PointPair>(&info.data)) {
             HORIZ_pairs.erase(*pair);
         }
         break;
     }
     case Mode::Vertical: {
-        if (auto* pair = std::get_if<PointPair>(&lastConstraint.data)) {
+        if (auto* pair = std::get_if<PointPair>(&info.data)) {
             VERT_pairs.erase(*pair);
         }
         break;
     }
     case Mode::Parallel: {
-        if (auto* pair = std::get_if<LinePair>(&lastConstraint.data)) {
+        if (auto* pair = std::get_if<LinePair>(&info.data)) {
             parallelPairs.erase(*pair);
         }
         break;
     }
     case Mode::Coincedent: {
-        if (auto* pair = std::get_if<PointPair>(&lastConstraint.data)) {
+        if (auto* pair = std::get_if<PointPair>(&info.data)) {
             P2Ppairs.erase(*pair);
         }
         break;
@@ -174,10 +179,7 @@ void Canvas::remove_constraints()
     default:
         break;
     }
-
-    // Сбрасываем информацию о последнем ограничении
-    lastConstraint.mode = Mode::None;
-    constraints_count--;
+    C_Info.erase(it);
 }
 
 // ===================================================================
@@ -244,15 +246,13 @@ void Canvas::mousePressEvent(QMouseEvent* event)
                 sys.addConstraintHorizontal(*found, constraints_count++);
                 auto pair = makeOrderedPair<PointPair>(found->start_ref, found->end_ref);
                 HORIZ_pairs.insert(pair);
-                lastConstraint.mode = Mode::Horizontal;
-                lastConstraint.data = pair;
+                C_Info[constraints_count - 1] = { Mode::Horizontal, pair };
             }
             else {
                 sys.addConstraintVertical(*found, constraints_count++);
                 auto pair = makeOrderedPair<PointPair>(found->start_ref, found->end_ref);
                 VERT_pairs.insert(pair);
-                lastConstraint.mode = Mode::Vertical;
-                lastConstraint.data = pair;
+                C_Info[constraints_count - 1] = { Mode::Vertical, pair };
             }
             update();
             after_constraint = true;
@@ -363,8 +363,7 @@ void Canvas::mousePressEvent(QMouseEvent* event)
             auto pair = makeOrderedPair<LinePair>(found, current_line);
             sys.addConstraintParallel(*found, *current_line, constraints_count++);
             parallelPairs.insert(pair);
-            lastConstraint.mode = Mode::Parallel;
-            lastConstraint.data = pair;
+            C_Info[constraints_count - 1] = { Mode::Parallel, pair };
 
             current_line = nullptr;
             mode = Mode::None;
@@ -434,8 +433,7 @@ void Canvas::mousePressEvent(QMouseEvent* event)
         sys.addConstraintP2PCoincident(*clickedPoint, *firstPoint, constraints_count++);
         auto pair = makeOrderedPair<PointPair>(clickedPoint, firstPoint);
         P2Ppairs.insert(pair);
-        lastConstraint.mode = Mode::Coincedent;
-        lastConstraint.data = pair;
+        C_Info[constraints_count - 1] = { Mode::Coincedent, pair };
 
         firstPoint = nullptr;
         mode = Mode::None;
@@ -554,7 +552,8 @@ void Canvas::paintEvent(QPaintEvent* event)
             // Ошибка решения: удаляем последнее добавленное ограничение
             QMessageBox::warning(this, QString("Error!"),
                 QString("Last constraint is unavailable!"));
-            remove_constraints();
+            remove_constraint(constraints_count - 1);
+            constraints_count--;
         }
         after_constraint = false;
     }

Canvas.h

@@ -97,7 +97,7 @@ private:
     // ====================== Методы работы с ограничениями ======================
 
     /// Удалить последние добавленные ограничения при ошибке солвера
-    void remove_constraints();
+    void remove_constraint(int);
 
     // ====================== Данные для перемещения объектов ======================
 
@@ -138,10 +138,10 @@ private:
 
     /// Структура для хранения информации о последнем добавленном ограничении
     /// (используется для отката при ошибке солвера)
-    struct LastConstraint {
-        Mode mode{ Mode::None };                     ///< Тип последнего ограничения
-        std::variant<LinePair, PointPair> data;      ///< Данные ограничения
-    };
 
-    LastConstraint lastConstraint;  ///< Информация о последнем добавленном ограничении
+    struct ConstraintInfo {
+        Mode mode;
+        std::variant<LinePair, PointPair> data;
+    };
+    std::map<int, ConstraintInfo> C_Info;
 };