MVP-Refactoring #139
Reference in New Issue
Block a user
Delete Branch "refactoring/68-mvp-refactoring"
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?
MVP-Refactoring
Zugehörige Issue(s):
Closes #68
Beschreibung
Großes Refactoring, inkl Ergänzung von Kommentaren, Optimisierungen von den
build()-Methoden, Anpassungen von ÜbersetzungenÄnderungen
build()-Methoden auf Optimierung überprüftCustomThemeerweitert und Änderungen in die App propagiertZusätzliche Anmerkungen
Die
MatchSummaryTilewurde wegen #137 nicht refactoredWIP: MVP-Refactoringto MVP-Refactoring@@ -19,4 +19,3 @@class _CreateGroupViewState extends State<CreateGroupView> {final _groupNameController = TextEditingController();late final AppDatabase db;warum für den rest comments aber nicht für db?
Habe Datenbank und
isLoadingnicht kommentiert, weil die so allgegenwärtig sind, dass die eigentlich keinen kommentar brauchen. Soll ich einen hinzufügen?@@ -53,3 +49,1 @@),centerTitle: true,),appBar: AppBar(title: Text(loc.create_new_group)),schatz irgendwas passt hier nicht:

warum hast du alles weggelassen außer den title??? das ganze design ist anders jetzt und kacke auch
@@ -21,7 +21,11 @@ class GroupsView extends StatefulWidget {class _GroupsViewState extends State<GroupsView> {late final AppDatabase db;hier ebenfalls kein comment?
@@ -21,9 +21,17 @@ class HomeView extends StatefulWidget {class _HomeViewState extends State<HomeView> {bool isLoading = true;hier ebenfalls kein comment?
@@ -38,4 +40,2 @@appBar: AppBar(backgroundColor: CustomTheme.backgroundColor,scrolledUnderElevation: 0,leading: IconButton(auch hier mit dem design weird alles aufeinmal anders wie sonst

@@ -58,3 +53,2 @@),centerTitle: true,title: Text(loc.choose_group),),@@ -138,3 +132,2 @@/// Filters the groups based on the search query./// TODO: Maybe implement also targetting player names?/// Filters the groups based on the search [query].warum query in klammern? entweder ganz oder garnicht aber wat soll dat hier
Weil das der Input Parameter der Funktion ist
@@ -40,4 +40,3 @@scrolledUnderElevation: 0,leading: IconButton(icon: const Icon(Icons.arrow_back_ios),onPressed: () {@@ -59,3 +54,2 @@),centerTitle: true,title: Text(widget.match.name),),@@ -28,6 +28,8 @@ class _MatchViewState extends State<MatchView> {late final AppDatabase db;kein comment?
@@ -28,6 +28,8 @@ class _MatchViewState extends State<MatchView> {late final AppDatabase db;bool isLoading = true;kein comment?
@@ -31,3 +43,1 @@List<Player> selectedPlayers = [];List<Player> suggestedPlayers = [];List<Player> allPlayers = [];late final AppDatabase db;kein comment?
@@ -32,3 +43,2 @@List<Player> suggestedPlayers = [];List<Player> allPlayers = [];late final AppDatabase db;bool isLoading = true;kein comment?
Sieht gut aus :)
gucke später nochmal genauer, hatte bei der Arbeit leider keine Zeit und muss auch jetzt schon direkt wieder weg. Aber hab nichts offensichtlich falsches im code gefunden, außer den Sachen, die Mathis schon gesagt hat.
Könnt also mergen :) wenn ich was finde mach ich einfach nen issue
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings