Solved How to search for players, as well as mobs?

[Funtime]

command /safe:
trigger:
loop players in radius 10 of player:
add 1 to {_n}
loop monsters in radius 10 of player:
add 1 to {_n}
if {_n} is more than or equal to 1:
send "&cThis is not a safe area!"
else:
send "&aThis is a safe area!"

Searching for players works but when I copied and pasted it and replaced players to monsters, it states that 'player' is not an item stack (safe.sk, line 5: loop monster in radius 10 of player:'. How do I fix this?

 

[ShaneBee]

You can create a variable to store the list of mobs you want to check.

example:

command /setupvariable <text>:
  trigger:
    if arg 1 is set:
      add arg 1 to {monsterlist::*}
      stop
    broadcast "%{monsterlist::*}%"

command /safe:
  trigger:
    loop players in radius 10 of player:
      add 1 to {_n}
    loop entities in radius 10 of player:
      loop {monsterlist::*}:
        if loop-entity = loop-value-2:
          add 1 to {_n}
    if {_n} is more than or equal to 1:
      send "&cThis is not a safe area!"
    else:
      send "&aThis is a safe area!"

 

[ShaneBee]

You can create a variable to store the list of mobs you want to check.

example:

command /setupvariable <text>:
  trigger:
    if arg 1 is set:
      add arg 1 to {monsterlist::*}
      stop
    broadcast "%{monsterlist::*}%"

command /safe:
  trigger:
    loop players in radius 10 of player:
      add 1 to {_n}
    loop entities in radius 10 of player:
      loop {monsterlist::*}:
        if loop-entity = loop-value-2:
          add 1 to {_n}
    if {_n} is more than or equal to 1:
      send "&cThis is not a safe area!"
    else:
      send "&aThis is a safe area!"

Although this is a good way around the problem, It won't work if you're including it inside a Skript. The Skript I hope to make is when a player does "/spawn" and there're players/mobs nearby, there will be a countdown to go to spawn.

 

[ShaneBee]

Edit it?

You need to use the first command just to setup the variable, after that you never touch it again, you can even delete the command.

 

[ShaneBee]

Edit it?

You need to use the first command just to setup the variable, after that you never touch it again, you can even delete the command.

I understand what you mean, but I'm planning on making this a public Skript, which having to set the mobs is not going to be popular.

 

[ShaneBee]

I understand what you mean, but I'm planning on making this a public Skript, which having to set the mobs is not going to be popular.

My god, YOU set the mob list ONE time that will be used for ALL players commands. You set the variable {monsterlist::*} and later later you use the variable "{monsterlist::*}".

Can't you understand that?

WIth the first command you set the variable, this variable will hold the mob types that should be checked. Then, you run the check for your tp plugin, or any other plugin you want to make.

Variables are global (instead of the variables starting with _ "underline"). You set them and use them as you want.

Just do this loop inside your tp plugin check:

    loop players in radius 10 of player:
      add 1 to {_n}
    loop entities in radius 10 of player:
      loop {monsterlist::*}:
        if loop-entity = loop-value-2:
          add 1 to {_n}

But you need to first set the monsters you want to detect with the command I gave you:

command /setupvariable <text>:
  trigger:
    if arg 1 is set:
      add arg 1 to {monsterlist::*}
      stop
    broadcast "%{monsterlist::*}%"