[Randomize]

Crosshair Type

Description:
Change your crosshair to different type

Console Commands:
cmd_show_menu_ch

Changelog:

V1.0 First release

Special thanks to <VeCo> he's smart
[IMG]http://img585.**************/img585/5272/chmenu.jpg[/IMG]

[kiki33hun]

Admin Commands: cmd_show_menu_ch

Not admin command and not valid command

[Devil259]

What about recoil ?

[Liverwiz]

Originally Posted by kiki33hun Not admin command and not valid command

register_concmd("show_menu_ch","cmd_show_menu_ch")

PHP Code:

register_clcmd("show_menu_ch", "cmd_show_menu_ch") 


Originally Posted by Randomize Console Commands: cmd_show_menu_ch

show_menu_ch

I also wouldn't make players, num, i, or id static; along with i'd keep i and id defined within the for loop

PHP Code:

new players[32],num
    get_players(players,num,"a") // get all alive players in the server
 
    for(new i=0;i<num;i++) // loop trough them...
    {
        new id = players[i] // store player index in a more convenient variable
 
        if(1 << get_user_weapon(id) & SNIPER_WEAPONS) continue // if this player is holding a sniper - continue with the next player
 
        // else, show the HUD message...
        
        set_hudmessage(0, 255, 0, -1.0, -1.0, .holdtime = 1.0) // skip default values
        ShowSyncHudMsg(id,sync, "%c",user_crosshair_type[id]) // show crosshair HUD  
    } 


Keeping i and id within the for loops will keep variable overhead down. Won't make much of a difference, but its good practice. IMHO

[Bilal Pro]

Originally Posted by Devil259 What about recoil ?

Answer that please

[Swaycher]

Very nice work!

[<VeCo>]

Originally Posted by Liverwiz I also wouldn't make players, num, i, or id static; along with i'd keep i and id defined within the for loop PHP Code: new players[32],num     get_players(players,num,"a") // get all alive players in the server       for(new i=0;i<num;i++) // loop trough them...     {         new id = players[i] // store player index in a more convenient variable           if(1 << get_user_weapon(id) & SNIPER_WEAPONS) continue // if this player is holding a sniper - continue with the next player           // else, show the HUD message...           set_hudmessage(0, 255, 0, -1.0, -1.0, .holdtime = 1.0) // skip default values         ShowSyncHudMsg(id,sync, "%c",user_crosshair_type[id]) // show crosshair HUD       }  Keeping i and id within the for loops will keep variable overhead down. Won't make much of a difference, but its good practice. IMHO

You should never create variables into a loop. It's better to leave them static because the function is called often (every second) and it will be bad if you create new variables on every call.

[Devil259]

Also, id is only used in the for loop, so you can do :

for( new i = 0, id; i < num; i++ )

[ConnorMcLeod]

Also, by default, players shoundt' have any additional crosshair, and they should be able to choose "none" in the menu, after having tested custom ones.
Also, why don't you remove the cs crosshair ?

[<VeCo>]

Also, return PLUGIN_HANDLED in cmd_show_menu_ch(id) to remove the "unknown command" message.