AlliedModders

AlliedModders (https://forums.alliedmods.net/index.php)
-   Scripting Help (https://forums.alliedmods.net/forumdisplay.php?f=11)
-   -   Some warnings (https://forums.alliedmods.net/showthread.php?t=159578)

KamiN 06-19-2011 04:04

Some warnings
 
1st Warning

Quote:

L 06/19/2011 - 00:09:06: [CSTRIKE] Non-player entity -1 out of range
L 06/19/2011 - 00:09:06: [AMXX] Displaying debug trace (plugin "jb_days_lr.amxx")
L 06/19/2011 - 00:09:06: [AMXX] Run time error 10: native error (native "cs_set_weapon_ammo")
L 06/19/2011 - 00:09:06: [AMXX] [0] jb_days_lr.sma::sub_choose_enemy (line 477)
Code:

public sub_choose_enemy(id, menu, item)
{
    if( item == MENU_EXIT )
    {
    menu_destroy(menu)
    return PLUGIN_HANDLED
    }
 
    if(catch)
        catch = false
 
    else if(hns)
        hns = false
   
    new Data[6], Name[64]
    new Access, Callback
    menu_item_getinfo(menu, item, Access, Data,5, Name, 63, Callback)
   
    new tempid = str_to_num(Data)
   
    new mapname[32], challenger[32], challenged[32], ct_origin[3], te_origin[3]

    get_mapname(mapname,31)
    get_user_name(id, challenger, 31)
    get_user_name(tempid, challenged, 31)
   
    // strip weapons
    strip_user_weapons(id)
    strip_user_weapons(tempid)
   
    // health
    set_user_health(id, 100)
    set_user_health(tempid, 100)
   
    // effects
    switch (get_pcvar_num(cvar_effect))
    {
        case 0: // glow
        {
            set_user_rendering(id, kRenderFxGlowShell, 250, 0, 0, kRenderNormal, 20)   
            set_user_rendering(tempid, kRenderFxGlowShell, 0, 0, 250, kRenderNormal, 20)
        }
        case 1: // beacon
        {
            set_task(1.0, "beacon", id)
            set_task(1.0, "beacon", tempid)
        }
        case 2: // glow n' beacon
        {
            set_user_rendering(id, kRenderFxGlowShell, 250, 0, 0, kRenderNormal, 20)   
            set_task(1.0, "beacon", id)
            set_user_rendering(tempid, kRenderFxGlowShell, 0, 0, 250, kRenderNormal, 20)
            set_task(1.0, "beacon", tempid)
        }
    }
   
    // player challenged
    player_challenged[tempid] = true
   
    // lr menu once until he kills the player challenged
    once[id] = true
   
    // unable pick up weapon (only the 2 players on duel)
    touch_weapons[id] = true
    touch_weapons[tempid] = true
           
    if(m3_duel[id])
    {

        give_item(id, "weapon_m3")
        give_item(tempid, "weapon_m3")
        cs_set_user_bpammo(id, CSW_M3, 30)       
        cs_set_user_bpammo(tempid, CSW_M3, 30)
               
        if(equali(mapname,"some1s_jailbreak"))
        {
            ct_origin[0] = -759, ct_origin[1] = 1047, ct_origin[2] = 100
            te_origin[0] = -585, te_origin[1] = 867, te_origin[2] = 100
            set_user_origin(id,te_origin)
            set_user_origin(tempid,ct_origin)
        }
       
        set_hudmessage( 0, 255, 0, -1.0, 0.40, 2, 5.0, 8.0, 0.0, 0.0, 10)
        show_hudmessage(0, "%L",LANG_PLAYER,"POMPA_DUEL", challenger, challenged)
       
    }
    else if(deagle_duel[id])
    {
        deagle_duel[tempid] = true
       
        cs_set_weapon_ammo(give_item(id, "weapon_deagle"), 1)
        cs_set_weapon_ammo(give_item(tempid, "weapon_deagle"), 1)
       
        if(equali(mapname,"some1s_jailbreak"))
        {
            ct_origin[0] = -1352, ct_origin[1] = 271, ct_origin[2] = 38
            te_origin[0] = -1338, te_origin[1] = -782, te_origin[2] = 38
            set_user_origin(id,te_origin)
            set_user_origin(tempid,ct_origin)
        }
       
        set_hudmessage( 0, 255, 0, -1.0, 0.40, 2, 5.0, 8.0, 0.0, 0.0, 10)
        show_hudmessage(0, "%L", LANG_PLAYER,"DEAGLE_DUEL", challenger, challenged)
    }
    else if(ump_duel[id])
    {
        give_item(id, "weapon_ump45")
        give_item(tempid, "weapon_ump45")
        cs_set_user_bpammo(id, CSW_UMP45, 60)       
        cs_set_user_bpammo(tempid, CSW_UMP45, 60)
       
        if(equali(mapname,"some1s_jailbreak"))
        {
            ct_origin[0] = -1756, ct_origin[1] = -2526, ct_origin[2] = 36
            te_origin[0] = -3020, te_origin[1] = -2529, te_origin[2] = 36
            set_user_origin(id,te_origin)
            set_user_origin(tempid,ct_origin)
        }
       
        set_hudmessage( 0, 255, 0, -1.0, 0.40, 2, 5.0, 8.0, 0.0, 0.0, 10)
        show_hudmessage(0, "%L", LANG_PLAYER,"UMP_DUEL", challenger, challenged)
    }
    else if(scout_duel[id])
    {
        give_item(id, "weapon_scout")
        give_item(tempid, "weapon_scout")
        cs_set_user_bpammo(id, CSW_SCOUT, get_pcvar_num(cvar_scout_bpammo))
        cs_set_user_bpammo(tempid, CSW_SCOUT, get_pcvar_num(cvar_scout_bpammo))
       
        if(equali(mapname,"some1s_jailbreak"))
        {
            ct_origin[0] = -2898, ct_origin[1] = -2040, ct_origin[2] = 37
            te_origin[0] = -2908, te_origin[1] = 905, te_origin[2] = 37
            set_user_origin(id,te_origin)
            set_user_origin(tempid,ct_origin)
        }
       
        set_hudmessage( 0, 255, 0, -1.0, 0.40, 2, 5.0, 8.0, 0.0, 0.0, 10)
        show_hudmessage(0, "%L", LANG_PLAYER,"SCOUT_DUEL", challenger, challenged)
    }
   
    duel_active = true
   
    menu_destroy(menu)
    return PLUGIN_HANDLED
}

---------------------

2nd Warning

Quote:

L 06/18/2011 - 12:07:33: [CSTRIKE] Invalid player 12
L 06/18/2011 - 12:07:33: [AMXX] Displaying debug trace (plugin "jb_days_lr.amxx")
L 06/18/2011 - 12:07:33: [AMXX] Run time error 10: native error (native "cs_get_user_team")
L 06/18/2011 - 12:07:33: [AMXX] [0] jb_days_lr.sma::beacon (line 536)
Code:

public beacon(id)
{
    if(cs_get_user_team(id) == CS_TEAM_CT)
    {
        static origin[3]
        get_user_origin(id, origin)
        message_begin(MSG_BROADCAST, SVC_TEMPENTITY)
        write_byte(TE_BEAMCYLINDER)    // TE id
        write_coord(origin[0])        // x
        write_coord(origin[1])        // y
        write_coord(origin[2]-20)    // z
        write_coord(origin[0])        // x axis
        write_coord(origin[1])        // y axis
        write_coord(origin[2]+200)    // z axis
        write_short(beacon_sprite)    // sprite
        write_byte(0)            // startframe 
        write_byte(1)            // framerate 
        write_byte(6)            // life
        write_byte(2)              // width
        write_byte(1)              // noise   
        write_byte(0)              // red 
        write_byte(0)              // green
        write_byte(250)        // blue
        write_byte(200)            // brightness
        write_byte(0)            // speed
        message_end()
    }
    else
    {
        static origin[3]
        get_user_origin(id, origin)
        message_begin(MSG_BROADCAST, SVC_TEMPENTITY)
        write_byte(TE_BEAMCYLINDER)    // TE id
        write_coord(origin[0])        // x
        write_coord(origin[1])        // y
        write_coord(origin[2]-20)    // z
        write_coord(origin[0])        // x axis
        write_coord(origin[1])        // y axis
        write_coord(origin[2]+200)    // z axis
        write_short(beacon_sprite)    // sprite
        write_byte(0)            // startframe 
        write_byte(1)            // framerate 
        write_byte(6)            // life
        write_byte(2)              // width
        write_byte(1)              // noise   
        write_byte(250)            // red 
        write_byte(0)              // green
        write_byte(0)            // blue
        write_byte(200)            // brightness
        write_byte(0)            // speed
        message_end()
    }
    set_task(1.0, "beacon", id)
       
}

Here i should add this in front of code?
Code:

if(!is_user_alive(id))
    return

-------------------------------

3rd Warning

Quote:

L 06/18/2011 - 15:22:05: [AMXX] Displaying debug trace (plugin "jb_days_lr.amxx")
L 06/18/2011 - 15:22:05: [AMXX] Run time error 4: index out of bounds
L 06/18/2011 - 15:22:05: [AMXX] [0] jb_days_lr.sma::fw_player_killed (line 330)
Quote:

L 06/18/2011 - 17:25:41: [FUN] Player out of range (0)
L 06/18/2011 - 17:25:41: [AMXX] Displaying debug trace (plugin "jb_days_lr.amxx")
L 06/18/2011 - 17:25:41: [AMXX] Run time error 10: native error (native "strip_user_weapons")
L 06/18/2011 - 17:25:41: [AMXX] [0] jb_days_lr.sma::fw_player_killed (line 335)
Code:

public fw_player_killed(victim, attacker, shouldgib)
{
    new name[32]
    get_user_name(attacker, name, 31)
           
    if(duel_active)
    {
        if(cs_get_user_team(victim) == CS_TEAM_CT && player_challenged[victim])
        {
            m3_duel[attacker] = false
            deagle_duel[attacker] = false
            ump_duel[attacker] = false
            scout_duel[attacker] = false
            once[attacker] = false
            strip_user_weapons(attacker)
            give_item(attacker, "weapon_knife")
            set_user_rendering(attacker)
            duel_menu(attacker)
        }
        else if(cs_get_user_team(victim) == CS_TEAM_CT && !player_challenged[victim])
        {
            set_task(0.4, "kill_player", attacker)
            set_hudmessage( 0, 255, 0, -1.0, 0.40, 2, 5.0, 8.0, 0.0, 0.0, 10)
            show_hudmessage(0, "%L", LANG_PLAYER,"DUEL_DOWN", name)
               
        }
    }
    remove_task(attacker)
    remove_task(victim)
}

--------------------------------------


4th Warning

Quote:

L 06/18/2011 - 14:40:31: [AMXX] Displaying debug trace (plugin "xredirect.amxx")
L 06/18/2011 - 14:40:31: [AMXX] Run time error 4: index out of bounds
L 06/18/2011 - 14:40:31: [AMXX] [0] xredirect.sma::welcome_message (line 2424)
Code:

public welcome_message(id[])
{
    new nID = id[0]
    if (is_user_connected(nID)) // make sure the player didn't already disconnect within the set_task delay
    {
        new nLastServer = g_nLastServer[nID - 1]
        if ((nLastServer >= 0) && (nLastServer != g_nOwnServer) && (nLastServer < MAX_SERVERFORWARDS))
        {
            new sAnnounceText[MAX_WELCOME_LEN]
            format(sAnnounceText, MAX_WELCOME_LEN - 1, "%L", nID, "MSG_REDIRFROM", g_saServerNames[g_nOwnServer], g_saServerNames[nLastServer])
            if ((get_pcvar_num(cvar_retry) == 1) && (get_pcvar_num(cvar_show) == 1))
                format(sAnnounceText, MAX_WELCOME_LEN - 1, "%s^n%L", sAnnounceText, nID, "MSG_RETRY_BACK_ANNOUNCE")
           
            set_hudmessage(000, 100, 255, -1.0, -1.0, 0, 0.0, 10.0, 0.5, 2.0, 1)
            show_hudmessage(nID, sAnnounceText)
        }
    }
}


bibu 06-19-2011 04:56

Re: Some warnings
 
Check till point 3 if the user is connected. For xredirect, your fourth problem, post it in the "xRedirect" forum.

fysiks 06-19-2011 05:06

Re: Some warnings
 
3. You have to check if the attacker is actually a player.
4. How is the function welcome_message() called?

KamiN 06-19-2011 06:56

Re: Some warnings
 
So 3rd warning fix would be this?

Code:

            if(is_user_alive(attacker))
            {
                strip_user_weapons(attacker)
                give_item(attacker, "weapon_knife")
                set_user_rendering(attacker)
                duel_menu(attacker)
            }


fysiks 06-19-2011 15:54

Re: Some warnings
 
That is probably one thing you should do but you still need to check if the attacker is an actual player. In your error, it shows that the attacker is worldspawn (the environment).


All times are GMT -4. The time now is 23:31.

Powered by vBulletin®
Copyright ©2000 - 2024, vBulletin Solutions, Inc.