HomePhabricator
Diffusion RTTB e9629b905d7d

Added additional dokumentation
Concern Raisede9629b905d7d

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 additional dokumentation

Details

Auditors
hentsch
Provenance
strubelpAuthored on Jun 13 2017, 4:08 PM
strubelpPushed on Jun 13 2017, 4:13 PM

Event Timeline

hentsch added a subscriber: hentsch.

Insgesamt sehr gute Arbeit! Vielen Dank 👍
Du kannst die Anmerkungen, soweit du sie sinnvoll findest, noch ändern und dann in den master mergen.

/code/algorithms/rttbDoseToVolumeMeasureCollectionCalculator.cpp
62

ich vermute, measureCollection ist eine Klassenvariable. Dann sollte sie _measureCollection heißen.

/code/algorithms/rttbDoseToVolumeMeasureCollectionCalculator.h
46

kann das passieren, dass diese nicht gesetzt werden? Dadurch, dass man den Konstruktor aufrufen muss und diese variablen festgesetzt sind, zwingen wir doch den Nutzer zu einer Initialisierung.

/code/algorithms/rttbVxDoseToVolumeMeasureCollectionCalculator.h
34

wie wird min denn gesetzt?

43

DoseToVolumeMeasureCollection::complexStatistics name ist doch bekannt, das muss nicht unbedingt gesetzt werden.

This commit now has outstanding concerns.Jun 19 2017, 11:45 AM

Commit No Longer Exists

This commit no longer exists in the repository.