[Stormsys]

Right i compile my plugin go in game, as soon as the bomb is finished planting it dosent drop and server frezzes+ crashes:

this is my code for bomb planted:

	else if (FStrEq(name, "bomb_planted"))
	{
		int userid = event->GetInt("userid");
		edict_t *pEntity = m_Engine->PEntityOfEntIndex(userid-1);
		if(pEntity)
		{
			if(FStrEq(pEntity->GetClassName(),"player"))
			{
				this->XPBombPlant(pEntity);

			}
			
		}

	}

any ideas why it crashes?

[c0ldfyr3]

event->GetInt("userid");

You are getting the users UserID and not their Entity Index. Search the forums, this has been answered a ton of times.

[Stormsys]

Originally Posted by c0ldfyr3 event->GetInt("userid"); You are getting the users UserID and not their Entity Index. Search the forums, this has been answered a ton of times.

um gonna search that now, how comes it only affects the bomb_planted event and not the player_spawn??

[c0ldfyr3]

It effects every event :\
Every event that you use has a userid and attacker id, they are both UserID's and not Entity Index's.

edict_t *EdictOfUserId( int UserId )
{
	int i;
	edict_t *pEnt;
	for  (i=1; i<MAX_PLAYERS; i++)
	{
		pEnt = m_Engine->PEntityOfEntIndex(i);
		if (pEnt && !pEnt->IsFree())
		{
			if (FStrEq(pEnt->GetClassName(), "player"))
			{
				if (m_Engine->GetPlayerUserId(pEnt)==UserId)
				{
					return pEnt;
				}
			}
		}
	}

	return NULL;
}

[API]

IMO, Valve should have put the index in the event, but looping through and checking works. I would also check if what was returned is NULL.

[Mani]

You can make it much faster and remove the loop by creating a small 64k array lookup buffer, user ids are defined as the short type.

The setup calls are identical to the VSP names, Load(), Unload(), which must be called in priority order. i.e don't call gpManiTrackUser->Load() as the last thing in the Load() callback.

To get the player index, use gpManiTrackUser->GetIndex(user_id), if it's -1 they don't exist.

mani_trackuser.h

#ifndef MANI_TRACK_USER_H
#define MANI_TRACK_USER_H

class ManiTrackUser
{

public:
	ManiTrackUser();
	~ManiTrackUser();

	void		ClientActive(edict_t *pEntity);
	void		ClientDisconnect(player_t *player_ptr);
	void		Load(void);
	void		Unload(void);
	void		LevelInit(void);
	int		 GetIndex(int user_id) { return (int) user_table[user_id]; }
}

private:

	// Table of user ids
	char		user_table[65536];
};

extern	ManiTrackUser *gpManiTrackUser;

#endif

mani_trackuser.cpp

// Add your includes here
#include "mani_trackuser.h"

ManiTrackUser::ManiTrackUser()
{
	// Setup table for search speed lookup
	for (int i = 0; i < 65536; i ++)
	{
		user_table[i] = -1;
	}

	gpManiTrackUser = this;
}

ManiTrackUser::~ManiTrackUser()
{
	// Cleanup
}

//---------------------------------------------------------------------------------
// Purpose: Plugin Loaded
//---------------------------------------------------------------------------------
void	ManiTrackUser::Load(void)
{
	for (int i = 0; i < 65536; i ++)
	{
		user_table[i] = -1;
	}

	for (int i = 1; i <= max_players; i++)
	{
		player_t player;

		player.index = i;
		if (!FindPlayerByIndex(&player)) continue;
		user_table[player.user_id] = i;
	}
}

//---------------------------------------------------------------------------------
// Purpose: Plugin Unloaded
//---------------------------------------------------------------------------------
void	ManiTrackUser::Unload(void)
{
	for (int i = 0; i < 65536; i ++)
	{
		user_table[i] = -1;
	}
}

//---------------------------------------------------------------------------------
// Purpose: Level Loaded
//---------------------------------------------------------------------------------
void	ManiTrackUser::LevelInit(void)
{
	for (int i = 0; i < 65536; i ++)
	{
		user_table[i] = -1;
	}
}

//---------------------------------------------------------------------------------
// Purpose: Client Active
//---------------------------------------------------------------------------------
void	ManiTrackUser::ClientActive(edict_t *pEntity)
{
	if(pEntity && !pEntity->IsFree() )
	{
		IPlayerInfo *playerinfo = playerinfomanager->GetPlayerInfo( pEntity );
		if (playerinfo && playerinfo->IsConnected())
		{
			user_table[playerinfo->GetUserID()] = engine->IndexOfEdict(pEntity);
		}
	}
}

//---------------------------------------------------------------------------------
// Purpose: Check Player on disconnect
//---------------------------------------------------------------------------------
void ManiTrackUser::ClientDisconnect(player_t	*player_ptr)
{
	user_table[player_ptr->user_id] = -1;
}


ManiTrackUser	g_ManiTrackUser;
ManiTrackUser	*gpManiTrackUser;

Mani

[c0ldfyr3]

Originally Posted by Mani i.e don't call gpManiTrackUser->Load() as the last thing in the Load() callback.

Im just wondering; why not...?

[Mani]

Originally Posted by c0ldfyr3 Originally Posted by Mani i.e don't call gpManiTrackUser->Load() as the last thing in the Load() callback. Im just wondering; why not...?

Because you might want to do a lookup via user id in your late loading handling. If the call to gpManiTrackUser->Load() has not happened, the array will not have been setup yet.

Same thing with ClientDisconnect(), it should be called last as you may have functionality which depends on it earlier in the ClientDisconnect() callback.

Mani