[Solved] vJass Duel System crashing and I don't know why

[LightRipper]

The map below has a sweet duel system and I want it in my map. I copied it and gives me a crash with something about an "i".

Any help is appreciated!

 

[Uncle]

As far as I understand, Jass doesn't support For Loops using that syntax. It should look like this:

public method getHeroId takes integer whichHero returns integer
    local integer i = 0
    loop
        if heroes[i] == whichHero then
            return i
        endif
        set i = i + 1
        exitwhen i > count
    endloop
    return i
endmethod

Or maybe try giving i a default value of 0 and keeping everything else the same.

 

[LightRipper]

It gave me this error now

 

[Uncle]

Forgot the word "then" in my original post, needs to say "if heroes == whichHero then"

 

[LightRipper]

Other error

 

[Uncle]

Same thing again. You need to adjust the For Loop to use the syntax I provided.

local integer i = 0
loop
 // put important code here
 set i = i + 1
 exitwhen i >= this.teams.getSize()
endloop

That one has two For Loops so you need to do it twice (a loop within a loop). The second loop uses the integer j instead of i but the same rules apply.

 

[LightRipper]

I don't seem to have an end to this. I tried to modify it, but it gives me other errors. Probably I didn't understand correctly.

Edit: I give up on this.. is too complicated. I will try to find another duel system.

 

[Uncle]

It's a lot of work, the code is 1500 lines long.

Anyway, here's a very basic duel system I threw together that allows for 1v1 duels and could be expanded to allow for team duels and other stuff like that. It's really just to give you an idea of how you could create this yourself. Type -duel # (player number, 1->24) to challenge someone to a 1v1. They need to type -duel # using your number for the duel to start. Only one duel can occur at a time. A duel ends when a hero dies. You can cancel a duel request by typing -cancel.

 

[LightRipper]

Thank you! I appreciate you.