AI for Wheat Player dosen't work.

[Pekisa7]

Hello everyone. I just finished programing AI for the Wheat Player enemy of my upcoming map. When i put the ai in the trigger edtior then tested the map units don't do anything. Watched the Warseeker tutorial on Jass ai did everything right dosent work. Can anyone help me with this? Best regards from Pekisa7.

 

[Chaosium]

Yeah you had quite a few errors in your script as it turns out :d

The problem is that a comma was missing in most lines, you need to do it like so

There were also errors (like KNGITH instead of KNIGHT or GRYPHON_RIDER instead of GRYPHON so the game doesn't know what's this)

There's an endloop at the end but no loop to mark the beginning of the loop, I've removed it

Finally, you need to declare what is commander Zolas. I've added this at the beginning:

Replace the question marks with the raw ID of your hero in the editor (it's a 4 digit code like H50Z) and it should be good

 

[Pekisa7]

Yeah you had quite a few errors in your script as it turns out :d

View attachment 494758

The problem is that a comma was missing in most lines, you need to do it like so

View attachment 494759

There were also errors (like KNGITH instead of KNIGHT or GRYPHON_RIDER instead of GRYPHON so the game doesn't know what's this)

There's an endloop at the end but no loop to mark the beginning of the loop, I've removed it

Finally, you need to declare what is commander Zolas. I've added this at the beginning:

View attachment 494760

Replace the question marks with the raw ID of your hero in the editor (it's a 4 digit code like H50Z) and it should be good

Thank you so much. Now it works!

 

[Pekisa7]

Yeah you had quite a few errors in your script as it turns out :d

View attachment 494758

The problem is that a comma was missing in most lines, you need to do it like so

View attachment 494759

There were also errors (like KNGITH instead of KNIGHT or GRYPHON_RIDER instead of GRYPHON so the game doesn't know what's this)

There's an endloop at the end but no loop to mark the beginning of the loop, I've removed it

Finally, you need to declare what is commander Zolas. I've added this at the beginning:

View attachment 494760

Replace the question marks with the raw ID of your hero in the editor (it's a 4 digit code like H50Z) and it should be good

Hi can you check the demon ai as well. I changed it a little bit. Do i have to do the same thing as last time? Best regards from Pekisa7

 

[Chaosium]

It wouldn't work because you cannot declare stuff like this

Instead you either do this (it will fetch the raw code from the game itself)

Or declare everything in the globals at the beginning

Note that you technically don't need the names in white, it's just for your own convenience because it's much easier to read instead of "nerw" and stuff like that. But your script is just fine using the raw codes if you give it only that

 

[Pekisa7]

It wouldn't work because you cannot declare stuff like this

View attachment 495622

Instead you either do this (it will fetch the raw code from the game itself)

View attachment 495621

Or declare everything in the globals at the beginning

View attachment 495623

Note that you technically don't need the names in white, it's just for your own convenience because it's much easier to read instead of "nerw" and stuff like that. But your script is just fine using the raw codes if you give it only that

what about this ai? Whats wrong?

 

[raypack]

what about this ai? Whats wrong?

You have this:

When you should do as Chaosium told you:

DECLARE THE VARIABLES.

The solution has already been given to you. Furthermore, you declare a variable for the player but not the units?..

 

[Chaosium]

Yup as raypack said, it's just a variable problem, you're missing the = sign that says "this means that"

But all the rest is correct I highly recommend using JassDemo to check for errors in your scripts, it tells you what goes wrong and where, super useful

 

[Pekisa7]

Yup as raypack said, it's just a variable problem, you're missing the = sign that says "this means that"

But all the rest is correct I highly recommend using JassDemo to check for errors in your scripts, it tells you what goes wrong and where, super useful

View attachment 495886

Thanks it works!

 

[Pekisa7]

You have this:
View attachment 495881

When you should do as Chaosium told you:
View attachment 495882
DECLARE THE VARIABLES.

The solution has already been given to you. Furthermore, you declare a variable for the player but not the units?..

Thank you so much!

 

[raypack]

I generally advise you to not use variables for units if it causes you issues. Player variables are good enough.
Learning the raw ID of wc3 units, spells, upgrades etc becomes muscle memory after you write 10-20 scripts for each race, including some neutral hostile units.
ID of custom units is pretty simple and you can just write a comment like this:

call CampaignDefenderEx( 1,2,3, 'ogru') // Grunt
call CampaignDefenderEx( 1,2,3, 'o001') // Custom Unit X -> for example:
call CampaignDefenderEx( 1,2,3, 'o002') // Crossbowman

Like this you at least won't risk the AI not working.

Edit:
I also recommend just copying AI from the campaign maps and adapting it to fit your map(s).