HomePhabricator
Diffusion RTTB 8431fe604bd4

Added all other Volume to Dose Measure Calculator
Concern Raised8431fe604bd4

This commit no longer exists in the repository. It may have been part of a branch which was deleted.This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref.

Description

Added all other Volume to Dose Measure Calculator

Details

Auditors
hentsch
Provenance
strubelpAuthored on Jun 6 2017, 3:09 PM
strubelpPushed on Jun 9 2017, 1:43 PM

Event Timeline

hentsch added a subscriber: hentsch.
hentsch added inline comments.
/code/algorithms/rttbMOCxVolumeToDoseMeasureCalculator.h
48

hier (und bei allen abgeleiteten Klassen) würde ich dir ans herzen legen, das override keyword zu benutzen: void computeSpecificValue(double xAbsolute) override;
siehe auch http://en.cppreference.com/w/cpp/language/override
so kann vom compiler gecheckt werden, ob die funktion tatsächlich überschrieben wird (sonst gibts nämlich keinen Fehler, aber passieren tut auch nix, was sehr unangenehm werden kann...)

/code/algorithms/rttbMOHxVolumeToDoseMeasureCalculator.h
26

map ist hier unnötig. DoseVoxelVolumeType ist bereits in VolumeToDoseMeasureCalculator enthalten, insofern braucht man es hier nicht.

/code/algorithms/rttbMaxOHxVolumeToDoseMeasureCalculator.cpp
1

bitte immer darauf achten, dass der RTTB header da ist 😸

/code/io/other/rttbDoseStatisticsXMLReader.cpp
104

folgendes liest sich etwas besser und ist kürzer:
rttb::algorithms::VolumeToDoseMeasure MOCx(rttb::algorithms::VolumeToDoseMeasure::complexStatistics::MOCx);

194

hier könnte man sich auch überlegen, ob man es nicht explizit angegben haben will:
MOHx.insertValue(boost::lexical_cast<double>(x)*volume / 100, boost::lexical_cast<double>(datum));
Der Unterschied ist, ob man es als std::pair verpackt angeben muss oder die rohwerte einfach als parameter übergeben möchte (ich würde mich für die Rohwerte aussprechen).
Noch was (hast du nicht geschrieben, aber ich sehe es gerade): x ist unsigned int, wird dann per boost:lexical_cast zu double gecastet. Da tuts auch ein static_cast<double>.

/code/io/other/rttbDoseStatisticsXMLWriter.cpp
119

what a beauty! Finde ich sehr viel übersichtlicher, da gekapselter!

/testing/algorithms/DoseStatisticsTest.cpp
84

auch wieder hier (s.o): algorithms::VolumeToDoseMeasure MOHx(...)

This commit now has outstanding concerns.Jun 12 2017, 4:16 PM

Commit No Longer Exists

This commit no longer exists in the repository.