Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Oettinger (unfinished) and add mood and friendliness of message #26

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Haydnspass
Copy link

Hi,

hab unfertigen Oettinger aber vor allem eine andere Frage: Hab einen optional in dem struct hinzugefügt mit dem man bei 'Message' auch die friendliness einer Nachricht setzen kann, die dann die 'mood' als property (?) der jeweiligen Klasse ändern kann.
Also beispielsweise mood ist anfangs 3 dann kommt eine Message mit friendliness -2 und dann ist mood der Person 1. Damit könnte man dann die jeweils folgenden Nachrichten handlen. Allerdings funktioniert das noch nicht so ganz und ich weiß nicht von wo ich die methode changeMood aufrufen muss. Hast du da Tipps?

@nilsvu
Copy link
Member

nilsvu commented May 1, 2016

Hi Lucas(?), coole Idee mit der mood und friendliness 👍 Und sehr gut, dass du das als optionales Attribut von Message implementiert hast.

2/2 Pt. schonmal 💯

  • Ich denke wir können das zusammen mit Neue Klasse: Zocker #17 lieber als Attribut tone mit Werten eines Enums Tone einbauen, dann ist das nicht nur eine Int-Skala. Dann ist das schonmal in jeder Message enthalten.
  • Um dann den Gemütszustand anzupassen brauchen wir eine Erweiterung der Superklasse Chatter, und zwar eine Methode, die vor responseForMessage(_:) aufgerufen wird. Ich schlage so etwas wie processMessage(_:) oder dergleichen vor, die Subklassen dann auch überschreiben können.
  • Die Methode processMessage(_:) kann dann bisher vom ViewController aufgerufen werden, eben vor responseForMessage(:_). Wir sollten das aber lieber wie in Implemented Categories #18 erwähnt in ein Conversation Struct auslagern.

Ich merge die PR erstmal nicht, bis wir das vereinheitlicht haben. Du kannst gern versuchen das zu implementieren, sonst mach ich das bei Gelegenheit mal.

required init(){
super.init(image: UIImage(named: "oettinger"))
// set initial mood of oettinger to 2 because he looks funny
self.mood = 2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants