myCSharp.de - DIE C# und .NET Community
Willkommen auf myCSharp.de! Anmelden | kostenlos registrieren
 
 | Suche | FAQ

» Hauptmenü
myCSharp.de
» Startseite
» Forum
» FAQ
» Artikel
» C#-Snippets
» Jobbörse
» Suche
» Regeln
» Wie poste ich richtig?
» Forum-FAQ

Mitglieder
» Liste / Suche
» Wer ist wo online?

Ressourcen
» openbook: Visual C#
» openbook: OO
» Microsoft Docs

Team
» Kontakt
» Übersicht
» Wir über uns

» myCSharp.de Diskussionsforum
Du befindest Dich hier: Community-Index » Diskussionsforum » Entwicklung » Code-Reviews » Code Review MVVM WPF Caliburn Project erwünscht
Letzter Beitrag | Erster ungelesener Beitrag Druckvorschau | Thema zu Favoriten hinzufügen

Antwort erstellen
Zum Ende der Seite springen  

Code Review MVVM WPF Caliburn Project erwünscht

 
Autor
Beitrag « Vorheriges Thema | Nächstes Thema »
chrshndl
myCSharp.de-Mitglied

Dabei seit: 26.08.2018
Beiträge: 2


chrshndl ist offline

Code Review MVVM WPF Caliburn Project erwünscht

Beitrag: beantworten | zitieren | editieren | melden/löschen       | Top

Hallo Community,

ich bin neu hier im Forum und melde mich an, da ich für mein Hobbyprojekt gerne ein wenig Input in Form von Codereview oder sogar PullRequests haben würde.

Das Projekt habe ich getreu dem Motto "Weeks of coding can save you hours of planing" einfach mal als Projekt zur Validierung der Struktur angefangen.
Jetzt bin ich an einem Punkt, an dem ich das letzte Feature implementieren könnte (und dann Tests schreiben), allerdings bin ich mir aktuell gar nicht mehr sicher, ob die Struktur so wirklich sinnvoll ist.

Es geht um ein Projekt, mit welchem man sich eine Sammlung an Fitnessübungen anlegen kann.
Hierfür gibt es ein Tabview, welches einmal das Hinzufügen/ Ändern von Übungen erlaubt und in einem weiteren Reiter alle angelegten Übungen anzeigt, welche man dann in eine ListView packen kann um daraus einen Trainingsplan zu erstellen.
Die Auswahl an Übungen soll dann im letzten Schritt per PDF Sharp zu einem Trainingsplan gedruckt werden können.

Link zu meinem Projekt:

 Plan2Fit

Vor allem die Art und Weise, wie ich das Reloaden der TabView Inhalte realisiert habe ist nicht schön, das habe ich in den Comitts auch extra vermerkt.
Beachtet bitte, dass bisher noch nicht getestet wurde, ErrorHandling ist quasi nicht vorhanden.

Ich hoffe auf rege Anteilnahme und viele Vorschläge, wie erwähnt auch gerne direkt im Repo.
Haltet euch nicht zurück, ich habe mich hier nicht angemeldet, damit mein Code schön geredet wird, sondern um Fehler aufgezeigt zu bekommen!

Liebe Grüße
Chris
26.08.2018 20:36 E-Mail | Beiträge des Benutzers | zu Buddylist hinzufügen
markl markl ist männlich
myCSharp.de-Mitglied

avatar-4095.gif


Dabei seit: 29.09.2016
Beiträge: 77
Entwicklungsumgebung: VS2017 and VSC


markl ist offline

Beitrag: beantworten | zitieren | editieren | melden/löschen       | Top

Schon mal eine kurze Anmerkung:

obj, bin und nupkg gehören nicht ins Repo.
27.08.2018 09:49 E-Mail | Beiträge des Benutzers | zu Buddylist hinzufügen
chrshndl
myCSharp.de-Mitglied

Dabei seit: 26.08.2018
Beiträge: 2

Themenstarter Thema begonnen von chrshndl

chrshndl ist offline

Beitrag: beantworten | zitieren | editieren | melden/löschen       | Top

Jap, habe zwar das .gitignore hinzugefügt aber vergessen die files nach dem ersten comit auch zu löschen.
Danke!
27.08.2018 10:23 E-Mail | Beiträge des Benutzers | zu Buddylist hinzufügen
Baumstruktur | Brettstruktur       | Top 
myCSharp.de | Forum Der Startbeitrag ist älter als ein Jahr.
Der letzte Beitrag ist älter als ein Jahr.
Antwort erstellen


© Copyright 2003-2020 myCSharp.de-Team | Impressum | Datenschutz | Alle Rechte vorbehalten. | Dieses Portal verwendet zum korrekten Betrieb Cookies. 01.04.2020 04:30