Potential NPE in Edge

Edge.java#L69 calls navigate(start, goal), but both arguments argument (start and goal) might be null and will cause navigate to throw an NPE.

On:

building being built
robber being moved
our resources being updated
our dev cards being updated (bought or used)

game.getScreen().render(1)

Potentially dodgy thread shit

The constructor of client.Client starts a new thread before the constructor of any subclass of client.Client is run.

	public Client()
	{
		setUpConnection();
		thread = new Thread(this);
		thread.start(); // here
	}

New game beautification

Make text boxes act like text boxes
Make header look like a header
Have sane (i.e., single player) defaults

Not moving on to game Screen

When hex-event merged with Master the game screen stopped loading. Sometimes it waits forever for client setup waiting for game information.

Null Pointer Exception on Client Shutdown

Getting a NPE when trying to shutdown server.

// in catan.SettlersOfCatan
public void dispose()
{
	client.shutDown(); // here
	skin.dispose();
}

java.lang.NullPointerException
	at connection.LocalServerConnection.getMessageFromServer(LocalServerConnection.java:34)
	at client.EventProcessor.processMessage(EventProcessor.java:262)
	at client.EventProcessor.run(EventProcessor.java:41)
	at java.lang.Thread.run(Thread.java:745)

Set logging level based on environment

Set to errors only for production, debug for development etc.

Add the ability for offline mode, by allowing a server to be run locally with a client. This then means a server could have 3 local AIs and a local player, allowing for play without an internet connection.
To do this, the type of connection between a server and client must be abstracted over. Likewise, the Client must be generalised to allow for remote or local connection with the server.

HUD elements

Essential

Beta

Alpha

Bank trade
Port trade
Player trade
Chat

Start server etc on menu button click

User input during gameplay

event listeners on board elements
write turn state
notify worker
render options menu when exists

(Don't forget about trade and development cards!)

Ask for # Ai when hosting multiplayer server

When wanting to host a multi-player server, there should be a box which pops up prompting the user to indicate a number of AIs (0, 1, or 2) for the server to run, with the implication that the remaining positions will be filled by online players. This will aid in testing and demo-ing immensely, as we would only have to spin up 1 remote client (in the case of 2 Ai players), instead of 3 (in the case of 0). In the code, I just need a way of retrieving a number the user entered from a UI element, which I then pass to an initialisation function for one of our LocalServers. Super easy. Perhaps a check-box option or something?

Crash when you don't select AI difficulty

100% CPU usage

Move to non-continuous rendering

We have lots of still scenes, lets not kill the CPU if we don't have to.

NPE on game exit when hosting multiplayer game

Reproduce:

Checkout master
Build and run
Choose Host New Multiplayer Game
Exit the game by trying to close the window
Profit (Infinite NPEs in console!)

Refactor Game class for client side usage

Wrapper?

Turn state
My colour

Build "random walking" AI

Build a simple AI that will:
-Determine all possible legal moves.
-Randomly assign a rank to each move.
-Pick the move with the highest value and play it.

Build macOS, Linux and Windows apps

Basic main menu

Background
Title
Button to host game
Button to join existing game
Button to quit game

Client logging

I am not aware of any point at which Gdx.app == null. Have you seen this happen?

	public void log(String tag, String msg)
	{
		if (Gdx.app == null)
		{
			System.out.println(tag + ": " + msg);
		}
		else
			Gdx.app.log(tag, msg);
	}

Move validation on Client - UI

@lmcmenemie and @bg3232 , please not the method Client.sendMoves(Turn) will need move validation. Simply just forgot to do it. I'll add it in tomorrow.

Likewise, what do you need from me for retrieving valid move possibilities? I have the master list of all possibilities stored in Turn objects. Do you want it in a different format? I swear one of you said something about wanting to retrieve valid possibilities based upon a given clicked object? Either way just let me know, and I'll get it to you tomorrow. You can continue working assuming whatever it is that you'll need will be there.

Tile assets made
Board drawn (code)
Tiles drawn on board
Camera/viewport configured

Ask for username when entering lobby.

Game world elements

Port
Robber
Chits

Create models and integrate.

Huge lag

UI issues

Minimum width and height
Stretch background on large window
Decide on chromeless window

Integer used in place of enum constant?

5284dff

Remote setup screen issues

Add background or border to text input
Make text input wider

Client worker

Finding possible moves given a turn thingy
Taking a complete action, building a protobuf and sending it to server

Implement Bank limits

Implement the bank on both the client and server side.

The server will prevent allocations / trades if there are no more resources. It will also prevent the building of settlements, roads, and cities if there are no more.
The client will maintain the status of the bank so that it will know when certain limits have been reached. Can be used for move validation for both AI and the client

Move .proto files to core/src/main/proto
Compile with gradle

Menu graphics

Game title
Buttons

Continuous integration (Travis)

@keananf, can you set up CI? It's super simple.

I will commit a config file, while you go to Travis CI and flip a switch on 😁

Indicating initial game preferences / game display questions

Hey @lmcmenemie and @bg3232 , these were just a few things I was thinking about, and that would be needed from your end:

When wanting to join a remote game, there should be a way for the player to indicate the host and port for the server its connecting to. As of now, there hasn't been any discussion of a server discovery protocol between the groups, so I think its safe to assume this is adequate.
When wanting to host a multi-player server, there should be a box which pops up prompting the user to indicate a number of AIs (0, 1, or 2) for the server to run, with the implication that the remaining positions will be filled by online players. This will aid in testing and demo-ing immensely, as we would only have to spin up 1 remote client (in the case of 2 Ai players), instead of 3 (in the case of 0). In the code, I just need a way of retrieving a number the user entered from a UI element, which I then pass to an initialisation function for one of our LocalServers. Super easy. Perhaps a check-box option or something?
Is there going to be a HUD of some type (indicating basic player info like colour, number of resources, cities, dev cards, settlements, etc.)? Given that its a board game, it could be cool to have each player's info displayed in a corner of the screen, perhaps? I can tell you how all this data is stored in the client and how you can access it in a thread safe manner.
Do we have art assets for settlements, cities, roads, or dev cards? If not, how long will these take to make for each colour?
Do Node and Edge objects accurately correspond to the gaps between the hexes? If not, do you know if the coordinate conversion you use for hexes would work for both of them too?

keananf / settlersofcatan Goto Github PK

settlersofcatan's Introduction

Settlers of Catan

Building

Linux and macOS

Running

Linux and macOS

Hosting a Game

Single Player against AI

AI Only Game

Host a Game Against Network Players

Joining a Game

settlersofcatan's People

Contributors

Watchers

settlersofcatan's Issues

Essential

Beta

Alpha

Recommend Projects

Recommend Topics

Recommend Org