[Trigger] Help

[ShadowDemonLord]

I cant seem to get this trigger to work.

Player-Player 1 (Red) types a chat message containing (secret) is an exact match

Player 1 (Red) Equal to (PlayerName)

Unit-Create 1 (Unit) for Player 1 (Red) at (Center of region) facing Default building facing degrees.

A little help please.

 

[Tiaan]

I'm assuming that the goal of this trigger is to have player red type a message and a unit spawns for player red as a result of this event.

Im a bit confused on what your condition is checking, maybe if you clarified on exactly what your goal of this trigger is we could help you more

 

[Dr Super Good]

Player 1 (Red) Equal to (PlayerName)
Is impossiable. You can not compair a player object to a string.

I think what you are after is a string comparision with pesudo code of.
If player name of Player(0) (Red) is equal to (PlayerName) then

And remember that string comparisons are case sensitive. Thus I advise converting the player's name to eithor pure upper or lower case before running the comparison on the string of the same case.

This is to compensate with the fact Battlenet kindly lets you change the case of the letters in your player name without having to create a new account as you know. Incase you did not, basically someone like me can log on as "dRsUPERgOOD" instead of my normal "DrSuperGood" without having to do nothing more than change the name I log on with. Many fools who make save systems completly ignore this obvious and vital fact resulting in people not being able to use the same codes when on the same account with different caseing. DarkDeeds 6 (the final version) suffered from a bug where upper case characters resulted in curropted codes which had to be fixed with this.

 

[Marcelo Hossomi]

Could you post the whole trigger (copied from the Trigger Editor using "Copy as text" command) in

• tags? It makes it a lot easier :D

 

[ShadowDemonLord]

Im trying to get it so when I am playing the game and I am in red slot that i can type a message and get a special unit.

 

[Smiddle]

• depositin' cans
  • Events
    • Player - Player 1 types a message containing Secret as An exact match
    • Player - Player 2 types a message containing Secret as An exact match
    • Player - Player 3 types a message containing Secret as An exact match
    • Player - Player 4 types a message containing Secret as An exact match
    • --- Do the same as above with all players. ---
  • Conditions
  • Actions
    • Set Point_Var = (Center of Region)
    • Unit - Create 1 Unit for (Triggering player) at Point_Var facing Default building facing degrees
    • Custom script: call removeLocation( udg_PointVar )

Point_Var is a point variable.

I didn't quite understand if you wanted only Player 1 to be able to get the unit, or all players.

 

[ShadowDemonLord]

Just player 1, but i wanted it to where it had to have my profile name inorder for the trigger to work, that way others cant really use that special unit.

 

[Dr Super Good]

...... Like I said.

set namestring to the player name of the triggering player
set namestring to lowercase namestring
if namestring is equal to your account name in lower case then
create your unit
endif

If you seriously can not make this, you should reconsider making a map in the first place and perhapse go back to fidgeting around with triggers until you can make this.

 

[ShadowDemonLord]

Events
Player - Player 1 (Red) types a chat message containing -my loves heart is my own as An exact match
Conditions
Player 1 (Red) Equal to sirandrews
Actions
Unit - Create 1 The Dead Knight for Player 1 (Red) at (Center of Spawning Zone <gen>) facing Default building facing degrees

Theres my trigger...

 

[Dr Super Good]

Player 1 (Red) Equal to sirandrews

Obviously that will not work. Like I said you can not compair a player to a string. You have to compair the name of player in lowercases to your account name in lower cases.

 

[ShadowDemonLord]

So what should I put instead? And how would I go by doing it?

 

[ALiEN95]

is it possible to do some action only if player is called (for example) PITER?

 

[Dr Super Good]

if StringCase(GetPlayerName(GetTriggerPlayer()),false) == "drsupergood" then
    //Create your unit here. . . 
else
    //Inform the user that it only works for you to be helpful (else is not required but advisable)
endif

Replace drsupergood with your account name (all letters in lower case) and you are good to go and can log in as any form of your account name casing and still use the command.

If you still can not use this code which will save / compile, I advise eithor giving up mapping or stopping your map project and learning how to trigger. It is pointless even trying to make a map if you are not familar with the basics as it is impossiable to make anything high quality without understanding it.

 

[ALiEN95]

Thx it workers great,+ rep
i could also ban some idiots

some people know how to edit maps but they just don´t work on JASS triggers

btw do you know why is WE so "stupid" to crash if you type custom script code wrong?
why not turning off trigger always?

 

[Dr Super Good]

Because WE was programmed badly and is not very robust.

Use JNGP as its syntax checker is way more reliable.

 

[Reaper2008]

Or Jasscraft, if you don't want all the fancy stuff/ have Kaspersky.

 

[Dr Super Good]

I have Kaspersky and use JNGP no problem... What are you on about...

 

[Reaper2008]

Really? Kaspersky removed half the JNPG files 3 times on 2 different PCs.

 

[Dr Super Good]

Add them to the exceptions list....

 

[ShadowDemonLord]

• Events
  • Player - Player 1 (Red) types a chat message containing -my loves heart is my own as An exact match
• Conditions
  • (Name of (Triggering player)) Equal to SirAndrews
• Actions
  • Unit - Create 1 The Dead Knight for Player 1 (Red) at (Center of Spawning Zone <gen>) facing Default building facing degrees

You have been proven wrong! Haha.

 

[Dr Super Good]

Now wait till the person logs in as "sirandrews" or "SIRANDREWS" and watch everyone laugh at him.

Really just do what I said. It will save you looking a fool if you change the case on your account name.

 

[ShadowDemonLord]

Ive already tested it and it works perfectly.

 

[Dr Super Good]

Well good for you, however be aware that the trigger you posted will not work if you change the case of any of your account name letters as "a" is not the same as "A" in string comparisons.

 

[ShadowDemonLord]

Ya I know that, ashame I never change accts so what would be the point.

 

[BlargHonk]

I don't really think we should be helping him really, seems like he is adding cheats to a map.

But anyways, do what Dr.Supergood said, save the location to a variable, create unit at variable, and destroy variable

Yes I am a leak hore.

(Nazi isn't "appropriate")

 

[ShadowDemonLord]

No it is not cheating, i know how to put cheats in a map. This is so the moderators can get there heros, there not that special, just look cooler and wouldnt be in any races hero section.

 

[BlargHonk]

so the moderators can win the game easier, huh.

 

[ShadowDemonLord]

I JUST said that they arnt stronger or anything they just look cooler and arnt in any race.

 

[BlargHonk]

Still don't like it.

 

[ShadowDemonLord]

If it makes you happier, the moderaors only have two spells compared to the races heros having 4 a complete set. And NO The moderator spells are not rigged... there simple spells.

 

[Dantark]

Try this using string array str[]

• set str[1] = (entered chat string)
• set str[2] = (name of (typing player))
• if
• str[1] = str[2]
• then
• unit - create big and angry unit
• else
• do nothing

if i understand right

 

[BlargHonk]

Ummmm, What?

That trigger makes my head hurt >.<

And not because it's complicated, it's not done right >.<

Edit: YAY 247 posts

24/7

 

[Dantark]

He want to check player name and i dont see any way to do it

 

[BlargHonk]

O my lawd

We have someone who knows Bananas about triggers >.<