spawnmob

[NiLSPACE]

maybe can someone create an plugin to spawn mobs? it would be great if someone could do that.

[NiLSPACE]

maybe can someone create an plugin to spawn mobs? it would be great if someone could do that.

??

[tonibm19]

I'll try to do that, but as you knoe I'm a noob so maybe I can't. Some time ago I made a
half-done plugin that let you to spawn spiders, creepers and other monsters so maybe I can do it. I'll try to make it like bukkit essentials spawnmob.

[tonibm19]

What I did wrong?

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

function HandleSpawnMobCommand ( Player, Split, World, OtherPlayer )     if Split[2] == nil then         Player:SendMessage( "Usage: /spawnmob <mobtype> <player>" )         return true     end     if Split[2] == creeper and Split[3] == nil then         Player:GetWorld():SpawnMob( Player:GetPosX(), Player:GetPosY(), Player:GetPosZ(), 50 )         Player:SendMessage( "Creeper spawned!" )         return true     end     if Split[2] == creeper and Split[3] == OtherPlayer then         OtherPlayer:GetWorld():SpawnMob( OtherPlayer:GetPosX(), OtherPlayer:GetPosY(), OtherPlayer:GetPosZ(), 50 )         Player:SendMessage( "Creeper spawned!" )         return true     end end

It gives me errors:
[20:01:28] LUA: Plugins/SpawnMob/main.lua:12: attempt to call method 'SendMessage' (a nil value)
[20:01:28] LUA error in cPlugin_NewLua::HandleCommand. Stack size: 3
and
[20:01:28] LUA: Plugins/SpawnMob/main.lua:12: attempt to call method 'GetWorld' (a nil value)
[20:01:28] LUA error in cPlugin_NewLua::HandleCommand. Stack size: 3

[NiLSPACE]

I think becouse OtherPlayer and creeper both are a nil value.

[tonibm19]

I think becouse OtherPlayer and creeper both are a nil value.

but if I execute /spawnmob creeper it should spawn a creeper on player pos (player who executes command) and it says GetWorld is a nil value.

[xoft]

Are you sure you have the arguments alright? The command handler callbacks take only two arguments, the first one is the split command, the second one is the cPlayer. Your function has more args and mixed up.

Secondly, to compare strings, you need to put them in quotes properly:

1	if (Split[2] == "creeper")

[NiLSPACE]

I don't see anywhere what OtherPlayer is. Its just a nil value I think. I usely use something like this: I quickly created it so there could be some mistakes in it.

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33

function ReturnMobIDFromString(Mob)     if Mob == nil then         return;     end     Mob = string.upper(Mob)     elseif Mob == "CREEPER" then         return 50     elseif Mob == "SKELETON" then         return 52     elseif Mob == "SPIDER" then         return 52     elseif Mob == "GIANT" then         return 53     elseif Mob == "ZOMBIE" then         return 52     -- ETC.     end end   Mob = ReturnMobIDFromString(Split[2]) if Mob == nil then     Player:SendMessage("Mob not found")     return true end if Split[3] ~= nil then     cRoot:Get():ForEachPlayer(     function(OtherPlayer)     if OtherPlayer:GetName() == Split[3] then         OtherPlayer:GetWorld():SpawnMob(Player:GetPosX(), Player:GetPosY(), Player:GetPosZ(), Mob)     end)     return true end Player:GetWorld():SpawnMob(Player:GetPosX(), Player:GetPosY(), Player:GetPosZ(), Mob)

[xoft]

I prefer to call the number "type", rather than "id". ID means unique, two spiders have different IDs So why not call the function StringToMobType(). If you write it in C/C++, you can also make it built in into MCS as an API (see StringToDamageType and DamageTypeToString functions in BlockID.cpp for inspiration)

Instead of doing a ForEachPlayer(), you can also do DoWithPlayer() that calls the callback only once. But then, if you have multiple players of the same name (perfectly legal in MCS), only the first one would match.

[NiLSPACE]

Patch:
  StringToMobType.patch (Size: 6.44 KB / Downloads: 377)
I didn't commit it becouse I'm not sure if I had to use int or something else.