Rank System v2.3

[Moderator]

12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.

Bribe

To improve readability, performance and map file size in Create Multiboard set "((Player number of (Matching player)) + 1)" to a variable before repeat-referencing it (set it to an integer at the top of the loop).

Also, do you notice anything silly here?:

(Name of (Player((Player number of (Matching player)))))

This can easily be simplified.

(Player number of (Owner of (Matching unit))) -> there is no matching unit, only matching player.

General efficiency problems in the indexing you have. You shouldn't be incrementing/decrementing arrays during the loop. Arrays are there for static reference, not volatile iterables.

Leaks

• Game - Display to (Player group((Player(MB_Index[3]))))...

 

[Dmazzz]

I just tried it... it is very lacking and simple.

 

[WherewolfTherewolf]

Ranking is a very tired concept (look at how the guy who made the newest inventory system got flamed).

 

[sourpet]

Their is a bug ...
-When you kill an unit it says : +1000exp but on the board it says +1exp
So the row should be name "Kills" and not "Exp", you should really fix that.

Suggestions :
-Maybe Add a Sound or/and Effect when the hero gets a new rank, because a little text saying : "Private II" its not very ... entertaining or catchy, you get it ?
-This spells will be confused if their plenty more of heroes from the same player, so maybe you should label your board like : "Red's Hero +*name of hero*" or "player 1's hero + *name of hero*" or "*player name*'s hero + *name of hero*"

Finaly :
For a final comment, beside the few stuff you could change its a nice map

 

[EvilCrizpy]

updated and fixed bugs enjoy

 

[Judgement_Dragon]

This sux and it is very simple made!

 

[skeleton]

I don't like it cause it is simple!

 

[Gaara]

Good!

 

[EvilCrizpy]

v1.2 is coming out now with updates:

Exp for when u cast spells
changed leaderboard abit agen
now lose 10exp wen die
and can get depromoted
Exp for spells is 2x level of it.

Will post v1.2 asap

 

[Spet]

Pretty cool idea but very simple

 

[EvilCrizpy]

I know but will come in handy for rpg type maps or people who dont want to do it themselves. Also very easy to understand.

 

[xXGuillermoXx]

very cool but you should be able to "-save" rank if its for save/load (like Survival)

 

[light bolt30]

this rank system sucks and doesnt work properly..

is there another rank system somewhere out there thats way better??

 

[real138]

it works perfectly fine idk wat everyone is talking about.
and so wat if its simple, some ppl might just need this a simple ranking system in gui thats easily changed.
im using it in my map but i changed it from a multiboard to so when someone types -rank they get their exp, rank, and exp needed for next rank.
If you'd like i can send you the chat based version for you too use some people might like better since it doesnt use up the space of the multiboard.
Simple yet effective GJ

 

[massiveguy]

Hmm yes i like it this was what i was looking for .. kinda.. erm but 1 ques how to change how much exp received? isit always 1? can it be changed?

 

[xctaek21]

i want something good with Save load...

 

[DarkZX]

Thank you very much for posting this, it was very easy to change around the ranks and implement this into my RPG map

+ Rep, good job and keep up the good work

 

[EvilCrizpy]

Redone the whole spell
check description for updates


Edit :

Version 2.0 is now released
Chat system
Colours added to multboard.

Update : 2.1 is now released
Fixed Multiboard/Game message rank bug
Now both are active just type -rank to see by Game message.
Also Taking suggestions.

 

[blek121]

cool!

 

[Bribe]

• Set Points[(Player number of (Owner of (Matching unit)))] = 0
• Set Level[(Player number of (Owner of (Matching unit)))] = 1
• Set CurrentRank[(Player number of (Player((Integer A))))] = 1

This will bug. (Player number of (Owner of (Matching unit))) needs to be changed to simply "Integer A".

Also, can you update the screenshot to show the updates?

 

[Bribe]

Create Multiboard

Multiboard - Create a multiboard with 4 columns and ((Number of players in (All players controlled by a User player)) + 1) rows, titled EXP Table V2.1

- Leaks

(Last created multiboard)

- That is spammed over and over, just use your variable RankBoard and that would make your trigger run more quickly.

Quote me if I'm wrong, but I believe you can do "0.00 seconds game time" as the event and it will still display

Player Group - Pick every player in (All players controlled by a User player) and do (Actions)

- This leaks a force. You also leak another "pick every player" force right below it. I advise:

• Set PlayerGroup = (All players matching ((Matching player) controller Equal to User) And ((Matching player) slot status) Equal to Is playing))
• Player Group - Pick every player in PlayerGroup and do (Actions)
  • Loop - Actions
• Custom script: call DestroyForce(udg_PlayerGroup)

(Player number of (Matching player)

- Spammed, should be set to an Integer variable at the top of the loop in order to maximize game-time performance.

Refresh Multiboard

Floating Text - Create floating text that reads |cff00ff00+1xp|r at (Position of (Killing unit))

- Leaks

(Player number of (Owner of (Killing unit)))

- Spammed, again, set this one to an integer variable an reference that instead.

Floating Text - Create floating text that reads |cff00ff00-1xp|r at (Position of (Killing unit))

- Leaks

(Dying unit)

- should just be (Triggering unit)

(Player number of (Owner of (Dying unit)))

- is also spammed.

(Last created multiboard)

- You're referencing this here again when you should just be referencing your Multiboard variable.

Display Stats

Game - Display to (Player group((Triggering player)))

- Leaks

Refresh Stats 2

(Casting unit)

- Should just be (Triggering unit)

Floating Text - Create floating text that reads (((|cff00ff00 + +) + (String((Level of (Ability being cast) for (Casting unit))))) + (xp + |r)) at (Position of (Casting unit))

- Leaks

Set CurrentRank[(Player number of (Owner of (Killing unit)))] = (CurrentRank[(Player number of (Owner of (Killing unit)))] + 1)

- There is no (Killing unit) in this event

You're again spamming (Player number of (Owner of (Casting unit))) which should be put into a variable.

 

[.VTZ.]

plz fix that bugs,
i really need this system

 

[EvilCrizpy]

All changes have been made,
changed format of triggering to kinda Indexing.

Also fixed a few leaks, removed Chat system for now, due to bugs and requires fixtures.

 

[GIANT_CRAB]

Some ideas for you, you should give different rank different bonuses, advantages.
For example, Recruit: Nothing, Private: 1% more exp, Sergeant: 2% more exp, 1% more bounty, stats bonuses etc.

 

[maddeem]

If you were to find or create an integer save/load system, this could be a very useful system.

 

[EvilCrizpy]

If you were to find or create an integer save/load system, this could be a very useful system.

Maybe later on i shall release a save/load system which has this implemented with it.
But not yet;

Evilcrizpy

 

[Foronisus]

looks nice by me

 

[EvilCrizpy]

Thankss for the feedback;

Evilcrizpy

 

[kurtgiogi]

I Tried it, it was good, good for maps like adventure, RPG MAps etc... Keep up the good work

 

[EvilCrizpy]

Thanks;
Going to add Buffs soon;
so each rank adds a different buff to the unit, this will be done once i finalize balances for it.

 

[HellHeaven]

Okay, I have a suggestion. Add titles to player's names, showing their rank because it is nicer than using a command.