You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Bin mir jetzt auch grae unsicher ob meine neu erstellte Klasse dabei ist. Sie heißt Zocker.
In der Konsole hab ich beim git push Befehl immer Error 403 angezeigt bekommen.
Mit der Desktop App ging das hochladen jetzt
Hallo Christoph, coole Idee, dass dein Zocker bei jeder Warum-Frage genervter wird 👍
2/2 Pt. schonmal 💯
Du musstest in Chatter eine default clause schreiben weil du neue MessageTypes hinzugefügt hast. Wenn ich das jetzt so merge müssen die anderen ihre Subklassen auch alle an die neuen MessageTypes anpassen. Nimm die neuen cases daher lieber wieder raus und versuch' die Logik auf deine Subklasse zu beschränken. Was du stattdessen auch machen könntest ist, ein neues Attribut tone oder dergleichen der Message hinzuzufügen, das Werte eines neuen Enums ´Tone` annimmt aber Optional ist.
Ich kann's also erst mergen wenn deine Logik auf die anderen Subklassen keine Auswirkung hat 😉
Hallo Nils,
Vielen Dank für die Tipps, ich versuche mal mein Programm dahin zu nachzubessern.
kann aber gut sein, dass es etwas dauern wird, weil ich mir erstmal anschauen muss, wie ich die Methoden benutze.
Hallo Christoph, coole Idee, dass dein Zocker bei jeder Warum-Frage genervter wird 👍
2/2 Pt. schonmal 💯
Du musstest in Chatter eine default clause schreiben weil du neue MessageTypes hinzugefügt hast. Wenn ich das jetzt so merge müssen die anderen ihre Subklassen auch alle an die neuen MessageTypes anpassen. Nimm die neuen cases daher lieber wieder raus und versuch' die Logik auf deine Subklasse zu beschränken. Was du stattdessen auch machen könntest ist, ein neues Attribut tone oder dergleichen der Message hinzuzufügen, das Werte eines neuen Enums ´Tone` annimmt aber Optional ist.
Ich kann's also erst mergen wenn deine Logik auf die anderen Subklassen keine Auswirkung hat 😉
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub #17 (comment)
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Zwei neue MessageTypes hinzugefügt