[Kidev]

And how can I count the number of bomb site without i?

[ConnorMcLeod]

You use it ?

Right, you use it to format menu, so then you need to make 2 while loops.


You have 1 global variable i, and a local i variable, for readability you should rename global var to something like BombSiteNum, g_iBombSiteNum or whatever.

[Kidev]

Yeah, in the array:

PHP Code:

new const bombsite[ ][ ] = {
    "A",
    "B",
    "C",
    "D",
    "E"
}; 


==>

PHP Code:

public plugin_init( ) {
    register_plugin( "I'll plant the bomb on...", "1.3", "Kid" );
    
    new target = -1, classnames[ ][ ] = { "func_bomb_target", "info_bomb_target" };

    while( ( ( target = find_ent_by_class( target, classnames[0] ) ) || ( target = find_ent_by_class( target, classnames[1] ) ) ) && i < 5 )
        i++;

    if( !i ) {
        pause("a");
    }
    register_logevent( "eventSpawnedWithBomb", 3, "2=Spawned_With_The_Bomb" );
    register_dictionary( "illplanton.txt" );
    printmode = register_cvar( "ipo_print_mode", "2" );
    useradcmd = register_cvar( "ipo_use_rad_cmd", "1" );

    return PLUGIN_CONTINUE;
} 


?

Originally Posted by ConnorMcLeod You use it to format menu, so then you need to make 2 while loops. You have 1 global variable i, and a local i variable, for readability you should rename global var to something like BombSiteNum, g_iBombSiteNum or whatever.

==> OK

[ConnorMcLeod]

And don't forget to reset target to -1 or 0 before the 2nd loop

[Kidev]

Originally Posted by ConnorMcLeod And don't forget to reset target to -1 or 0 before the 2nd loop

Ok ok, thank you

Done, update v1.4.

[Dolph_Ziggler]

nice

[alonelive]

mmm... add a cvar:
only on round startup player (spawned with C4) have a menu with choose a bombplace
or
each player, which got the bomb during the game process.

[ConnorMcLeod]

Lol, you doesn't seem to understand, or all i said made you confused.

PHP Code:

    new target = -1;

    if( !find_ent_by_class( -1, "func_bomb_target" ) && !find_ent_by_class( -1, "info_bomb_target" ) )
        pause("a");

    target = -1;

    while( ( ( target = find_ent_by_class( target, "func_bomb_target" ) ) || ( target = find_ent_by_class( target, "info_bomb_target" ) ) ) && bombsitenum < 5 )
        bombsitenum++; 


1st check is useless since you need to count sites.
while loop has the same error i pointed, you have to separate each classname into 2 while loops.

[Kidev]

Like this?

PHP Code:

    new target = -1;

    if( !find_ent_by_class( -1, "func_bomb_target" ) && !find_ent_by_class( -1, "info_bomb_target" ) )
        pause("a");

    target = -1;

    while( ( target = find_ent_by_class( target, "func_bomb_target" ) ) && bombsitenum < 5 )
        bombsitenum++; 
        
    while( ( target = find_ent_by_class( target, "info_bomb_target" ) ) && bombsitenum < 5 )
        bombsitenum++; 


[ConnorMcLeod]

Remove :

if( !find_ent_by_class( -1, "func_bomb_target" ) && !find_ent_by_class( -1, "info_bomb_target" ) )
pause("a");

And do how you did previously by checking if bombsitenum is not null.