source: src/Actions/ActionTraits.cpp@ e4afb4

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 v1.0.6
Last change on this file since e4afb4 was e4afb4, checked in by Frederik Heber <heber@…>, 15 years ago

Huge refactoring: Introduction of Traits to Actions.

This change is really big but the introduction of the Trait concept (at least
in its current light form) is so fundamental that lots of pieces had to be
changed in order to get everything working.

The main point why it was necessary to add these traits in the first place was
to comfortably allow for adding extension of Actions information-wise, i.e.
with stuff that is only important for the QtUI, such as icons, or tooltips, ...
This extra information should not be stored with Action itself, as it has
nothing to do with the workings of the Action. And neither should it get
stored with some blown-out-of-proportions MapOfActions class ...

The gist of the change is as follows:

  • OptionTrait contains the token, description, shortform and type of an option, such as ("position", "position in space, none, typeid(Vector)).
  • ActionTrait is the derived form for actions where additionally MenuPosition and MenuName are stored (and probably more to come for the GUI), also we have a set of OptionTrait instances, one for each option of the Action.
  • Action then contains this ActionTrait, specialized for each Action.
  • the preprocessor macros have been enhanced to gather all this information from the .def files.
  • MapOfActions is gone. Completely. Most of its use was to store this extra information and the ValueStorage part now is just in class ValueStorage.
  • ValueStorage is no more an interface to MapOfActions but as the name says a (type-safe) ValueStorage.

Listing the (remaining) changes in alphabetical order of the class:

  • Action
    • member value ::name dropped, ::getName() uses ActionTraits::getName()
    • new define NODEFAULT which is used in paramdefaults in .def files
    • all derived actions classes such as Process, Calculations, MakroAction,... have been adapated to use the ActionTrait concept as well.
  • ActionHistory
    • extraced RedoAction and UndoAction, shifted implementation into their own object files and they use .def files as well (i.e. streamlined with method used for other actions)
  • MenuDescription
    • contain information on Menus such as name, ...
    • new unit test checks for consistency
  • molecule
    • const member functions: Copy(), Output() and OutputBonds()
  • OptionRegistry
    • new registry class for options only
    • we want the same type throughout the code for each token, e.g. "position"
    • the registry containts checks for consistency
  • OptionTrait
    • default values are specified in paramdefaults, none are given by NODEFAULT
    • introduced default for translate-atoms, point-correlation, pair-correlation
  • Registry pattern
    • new unit test, but only sceleton code so far
  • ...Query, also ...Pipe
    • atoms, molecule and elements are now all const
    • also ValueStorage's signatures all have const therein
  • ValueStorage
    • set/queryCurrentValue from MapOfActions
    • at times VectorValue has been in .def files where Vector was in the signature. This is cleared. Such stuff is only present for e.g. BoxVector being queried as a Vector. But this is a feature and intended.
  • World
    • most of the (un)selection functions now work on const atoms and molecules
    • in one case we need a const_cast to remove this, but this is intentional, as the vector of selected atoms stores non-const pointers and this is ok.

There is only one test which had to be changed slightly because a specific
option token as "position" must now have the same type everywhere, e.g. always
Vector.

  • TESTFIX: Simple_configuration/2: --position -> --domain-position (and associated to BoxVector)
  • Property mode set to 100644
File size: 4.5 KB
Line 
1/*
2 * Project: MoleCuilder
3 * Description: creates and alters molecular systems
4 * Copyright (C) 2010 University of Bonn. All rights reserved.
5 * Please see the LICENSE file or "Copyright notice" in builder.cpp for details.
6 */
7
8/*
9 * ActionTraits.cpp
10 *
11 * Created on: Oct 26, 2010
12 * Author: heber
13 */
14
15#include "Actions/ActionTraits.hpp"
16
17#include "Helpers/Assert.hpp"
18
19#include <iostream>
20#include <sstream>
21#include <string>
22#include <typeinfo>
23
24/** Copy constructor for base class ActionTraits.
25 * \param &_Traits source ActionTraits class to copy
26 */
27ActionTraits::ActionTraits(const std::string &_token) :
28 OptionTrait(_token,
29 &typeid(void),
30 "this is an invisible action",
31 "",
32 ""
33 )
34{
35 std::cout << "ActionTraits::ActionTraits(string &) with " << getName() << ", type " << getTypeName() << " and description " << getDescription() << std::endl;
36}
37
38/** Copy constructor for base class ActionTraits.
39 * we have to make our own copy in order to avoid references and deep-copy everything.
40 * \param &_Traits source ActionTraits class to copy
41 */
42ActionTraits::ActionTraits(const ActionTraits &_Traits) :
43 OptionTrait(_Traits.getName(),
44 _Traits.getType(),
45 _Traits.getDescription(),
46 _Traits.hasDefaultValue() ? _Traits.getDefaultValue() : "",
47 _Traits.hasShortForm() ? _Traits.getShortForm() : ""
48 )
49{
50 MenuTitle = _Traits.MenuTitle;
51 MenuPosition = _Traits.MenuPosition;
52 for (options_const_iterator iter = _Traits.Options.begin(); iter != _Traits.Options.end(); ++iter) {
53 Options.insert(
54 std::pair< std::string, OptionTrait *> (
55 iter->first,
56 new OptionTrait(iter->second->getName(),iter->second->getType(),iter->second->getDescription(), iter->second->getDefaultValue(), iter->second->getShortForm())
57 )
58 );
59 }
60 std::cout << "ActionTraits::ActionTraits(ActionTraits &) with " << getName() << ", type " << getTypeName() << " and description " << getDescription() << std::endl;
61}
62
63/** Copy constructor for base class ActionTraits.
64 * we have to make our own copy in order to avoid references and deep-copy everything.
65 * \param &_Traits source OptionTrait class to copy
66 */
67ActionTraits::ActionTraits(const OptionTrait &_Traits) :
68 OptionTrait(_Traits)
69{
70 std::cout << "ActionTraits::ActionTraits(OptionTrait &) with " << getName() << ", type " << getTypeName() << " and description " << getDescription() << std::endl;
71}
72
73/** Constructor for base class ActionTraits.
74 * Deletes all present Options.
75 */
76ActionTraits::~ActionTraits()
77{
78 for (options_iterator iter = Options.begin(); !Options.empty(); iter = Options.begin()) {
79 delete (iter->second);
80 Options.erase(iter);
81 }
82}
83
84
85/** Returns menu title for this ActionTrait.
86 * \return ActionTraits::MenuTitle as std::string
87 */
88const std::string& ActionTraits::getMenuName() const
89{
90 return MenuTitle;
91}
92
93/** Returns menu title for this ActionTrait.
94 * \return ActionTraits::MenuPosition as std::string
95 */
96const int ActionTraits::getMenuPosition() const
97{
98 return MenuPosition;
99}
100
101/** Returns Description for the given option of this ActionTrait.
102 * \param &token token of option
103 * \return reference to OptionTrait
104 */
105OptionTrait const & ActionTraits::getOption(const std::string &token) const
106{
107 ASSERT(Options.find(token) != Options.end(),
108 "ActionTraits::getOption() - Option not found!");
109 return *(Options.find(token)->second);
110}
111
112/** States whether given option (token) is present or not.
113 * \param &token name of option
114 * \return true - option present, false - not
115 */
116bool ActionTraits::hasOption(const std::string &token) const
117{
118 return (Options.find(token) != Options.end());
119}
120
121/** States whether this Action has options at all.
122 * \return true - options present, false - no options
123 */
124bool ActionTraits::hasOptions() const
125{
126 return (Options.begin() != Options.end());
127}
128
129/** Forward iterator from beginning of list of options.
130 * \return iterator
131 */
132ActionTraits::options_iterator ActionTraits::getBeginIter()
133{
134 return Options.begin();
135}
136
137/** Forward iterator at end of list of options.
138 * \return iterator
139 */
140ActionTraits::options_iterator ActionTraits::getEndIter()
141{
142 return Options.end();
143}
144
145/** Constant forward iterator from beginning of list of options.
146 * \return constant iterator
147 */
148ActionTraits::options_const_iterator ActionTraits::getBeginIter() const
149{
150 return Options.begin();
151}
152
153/** Constant forward iterator at end of list of options.
154 * \return constant iterator
155 */
156ActionTraits::options_const_iterator ActionTraits::getEndIter() const
157{
158 return Options.end();
159}
160
161
Note: See TracBrowser for help on using the repository browser.