[Realm Relay] I'm bad at coding and I should feel bad

[Alde.]

What is wrong with this? ;

		if (packet.id() == ID_USEITEM) {$.echo(">! Sent a useitem packet.");
		
		var time = packet.time
		
		var SlotObject = event.SlotObject
		var Location = event.Location
		
		var objectId = SlotObject.objectId
		var slotId = SlotObject.slotId
		var objectType = SlotObject.objectType
		
		var Locationx = Location.x
		var Locationy = Location.y
		
		$.echo("# time : "+time);
		$.echo("# objectId : "+objectId);
		$.echo("# slotId : "+slotId);
		$.echo("# objectType : "+objectType);
		$.echo("# Locationx : "+Locationx);
		$.echo("# Locationy : "+Locationy);		
		
		}

[Alde.]

Bumpty bump!

[Notsomeone]

What is wrong with this? ;

		if (packet.id() == ID_USEITEM) {$.echo(">! Sent a useitem packet.");
		
		var time = packet.time
		
		var SlotObject = event.SlotObject
		var Location = event.Location
		
		var objectId = SlotObject.objectId
		var slotId = SlotObject.slotId
		var objectType = SlotObject.objectType
		
		var Locationx = Location.x
		var Locationy = Location.y
		
		$.echo("# time : "+time);
		$.echo("# objectId : "+objectId);
		$.echo("# slotId : "+slotId);
		$.echo("# objectType : "+objectType);
		$.echo("# Locationx : "+Locationx);
		$.echo("# Locationy : "+Locationy);		
		
		}

Why don't you tell us whats wrong? Errors usually come with error logs.

[infern000]

why are you trying to get the SlotObject and Location from event and not from packet?

		var SlotObject = event.SlotObject
		var Location = event.Location

[Notsomeone]

why are you trying to get the SlotObject and Location from event and not from packet?

		var SlotObject = event.SlotObject
		var Location = event.Location

True. ZBORNOX, you make good scripts but please use $.echo and try/catch blocks to debug before asking here.

[Alde.]

@Both guys ahead ; I have already tried using x.packet, and it didnt worked either.

[Alde.]

Why don't you tell us whats wrong? Errors usually come with error logs.

[Notsomeone]

Im not trying to demotivate you, or to be mean, but you should really study on the concept of classes and instances of classes. The variable "event" has no property "Location".

[Alde.]

Im not trying to demotivate you, or to be mean, but you should really study on the concept of classes and instances of classes. The variable "event" has no property "Location".

:'(

Then who has?



Also, thanks ; I didnt knew what their name was (I tried looking away before coming here). Classes!

[Notsomeone]

You just need to grasp that Location and Slotobject are classes. Not instances.

[Alde.]

You just need to grasp that Location and Slotobject are classes. Not instances.

Can you please teach me quickly (if it can be done) how to grap the data from the classes?

[Notsomeone]

Class is like a type for a variable. Int, String etc.
You have to look at the UseItem packet at script_help.txt. It will specify the properties (variables) of the packet and their class.

[Alde.]

Class is like a type for a variable. Int, String etc.
You have to look at the UseItem packet at script_help.txt. It will specify the properties (variables) of the packet and their class.

Finaly some light at the end of the tunnel!


Its

var SlotObject = packet.slotObject;

NOT

var SlotObject = packet.SlotObject;

Thanks a lot Notsomeone for teaching me instead of doing it for me!

Solved! /aww

[JustAnoobROTMG]

@ZBORNOX

Classes :
OOP in JS, Part 1 : Public/Private Variables and Methods

[Notsomeone]

This link is very useful too, I like the easily understandable examples. Lesson: Classes and Objects (The Java™ Tutorials > Learning the Java Language)