| Class | Class Todos Count | Method with Todos |
|---|---|---|
| org.cb.jset.CardProperties | 0 | 1 |
| org.cb.jset.CardSet | 0 | 2 |
| org.cb.jset.JSetGameImpl.GameConnectionImpl | 1 | 0 |
| org.cb.jset.client.GameEvent | 1 | 0 |
| org.cb.jset.client.ui.JSetCardComponent.OddShape | 1 | 0 |
| org.cb.jset.client.ui.JSetLogo | 0 | 1 |
| org.cb.jset.server.JSetGameServer.Games | 0 | 1 |
| org.cb.jset.server.RemoteSetGameServer | 0 | 1 |
| Method Todos |
|---|
| findCard(): this should not be needed if equals is implemented correctly. |
| Method Todos |
|---|
| getCards(): do not expose the array like that. An Immutabble list could do it. |
| Method Todos |
|---|
| main(): move to unit test |
| Class Todos |
|---|
| we should probably turn this inner into a static inner class and make the reference to the associated Game transient |
| Class Todos |
|---|
| start -> stop and begin -> end. Am I not mixing it all here? |
| Class Todos |
|---|
| make it a subclass of RectangularShape ? |
| Method Todos |
|---|
| saveLogo(): implement the save image functionality. We probably need to specify image type (e.g. png) |
| Method Todos |
|---|
| removeGame(): ending remote games should deregister themselves |
| Method Todos |
|---|
| createGame(): specify what happens if exising name with same name exists. |