Verbesserung
Top-Thema
Wichtig
Kleinigkeit
Wichtig
Verbesserung
Der aktuell implementierte DynamicFormBuilder ist anhand mehrerer Probleme nicht direkt als FormBuilder verwendbar:
- Er implementiert zwar ModelBuilder, aber erstellt in getModel(..) eine FormGroup, die er auch zurückgibt. Jedoch wird in der EditComponent bzw. FormComponent in der Methode createFormContext(..) explizit hart gecastet auf FormContext. Dies führt folglich zu einer ClassCastException.
- Der Konstruktor erwartet als Argument einen contextType`und `contextModel. contextType wird an keiner Stelle verwendet, ist also anscheinend überflüssig. contextModel wird an der Stelle benutzt, wo das Modell der Komponente benutzt werden müsste. Außerdem wird das Modell der Komponente als contextType benutzt. Das macht so keinen Sinn.
- Aktuell wird der DynamicFormBuilder an drei Stellen benutzt, die wiederum nur die FormGroup benötigen und keinen FormBuilder, somit würde es ausreichen den benötigten Code, zum Erstellen der FormGroup in eine Utility-Methode rauszufactoren.
Test
Kein Test.