Secret Experience

Secret Experience (https://secretexperience.net/index.php)
-   [Lineage] Java Mods (https://secretexperience.net/forumdisplay.php?f=636)
-   -   [L2JServer] Fix error duplicate Subclass (https://secretexperience.net/showthread.php?t=32604)

allanalcantara 16-08-2011 12:27 PM
Fix error duplicate Subclass
 
1 Anexo(s)
sempre com a galera, rsrs! :D postando o que há de melhor. *-*

Código:
--- trunk/L2J-Mack_IL/L2J-Mack_GameServer/java/com/l2dot/gameserver/model/actor/instance/L2PcInstance.java (revision 14)
+++ trunk/L2J-Mack_IL/L2J-Mack_GameServer/java/com/l2dot/gameserver/model/actor/instance/L2PcInstance.java (revision 32)
@@ -8977,7 +8977,7 @@
      * @param int classIndex
      * @return boolean subclassAdded
      */
-    public boolean addSubClass(int classId, int classIndex)
+        public synchronized boolean addSubClass(int classId, int classIndex)
    {
            if (getTotalSubClasses() == Config.MAX_SUBCLASS || classIndex == 0)
                    return false;

paytaly 16-08-2011 01:11 PM
Só pra constar: Esse não é o melhor método de prevenir Multiclass.
Para mais segurança é melhor usar um Lock que é habilitado no começo do método e desabilitado no final.

O modificador synchronized faz com que o método seja executado por apensa um thread de cada vez. Não estou dizendo que nesse caso específico acontecerá, mais que tomem cuidado ao usá-lo pois ele pode matar ou travar threads e causar mais Bugs do que conserta.

A dica é testar bem antes de adicionar qualquer coisa.

allanalcantara 16-08-2011 02:06 PM
Citação:
Só pra constar: Esse não é o melhor método de prevenir Multiclass.
Para mais segurança é melhor usar um Lock que é habilitado no começo do método e desabilitado no final.

O modificador synchronized faz com que o método seja executado por apensa um thread de cada vez. Não estou dizendo que nesse caso específico acontecerá, mais que tomem cuidado ao usá-lo pois ele pode matar ou travar threads e causar mais Bugs do que conserta.

A dica é testar bem antes de adicionar qualquer coisa.
concordo plenamente. rsrs! :]
em algumas packs isso pode resolver, em outras pode dar problemas, e não são poucos. :wacko: mas fica no gosto do "adicionador" né! ^^

ManoeL-ADM 16-08-2011 03:41 PM
@paytaly

que tal posta essa correção que vc disse pra galera?

paytaly 16-08-2011 05:14 PM
Já tem postada por ai. Só procurar.
Sei que já ví, agora onde, é outra história. :P


Horários baseados na GMT -3. Agora são 10:47 PM.

Powered by vBulletin® Version 3.8.4
Copyright ©2000 - 2026, Jelsoft Enterprises Ltd.