Jump to content

Recommended Posts

Posted

For those that wanna implement Callbacks in java for better source control, or use bytecode manipulation for other ( evil ? ) purposes:

http://www.csg.is.titech.ac.jp/~chiba/javassist/tutorial/tutorial.html

 

For those that wanna learn code design and architecture to be able to make things like Phoenix-Engine easily:

http://www.javacamp.org/designPattern/

 

For those that wanna find or fix exploits in l2j based on race conditions and general thread synchronization issues:

http://www.javaworld.com/jw-10-1998/jw-10-toolbox.html

Posted

For those that wanna implement Callbacks in java for better source control, or use bytecode manipulation for other ( evil ? ) purposes:

http://www.csg.is.titech.ac.jp/~chiba/javassist/tutorial/tutorial.html

 

For those that wanna learn code design and architecture to be able to make things like Phoenix-Engine easily:

http://www.javacamp.org/designPattern/

 

callback is a highly not recommended thing to implement into l2j but anyway if someone wants to make web apps than its usefull... :)

Posted

callback is a highly not recommended thing to implement into l2j but anyway if someone wants to make web apps than its usefull... :)

 

If implemented correctly, they add absolutelly no overhead in the application. We are using them in our private svn and they work pretty well. Its a rework of the aionemu callback system, with the annotations removed. Saves you from applying 100 core patches and adds zero performance downgrade.

Posted

If implemented correctly, they add absolutelly no overhead in the application. We are using them in l2jsaver private svn and they work pretty well. Saves you from applying 100 core patches and adds zero performance downgrade.

 

Well it still creates a bunch of objects for nothing in many case(which increase the memory load), and on top of that it gives a really bad control over the source since you need to create an interface for most things separated.

 

The reason why people use callbacks are because they want an event notifier for web apps its not usefull for other things.

 

I know tho you picked to choose the callbacks from aion to the advice of a noob, but unless you want many many many small and useless objects for something that can be solved with a simple global task for example than you should not implement a callback. :)

Posted

We are probably talking about different things. I do not mean to hook interface lists inside functions in the source code and add objects that implement them. My callback system is based on javassist. It doesnt create any objects at all, it just applies the "patch" not in the source code, but on the bytecode when the JVM loads the class. For example.

 

public static void blabla()

{

 

System.out.println("hi1");

 

}

 

When the class containing that method gets loaded in the jvm, the jvm checks the javassist agent and if i have declared to do + testtest() inside blabla() , during class loading it manipulates and loads the following instead:

 

 

public static void blabla()

{

 

testtest();

System.out.println("hi1");

 

}

 

From then on, it juses the new class and voila, no extra objects, no extra memory, no extra cpu usage ;)

 

So the process of loading a class is :

 

New class requested ---> JVM gets the .class file in a byte[] array --> javassist agent transforms the class by patching --> ClassLoader loads the class and the application uses the patched one.

Posted

We are probably talking about different things. I do not mean to hook interface lists inside functions in the source code and add objects that implement them. My callback system is based on javassist. It doesnt create any objects at all, it just applies the "patch" not in the source code, but on the bytecode when the JVM loads the class. For example.

 

public static void blabla()

{

 

System.out.println("hi1");

 

}

 

When the class containing that method gets loaded in the jvm, the jvm checks the javassist agent and if i have declared to do + testtest() inside blabla() , during class loading it manipulates and loads the following instead:

 

 

public static void blabla()

{

 

testtest();

System.out.println("hi1");

 

}

 

From then on, it juses the new class and voila, no extra objects, no extra memory, no extra cpu usage ;)

 

 

But you know that a Java/Native call is in general 3 times slower than a Java/Java call right? :)

Posted

Its not a native call, why is it a native call ? o0 ? All it does is load a patched bytecode in the jvm. From then on JVM handles it like a normal class without knowing if its patched or not.

Posted

Its not a native call, why is it a native call ? o0 ? All it does is load a patched bytecode in the jvm.

 

Editing the byte code makes the code runs slower than if its in java and gets loaded into the jvm only without modifying.

 

This API allows you to read/write the bytecode on the fly which is not a good thing at all for l2j, not to mention its reflection based which is again a useless and really 1 of the worse ideas to ever use in l2j at high level. It makes understanding the code and coding harder and way more unfriendly.

 

There are high chance to get a really huge memory usage and even out of memory errors if the numbers of classes that needs to be loaded into the class pool even if the API try to reduce it.

 

it created instances "for ever" (imho until you replace the jar) so basically even if you use something rarely it leaves an instance there without allowing the garbage collector to get it since its blocked from it to keep the object for later usage.

 

The class loader of it is in general useless the current way of l2j is much better.

 

1 advantage i can say is you can modify the source on runtime but that can be done with other much better ways also(altho you would need a strong pc for it).

Posted
Editing the byte code makes the code runs slower than if its in java and gets loaded into the jvm only without modifying.

 

Where did you get that ? I tested two big loops of printing a string. One with the code in the class normally, the other way with an empty function that gets patched on runtime. The miliseconds that both loops took, to complete were approximatelly the same.

Posted

Where did you get that ?

 

Its a general fact a method call happens much faster if its a java/java call than if its a java from bytecode call ok thats not a real problem for you(i looked into other API not into what you use).

 

What i posted above at the reply 22 is what i found on your api so far.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.



  • Posts

    • Where I can buy a cheap domain .com? cheapest I found was on Godaddy for 12 euro and Hostinger for 10 euro.
    • Hello everyone, here's a simple and useful idea for any type of server.   This code applies a discount when a player makes a purchase inside a clan’s castle or clan hall, offering a benefit to clan members who own a castle or clan hall. Important: Merchant transactions must be handled through multisell, not buylist. The discount is directly applied within the multisell, so the price shown is already reduced.   "For example, if a scroll costs 1000 Adena and you set a 20% discount in the config, the final price when purchasing inside a castle or clan hall will be 800 Adena."   This code is developed on the public aCis 401 revision.   public static int CLAN_BASE_OWNERSHIP_MERCHANT_DISCOUNT; CLAN_BASE_OWNERSHIP_MERCHANT_DISCOUNT = clans.getProperty("ClanBaseOwnershipMechantDiscount", 20); # If clan owns a clan hall or castle, all members have a discount of X% at merchant transactions (multisell). # Discount applies only inside the base (castle or clan hall). ClanBaseOwnershipMechantDiscount = 20   /** diff --git a/aCis_gameserver/java/net/sf/l2j/gameserver/data/xml/MultisellData.java b/aCis_gameserver/java/net/sf/l2j/gameserver/data/xml/MultisellData.java index 556e111..bbf8e69 100644 --- a/aCis_gameserver/java/net/sf/l2j/gameserver/data/xml/MultisellData.java +++ b/aCis_gameserver/java/net/sf/l2j/gameserver/data/xml/MultisellData.java @@ -101,7 +101,7 @@ do { // send list at least once even if size = 0 - player.sendPacket(new MultiSellList(list, index)); + player.sendPacket(new MultiSellList(list, index, player)); index += PAGE_SIZE; } while (index < list.getEntries().size()); diff --git a/aCis_gameserver/java/net/sf/l2j/gameserver/network/clientpackets/MultiSellChoose.java b/aCis_gameserver/java/net/sf/l2j/gameserver/network/clientpackets/MultiSellChoose.java index 7c82c5b..1654abc 100644 --- a/aCis_gameserver/java/net/sf/l2j/gameserver/network/clientpackets/MultiSellChoose.java +++ b/aCis_gameserver/java/net/sf/l2j/gameserver/network/clientpackets/MultiSellChoose.java @@ -6,6 +6,7 @@ import net.sf.l2j.Config; import net.sf.l2j.gameserver.enums.FloodProtector; import net.sf.l2j.gameserver.enums.StatusType; +import net.sf.l2j.gameserver.enums.ZoneId; import net.sf.l2j.gameserver.enums.items.CrystalType; import net.sf.l2j.gameserver.model.Augmentation; import net.sf.l2j.gameserver.model.actor.Player; @@ -225,6 +226,20 @@ return; } + if (player.isInsideZone(ZoneId.CLAN_HALL) && player.getClan() != null && player.getClan().hasClanHall()) + { + e.setItemCount(e.getItemCount() * (100 - Config.CLAN_BASE_OWNERSHIP_MERCHANT_DISCOUNT) / 100); + if (e.getItemCount() == 0) + e.setItemCount(1); + } + + if (player.isInsideZone(ZoneId.CASTLE) && player.getClan() != null && player.getClan().hasCastle()) + { + e.setItemCount(e.getItemCount() * (100 - Config.CLAN_BASE_OWNERSHIP_MERCHANT_DISCOUNT) / 100); + if (e.getItemCount() == 0) + e.setItemCount(1); + } + if (Config.BLACKSMITH_USE_RECIPES || !e.getMaintainIngredient()) { // if it's a stackable item, just reduce the amount from the first (only) instance that is found in the inventory diff --git a/aCis_gameserver/java/net/sf/l2j/gameserver/network/serverpackets/MultiSellList.java b/aCis_gameserver/java/net/sf/l2j/gameserver/network/serverpackets/MultiSellList.java index 9269b06..c6102a0 100644 --- a/aCis_gameserver/java/net/sf/l2j/gameserver/network/serverpackets/MultiSellList.java +++ b/aCis_gameserver/java/net/sf/l2j/gameserver/network/serverpackets/MultiSellList.java @@ -2,6 +2,9 @@ import static net.sf.l2j.gameserver.data.xml.MultisellData.PAGE_SIZE; +import net.sf.l2j.Config; +import net.sf.l2j.gameserver.enums.ZoneId; +import net.sf.l2j.gameserver.model.actor.Player; import net.sf.l2j.gameserver.model.multisell.Entry; import net.sf.l2j.gameserver.model.multisell.Ingredient; import net.sf.l2j.gameserver.model.multisell.ListContainer; @@ -15,7 +18,9 @@ private boolean _finished; - public MultiSellList(ListContainer list, int index) + private Player _player; + + public MultiSellList(ListContainer list, int index, Player player) { _list = list; _index = index; @@ -28,6 +33,8 @@ } else _finished = true; + + _player = player; } @Override @@ -74,7 +81,14 @@ { writeH(ing.getItemId()); writeH(ing.getTemplate() != null ? ing.getTemplate().getType2() : 65535); - writeD(ing.getItemCount()); + + if (_player.isInsideZone(ZoneId.CLAN_HALL) && _player.getClan() != null && _player.getClan().hasClanHall()) + writeD((ing.getItemCount() * (100 - Config.CLAN_BASE_OWNERSHIP_MERCHANT_DISCOUNT) / 100) < 1 ? 1 : ing.getItemCount() * 80 / 100); + else if (_player.isInsideZone(ZoneId.CASTLE) && _player.getClan() != null && _player.getClan().hasCastle()) + writeD((ing.getItemCount() * (100 - Config.CLAN_BASE_OWNERSHIP_MERCHANT_DISCOUNT) / 100) < 1 ? 1 : ing.getItemCount() * 80 / 100); + else + writeD(ing.getItemCount()); + writeH(ing.getEnchantLevel()); writeD(0x00); // TODO: i.getAugmentId() writeD(0x00); // TODO: i.getManaLeft()  
    • DISCORD : utchiha_market telegram : https://t.me/utchiha_market SELLIX STORE : https://utchihamkt.mysellix.io/ Join our server for more products : https://discord.gg/uthciha-services https://campsite.bio/utchihaamkt
    • WTB EXP ETERNAL 10x new dm.
    • This project is based on the latest public aCis sources (revision 401) and supports a multi-client system (C4 & IL), making it suitable for custom usage but not for retail.   You can configure the SelectedClient option in server.properties and loginserver.properties to switch between C4 and IL.  Both clients are fully synchronized, including login, server selection, packets, and geodata.   Notable Features: - Completed the login and server selection phase for both clients. - Synchronized all packets to support both clients (including some specific features). - Reworked the datapack and SQL files (excluding HTML files) to work seamlessly with both clients. - Added geodata support for both clients. - Adapted nearly all AI, scripts, bosses, HTML, and MULTISELL files to match C4 functionality. - Reduced the maximum clan level from 8 to 5 (C4 feature). - Rewrote clan HTML to remove C5-C6 features.   Disabled the following C5 and C6 features: - Divine Inspiration (C6 feature). - Clan skills and clan reputation points (C5 feature). - Pledge class (C5 feature). - Hero skills (C5 feature). - Dueling system (C6 feature). - Augmentations (C6 feature). - Cursed weapons (C5-C6 feature).   General Improvements: - Performed a general HTML cleanup and optimized features based on the client version. - Added an option to display the remaining time of disabled skills. - Skill timestamps now update when using the skill list.   This flexibility allows you to create a unique progression system tailored to your needs. The price for the diff patch, which can be applied to aCis public sources, is €150. For inquiries, please contact me via PM or Discord (ID: @Luminous).
  • Topics

×
×
  • Create New...