Solved What is wrong here (trying to spawn an entity)?

Code:

PHP:

                        $player = $this->getServer()->getPlayer($args[0]);
                        $pk = new AddEntityPacket();
                        $pk->entityRuntimeId = Entity::$entityCount++;
                        $pk->type = 66;
                        $motion = new Vector3($player->getX(), $player->getY(), $player->getZ());
                        $pk->motion = $motion;
                        $pk->yaw = 0;
                        $pk->pitch = 0;
                        $pk->metadata = [
                            15 => [0, 1],
                            20 => [2, 0]
                        ];
                        $player->dataPacket($pk);


Error:

Code:

Argument 1 passed to pocketmine\network\mcpe\NetworkBinaryStream::putVector3() must be an instance of pocketmine\math\Vector3, null given, called in phar:///custom.phar/src/pocketmine/network/mcpe/protocol/AddEntityPacket.php on line 96 on MalakasPlayzMC\LobbyCore

 

PHP:

$zombie = Entity::createEntity("Zombie", $player->getLevel());
$zombie->spawnToAll();

 

WTF No it's a packet.

$motion ??? wtf
replace $motion to $pk->position

Edited your code.

PHP:

  $player = $this->getServer()->getPlayer($args[0]);
                        $pk = new AddEntityPacket();
                        $pk->entityRuntimeId = Entity::$entityCount++;
                        $pk->type = 66;
                        $pk->position = new Vector3($player->getX(), $player->getY(), $player->getZ());
                        $pk->motion = $player->getMotion();
                        $pk->yaw = 0;
                        $pk->pitch = 0;
                        $pk->metadata = [
                            15 => [0, 1],
                            20 => [2, 0]
                        ];
                        $player->dataPacket($pk);

 

Why not use asVector3?

PHP:

$pk->position = $player->asVector3();


 

What if I want to spawn the entity on player's X + 3?

 

PHP:

$pk->position = $player->add(3, 0, 0);
//", 0, 0" is actually not required, I added it for illustrative purposes

 

Wtf read the post.

 

PHP:

  $player = $this->getServer()->getPlayer($args[0]);
                        $pk = new AddEntityPacket();
                        $pk->entityRuntimeId = Entity::$entityCount++;
                        $pk->type = 66;
                        $pk->position = new Vector3($player->getFloorX()+3, $player->getFloorY(), $player->getFloorZ());
                        $pk->motion = $player->getMotion();
                        $pk->yaw = 0;
                        $pk->pitch = 0;
                        $pk->metadata = [
                            15 => [0, 1],
                            20 => [2, 0]
                        ];
                        $player->dataPacket($pk);

 

getFloorX() or just getX()?

 

And what exactly in the post do you mean? I don't see a problem with his suggestion.

 

Well, I used player->getX() + 3 and it is solved. Thank you @Marabou

 

return vector of player I don't like that. the OP ask to ajust +3 in getX()

 

That is your problem. In fact, it makes the code shorter and cleaner (if you want to use player's position, if you want x+3, then you can use add).

 

Wtf if this person has solved his post is that I fix his style of code and you can never judge his style of code he decided to code as his I decide to fix with his code? I edit with its code style and I assure you that its code style and your code style works I do not see the difference.

It's not because PM added a vector return on Player.php that the code is not clear I assure you that here, I show you my code I am sure you will understand nothing.

 

Oh, lol, that escalated quickly.

1.Know the difference between style and writing code that can be and should be improved.
2.You should not reinvent stuff, when it's possible.
3.

Is that supposed to prove that you write shitcode? L, m, a, o. At least you are right in something. I already don't understand a half of things you're saying.

P.S. So sad that you removed the part about your "large PHP experience", I had so much to pick on there. But it seems that you realized it, well, good for you.

 

You are the type of person who is hitting fast please avoid spamming this post because it has been solved. if you want to talk open an off-topic and talk about your life you're new and you already want to get noticed.

here does not accept anyone who spam to say anything and you judge the codes of people without knowing the true personality and if I edited my post to avoid creating 'drama' as you were excited about to wait drama.

 

To avoid "drama" you wouldn't post that "Wtf read the post". I'll recede, because you don't understand what I am saying and don't bother to make yourself understood.

 

Your explanations and your arguments do not mean anything just read I do not even understand you so much that you say sh*t.

 

Does this mean that you want to continue this debate? Look, I suggest you to finish this. It does make sense that my words don't reach you, when you ask questions like:

 

What is the relationship with this post ? Please do not mix anything.
you will be banished. you sink more and more