[Supremache]

Originally Posted by iceeedr I think your problem is in szTarget, take this post by example. https://forums.alliedmods.net/showpo...54&postcount=8

This way doesn't work with rehlds ?

PHP Code:

new iUserId = str_to_num(szTargetId);
    iSetTarget[id] = find_player("k", iUserId); 


[Black Rose]

If messagemode is the issue (should be pretty easy to test it, no idea why you haven't...) you could just use regular chat but only enable it whenever your donation menu is open.

https://forums.alliedmods.net/showth...stmenu_2033759

[Supremache]

It's not donation menu, it's new idea i tried to make plugin for give/set/take money from players without using "say, say_team, console" just open menu and follow steps and i did it but what you gave me is new style, you want me remove msg mod and use something diffrent like select from random amount that are already found in plugin but if i want to give/set/take different amount than i have to use the menu many time, it's bad idea

[Black Rose]

Originally Posted by Supremache It's not donation menu, it's new idea i tried to make plugin for give/set/take money from players without using "say, say_team, console" just open menu and follow steps and i did it but what you gave me is new style, you want me remove msg mod and use something diffrent like select from random amount that are already found in plugin but if i want to give/set/take different amount than i have to use the menu many time, it's bad idea

My point was not to use my plugin but to adapt yours using normal chat instead of messagemode. Making menus that can navigate differently is really not that hard. YOU as a programmer decide when the menu is closed or not.

[Supremache]

I know what you was mean but i dont like this idea like calling menu handler many time and as i know and as Bugsy told me:

Originally Posted by Bugsy @Supremache: The goal is to call as few natives as possible in a plugin for efficiency.

So i see the better is calling menu one time...

Spoiler

PHP Code:

donatemenu(id) {
    
    if ( ! is_user_connected(g_donation_receiver[id][0]) || g_donation_receiver[id][1] != get_user_userid(g_donation_receiver[id][0]) ) {
        client_print(id, print_chat, "The receiver has left the game.");
        return;
    }
    
    new menubody[512], name[32], keys = MENU_KEY_1|MENU_KEY_2|MENU_KEY_3|MENU_KEY_4|MENU_KEY_5|MENU_KEY_6|MENU_KEY_7|MENU_KEY_8|MENU_KEY_0;
    get_user_name(g_donation_receiver[id][0], name, charsmax(name));
    
    new len = formatex(menubody, charsmax(menubody), "\
\yReceiver of donation: \r%s^n\ySelect an amount or enter one in chat.^n^nCurrent amount: \r$%d^n^n\
\r1. \w+5000^n\
\r2. \w-5000^n\
\r3. \w+1000^n\
\r4. \w-1000^n\
\r5. \w+500^n\
\r6. \w-500^n\
\r7. \w+100^n\
\r8. \w-100^n^n", name, g_donation_amount[id]);

    if ( g_donation_amount[id] ) {
        len += formatex(menubody[len], charsmax(menubody) - len, "\r9. \wSend donation^n");
        keys |= MENU_KEY_9;
    }
    else {
        len += formatex(menubody[len], charsmax(menubody) - len, "\d9. Send donation^n");
    }
    
    formatex(menubody[len], charsmax(menubody) - len, "\r0. \wExit");

    show_menu(id, keys, menubody, _, "donatemenu");
}

public donatemenu_handler(id, key) {
    
    if ( ! is_user_connected(g_donation_receiver[id][0]) || g_donation_receiver[id][1] != get_user_userid(g_donation_receiver[id][0]) ) {
        client_print(id, print_chat, "The receiver has left the game.");
        return;
    }
    
    switch ( key ) {
        case 0 : g_donation_amount[id] += 5000;
        case 1 : g_donation_amount[id] -= 5000;
        case 2 : g_donation_amount[id] += 1000;
        case 3 : g_donation_amount[id] -= 1000;
        case 4 : g_donation_amount[id] += 500;
        case 5 : g_donation_amount[id] -= 500;
        case 6 : g_donation_amount[id] += 100;
        case 7 : g_donation_amount[id] -= 100;
    
        case 8 : {
            donate(id, g_donation_receiver[id][0], clamp(g_donation_amount[id], 0, cs_get_user_money(id)));
            return;
        }
        
        case 9 : return;
    }
    
    g_donation_amount[id] = clamp(g_donation_amount[id], 0, cs_get_user_money(id));
    donatemenu(id);
} 


[Black Rose]

I give up.

[Supremache]

Thanks anyway i will try to do another codes idea

[fysiks]

Originally Posted by Supremache I know what you was mean but i dont like this idea like calling menu handler many time and as i know and as Bugsy told me: Originally Posted by Bugsy @Supremache: The goal is to call as few natives as possible in a plugin for efficiency.

While that is technically true, some people tend to go over the top with it. Calling a menu a few times is not something that will affect performance in any significant way. Generally, you only need to consider it in function that are called every time step (like prethink) or in for and while loops that require a lot (and I mean a lot) of iterations.

[Supremache]

Well !!