Show Posts
This section allows you to view all posts made by this member. Note that you can only see posts made in areas you currently have access to.
I don't know if you crossed that out because you unlocked or, or because you remembered you shouldn't. But know that you shouldn't lock threads in coding help just because the question has been answered. Someone may have a better solution or see an issue with the "solution" but they can't give it to you if it's locked.
I don't know how much you tried, but the error report gives you a line number (most text editors allow your to press ctrl+G to go to a certain line) and a general location of the error, you should practice finding and fixing the error yourself.
Also, I don't know what you have now, but in the code in this post, you can get rid of either the return or the else
it's so you can send a bit of information back out of the function
for example (poor example)
function getNumber1()
{
return 1;
}
function doMaths(%addme)
{
echo(getNumber1() + %addme);
}
then you go in your console and do doMaths(4); and it tells you "5"
in this instance getNumber1() is usable in exactly the same way as the number 1
now think what happens when you use variables and math in the getNumber1() function!
all of a sudden you can do math and stuff outside of your main function, and use your calculations elsewhere! it's a good deal.
you're calling the VARIABLE "getname" on the PLAYER
the player does not have this.
use this instead: messageclient(%client, '',"\c6" @ %client.getPlayerName() @ "\c6 killed you.");
it calls the METHOD (function) called "getPlayerName" on a client, which returns (lets you use) the name.
You should also replaceCode: [Select]if(%target $= "")with this:
{
messageclient(%client, '',"\c6No target specified.");
}Code: [Select]if(!isObject(findClientByName(%target).player)){Make sure to use findClientByName(%target) and not just %target.
messageClient(%client, '', "Invalid target specified.");
return;
}
function servercmdkill(%client, %target)
{
if(!isObject(findclientbyname(%target).player))
{
messageclient(%client,'',"\c6Invalid target specified.");
return;
}
else
{
findclientbyname(%target).player.kill();
messageclient(%client, '',"\c6" @ %client.player.getname @ "\c6 killed you.");
}
}
function servercmdkill(%client, %target)
{
if(%target $= "")
{
messageclient(%client, '',"\c6No target specified.");
}
else
{
%killer = %client.player.getname();
findclientbyname(%target).player.kill();
messageclient(%client, '',"\c6" @ %killer "\c6 killed you.");
}
}
Your just giving a name to it,
You could name it anything you want but it's still the same value.
They're just names
Code: [Select]function servercmdKill(%client,%input)
{
%Target = findclientbyname(%input);
%Target.player.kill();
}
That's a very, very crude way of doing it.
Code: [Select]function servercmdkill(%cl, %ta)
{
if(!%cl.isadmin)
return;
findclientbyname(%ta).player.kill();
}
function servercmdkill(%client, %target)
{
findclientbyname(%target);
%target.kill();
}function servercmdkill(%client, %target)
{
findclientbyname(%target).kill();
}function servercmdkill(%client, %target)
{
findclientbyname(%target);
%client.kill();
}
A lot of people on BL are nice and friendly.