[UnknownGamer]

Hello again,

I've got stuck on another command refusing to work, just simply trying to define the username of the person who does "/ready"

Код:

    // NEW GUN RELOAD COMMAND - MADE BY MATT //
    if(strcmp(cmd, "/ready", true) == 0)
    {
        // Gets the players name
        new string1[256], sendername[MAX_PLAYER_NAME];

        // The animations for the reload guns.
		OnePlayAnim(playerid,"UZI","UZI_reload",4.0,0,0,0,0,0);

		// Sends the client message, to all.
		format(string1, sizeof(string1), "%s has clicked off his safety.", sendername);
		ProxDetector(30.0, playerid, string1, COLOR_LIGHTBLUE);

    	return 1;
    }
	// END OF NEW RELOAD GUN COMMAND //

Errors:

Код:

C:\Users\matthew\Desktop\Sep 2012\gamemodes\larp.pwn(27923) : warning 219: local variable "sendername" shadows a variable at a preceding level
C:\Users\matthew\Desktop\Sep 2012\gamemodes\larp.pwn(27930) : warning 202: number of arguments does not match definition
C:\Users\matthew\Desktop\Sep 2012\gamemodes\larp.pwn(27930) : warning 202: number of arguments does not match definition
C:\Users\matthew\Desktop\Sep 2012\gamemodes\larp.pwn(27930) : warning 202: number of arguments does not match definition
C:\Users\matthew\Desktop\Sep 2012\gamemodes\larp.pwn(27930) : warning 202: number of arguments does not match definition
Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


5 Warnings.

Please can somebody write me the fixed command?

Thanks

[clarencecuzz]

pawn Код:

// NEW GUN RELOAD COMMAND - MADE BY MATT //
    if(strcmp(cmd, "/ready", true) == 0)
    {
        // Gets the players name
        new string1[256], sendname[MAX_PLAYER_NAME];

        // The animations for the reload guns.
        OnePlayAnim(playerid,"UZI","UZI_reload",4.0,0,0,0,0,0);

        // Sends the client message, to all.
        format(string1, sizeof(string1), "%s has clicked off his safety.", sendname);
        ProxDetector(30.0, playerid, string1, COLOR_LIGHTBLUE,COLOR_LIGHTBLUE,COLOR_LIGHTBLUE,COLOR_LIGHTBLUE,COLOR_LIGHTBLUE);

        return 1;
    }
    // END OF NEW RELOAD GUN COMMAND //

-You have the variable 'sendername' in another place, possibly the same code space that is interfering, so you are introducing a new variable called sendername twice. There are two alternatives, renaming one of the variables or removing the other variable with caution, being aware not to break any code.

-There are 5 colours in a ProxDetector function. (So I've been told. Not tested)

Код:

ProxDetector(Float:radi, playerid, string[],col1,col2,col3,col4,col5);

[UnknownGamer]

Thankyou. I got it working now.

Anybody who may want the script, working code is.

Код:

    // NEW GUN RELOAD COMMAND - MADE BY MATT //
    if(strcmp(cmd, "/ready", true) == 0)
    {
        // Defines the string
        new string1[256];

        // The animations for the reload guns.
		OnePlayAnim(playerid,"UZI","UZI_reload",4.0,0,0,0,0,0);

		// Sends the client message, to all.
		format(string1, sizeof(string1),"%s has clicked off his safety.", sendername);
		SendClientMessageToAll(COLOR_LIGHTBLUE, string1);

		format(string1, sizeof(string1),"%s reloads his weapon.", sendername);
		SendClientMessageToAll(COLOR_LIGHTBLUE, string1);

    	return 1;
    }
	// END OF NEW RELOAD GUN COMMAND //