[zakt4n]

I'm getting this error:

L 09/05/2011 - 14:32:21: [SM] Displaying call stack trace for plugin ".smx":
L 09/05/2011 - 14:32:21: [SM]   [0]  Line 193, rdmpower.sp::OnGameFrame()
L 09/05/2011 - 14:32:21: [SM] Plugin encountered error 11: Invalid memory access

What is the problem with my code:

public OnGameFrame()
{
    for (new i=1;i<=MaxClients;i++)
    {
        decl color[4];
        decl toSend[MaxClients];
        decl pCount;
        decl team;
        decl health;
        if (IsClientInGame(i) && IsPlayerAlive(i))
        {
            team = GetClientTeam(i);
            for (new x=1;x<=MaxClients && GetClientTeam(x) == team;x++)
            {
                pCount++;
                toSend[pCount] = x;
            }
            color[3] = 255;
            health = GetClientHealth(i);
            if (health > 99){color[0] = 95;color[1] = 251;color[2] = 0;}
            else if (health < 99 && health > 70){color[0] = 249;color[1] = 149;color[2] = 0;}
            else if (health < 69 && health > 50){color[0] = 228;color[1] = 116;color[2] = 3;}
            else if (health < 49 && health > 20){color[0] = 235;color[1] = 75;color[2] = 0;}
            else if (health < 19 && health > 0){color[0] = 255;color[1] = 0;color[2] = 0;}
            decl Float:origin[3];
            origin[2] += 60;
            GetClientAbsOrigin(i, origin);TE_Start("BeamRingPoint");TE_WriteVector("m_vecCenter", origin);TE_WriteFloat("m_flStartRadius", 40.0);TE_WriteFloat("m_flEndRadius", 45.0);TE_WriteNum("m_nModelIndex", lgtning);TE_WriteNum("m_nHaloIndex", lgtning);TE_WriteNum("m_nStartFrame", 0);TE_WriteNum("m_nFrameRate", 100);TE_WriteFloat("m_fLife", 3.0);TE_WriteFloat("m_fWidth", 45.0);TE_WriteFloat("m_fEndWidth", 45.0);TE_WriteFloat("m_fAmplitude", 0.0);TE_WriteNum("r", color[0]);TE_WriteNum("g", color[1]);TE_WriteNum("b", color[2]);TE_WriteNum("a", color[3]);TE_WriteNum("m_nSpeed", 5);TE_WriteNum("m_nFlags",    0);TE_WriteNum("m_nFadeLength", 100);TE_Send(toSend, pCount);
        }
    }
}

Thanks for everyone!

[mjmfighter]

What line is 193?

Also, its never good to have more than one statement per line. It makes it hard to debug.

Im guessing the error is somewhere in here:

PHP Code:

GetClientAbsOrigin(i, origin);TE_Start("BeamRingPoint");TE_WriteVector("m_vecCenter", origin);TE_WriteFloat("m_flStartRadius", 40.0);TE_WriteFloat("m_flEndRadius", 45.0);TE_WriteNum("m_nModelIndex", lgtning);TE_WriteNum("m_nHaloIndex", lgtning);TE_WriteNum("m_nStartFrame", 0);TE_WriteNum("m_nFrameRate", 100);TE_WriteFloat("m_fLife", 3.0);TE_WriteFloat("m_fWidth", 45.0);TE_WriteFloat("m_fEndWidth", 45.0);TE_WriteFloat("m_fAmplitude", 0.0);TE_WriteNum("r", color[0]);TE_WriteNum("g", color[1]);TE_WriteNum("b", color[2]);TE_WriteNum("a", color[3]);TE_WriteNum("m_nSpeed", 5);TE_WriteNum("m_nFlags",    0);TE_WriteNum("m_nFadeLength", 100);TE_Send(toSend, pCount); 


make each of these statements on its own line and then see which one is giving an error. Thats probably where your problem will be and you can fix it.

[zakt4n]

Is not, dont have any error in an effect. but here it go:

public OnGameFrame()
{
    for (new i=1;i<=MaxClients;i++)
    {
        decl color[4];
        decl toSend[MaxClients];
        decl pCount;
        decl team;
        decl health;
        if (IsClientInGame(i) && IsPlayerAlive(i))
        {
            team = GetClientTeam(i);
            for (new x=1;x<=MaxClients && GetClientTeam(x) == team;x++)
            {
                pCount++;
                toSend[pCount] = x;
            }
            color[3] = 255;
            health = GetClientHealth(i);
            if (health > 99){color[0] = 95;color[1] = 251;color[2] = 0;}
            else if (health < 99 && health > 70){color[0] = 249;color[1] = 149;color[2] = 0;}
            else if (health < 69 && health > 50){color[0] = 228;color[1] = 116;color[2] = 3;}
            else if (health < 49 && health > 20){color[0] = 235;color[1] = 75;color[2] = 0;}
            else if (health < 19 && health > 0){color[0] = 255;color[1] = 0;color[2] = 0;}
            decl Float:origin[3];
            origin[2] += 60;
            GetClientAbsOrigin(i, origin);
            TE_Start("BeamRingPoint");
            TE_WriteVector("m_vecCenter", origin);
            TE_WriteFloat("m_flStartRadius", 40.0);
            TE_WriteFloat("m_flEndRadius", 45.0);
            TE_WriteNum("m_nModelIndex", lgtning);
            TE_WriteNum("m_nHaloIndex", lgtning);
            TE_WriteNum("m_nStartFrame", 0);
            TE_WriteNum("m_nFrameRate", 100);
            TE_WriteFloat("m_fLife", 3.0);
            TE_WriteFloat("m_fWidth", 45.0);
            TE_WriteFloat("m_fEndWidth", 45.0);
            TE_WriteFloat("m_fAmplitude", 0.0);
            TE_WriteNum("r", color[0]);
            TE_WriteNum("g", color[1]);
            TE_WriteNum("b", color[2]);
            TE_WriteNum("a", color[3]);
            TE_WriteNum("m_nSpeed", 5);
            TE_WriteNum("m_nFlags",    0);
            TE_WriteNum("m_nFadeLength", 100);
            TE_Send(toSend, pCount);
        }
    }
}

line 193:

                pCount++;

[mjmfighter]

its probably the for loop. i dont think you can have the && inside a for loop. Try something like this instead:

PHP Code:

            if(GetClientTeam(x) == team)
            {
                for (new x=1;x<=MaxClients;x++)
                {
                    pCount++;
                    toSend[pCount] = x;
                }
            } 


[zakt4n]

but i need to put the pCount++ and toSend[pCount] = x only for a team clients.
See if this can:

            team = GetClientTeam(i);
            for (new x=1;x<=MaxClients;x++)
            {
                if (GetClientTeam(x) == team)
                {
                    pCount++;
                    toSend[pCount] = x;
                }
            }

[mjmfighter]

yeah i just relized my way wouldnt work because x isnt declared. your way would work.

[zakt4n]

it keep sending the same error

[mjmfighter]

maybe its because you are not checking to see if that client is in the game or not.

PHP Code:

            team = GetClientTeam(i);
            for (new x=1;x<=MaxClients;x++)
            {
                if (IsClientConnected(x) && IsClientInGame(x) && GetClientTeam(x) == team)
                {
                    pCount++;
                    toSend[pCount] = x;
                }
            } 


also, you might need to declare pCount = 0; or whatever value u need it to be.

[Peace-Maker]

You didn't initialize the pCount integer. use new instead of decl to set it to 0 initially.
if you just use decl pCount; and pCount++ you try to increase some garbage data.

[zakt4n]

now it is giving me this error:

L 09/05/2011 - 17:23:56: [SM] Native "GetClientTeam" reported: Client 2 is not in game
L 09/05/2011 - 17:23:56: [SM] Displaying call stack trace for plugin "rdmpower.smx":
L 09/05/2011 - 17:23:56: [SM]   [0]  Line 194, rdmpower.sp::OnGameFrame()