[Napoleon_be]

I've seen this a long time ago on a ZM server. I used to play a lot there just because that plugin was a big reason. Good job!

[rak]

you can add upgrates for hp, armor, speed and gravity;

nice idea

[claudiuhks]

Originally Posted by Blue Snake. Any sugestions?

What about integer divisions?

You could better use MySQL queries.
Sending TeamInfo messages for each chat message is not a brilliant idea.

[Blue Snake.]

I'll do integer division in the next version, thanks.

About team info, it's not my function, i've copied it from colorchat.inc because it's not on the amxmodx compiler.

[claudiuhks]

Originally Posted by Blue Snake. About team info, it's not my function, i've copied it from colorchat.inc because it's not on the amxmodx compiler.

I said sending TeamInfo messages for each chat message is not a brilliant idea. Your reply is not very descriptive about what you mean.
You may send only SayText messages instead.

[Blue Snake.]

You mean i should not hook say_team, only say?

[GuTo]

nice man thanks

[HamletEagle]

You should start the task when a player join the game, not from plugin_init.
new nr1, nr2, type; should be declared as local variables, not global.
This could be reduced a lot:

Spoiler

PHP Code:

inWaitTime = false
type = random_num(1, 4);
switch(type)
{
    case 1: // == +
    {
        nr1 = random_num(1,1000);
        nr2 = random_num(1,1000);
            new players[32], limit;
            get_players(players, limit, "");
            for(new i=0 ; i<limit ; ++i)
                if(eqEnabled[players[i]])
                    ColorChat(players[i], BLUE, "^x04[EQ]^x01 Solve the equation^x03 %d^x01 +^x03 %d^x01. The faster you answer correctly, the more points you earn...", nr1, nr2)
            num_to_str(nr1 + nr2, correctAnswer, 9);
        }
        case 2: // == -
        {
            nr1 = random_num(1,1000);
            nr2 = random_num(1,nr1);
            new players[32], limit;
            get_players(players, limit, "");
            for(new i=0 ; i<limit ; ++i)
                if(eqEnabled[players[i]])
                    ColorChat(players[i], BLUE, "^x04[EQ]^x01 Solve the equation^x03 %d^x01 -^x03 %d^x01.The faster you answer correctly, the more points you earn...", nr1, nr2)
            num_to_str(nr1 - nr2, correctAnswer, 9);
        }
        case 3: // == *
        {
            nr1 = random_num(2,100);
            nr2 = random_num(2, 30);
            new players[32], limit;
            get_players(players, limit, "");
            for(new i=0 ; i<limit ; ++i)
                if(eqEnabled[players[i]])
                    ColorChat(players[i], BLUE, "^x04[EQ]^x01 Solve the equation^x03 %d^x01 x^x03 %d^x01. The faster you answer correctly, the more points you earn...", nr1, nr2)
            num_to_str(nr1 * nr2, correctAnswer, 9);
        }
        case 4: // == /
        {
            nr2 = random_num(2, 30);
            nr1 = random_num(2,100) * nr2;
            new players[32], limit;
            get_players(players, limit, "");
            for(new i=0 ; i<limit ; ++i)
                if(eqEnabled[players[i]])
                    ColorChat(players[i], BLUE, "^x04[EQ]^x01 Solve the equation^x03 %d^x01 /^x03 %d^x01. The faster you answer correctly, the more points you earn...", nr1, nr2)
            num_to_str(nr1 / nr2, correctAnswer, 9);
        }
    }
    set_task(get_pcvar_float(timeCvar), "timeOut", taskParam);
    questionTime = floatround(halflife_time()); 


To something like:

inWaitTime = false
    nr1 = random_num(1, 1000)
    nr2 = random_num(1, 1000)
    type = random_num(0,3)
    
    new const math_sign[] = {"+", "-", "*", "/"}
    new players[32], limit, operation, id;
    get_players(players, limit);
    for(new i=0 ; i<limit ; ++i)
    {
        id = players[i]
        if(eqEnabled[id])
            ColorChat(id, BLUE, "^x04[EQ]^x01 Solve the equation^x03 %d^x01 %s^x03 %d^x01. The faster you answer correctly, the more points you earn...", nr1, nr2, math_sign[type])
    }
    switch(type)
    {
        case 0:
        {
            operation = nr1 + nr2
        }
        case 1:
        {
            operation = nr1 - nr2
        }
        case 2:
        {
            operation = nr1 * nr2
        }
        case 3:
        {
            operation = nr1 / nr2
        }
    }
    num_to_str(operation, correctAnswer, charsmax(correctAnswer));
    
    set_task(get_pcvar_float(timeCvar), "timeOut", taskParam);
    questionTime = floatround(halflife_time());

Notice that I cached players[i], you should do the same in other places.

In answer:

• read_args(say,192) is wrong, you should go until sizeof - 1, not until sizeof. You can directly use charsmax.
• Do not create variables in a loop, initialize them before.
• Use formatex, it's faster. But, after all, you don't need to format, just use copy() native.
  

  copy(sTemp, charsmax(sTemp), topNames[j]);
  copy(topNames[j], charsmax(topNames[]), topNames[j+1]);
  copy(topNames[j+1], charsmax(topNames[]), sTemp);

I'll just stop here because it's enough for now. Also, you should just attach or link to the needed include file, don't just copy paste it inside your plugin.

Unapproved until you make some changes.