[LemoNSK]

Hello, I've made this script so far.

PHP Code:

#include <amxmodx>
#include <amxmisc>

#define PLUGIN "PM Message"
#define VERSION "1.0"
#define AUTHOR "Reddington"

new menu;

public plugin_init() {
    register_plugin(PLUGIN, VERSION, AUTHOR)
    register_clcmd("say /pm", "SendMessage");
}

public SendMessage( id )
{
    new playerOnline = ConnectedPeople();
    menu = menu_create("\ySelect a person to send message to", "sendmessage_handler");
    for(new x = 1; x < playerOnline + 1; x++)
    {
        new name[70]; new msg[255];
        get_user_name(x, name, sizeof(name));
        format(msg, sizeof(msg), "%s", name);
        menu_additem(menu, msg, "player", 0);
    }
    menu_display(id, menu, 0);
    return PLUGIN_HANDLED;
}

public sendmessage_handler(id, menu, item)
{
    new playerOnline = ConnectedPeople();
    switch( item )
    {    
           for(new x = 1; x < playerOnline; x ++) //Here's an error.. i thought i could do something like this.. it doesn't work.. help here.
           {
                case x:
                {
                          
                }
           }

        
    }
    return 1;
}

ConnectedPeople()
{
    new connectedPeople;
    for(new x = 1; x < 33; x++)
    {
        if(is_user_connected(x))
        {
            connectedPeople++
        }
    }
    return connectedPeople;
} 


Can you tell me how am I supposed to perfom some action according to their ID's when I select for example one player from the menu? I can't do anything, because if I try to use FOR loop then it says error. Any suggestions?

[maqi]

What kind of help do you need ? Because most of this code is really bad, maybe you should brush up on some basic pawn tutorials, no offense

Anyway, you can't create a player menu without passing the uid for example, more info here https://forums.alliedmods.net/showthread.php?t=46364

Syntax for the switch in pawn is:

switch( condition ) {
    case 0: {
        // do something    
    }
    case 1: {
        // do something   
    }
    default: {
        // do something
    }
}

Also, a proper place for this is the Scripting Help section, just so u know in the future.

[LemoNSK]

Originally Posted by maqi What kind of help do you need ? Because most of this code is really bad, maybe you should brush up on some basic pawn tutorials, no offense Anyway, you can't create a player menu without passing the uid for example, more info here https://forums.alliedmods.net/showthread.php?t=46364 Syntax for the switch in pawn is: switch( condition ) { case 0: { // do something } case 1: { // do something } default: { // do something } } Also, a proper place for this is the Scripting Help section, just so u know in the future.

I know about this, mate.. Look inside my code and properly.. I wanted to answer a question whether it is possible to for loop inside the switch because it says me error. As you can see, I tried to write all the players connected to the server in the menu, that works.. but If I click on the certain player after they've been loaded in menu, it should do some function according to the player's id.. but I can't for loop inside so I can't continue on...

PS: I've been programming in PHP and HTML,CSS for a long time.. that's why I almost forgot everything.

[Relaxing]

https://forums.alliedmods.net/showthread.php?t=46364
You'll find a lot of information here about how stuff are done the proper way.

[LemoNSK]

Originally Posted by Relaxing https://forums.alliedmods.net/showthread.php?t=46364 You'll find a lot of information here about how stuff are done the proper way.

Alright, thank you. I'll look into it.

Edit: This is what I was asking. Thank you.