[ROZWIĄZANE] Używanie get_user_team zamiast is_user_connected + get_user_team

Witam,

pytanie takie z ciekawości. Patrzę sobie w kod źródłowy AMXX'a i znalazłem:

static cell AMX_NATIVE_CALL is_user_connected(AMX *amx, cell *params) /* 1 param */
{
      int index = params[1];
      if (index < 1 || index > gpGlobals->maxClients)
            return 0;
      CPlayer* pPlayer = GET_PLAYER_POINTER_I(index);
      return (pPlayer->ingame ? 1 : 0);
}

static cell AMX_NATIVE_CALL get_user_team(AMX *amx, cell *params) /* 3 param */
{
      int index = params[1];
      if (index < 1 || index > gpGlobals->maxClients)
            return -1;
      CPlayer* pPlayer = GET_PLAYER_POINTER_I(index);
      if (pPlayer->ingame)
      {
            // SidLuke, DoD fix
            if (g_bmod_dod)
            {
                  int iTeam = pPlayer->pEdict->v.team;
                  if (params[3])
                  {
                        const char *szTeam = "";
                        switch (iTeam)
                        {
                        case 1:
                              szTeam = "Allies";
                              break;
                        case 2:
                              szTeam = "Axis";
                              break;
                        }
                        set_amxstring(amx, params[2], szTeam, params[3]);
                  }
                  return iTeam;
            }
            //
            if (params[3])
            {
                  set_amxstring(amx, params[2], pPlayer->team.c_str(), params[3]);
            }
            return pPlayer->teamId;
      }
      return -1;
}

Wnioskuje, że gdy mam taki kod:

if(is_user_connected(id) && get_user_team(id) == 2)

To mogę użyć:

if(get_user_team(id) == 2)

bez obawy, że wywali błąd o graczu invalidzie? Z tego co widzę to kod się powtarza tam...

Tak, można tego użyć w ten sposób. Funkcja sprawdzania gracza nie zawierza ślepo w istnienie gracza na serwerze. Jednak cs_get_user_team chyba już się w tym momencie wysypie.

static cell AMX_NATIVE_CALL cs_get_user_team(AMX *amx, cell *params) // cs_get_user_team(index); = 1 param
{
// Get user team
// params[1] = user index
// Valid entity should be within range
cell *model;
CHECK_PLAYER(params[1]);
// Make into edict pointer
edict_t *pPlayer = MF_GetPlayerEdict(params[1]);
if ((params[0] / sizeof(cell)) >= 2)
{
model = MF_GetAmxAddr(amx, params[2]);
*model = *((int *)pPlayer->pvPrivateData + OFFSET_INTERNALMODEL);
}
return *((int *)pPlayer->pvPrivateData + OFFSET_TEAM);
}

Chyba, że CHECK_PLAYER sprawdza.

Tak samo jest z 

if(is_user_connected(id) && is_user_alive(id))

 

V

if(is_user_alive(id))

 

Bo Gracz niepołączony nie może być żywy. 

 

 

 

 

static cell AMX_NATIVE_CALL cs_get_user_team(AMX *amx, cell *params) // cs_get_user_team(index); = 1 param
{
// Get user team
// params[1] = user index
// Valid entity should be within range
cell *model;
CHECK_PLAYER(params[1]);
// Make into edict pointer
edict_t *pPlayer = MF_GetPlayerEdict(params[1]);
if ((params[0] / sizeof(cell)) >= 2)
{
model = MF_GetAmxAddr(amx, params[2]);
*model = *((int *)pPlayer->pvPrivateData + OFFSET_INTERNALMODEL);
}
return *((int *)pPlayer->pvPrivateData + OFFSET_TEAM);
}

Chyba, że CHECK_PLAYER sprawdza.

 

 

Sądzac po tym, moim zdaniem nawet z 

cs_get_user_team

 

nie będzie problemu. Ale to oczywiście moje zdanie i moja perspektywa.

CHECK_PLAYER sprawdza z tego co wiem wyłącznie czy ID gracza jest w przedziale od 1 do ilości slotów serwera, nie sprawdza czy gracz istnieje. Więc cs_get_user_team może się wysypać.

Z connected i alive to wiem.

 

Ale są przypadki gdzie trzeba oba. Np. Gracz jest martwy. Bo tam jest alternatywa

Jednak coś chyba jest nie tak

 

Wywala

Run time error 4: index out of bounds 

W tym kodzie:

public Damage(id)
{
	new idattacker = get_user_attacker(id);
	
	if(get_user_team(id) == get_user_team(idattacker))
		return PLUGIN_CONTINUE;
	
	if(ma_perk[idattacker] && !random(2)) //tutaj
	{
        }
	
	return PLUGIN_CONTINUE;
}

Czyżby nie sprawdzało? oO

Nie sprawdzasz czy idattacker jest graczem, a może przecież nim nie być...

a get_user_team?

      if (index < 1 || index > gpGlobals->maxClients)
            return -1;

Nie filtruje tego?

A czy gdzieś w kodzie masz przerwanie kodu wtedy gdy atakujący nie jest graczem?

Czekaj czekaj, dobrze kminię?

 

Jeżeli gracz nie jest podłączony albo nie jest graczem to get_user_team zwraca -1 i w tym momencie dzieje się to:

if(get_user_team(id) == -1)
		return PLUGIN_CONTINUE;

?

 

Czyli ofiara też musiałaby być niepodłączona albo nie być graczem?

 

Wiadomość wygenerowana automatycznie


Ten temat został zamknięty przez moderatora.

Powód: Pomoc udzielona

Jeśli się z tym nie zgadzasz, raportuj ten post, a moderator lub administrator rozpatrzy go ponownie.


Z pozdrowieniami,
Zespół AMXX.PL