Packets: PHP to NodeJS

[shadowvzs]

i started the play with nodeJS 10+ and binary/buffer/packet sending, if someone got idea or advice then here the mail sending & chat server (well i tryed on 1.4.2 not on newer pw but nah):

Features (testing on 1.4.2 v27/29):

* Chat:
---- send

* GM
---- getOnlineList
---- ban

* Guild
---- list all guild
---- load current guild
---- change rank in guild
---- create guild

* Mail
---- sendSysMail
---- sendGold

* Role
---- load
---- save
---- reset
---- delete
---- ban
---- rename
---- getGuild

* Territory
---- getList

* User
---- roleList
---- setGold (instant set shop gold and wipe the gold history)
---- getInfo (gold bought, spent, roleid etc)

You must be registered to see links

 

[w3bm4ster]

Don't you need to sum the value in CUint function, like in PHP

($value | 0x8000) for example?
Like so: 
[PHP]return this.WriteUByte(value + 0x80000, method);

And what about the ReadPacket? Looks more easier then Write. Anyway, amazing job! I like to see how Node is powerful tt:

 

[shadowvzs]

honestly this isn't finished yet, i still working on it

last mofications:
- i restructured how role read work
- changed read packets
- added async/await for roles because but be useable for api

You must be registered to see links

 

[w3bm4ster]

honestly this isn't finished yet, i still working on it

last mofications:
- i restructured how role read work
- changed read packets
- added async/await for roles because but be useable for api

You must be registered to see links

I see the GitHub. Well, even if you haven't finished It's an awesome progress!

My suggestions:
- Clean up Promises a little bit and structure of packets (it's completely bind to functions name)
- Add some checks to the Read Packet
- Instead of put service IP/port at index, leave directly in Packet File

 

[shadowvzs]

I see the GitHub. Well, even if you haven't finished It's an awesome progress!

My suggestions:
- Clean up Promises a little bit and structure of packets (it's completely bind to functions name)
- Add some checks to the Read Packet
- Instead of put service IP/port at index, leave directly in Packet File

Thank you for constructive feedback
- i noticed my silly mistakes at promise, i fixed the promise.all
- what checks do you mean? i added schemes which extract data based on scheme and i use try catch now
- host i can add in packet.js but port i can't since at moment i have 3-4 stuff (chat/mail/read user roles/ read role data) and totally 3 different ip (29100, 29300, 29400)

 

[shadowvzs]

well i updated my list, if you have other idea or suggestion/advice then that is welcome/so tell me

 

[w3bm4ster]

I see. And yes, it's only IP (I don't know why I typed host) >_>

I believe you can improve the scheme and this is not actually my tip, because it's a pattern from an old panel shared throughout PWPS forums. Just simply use ["int", "cuint", "float"] and then create a function that receives the struct and the data, iterate through them and use a switch case to unpack based on data type. With this approach, I believe will look more cleaner.

 

[shadowvzs]

I see. And yes, it's only IP (I don't know why I typed host) >_>

I believe you can improve the scheme and this is not actually my tip, because it's a pattern from an old panel shared throughout PWPS forums. Just simply use ["int", "cuint", "float"] and then create a function that receives the struct and the data, iterate through them and use a switch case to unpack based on data type. With this approach, I believe will look more cleaner.

it is similiar now too except scheme have:
- octets,
- array,
- string,
- uint32,
- uint16,
- int8,
- byte,
- float
example for role data (included unit16 - storesize)

You must be registered to see links

i need to have array too because its exact a group of data with a simple/signle scheme like items (same pattern if inventory/equipment/fashion/material or storehouse, task items or cards)

You must be registered to see links

WritePacket.PackAll(scheme, data, protocol) - use same scheme like
ReadPacket.UnpackAll(scheme, data) - same scheme

idk, so you say current system is inefficient?

[EDIT]
btw added more function and screenshot from last 2

 

[w3bm4ster]

it is similiar now too except scheme have:
- octets,
- array,
- string,
- uint32,
- uint16,
- int8,
- byte,
- float
example for role data (included unit16 - storesize)

You must be registered to see links

i need to have array too because its exact a group of data with a simple/signle scheme like items (same pattern if inventory/equipment/fashion/material or storehouse, task items or cards)

You must be registered to see links

WritePacket.PackAll(scheme, data, protocol) - use same scheme like
ReadPacket.UnpackAll(scheme, data) - same scheme

idk, so you say current system is inefficient?

[EDIT]
btw added more function and screenshot from last 2

Nop. My suggestion was merely aesthetic.

 

[shadowvzs]

updated and added tw land info listing, get role guild etc