Kara'mos Tower Defence

[Moderator]

18:08, 16th Nov 2009
ap0calypse: Rejected

See post #4 for my review

 

[ironinferno]

Review?

Good:
1.

Bad:

1. All selling from all players goes to RED:
2. To keep them from blocking, have creep set atk to 10000 and hp of the towers to 10 and if the tower block, the creep auto atk it.
3. TOO MUCH SPAWNS- cause lag even on a 100mb ghost++
4. only 1 builder? lame
5. you could set a trigger stating if a player left, all units in his region to be removed.
6. do u think , throwing cages/barrel is a little weird?

 

[Full-Iron]

Your review sucks,
2.)please you could of explained how to fix the blocking with triggers and not just explained it, i'd rather have it to auto sell the tower instead of killing it.
3.)It's 10 units each 7 seconds with a break, will add a units in map thing or something.
4.)Read what i wrote. -.-
5.)Think about, they teleport down, then they would get removed if any leaked.
6.)Read the Towers description.
Thanks for the other bugs though.

 

[ap0calypse]

Thanks for the other bugs though, dont review again and uninstall warcraft 3, thanks. (Unless you can re-do the review better.

You already have -17 rep... seriously, edit the post yourself this time and don't be rude against people who are just trying to review your map.
Not every review will be an awesome one, keep in mind that some people just don't have enough time for that, or just can't be bothered to do so.

Review:

​

Triggers:

​

The triggering is poorly done, you didn't remove leaks (the reason why it lags!), the triggers are pretty basic.
The pathing system is done in a very bad way: it creates a lot of lag and requires loads of triggers just to get the correct movement.
The countdown timer is bad: it gives time in 0.001 seconds, while it checks the time every second (creating something stupid, as you can see) - change that (I believe I have given you the timer-map... Five Tower Defence, it clearly says what you have to do for that).
For a few systems you requested, check below.

- Rating: 1/5

Objects:

​

Apparantly you do not know the damage formula (see below).
Though the description looks quite good (only the yellow color code bothers me, it's fine for the rest), the stats weren't correct (at least the damage wasn't).
At the moment, there is only 1 tower (with 2 upgrades), which goes against the map rules: "All maps must be close enough to completion to allow for a good experience by the player" (one tower with 2 upgrades is not even close to completion and does not allow a good experience).
The builder still has the unsummon/sacrifice-ability, remove those.
The Elder Loger doesn't have splash (Reason: Weapon Type = Missile) and if it did, it would deal 600% damage to everyone within 107 AoE.

- Rating: 1/5

Terrain:

​

The terrain is bad... there is nearly no tile variation, the leaves-tileset does not fit well in the map and it doesn't look good with Abyss.
At some points, the Abyss seemed randomly places (e.g.: near Red's starting location).
There is nothing to smoothen up the terrain, nothing to make it look better.

- Rating: 1/5

General:

​

The gameplay isn't orginal, but it's a TD, so I won't say too much about it. (e.g.: in my - pretty old - TD, you have the option to change builders as he levels, so this is basically less good, since you can change builders whenever you like).
There are some typo's (e.g.: "enemeys" should be "enemies").
I don't like the loading screen for a couple of reasons: 1) the lay-out looks a bit messy, 2) nearly everything that it says is not true.
The description on the hive could be a lot better, use BB-codes, create some kind of general overview, don't just mass a few things together.
No description in-game.

Overall:

​

Bad, incomplete map.
Every aspect of it is either not done, or just plain bad...
There are currently 3 towers, a bad terrain, some basic triggers and that's it.

- Rating: 1/5
- Status: Rejected

---

Block-Triggers:

Create an invisible unit which follows the pathing of the buildings.
Every X seconds, spawn this unit and when he stops, sell every building within 130 AoE of the unit.
Afterwards, make him run to the end again.

In triggers:

Trigger

• Anti Block
  • Events
    • Unit - A unit Is issued an order with no target
  • Conditions
    • (Unit-type of (Triggering unit)) Equal to Dummy Runner
  • Actions
    • Unit - Order (Random unit from (Units within 130.00 of (Position of (Triggering unit)))) to Orc Blademaster - Wind Walk
    • For each (Integer I) from 1 to 5, do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (PlayerArea[I] contains (Triggering unit)) Equal to True
          • Then - Actions
            • Set TempLoc[6] = (Center of EndRegion[I])
            • Unit - Order (Triggering unit) to Move To TempLoc[6]
            • Custom script: call RemoveLocation(udg_TempLoc[6])
          • Else - Actions

The PlayerArea[X] is the region in which a player can build.
The EndRegion[X] is the region where the creeps should run to.

Orc Blademaster - Wind Walk is the sell-command for the buildings, I have done it like this so selling takes no time at all and multiple buildings can sell at the same time.
Most TD's either take time to sell, or/and they can't sell multiple buildings at the same time (which I don't like).

Do not forget to add this action in the trigger that checks when a creep has made it to the end:

Important

• Actions
  • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
    • If - Conditions
      • (Unit-type of (Entering unit)) Equal to Dummy Runner
    • Then - Actions
      • Unit - Remove (Entering unit) from the game
      • Skip remaining actions
    • Else - Actions

Kick/Leave-triggers:

Kick-Trigger

• Init
  • Events
    • Map initialization
  • Conditions
  • Actions
    • Set Color[1] = blue
    • Set Color[2] = teal
    • Set Color[3] = purple
    • Set Color[4] = yellow
    • Set Color[5] = orange
    • Set Color[6] = green

This sets up the colors... obviously (it's best to merge this with an already existing map init trigger)

• Kick
  • Events
    • Player - Player 1 (Red) types a chat message containing -kick as A substring
  • Conditions
  • Actions
    • For each (Integer i) from 1 to 6, do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (String((Entered chat string)) as Lower case) Equal to (-kick + Color[i])
          • Then - Actions
            • Game - Defeat (Player((i + 1))) with the message: You have been kicked
            • Game - Display to (All players) for 7.00 seconds the text: (|c00444488 + ((Name of (Player((i + 1)))) + |r has been kicked))
            • Multiboard - Set the text for (Last created multiboard) item in column 1, row (1 + i)) to |c00666666[Kicked]|r
            • Set PlayerLeft[i] = True
          • Else - Actions

And this is the kick-trigger, if the message is -kick + (color defined in map init), then the player matching that color will be kicked.
Easy and efficient...

Leave-Trigger

• Player Leaves
  • Events
    • Player - Player 1 (Red) leaves the game
    • Player - Player 2 (Blue) leaves the game
    • Player - Player 3 (Teal) leaves the game
    • Player - Player 4 (Purple) leaves the game
    • Player - Player 5 (Yellow) leaves the game
  • Conditions
  • Actions
    • Game - Display to (All players) for 12.00 seconds the text: (((|c00005599 + (Name of (Triggering player))) + |r has Left the game.) + <Empty String>)
    • Multiboard - Set the text for (Last created multiboard) item in column 1, row (1 + (Player Number of (Triggering Player))) to |c00666666[Left]|r
    • Set PlayerLeft[(Player Number of (Triggering Player))] = True

In your spawn-trigger:

Action

• Actions
  • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
    • If - Conditions
      • PlayerLeft[X] = False
    • Then - Actions
      • Spawn Units
    • Else - Actions

It's best to do this with a loop... makes it easier

Damage Formula

You can see 3 damage data:

• Combat - Attack X - Damage Base (A)
• Combat - Attack X - Damage Number of Dice (B)
• Combat - Attack X - Damage Sides per Die (C)

I have given letters to each data, to make it look a bit better.
The rough formula is:

(A+B)-(A + (B * C))​

Of course, nobody feels like doing this every time...

The first number of the damage is very simple, let's say we want 50-60 damage.
The first number is 50, so we need to create 50...

A = 49
B = 1
49 + 1 = 50​

Everyone agrees with me here I hope.
Now the second damage, we have already defined A and B, so the only thing left is C.

I have made it really simple for us, since I've made B = 1.
The formula now says:

50-(A + (1 * C))
OR
50-(a + c)​

Very simple: 49 + 11 = 60, so C needs to be 11...

Overview:

• Combat - Attack X - Damage Base (49)
• Combat - Attack X - Damage Number of Dice (1)
• Combat - Attack X - Damage Sides per Die (11)

With this we have 50-60 damage.

To get simple damages, like 50-50, 45-45, 100-100, etc...
You just need to set A = (damage - 1) and B and C = 1.
Then you get something like:

• Combat - Attack X - Damage Base (49)
• Combat - Attack X - Damage Number of Dice (1)
• Combat - Attack X - Damage Sides per Die (1)

Which gives 50-50 damage.

That's all there is to it, to make it easy for yourself: try to set B=1 as much as possible.
I don't do it myself, but that's because my math isn't bad...

 

[Full-Iron]

The PlayerArea[X] is the region in which a player can build.
The EndRegion[X] is the region where the creeps should run to.

Do you mean it's a varibal or the actural area the player builds at? ^^' (If you understand what i mean. D: )

 

[ap0calypse]

Do you mean it's a varibal or the actural area the player builds at? ^^' (If you understand what i mean. D: )

Yeah, it's a variable... you set them up at map init (so EndAreaRed will become EndArea[1] etc...).

 

[Full-Iron]

Yeah, it's a variable... you set them up at map init (so EndAreaRed will become EndArea[1] etc...).

mmmm, iit still doesn't seem to work. :O