Code-Review-Tasks

[ghorwin]

• IBKMK::Polygon3D (computeNormal() und updateLocalCoordinateSystem()) -> Konsistenzforderung
• Member-Dokumentation (fast überall, wo Quelltext hinzugefügt wurde, fehlt die Dokumentation der Membervariablen und -funktionen)
• NANDRAD::ConstructionInstance - Konsistenzprüfungen + Überarbeitung der ConnectedWindows/EmbeddedObject Referenzierungen...
• NM_ConstructionStatesModell - LW Radiation-Exchange unter Einbeziehung der Fenster; hier müsste man erstmal noch das Berechnungsmodell klären; Bitte kein Schnellschuss einer Implementierung hier!
• ThermalNetworkStatesModel::calculateErrorWeightFactors() -> review/aufräumen
• RC_VicusClipping : Algorithmus durchsehen/überarbeiten (aufwändig!); Fehlerbehandlung
• SVDatabase : Generell: löschen anderweitig referenzierter Elemente klären; Konsistenzforderung lockern und ÜBERALL von ungültigen Querverweisen ausgehen?
• SVDatabaseEditDialog : review
• Network-Edit-Property-Widgets (da jetzt in öffentlicher Verwendung/Usability Review und Implementierung durchsehen)
• SVZoneTemplateEditDialog : Behandlung ungültiger Referenzen (siehe SVDatabase Task)
• VICUS Datenmodellkonsistenz; generell verhindern, dass bei Modifikationen des Datenmodells identische IDs entstehen -> alle nachgelagerte "Fixes/Prüfungen" entfallen
• SVPropBuildingAcousticTemplatesWidget - grundlegendes Thema "Raumakustik" -> klären ob Zoneneigenschaften oder unabhängige, assoziierte Parametrierung (wird auch nur für die Oberfläche benötigt, da nicht Teil von NANDRAD); ist das wirklich ein Zonensubtemplate?
• SVPropBuildingSurfaceConnectionWidget : Fehlerbehandlung / entfernen der inkonsistenzchecks -> siehe VICUS Datenmodellkonsistenz
• SVView3DDialog - review + Fehlerbehandlung
• Vic3D::Scene -> Problem Inkonsistenz "nullptr" bei Triangulation data; das müsste grundlegend umgebaut werden; entweder es gibt die Triangulierung und dann muss sie auch gültig sein, oder nicht... der Hack mit dem "== nullptr" ist nur ein lokale Fix eines größeren konzeptionellen Problems
• VICUS_ComponentInstance::compare() warum und wozu? Ist durch Konsistenzforderung ausgeschlossen und wird beim Einlesen geprüft (einzigartige ID für jede ComponentInstance)
• VICUS::Project - Dokumentation der neuen Funktionen (vor allem map key und value!)
• VICUS::ProjectGenerator - Fehlerbahandlungen
• VICUS::Window - U-Wert Berechnung korrigieren (Rahmen)
• Übergeordnet: Klärung der Datenstruktur Surface + Subsurface und resultierender Triangulierung und Umgang mit ungültigen Polygonen/Subpolygonen; Überarbeitung "Kopieroperation"
• NetworkEdge - Dokumentation/TODOs/Klassenschnittstelle

[hirseboy]

VICUS_ComponentInstance::compare() warum und wozu? Ist durch Konsistenzforderung ausgeschlossen und wird beim Einlesen geprüft (einzigartige ID für jede ComponentInstance) --> War nur ein schneller fix, sollte aber durch Konsistenzprüfung ersetzt werden. Hast du ja bereits gelöscht...