public class TowerFactory 
    { 

Bestand: https://github.com/jscotty/TowerDefAS3/blob/master/src/game/tower/TowerFactory.as

Plaats factories in een eigen map.

Vaak verdeel je je code:

• of aan de hand van de verschillende objecten (enemy, turret, etc.)
• of aan de hand van de design pattern (dus factories in de map 'factories'). Later dit jaar gaan we naar andere design patterns kijken.

s 

Bestand: https://github.com/jscotty/TowerDefAS3/blob/master/src/game/levelCreator/LevelCreator.as

Gebruik hele namen voor je variabelen. De compiler maakt ze wel weer klein bij het exporteren. Tot die tijd geldt: leesbaarheid boven alles.

Een uitzondering hiervoor zijn de indexes van een loop (bijvoorbeeld var i )

NormalBulletType 

Bestand: https://github.com/jscotty/TowerDefAS3/blob/master/src/bullets/NormalBulletType.as

Gebruik geen 'type' in je Class names. Het is een object met de naam NormalBullet.

soundSystem = new SoundSystem();
            soundSystem.addMusic(clickSfx);
            soundSystem.addMusic(backgroundSfx); 

Bestand: https://github.com/jscotty/TowerDefAS3/blob/master/src/Main.as

Wees duidelijk in welke Class welke logica zou moeten hebben. De logica die je nu hier hebt staan:
soundSystem = new SoundSystem();
soundSystem.addMusic(clickSfx);
soundSystem.addMusic(backgroundSfx);

dat behoort niet tot de verantwoordelijkheid van Main.as

Je schrijft pas echt Object Oriented als je de logica bij de juiste Object/Class onderbrengt.

/*graphics.beginFill(0x00ff00, 1);
            graphics.drawRect(0, -5, 150, 100);
            graphics.endFill();*/ 

Bestand: https://github.com/jscotty/TowerDefAS3/blob/master/src/assets/ButtonPH.as

Code die je niet meer gebruikt moet je weggooien. Dit houdt je repos en codebase schoon.

var enemy:Array = Game.enemyArray;
                for (var i:int = 0; i < enemy.length; i++) {
                    if (this.hitTestObject(enemy[i])) {
                        count ++;
                        if (count == 1) {
                            enemyNum = i;
                            enemy[i].particle = true;
                            enemy[i].bullet = type;
                            enemy[i].enemyId = i;
                            dispatchEvent(new Event(bulletHit));
                        }
                    }else {
                        count = 0;
                    }
                } 

Bestand: https://github.com/jscotty/TowerDefAS3/blob/master/src/game/bullets/WeakBullet.as

WeakBullet.as weet teveel van de wereld om zich heen. Hij spreekt hard properties aan van andere Classes. Dit mag niet. Je doorbreekt zo de regel van encapsulation: elk object heeft een eigen verantwoordelijkheid en denkt alleen voor zichzelf.

Dit soort logica moet je onderbrengen in een andere (aparte) Class.

private var cursor:MouseC;
        private var easy:String = "easy";
        private var medium:String = "medium";
        private var hard:String = "hard"; 

Bestand: https://github.com/jscotty/TowerDefAS3/blob/master/src/Main.as

Wees constant in je benamingen. Ik zie privates met een underscore en privates zonder een underscore. Kies één van de manieren om dit te schrijven en voer dit overal door.

trace("LOADING... <3");
            var tf:TextFormat = new TextFormat("Papyrus", 82, 0x000000,true); // variable voor de text style.
            _text = new TextField();
            _text.text = "Loading...";
            _text.y = 200; // text positie y
            _text.x = stage.stageWidth / 2.5; // text positie x
            addChild(_text);
            _text.selectable = false;
            _text.width = _text.textWidth + 300; // maximale grootte text
            _text.setTextFormat(tf);
            _text.visible = true;

Bestand: https://github.com/jscotty/TowerDefAS3/blob/master/src/PreLoader.as

Je code zou nog mooier worden als je je logica los houdt van hoe het eruit ziet. Elk project is wel geïnteresseerd in een preloader, alleen bij elk project zal het preload scherm er iets anders uitzien.
Wat dacht je van 2 Classes, 1 voor de Preloader logica en 1 voor de PreloadScreen? De preloader dispatcht events o.a. voor de PROGRESS en COMPLETED.

TowerDefAS3/src 

Bestand: https://github.com/jscotty/TowerDefAS3/blob/master/src/Main.as

Je repository wordt nagekeken. We proberen zoveel mogelijk tips en verbeterpunten te geven voor jouw code. Dit verwerken we in losse issues.

particle1.png       
witte bloed lichaam partical 2.png      
witte bloed lichaam partical 3.png      
witte bloed lichaam partical 4.png 

Bestand: https://github.com/jscotty/TowerDefAS3/blob/master/src/assets/NEnemyPH.as

Zet geen art in je map met scripts. Hanteer een nette ondervedeling in je project.