Changeset 6d1e0a for src/UIElements/Views/Qt4/MoleculeList/QtMoleculeItem.cpp

Timestamp:

Jan 28, 2015, 7:07:01 PM (10 years ago)

Author:

Frederik Heber <heber@…>

Branches:

Action_Thermostats, Add_AtomRandomPerturbation, Add_FitFragmentPartialChargesAction, Add_RotateAroundBondAction, Add_SelectAtomByNameAction, Added_ParseSaveFragmentResults, AddingActions_SaveParseParticleParameters, Adding_Graph_to_ChangeBondActions, Adding_MD_integration_tests, Adding_ParticleName_to_Atom, Adding_StructOpt_integration_tests, AtomFragments, Automaking_mpqc_open, AutomationFragmentation_failures, Candidate_v1.5.4, Candidate_v1.6.0, Candidate_v1.6.1, ChangeBugEmailaddress, ChangingTestPorts, ChemicalSpaceEvaluator, CombiningParticlePotentialParsing, Combining_Subpackages, Debian_Package_split, Debian_package_split_molecuildergui_only, Disabling_MemDebug, Docu_Python_wait, EmpiricalPotential_contain_HomologyGraph, EmpiricalPotential_contain_HomologyGraph_documentation, Enable_parallel_make_install, Enhance_userguide, Enhanced_StructuralOptimization, Enhanced_StructuralOptimization_continued, Example_ManyWaysToTranslateAtom, Exclude_Hydrogens_annealWithBondGraph, FitPartialCharges_GlobalError, Fix_BoundInBox_CenterInBox_MoleculeActions, Fix_ChargeSampling_PBC, Fix_ChronosMutex, Fix_FitPartialCharges, Fix_FitPotential_needs_atomicnumbers, Fix_ForceAnnealing, Fix_IndependentFragmentGrids, Fix_ParseParticles, Fix_ParseParticles_split_forward_backward_Actions, Fix_PopActions, Fix_QtFragmentList_sorted_selection, Fix_Restrictedkeyset_FragmentMolecule, Fix_StatusMsg, Fix_StepWorldTime_single_argument, Fix_Verbose_Codepatterns, Fix_fitting_potentials, Fixes, ForceAnnealing_goodresults, ForceAnnealing_oldresults, ForceAnnealing_tocheck, ForceAnnealing_with_BondGraph, ForceAnnealing_with_BondGraph_continued, ForceAnnealing_with_BondGraph_continued_betteresults, ForceAnnealing_with_BondGraph_contraction-expansion, FragmentAction_writes_AtomFragments, FragmentMolecule_checks_bonddegrees, GeometryObjects, Gui_Fixes, Gui_displays_atomic_force_velocity, ImplicitCharges, IndependentFragmentGrids, IndependentFragmentGrids_IndividualZeroInstances, IndependentFragmentGrids_IntegrationTest, IndependentFragmentGrids_Sole_NN_Calculation, JobMarket_RobustOnKillsSegFaults, JobMarket_StableWorkerPool, JobMarket_unresolvable_hostname_fix, MoreRobust_FragmentAutomation, ODR_violation_mpqc_open, PartialCharges_OrthogonalSummation, PdbParser_setsAtomName, PythonUI_with_named_parameters, QtGui_reactivate_TimeChanged_changes, Recreated_GuiChecks, Rewrite_FitPartialCharges, RotateToPrincipalAxisSystem_UndoRedo, SaturateAtoms_findBestMatching, SaturateAtoms_singleDegree, StoppableMakroAction, Subpackage_CodePatterns, Subpackage_JobMarket, Subpackage_LinearAlgebra, Subpackage_levmar, Subpackage_mpqc_open, Subpackage_vmg, Switchable_LogView, ThirdParty_MPQC_rebuilt_buildsystem, TrajectoryDependenant_MaxOrder, TremoloParser_IncreasedPrecision, TremoloParser_MultipleTimesteps, TremoloParser_setsAtomName, Ubuntu_1604_changes, stable

Children:

53c1ff

Parents:

bed759

git-author:

Frederik Heber <heber@…> (01/18/15 13:33:42)

git-committer:

Frederik Heber <heber@…> (01/28/15 19:07:01)

Message:

Added a slew of specific QtMoleculeItem for each of the COLUMNs.

• all are produced solely by the factory and are very specific.

File:

• src/UIElements/Views/Qt4/MoleculeList/QtMoleculeItem.cpp (modified) (2 diffs)

src/UIElements/Views/Qt4/MoleculeList/QtMoleculeItem.cpp

@@ -43 +43 @@
 #include "CodePatterns/Observer/Notification.hpp"
 
+#include <algorithm>
+
 #include "molecule.hpp"
 
-// some attributes need to be easier to find for molecules
-// these attributes are skipped so far
-const int QtMoleculeItem::COLUMNCOUNT = COLUMNTYPES_MAX;
-const char *QtMoleculeItem::COLUMNNAMES[QtMoleculeItem::COLUMNCOUNT]={"Name","Visibility", "Atoms","Formula","Occurrence"/*,"Size"*/};
-
-
-QList<QStandardItem *> createMoleculeItemRow(const molecule *_mol)
+QtMoleculeItem::QtMoleculeItem(
+     const molecule *_mol,
+     const channellist_t &_channellist,
+     const enum MoveTypes _movetype,
+     const emitDirtyState_t _emitDirtyState) :
+  Observer("QtMoleculeItem"),
+  mol(_mol),
+  movetype(_movetype),
+  channellist(_channellist),
+  IsSignedOn(false),
+  dirty(true),
+  emitDirtyState(_emitDirtyState)
 {
-  QList<QStandardItem *> molItems;
-  QStandardItem *mainitem = new QtMoleculeItem(_mol);
-  mainitem->setText(QString(_mol->getName().c_str()));
-  mainitem->setFlags(mainitem->flags() | Qt::ItemIsSelectable);
-  molItems << mainitem;
-
-  QStandardItem *visitem = new QStandardItem();
-  visitem->setCheckState(Qt::Unchecked);
-  visitem->setFlags(visitem->flags() | Qt::ItemIsUserCheckable);
-  molItems << visitem;
-
-  molItems << new QStandardItem(QString::number(_mol->getAtomCount()));
-  molItems << new QStandardItem(QString(_mol->getFormula().toString().c_str()));
-  molItems << new QStandardItem(QString(""));
-  return molItems;
+  signOnToMolecule();
+  setFlags(flags() | Qt::ItemIsSelectable);
 }
 
-QtMoleculeItem::QtMoleculeItem(const molecule *_mol) :
-  Observer("QtMoleculeItem"),
-  mol(_mol),
-  IsSignedOn(false)
+void QtMoleculeItem::signOnToMolecule()
 {
-  mol->signOn(this, molecule::AtomInserted);
-  mol->signOn(this, molecule::AtomRemoved);
-  mol->signOn(this, molecule::MoleculeNameChanged);
+  if (!IsSignedOn)
+    for (channellist_t::const_iterator channeliter = channellist.begin();
+        channeliter != channellist.end(); ++channeliter)
+      mol->signOn(this, *channeliter);
   IsSignedOn = true;
+}
+
+void QtMoleculeItem::signOffFromMolecule()
+{
+  if (IsSignedOn)
+    for (channellist_t::const_iterator channeliter = channellist.begin();
+        channeliter != channellist.end(); ++channeliter)
+      mol->signOff(this, *channeliter);
+  IsSignedOn = false;
 }
 
 QtMoleculeItem::~QtMoleculeItem()
 {
-  if (IsSignedOn) {
-    mol->signOff(this, molecule::AtomInserted);
-    mol->signOff(this, molecule::AtomRemoved);
-    mol->signOff(this, molecule::MoleculeNameChanged);
+  signOffFromMolecule();
+}
+
+void QtMoleculeItem::updateState()
+{
+  if (dirty) {
+    internal_updateState();
+    dirty = false;
   }
 }
-
 
 void QtMoleculeItem::update(Observable *publisher)
@@ -97 +101 @@
 {
   if (dynamic_cast<molecule *>(publisher) != NULL) {
-    switch (notification->getChannelNo()) {
-      case molecule::AtomInserted:
-      case molecule::AtomRemoved:
-      {
-        // change atomcount
-        QStandardItem *count_item = parent()->child(index().row(), ATOMCOUNT);
-        count_item->setText(QString::number(mol->getAtomCount()));
-
-        // change formula
-        const std::string molecule_formula = mol->getFormula().toString();
-        QStandardItem *formula_item = parent()->child(index().row(), FORMULA);
-        formula_item->setText(QString(molecule_formula.c_str()));
-
-        // re-position in model's tree
-//        static_cast<QtMoleculeList*>(model())->readdItem(this, molecule_formula);
-        break;
+    if (notification->getChannelNo() == molecule::AboutToBeRemoved) {
+      signOffFromMolecule();
+      // prevent any remaining updates from accessing the molecule
+      mol = NULL;
+      dirty = false;
+    } else {
+      channellist_t::const_iterator iter =
+          std::find(channellist.begin(), channellist.end(), notification->getChannelNo());
+      if (iter != channellist.end()) {
+        dirty = true;
+        emitDirtyState(this, movetype);
+      } else {
+        ASSERT(0,
+            "QtMoleculeItem::recieveNotification() - received notification to channel "
+            +toString(notification->getChannelNo())+" we are not subscribed to.");
       }
-      case molecule::MoleculeNameChanged:
-      {
-        // change name
-        QStandardItem *name_item = parent()->child(index().row(), NAME);
-        name_item->setText(QString(mol->getName().c_str()));
-        break;
-      }
-      default:
-        ASSERT(0, "QtMoleculeItem::recieveNotification() - cannot get here, not subscribed to channel "
-            +toString(notification->getChannelNo()));
-        break;
     }
   }