Triggering weapons

[cbk1994]

Anyone know whats wrong with this?

PHP Code:

//#CLIENTSIDE
function onCreated( NULL )
{
  setTimer( 0.05 );
}
function onTimeOut( NULL )
{
  if ( keydown2( getkeycode( "d" ), false ) )
  {
    if ( this.kpress == false )
    {
      doTriggerWeapon( NULL );
    }
    this.kpress = true;
  }
  else
  {
    this.kpress = false;
  }
  setTimer( 0.05 );
}
function doTriggerWeapon( NULL )
{
  temp.i = findWeapon( player.weapons[selectedweapon].name );
  temp.i.onWeaponFired();
} 


GraalScript: Function Weapon Weapons/Basic Sword.onWeaponFired not found in script of Weapon -Player/Trigger

is the error I get in F2 log. Please help :C

[JkWhoSaysNi]

Is onWeaponFired() public in the weapon NPC?

[cbk1994]

Weapons/Basic Sword

PHP Code:

join( "functions_weapon" );
//#CLIENTSIDE
function onCreated( NULL )
{
  this.item_type = "weapon";
  this.item_name = "Basic Sword";
  this.item_var = "basicsword";
  
  this.sword_damage = 10;
  this.sword_freezetime = .5;
  this.sword_gani_idle = "idle";
  this.sword_gani_walk = "walk";
  this.sword_gani_attack = "sword";
  this.sword_image = "sword2.png";
} 


in the class

PHP Code:

[...]
//#CLIENTSIDE
[...]
function onWeaponFired( NULL )
{
  equipWeapon( NULL );
} 


[JkWhoSaysNi]

change it to:

PHP Code:

public function onWeaponFired( NULL )
{
  equipWeapon( NULL );
} 


Is there any reason for passing NULL to the function?

[Rapidwolve]

Originally Posted by JkWhoSaysNi

Is there any reason for passing NULL to the function?

I was thinking the same.

[cbk1994]

Makes it look better

[Deadly_Killer]

Originally Posted by cbkbud

Makes it look better

It looks retarded, to be perfectly honest with you.

It seems like a lot of useless space, and is a horrible habit if you plan to go into other languages such as C/C++/C# due to functions needing to take specific parameters.

[cbk1994]

Originally Posted by Deadly_Killer

It looks retarded, to be perfectly honest with you.

It seems like a lot of useless space, and is a horrible habit if you plan to go into other languages such as C/C++/C# due to functions needing to take specific parameters.

You sound retarded, to be perfectly honest with you.

Please read my post on Clean Coding: Common Sense, and you will find that everyone has their own scripting style, and that if it's okay with them, it works. I've already stated my mind on people who say that lines of code matter, and that space matters. You can't waste space because I can put as many of those as I want.

I can already script in C++, so get off of my ass.

[Deadly_Killer]

Originally Posted by cbkbud

You sound retarded, to be perfectly honest with you.

Please read my post on Clean Coding: Common Sense, and you will find that everyone has their own scripting style, and that if it's okay with them, it works. I've already stated my mind on people who say that lines of code matter, and that space matters. You can't waste space because I can put as many of those as I want.

I can already script in C++, so get off of my ass.

1) C++ is not a 'script'.
2) I won't argue with you, it seems like a waste of time.

[Tigairius]

Originally Posted by JkWhoSaysNi

Is there any reason for passing NULL to the function?

Yeah, seriously..? The script looks retarded, it's inefficient, the spacing between... everything is very bad looking, you're putting things where they don't need to be, and in my opinion, that script is terrible. I have done some bad scripting in my day, but nothing amounts to the poor quality of that specific script.

[ff7chocoboknight]

Ok When Did This Thread Have Anything To Do With Attacking Eachother About How Ppl Script? I Mean Really Ppl, Lets All Argue About The Way He Scripts Instead Of Helping Him!!

[Chandler]

You need to use public function,
public function onWeaponFired
should fix it.

Nice code btw,

[Novo]

Don't bite me.

Do this:

weapon.trigger("WeaponFired", params );

This'll trigger onWeaponFired, but if it doesn't exist, it doesn't do anything.

Another is to make onWeaponFired a public function and doing:
weapon.onWeaponFired();

[cbk1994]

Originally Posted by Tigairius

Yeah, seriously..? The script looks retarded, it's inefficient, the spacing between... everything is very bad looking, you're putting things where they don't need to be, and in my opinion, that script is terrible. I have done some bad scripting in my day, but nothing amounts to the poor quality of that specific script.

Originally Posted by cbk1994

Please read my post on Clean Coding: Common Sense, and you will find that everyone has their own scripting style, and that if it's okay with them, it works. I've already stated my mind on people who say that lines of code matter, and that space matters. You can't waste space because I can put as many of those as I want.

Thank you Novo and Chandler for being nice about respecting other people's code, and for the suggestions. The weapon.trigger is working perfectly.

Would a moderator please lock this topic, it has no more purpose and I don't want an even bigger arguement.

[zokemon]

I agree the NULL may be a little useless and does get rather in the way in my opinion but I don't see his script as looking horribly structured o_o

He likes to space his stuff out alot... So?

Personnaly, when I get into immensly long scripts, styling is important yes but any script style (as long as it is still organized and consistant) mean nothing as the script just a big haze (Have to remember where certain functions are defined in the script and such)