[Lukass]

    register_clcmd("say /noriuvip","noriu_vip",0,"- Jeigu nori VIP")
    register_clcmd("say /viprules","vip_rules",0,"- VIP taisykles")
    register_clcmd("say /rules","rules",0,"- Taisykles")


public noriu_vip(id) {

    
    show_motd(id,"noriuvip.txt","Nori VIP?")
  
}

public vip_rules(id) {

    
    show_motd(id,"viprules.txt","VIP taisykles")
  
}



public rules(id) {

    
    show_motd(id,"rules.txt","Taisykles")
  
}

It`s possible to use one function instead these three?

[danielkza]

Originally Posted by Lukass register_clcmd("say /noriuvip","noriu_vip",0,"- Jeigu nori VIP") register_clcmd("say /viprules","vip_rules",0,"- VIP taisykles") register_clcmd("say /rules","rules",0,"- Taisykles") public noriu_vip(id) { show_motd(id,"noriuvip.txt","Nori VIP?") } public vip_rules(id) { show_motd(id,"viprules.txt","VIP taisykles") } public rules(id) { show_motd(id,"rules.txt","Taisykles") } It`s possible to use one function instead these three?

Yes, but it would make the code bigger, not smaller, and would bring you no benefit unless there is some common logic among the 3 commands (actually there's no logic at all).

[Bad_Bud]

PHP Code:

register_clcmd("say","rules")
 
public rules(id)
{
     new arg[33]
     read_argv(1,arg,32)
 
     if(equali(arg,"/noriuvip"))
     {
          show_motd(id,"noriuvip.txt","Nori VIP?")
     }
     else if(equali(arg,"/viprules"))
     {
          show_motd(id,"viprules.txt","VIP taisykles")
     }
     else if(equali(arg,"/rules"))
     {
          show_motd(id,"rules.txt","Taisykles")
     }
     else
     {
           return PLUGIN_CONTINUE
     }
 
     return PLUGIN_HANDLED
} 


Not sure how/what you're doing with flags on the clcmd, because I've never used them, but here's what I would do if you had to have it in a single function to satisfy yourself.

If the flags meant anything important, you'll have to add another check to the if statements.

[Lukass]

Thanks dude, i want exacaly it.

[SnoW]

That way it will check every single command or text what you say to chat. Ofc it won't matter if you do it only in this plugin. But just notice that if you have 20 plugins that checks all say commands: Always when someone says something to chat it will run to 20 function. That's why it's -well someone would say- better to use exact chat checks.

[anakin_cstrike]

Or try this, i like it more

PHP Code:

#include <amxmodx>

new g_Commands[ ][ ] =
{
    "/noriuvip",
    "/viprules",
    "/rules"
};

new g_Motds[ ][ ] =
{
    "noriuvip.txt",
    "viprules.txt",
    "rules.txt"
};

new g_Messages[ ][ ] =
{
    "Nori VIP?",
    "VIP taisykles",
    "Taisykles"
};

public plugin_init() 
{
    register_clcmd( "say", "hook_say" );
    register_clcmd( "say_team", "hook_say" );
}

public hook_say( id )
{
    new args[ 32 ];
    read_args( args, charsmax( args ) );
    remove_quotes( args );
    
    for( new i = 0; i < sizeof g_Commands; i++ )
    {
        if( equali( args, g_Commands[ i ] ) )
            show_motd( id, g_Motds[ i ], g_Messages[ i ] );
    }
    
    return 0;
} 


[Lee]

anakin_cstrike, if you iterate over g_Commands in plugin_init() and use said loop to call register_clclmd() and still retain one public fuction, you keep the extensibility without having to pass through the VM every time someone uses chat.

[Exolent[jNr]]

But if you hook them to the same function, you would have to read the command to figure out which to display.

[Lee]

Yes, of course, but he's currently doing the same thing whether it's a command we're interested in or someone talking about their cat that just got run over.