Updates Unterwegs

[Marti Marcel]

Die updates am communication layer sollten um ca. 14:00 Uhr online sein.

Grüsse, Marcel

[marmarti]

Wie wollt ihr das eigentlich haben?

Der Client verschickt Random invites und Targetted (direkte) invites.

Was wollt ihr beim Client auf der Empfangen - Seite haben? Ist es okay, wenn auf client seite ebenfalls zwei verschiedene invites ankommen, oder wollt ihr die einheitlich behandeln?

Wollt ihr:

A -> S [InviteDirectMessage (A, B)]

S -> B [InviteDirectMessage (A, B)]

A -> S [InviteRandomMessage (A)]

S -> B [InviteRandomMessage (A)]

oder

A -> S [InviteDirectMessage (A, B)]

S -> B [InviteMessage (A)]

A -> S [InviteRandomMessage (A)]

S -> B [InviteMessage (B)]

?

[Sebastian Wicki]

Sorry für die späte Antwort, die Mail ist hier irgendwo hängen geblieben.

Wir möchten keine der beiden Nachrichten. Random-Invites werden nicht
an Clients versandt, sie sind nur vom Client zum Server:

A -> S [InviteRandomMessage (A)]

B -> S [InviteRandomMessage (B)]
C -> S [InviteRandomMessage (C)]

S -> A [NewGame(A,B)]
S -> B [NewGame(A,B)]

D -> S [InviteRandomMessage (D)]

S -> C [NewGame(C,D)]
S -> D [NewGame(C,D)]


Random-Requests werden also nicht vom Client bestätigt oder rejected.

Dafür habe ich zwei Feature-Requests die ich Thomas bereits gesagt habe:

1. Difficulty-String für Schwierigkeitsgrad (bei der InviteMessage wie
beim RandomRequest)
2. Falls noch nicht zu spät (falls ihr es schon geändert habt ist es
nicht so tragisch): Damit wir nicht überall die Funktionsignaturen
ändern müssen, wärs cool wenn die DirectInvite eigentlich nach wie vor
InviteMessage heisst.

> --
> You received this message because you are subscribed to the Google Groups
> "VS_Sudoku" group.
> To post to this group, send email to vs_s...@googlegroups.com.
> Visit this group at http://groups.google.com/group/vs_sudoku?hl=en.
>
>

[Sebastian Wicki]

Am 13. Dezember 2012 14:31 schrieb Sebastian Wicki <swi...@student.ethz.ch>:
> Wir möchten keine der beiden Nachrichten.

Ich meinte natürlich wir möchten keine der beiden Varianten. Sorry :D

[Marti Marcel]

Alles klar. Das lässt sich problemlos umsetzen.
Nur noch kurz zum sichergehen: Was passiert bei einem stinknormalen invite?

A -> S [InviteMessage(A, B)]
S -> B [InviteMessage(A, B)]

richtig?

Dann wäre da noch ein Problem mit den Namen. Wo genau braucht ihr die Namen und wo nicht?
Auf server seite braucht ihr doch:

- bei einem Random invite:
* keinen Namen, da ihr die Namen selbst bereits mit dem client assoziiert?

- bei einem normalen invite:
* den Namen des spielers, der eingeladen werden soll (begründung ist die gleiche wie beim random invite)

Auf Client seite:

- bei einem normalen invite:
* den namen des spielers, der euch eingeladen hat.

Auf client seite muss ich also im grunde genommen nicht verlangen dass der eigene name noch mitgeschickt wird, da das ja bei der Registrierung bereits geschehen sein sollte.

Ich will einfach auf Nummer sicher gehen, nicht dass ich noch unnötig viel am interface ändere.
________________________________________
Von: vs_s...@googlegroups.com [vs_s...@googlegroups.com]&quot; im Auftrag von &quot;Sebastian Wicki [swi...@student.ethz.ch]
Gesendet: Donnerstag, 13. Dezember 2012 14:32
An: vs_s...@googlegroups.com
Betreff: Re: [VS_Sudoku] Re: Updates Unterwegs

Am 13. Dezember 2012 14:31 schrieb Sebastian Wicki <swi...@student.ethz.ch>:
> Wir möchten keine der beiden Nachrichten.

Ich meinte natürlich wir möchten keine der beiden Varianten. Sorry :D

--

[Marti Marcel]

Also Random invite:

A -> S [InviteRandomMessage()]
S -> B [NewGame(A, B)]

Normaler invite:

A -> S [InviteMessage(B)] *
S -> B [InviteMessage(A)] *
B -> S [ACKMessage(inviteMessage)]

S -> A [NewGame(A, B)]
S -> B [NewGame(A, B)]

* InviteMessage ist also die einzige Nachricht, die mit einem Namen assoziiert sein müsste?
________________________________________
Von: vs_s...@googlegroups.com [vs_s...@googlegroups.com]&quot; im Auftrag von &quot;Marti Marcel [marm...@student.ethz.ch]
Gesendet: Donnerstag, 13. Dezember 2012 14:41
An: vs_s...@googlegroups.com
Betreff: AW: [VS_Sudoku] Re: Updates Unterwegs

[Sebastian Wicki]

Ja stimmt, das passt so.

[marmarti]

Die updates sind vorläufig im branch invitemsg-impl auf github. Schauts euch bitte an bevor ich euch durch den merge den code zerschiesse.

Gruss,

Marcel

[Sebastian Wicki]

Sieht von meiner Seite her gut aus. Code muss so oder so angepasst
werden, aber weil die InviteMessage immer noch gleich heisst, ist es
nur sehr wenig.. passt also.

@GUI-Team: Ist der GUI-Code von gestern noch aktuell? Weil ich den
Service anpassen muss, werde damit testen, da wäre es cool eine
aktuelle Version zu haben.

[Marti Marcel]

Da wäre noch eine letzte Sache, die euch betrifft.
Nicu hat mir noch gesagt es wäre gut, wenn in der GameOverMessage eine Liste von Punkten für die verschiedenen Spieler mitgegeben werden könnte (Siehe dazu auch im Github issues Bereich).

Wie würdet ihr die punkte gerne übergeben?

Mein Vorschlag für euch wäre, dass wir einfach eine Map<String, Integer> in der GameOverMessage speichern. Dann wäre die Nachricht auch gerade noch für Modi abseits von 1vs1 gerüstet (falls wir jemals noch dazu kommen).

________________________________________
Von: vs_s...@googlegroups.com [vs_s...@googlegroups.com]&quot; im Auftrag von &quot;Sebastian Wicki [swi...@student.ethz.ch]

Gesendet: Donnerstag, 13. Dezember 2012 15:49

An: vs_s...@googlegroups.com
Betreff: Re: [VS_Sudoku] Re: Updates Unterwegs

Sieht von meiner Seite her gut aus. Code muss so oder so angepasst

[Sebastian Wicki]

Ja, eine Map wäre prima, so speichern wir das auf dem Server intern auch.

Liegt allerdings am GUI-Team (und an der Sudoku-Logic, die die
Nachricht ans GUI weiterleitet) das auch anzuzeigen, aber notfalls
können die das Feld ja auch einfach ignorieren.

[Marti Marcel]

Passt, dann baue ich das so noch ein.

Werde bei der GameOver message nichts mehr rausnehmen, sondern einfach noch die map hinzufügen, dann sollten die Abhängigkeiten beim GUI - Team nicht gestört werden.

Schönen Abend noch!

________________________________________
Von: vs_s...@googlegroups.com [vs_s...@googlegroups.com]&quot; im Auftrag von &quot;Sebastian Wicki [swi...@student.ethz.ch]

Gesendet: Donnerstag, 13. Dezember 2012 16:15

[Sebastian Wicki]

Habe mal wieder eine neue Version verdrahtet. Habe dafür die
GUI-Klassen von gestern genommen und auf das neue API angepasst. Was
funktioniert: Invite Direct (inkl. reject), Random Match (beide mit
"medium"-Difficulty hardcoded), Set Field, Player Left.

Noch nich gestestet ist Game Over, glaube da ist noch ein Wurm in der
Lösungs-DB drin.

Was noch nicht funktioniert: Löschen eigener Werte (sendet die Logik
glaube noch nicht an den Server), Scores anzeigen (ist ja auch noch
nicht implementiert), alles andere woran ich gerade nicht denke.

[Giger Michael]

Sorry für die späte Antwort. Der code von gestern ist noch aktuell. Bei der Anzeige der Scores haben wir uns doch geeinigt, dass wir das nicht wirklich machen, sondern nur beim Führenden die Hintergrundfarbe ändern? Das sollte implementiert.

________________________________________
Von: vs_s...@googlegroups.com [vs_s...@googlegroups.com]&quot; im Auftrag von &quot;Sebastian Wicki [swi...@student.ethz.ch]

Gesendet: Donnerstag, 13. Dezember 2012 18:05

[Marti Marcel]

So viel wie ich mitbekommen habe geht es doch mehr darum die scores am _Ende_ des Spiel anzuzeigen, damit beide Spieler wissen wie knapp das Spiel verloren / gewonnen wurde.

Das mit den Hintergrundfarben ist soweit ich weiss auch noch aktuell. Zumindest habe ich nichts anderes mitbekommen :]
________________________________________
Von: vs_s...@googlegroups.com [vs_s...@googlegroups.com]&quot; im Auftrag von &quot;Giger Michael [micg...@student.ethz.ch]
Gesendet: Donnerstag, 13. Dezember 2012 18:09

An: vs_s...@googlegroups.com
Betreff: AW: [VS_Sudoku] Re: Updates Unterwegs

Sorry für die späte Antwort. Der code von gestern ist noch aktuell. Bei der Anzeige der Scores haben wir uns doch geeinigt, dass wir das nicht wirklich machen, sondern nur beim Führenden die Hintergrundfarbe ändern? Das sollte implementiert.

[Sebastian Wicki]

Der Score ist nur bei GameOver. Das mit der Hintergrundfarbe ist
zusätzlich drin und funktioniert jetzt auch.

Der GameOver-Score Ist eine Tabelle mit wer wieviel Punkte hat. Falls
ihr die nicht anzeigen könnt/wollt braucht ihr das auch nicht. Der
Server schickt sie jedenfalls jetzt mit.