[cessil]

you should be using hooks for GivePlayerWeapon so you don't have to edit anything in your script
you've also skipped over any issues players could have like lag or desync, (there's a link to the anti cheat tips thread in my signature)
for example if a player has an ak-47 and then you give them an m4, unless they update instantaneously (not going to happen) your script will consider them using cheats

also your script is very messy, here's a trimmed down version based on copy + pasting other peoples work

pawn Код:

#define WEAPON_SLOT_COUNT 13 //0 - 12

new Weapons[MAX_PLAYERS][WEAPON_SLOT_COUNT];

stock GetWeaponSlot(weaponid)
{
    new slot;
    switch(weaponid)
    {
        case 0,1: slot = 0;
        case 2 .. 9: slot = 1;
        case 10 .. 15: slot = 10;
        case 16 .. 18, 39: slot = 8;
        case 22 .. 24: slot = 2;
        case 25 .. 27: slot = 3;
        case 28, 29, 32: slot = 4;
        case 30, 31: slot = 5;
        case 33, 34: slot = 6;
        case 35 .. 38: slot = 7;
        case 40: slot = 12;
        case 41 .. 43: slot = 9;
        case 44 .. 46: slot = 11;
    }
    return slot;
}
//https://sampforum.blast.hk/showthread.php?pid=782197#pid782197

stock CheckWeapons(playerid)
{
    new weaponid = GetPlayerWeapon(playerid);
    new slot = GetWeaponSlot(weaponid);
    if(Weapons[playerid][slot] != weaponid)
    {
        //not equal to the server's data
    }
    //don't even need to return anything
}

stock ServerWeapon(playerid, weaponid, ammo)
{
    new slot = GetWeaponSlot(weaponid);
    Weapons[playerid][slot] = weaponid;
    GivePlayerWeapon(playerid, weaponid, ammo);
}