Fixed all tests and programs for Dx
Concern Raisedc9d105ac55b1
Actions

Description

Fixed all tests and programs for Dx

Details

Auditors

Provenance

• strubelp	Authored on Jun 6 2017, 11:52 AM
• strubelp	Pushed on Jun 9 2017, 1:43 PM

Event Timeline

• strubelp committed rRTTBc9d105ac55b1: Fixed all tests and programs for Dx (authored by • strubelp).Jun 6 2017, 11:52 AM

lieber weniger Default parameter, auch wenn man dann nicht alles durchtesten kann. Wenn man den Nutzer zwingt, parameter explizit anzugeben, stellt man auch sicher, dass er keine Berechnungen mit einem volume -1 machen kann 😄

/code/algorithms/rttbDoseStatistics.h
93	Dx anstatt _Dx als parametername (_Dx ist klassenvariable)
/code/algorithms/rttbVolumeToDoseMeasure.h
48	gibt es einen guten Grund für die default parameter? Sonst würde ich die weglassen. -1 ist kein guter default parameter, der zeigt ja nur an, dass der parameter noch "fehlt". Besser ist es, wenn man alle parameter explizit angeben muss. Wenn es wirklich gebraucht wird, würde ich lieber zwei verschiedene Konstruktoren machen. Und dann bei der berechnung abchecken, ob alles gesetzt ist, was gesetzt werden muss.

This commit now has outstanding concerns.Jun 12 2017, 3:27 PM

• strubelp marked 2 inline comments as done.Jun 13 2017, 12:05 PM

Commit No Longer Exists

This commit no longer exists in the repository.