[L2JServer] Fix error duplicate Subclass

[allanalcantara]

sempre com a galera, rsrs! :D postando o que há de melhor. *-*

--- 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]

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]

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. mas fica no gosto do "adicionador" né! ^^

[ManoeL-ADM]

@paytaly

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

[paytaly]

Já tem postada por ai. Só procurar.
Sei que já ví, agora onde, é outra história. :P