feature/config #78
No reviewers
Labels
No Label
bug
dependencies
duplicate
enhancement
help wanted
invalid
missing feature
question
refactoring
retest
review
tschüss
ui
wontfix
No Milestone
No Assignees
2 Participants
No due date set.
Dependencies
No dependencies set.
Reference: localhorst/e-commerce#78
Loading…
Reference in New Issue
No description provided.
Delete Branch "feature/config"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ -0,0 +123,4 @@return data;}public static String getInstallationName() {In den Getter ist nicht sichergestellt, dass die werte schon initialisiert sind. Andere Komponenten könnten schon vorher PostConstruct ausgeführt haben.
@ -15,3 +16,2 @@// TODO: use values form cfg.private final int NUM_SLOTS = 50;private final int NUM_SLOTS = AppSettings.getNumberOfStorageSpaces();Muss in init() eingelesen werden wegen obigen Problem
closes #9
WIP: feature/configto feature/configs.o.
@ -0,0 +1,21 @@!!org.hso.ecommerce.app.config.YAMLDataWürde eher nicht unter dem "echten" Dateinamen mit falschen Daten ins Repo commiten.
Vlcht eher "example_config.yml" oder in ne Art readme rein pasten und drüber den Pfad.
Oder Alternativ eine funktionierende Konfiguration.
Ohne die Datei startet die Anwendung halt nicht mehr.
@ -0,0 +106,4 @@public YAMLData readConfig() {YAMLData data = new YAMLData();try (InputStream inputStream = new FileInputStream("./eCommerce_config.yml")) {Vlcht besser als private/public static final String in der Klasse.
wollen wir wirklich "./eCommerce_config.yml" als Dateinamen? Hat merkwürdiges Caseing (underscore und Groß/klein-schreibung.).
@ -0,0 +116,4 @@e.printStackTrace();}return data;Führt nach im catch(...) Fall evtl zu null pointer deref. Vlcht im catch leere werte setzen oder app beenden.
Wollen wir mit leeren Werten starten oder beenden? Ich hab jetzt mal ein
System.exit()eingebaut.NAICEEE