UltimateTicTacToe - ActionScript Style!

In an effort to complete this month's code challenge, I've started off with the basics. Below I have for review, a class that I can call to create the square/button a user would click to mark a spot on a Tic Tac Toe board. I haven't done any game logic, so I could very easily be forgetting something my buttons need to have.

• Would this package stand up to any future additions?

• How is my objected orientation here?

• Is there anything I can do to compact it?

• Am I following common practices?

```
package
{

import flash.display.MovieClip;
import flash.events.MouseEvent;

public class SquareSpace extends MovieClip
{

public static const SQUARE_LENGTH:int = 30;
public static const PLAYER_A_OWNED_SQUARE:int = 0;
public static const PLAYER_B_OWNED_SQUARE:int = 1;
public static const NEUTRAL_SQUARE:int = 2;
public static const AVAILABLE_SQUARE:int = 3;
public static const STATE_COLORS:Array = [0xFF3333,0x3333FF,0xDDDDDD,0x33FF33];

private var _state:int = AVAILABLE_SQUARE;
private var _square:MovieClip;

public function SquareSpace(boxX:int, boxY:int)
{
_square = formSquare();
_square.x = boxX;
_square.y = boxY;
_square.buttonMode = true;
}

public function addSquare():MovieClip
{
_square.addEventListener(MouseEvent.CLICK, sendClick);
return _square;
}

private function sendClick(event:MouseEvent):void
{
dispatchEvent(new MouseEvent(MouseEvent.CLICK));
}

public function changeSquareState(sqrState:int):void
{
if (sqrState >= PLAYER_A_OWNED_SQUARE && sqrState <= AVAILABLE_SQUARE) {
_state = sqrState;
_square.graphics.beginFill(STATE_COLORS[sqrState]);
_square.graphics.drawRect(0,0,SQUARE_LENGTH,SQUARE_LENGTH);
_square.graphics.endFill();

Overall this is looking pretty good. I do have a few questions and suggestions based on what I see.

-
Constructor

Your class extends MovieClip therefore it IS a Movieclip. What is the purpose of the inner _square class variable? Is SquareSpace meant to be purely a logic controller class with _square being the visual representation of the square? If so there is no need for SquareSpace to extend MovieClip. I would look at EventDispatcher or just plain Object.

-
addSquare/sendClick

I'm not sure I'm understanding the purpose of these methods. Can you explain your thought behind them?

-
changeSquareState

For state control/detection you generally want to use a switch statement. That way adding more states in the future is much easier.

-
The State Constants

This is more of a cosmetic comment but I usually prepend any constants with the word STATE_ so that you/other developers can easily tell that those are values intended to be used for state control and passed into changeSquareState.

-
Argument Error

A really minor comment but I like to include the state which is not acceptable in the error statement. Makes debugging in the future easier. Not a big deal though.

-
Future Design

As you build out this game do you plan on using the Flash timeline to position and configure your assets or are you going pure AS3?

-
Package

Generally you want to organize each of your classes into a package. If you're not using a proper IDE like FlashBuilder, FlashDevelop, or FDT this will be a bit of a pain but it's good practice none the less. Packages generally look like reversed domain names so for example your SquareSpace class might be in the following package com.alex.ultimatetictactoe.square

StackExchange Code Review Q#42642, answer score: 8