Setweather and Settime.

[Cool]

yes you are right but leave him alone its making the topic spammy

[Xmair]

Code Select Expand

function onPlayerCommand ( player , cmd , text )
{
cmd = cmd.tolower ( );

if ( cmd == "setweather" )
{
if ( !text ) MessagePlayer ( "/setweather WeatherID." , player );
else if ( !IsNum ( text ) ) MessagePlayer ( "The ID must be an integer." , player );
else if ( GetWeather( ) == text.tointeger( ) ) MessagePlayer( "The ID you put is already used!" , player );
else
{
SetWeather ( text.tointeger ( ) );
Message ( "The weather has been set to " + text + " by " + player.Name + "." );
}
}

else if ( cmd == "settime" )
{
if ( !text ) MessagePlayer ( "/settime Hour & Minute." , player );
else if ( split ( text , " " ).len ( ) < 2 ) MessagePlayer ( "/settime Hour & Minute." , player );
else if ( !IsNum ( split ( text , " " ) [ 0 ] ) || !IsNum ( split ( text , " " ) [ 1 ] ) ) MessagePlayer ( "The text must be an integer." , player );
else
{
local splittext = split ( text , " " ),
hour = splittext [ 0 ],
minute = splittext [ 1 ];
SetHour ( hour.tointeger( ) );
SetMinute ( minute.tointeger( ) );
Message( "The time has been set to " + GetHour( ) + ":" + GetMinute( ) + " by " + player.Name + "." );
}
}
else MessagePlayer ( "Unknown command!" , player )
}

[ysc3839]

Code Select Expand

function onPlayerCommand ( player , cmd , text )
{
cmd = cmd.tolower ( );

if ( cmd == "setweather" )
{
if ( !text ) MessagePlayer ( "/setweather WeatherID." , player );
else if ( !IsNum ( text ) ) MessagePlayer ( "The ID must be an integer." , player );
else if ( GetWeather( ) == text.tointeger( ) ) MessagePlayer( "The ID you put is already used!" , player );
else
{
SetWeather ( text.tointeger ( ) );
Message ( "The weather has been set to " + text + " by " + player.Name + "." );
}
}

else if ( cmd == "settime" )
{
if ( !text ) MessagePlayer ( "/settime Hour & Minute." , player );
else if ( split ( text , " " ).len ( ) < 2 ) MessagePlayer ( "/settime Hour & Minute." , player );
else if ( !IsNum ( split ( text , " " ) [ 0 ] ) || !IsNum ( split ( text , " " ) [ 1 ] ) ) MessagePlayer ( "The text must be an integer." , player );
else
{
local splittext = split ( text , " " ),
hour = splittext [ 0 ],
minute = splittext [ 1 ];
SetHour ( hour.tointeger( ) );
SetMinute ( minute.tointeger( ) );
Message( "The time has been set to " + GetHour( ) + ":" + GetMinute( ) + " by " + player.Name + "." );
}
}
else MessagePlayer ( "Unknown command!" , player )
}

Code Select Expand

local splittext = split ( text , " " ),
hour = splittext [ 0 ],
minute = splittext [ 1 ];This may cause error.

Code Select Expand

local splittext = split ( text , " " );
if ( splittext.len() != 0 )
......


[Xmair]

Code Select Expand

function onPlayerCommand ( player , cmd , text )
{
cmd = cmd.tolower ( );

if ( cmd == "setweather" )
{
if ( !text ) MessagePlayer ( "/setweather WeatherID." , player );
else if ( !IsNum ( text ) ) MessagePlayer ( "The ID must be an integer." , player );
else if ( GetWeather( ) == text.tointeger( ) ) MessagePlayer( "The ID you put is already used!" , player );
else
{
SetWeather ( text.tointeger ( ) );
Message ( "The weather has been set to " + text + " by " + player.Name + "." );
}
}

else if ( cmd == "settime" )
{
if ( !text ) MessagePlayer ( "/settime Hour & Minute." , player );
else if ( split ( text , " " ).len ( ) < 2 ) MessagePlayer ( "/settime Hour & Minute." , player );
else if ( !IsNum ( split ( text , " " ) [ 0 ] ) || !IsNum ( split ( text , " " ) [ 1 ] ) ) MessagePlayer ( "The text must be an integer." , player );
else
{
local splittext = split ( text , " " ),
hour = splittext [ 0 ],
minute = splittext [ 1 ];
SetHour ( hour.tointeger( ) );
SetMinute ( minute.tointeger( ) );
Message( "The time has been set to " + GetHour( ) + ":" + GetMinute( ) + " by " + player.Name + "." );
}
}
else MessagePlayer ( "Unknown command!" , player )
}

Code Select Expand

local splittext = split ( text , " " ),
hour = splittext [ 0 ],
minute = splittext [ 1 ];This may cause error.

Code Select Expand

local splittext = split ( text , " " );
if ( splittext.len() != 0 )
......


That's why I put

Code Select Expand

else if ( split ( text , " " ).len ( ) < 2 ) MessagePlayer ( "/settime Hour & Minute." , player );

[ysc3839]

Code Select Expand

function onPlayerCommand ( player , cmd , text )
{
cmd = cmd.tolower ( );

if ( cmd == "setweather" )
{
if ( !text ) MessagePlayer ( "/setweather WeatherID." , player );
else if ( !IsNum ( text ) ) MessagePlayer ( "The ID must be an integer." , player );
else if ( GetWeather( ) == text.tointeger( ) ) MessagePlayer( "The ID you put is already used!" , player );
else
{
SetWeather ( text.tointeger ( ) );
Message ( "The weather has been set to " + text + " by " + player.Name + "." );
}
}

else if ( cmd == "settime" )
{
if ( !text ) MessagePlayer ( "/settime Hour & Minute." , player );
else if ( split ( text , " " ).len ( ) < 2 ) MessagePlayer ( "/settime Hour & Minute." , player );
else if ( !IsNum ( split ( text , " " ) [ 0 ] ) || !IsNum ( split ( text , " " ) [ 1 ] ) ) MessagePlayer ( "The text must be an integer." , player );
else
{
local splittext = split ( text , " " ),
hour = splittext [ 0 ],
minute = splittext [ 1 ];
SetHour ( hour.tointeger( ) );
SetMinute ( minute.tointeger( ) );
Message( "The time has been set to " + GetHour( ) + ":" + GetMinute( ) + " by " + player.Name + "." );
}
}
else MessagePlayer ( "Unknown command!" , player )
}

Code Select Expand

local splittext = split ( text , " " ),
hour = splittext [ 0 ],
minute = splittext [ 1 ];This may cause error.

Code Select Expand

local splittext = split ( text , " " );
if ( splittext.len() != 0 )
......


That's why I put

Code Select Expand

else if ( split ( text , " " ).len ( ) < 2 ) MessagePlayer ( "/settime Hour & Minute." , player );

Sorry. But I think it's better not to split twice.

[hotdogcat]

i really don't understand how this "snippet" will contribute this community

[KAKAN]

Code Select Expand

else if ( cmd == "settime" )
 {
 if ( !text ) MessagePlayer ( "/settime Hour & Minute." , player );
local mada = split(text, " ");
 if ( mada.len ( ) < 2 ) MessagePlayer ( "/settime Hour & Minute." , player );
 else if ( !IsNum( mada[ 0 ] ) || !IsNum ( mada[ 1 ] ) ) MessagePlayer ( "The text must be an integer." , player );
 else
 {
 local hour = mada[ 0 ],
 minute = mada [ 1 ];
 SetHour ( hour.tointeger( ) );
 SetMinute ( minute.tointeger( ) );
 Message( "The time has been set to " + GetHour( ) + ":" + GetMinute( ) + " by " + player.Name + "." );
 }
 }There you go, @ysc3839 only 1 split used

[Xmair]

I like my way.

[DizzasTeR]

Split is more efficient than your 'GetTok' just so that you guys know.

[Stormeus]

Just go and f*ing kill your self.

@S.L.C Chill out, feel free to voice your criticism but don't tell people to go kill themselves.

[KAKAN]

Split is more efficient than your 'GetTok' just so that you guys know.

Both are same, still then I prefer split more than GetTok, but in some cases, I prefer GetTok.
It all depends on what you want to do and how much time you got to do it.
Most of the commands we use needs only 1 parameter, so there's less use of split either.

[DizzasTeR]

@KAKAN, Justify your statement, "Both are same"

[.]

Just go and f*ing kill your self.

@S.L.C Chill out, feel free to voice your criticism but don't tell people to go kill themselves.

@Stormeus Yeah, I did kinda snapped. It's just annoying AF to look for a snippet and having to go through pages and pages and pages of crap like that that has no real contribution other than to piss people off.

[KAKAN]

@KAKAN, Justify your statement, "Both are same"

Try this code:

Code Select Expand

local string = "I am a good string!", mada = split(string," ")[2], faka = GetTok(text, " ", 2 );
print( mada );
print( faka );
The result will be same. The difference is: split returns an array or null on failure, while get tok returns string or null on failure

[Xmair]

@KAKAN , The index text doesnt exists.