Jump to content
  • 0

AutoLoot


nakashimi

Question

 Could any good soul adapt this mode to l2jfrozen?

package handlers.voicedcommandhandlers;

import net.sf.l2j.gameserver.handler.IVoicedCommandHandler;
import net.sf.l2j.gameserver.model.actor.instance.L2PcInstance;

/**
* @author Cobra
*/

public class autoloot implements IVoicedCommandHandler
{
private static final String[] VOICED_COMMANDS =
private static String[] _voicedCommands =
{
    "autolooton",
    "autolootoff"
};

public String[] getVoicedCommandList()
{
    return VOICED_COMMANDS;
}

public boolean useVoicedCommand(String command, L2PcInstance activeChar, String target)
{
    if (command.equalsIgnoreCase("autolooton"))
    {
        activeChar.setAutoLootEnabled(true);
        activeChar.sendMessage("Auto loot is now enabled.");
    }
    else if (command.equalsIgnoreCase("autolootoff"))
    {
        activeChar.setAutoLootEnabled(false);
        activeChar.sendMessage("Auto loot is now disabled.");
    }
    return true;
}
}

It is for l2jserver but it is missing the L2PCInstance file from setAutoLootEnabled

I even tried it but when I tested it it didn't run the code I created it was this

/** AutoLoot parameters*/
private boolean _autoLootEnabled = false;

public void setAutoLootEnabled(final boolean autoLootEnabled)
{
_autoLootEnabled = autoLootEnabled;

}

 

/**
* @param reference
* @return Returns the autoLootEnabled.
*/

public boolean isAutoLootEnabled(final L2Object reference)
{
return _autoLootEnabled && !(reference instanceof L2GrandBossInstance)
&& !(reference instanceof L2RaidBossInstance)
&& !(reference instanceof L2MinionInstance);

}

 

Link to comment
Share on other sites

1 answer to this question

Recommended Posts

  • 0

If you sit down and study the code for several minutes you will understand how it work

and you won't need any help or copy paste the exact code.

 

This code:

public boolean useVoicedCommand(String command, L2PcInstance activeChar, String target)
{
    if (command.equalsIgnoreCase("autolooton"))
    {
        activeChar.setAutoLootEnabled(true);
        activeChar.sendMessage("Auto loot is now enabled.");
    }
    else if (command.equalsIgnoreCase("autolootoff"))
    {
        activeChar.setAutoLootEnabled(false);
        activeChar.sendMessage("Auto loot is now disabled.");
    }
    return true;
}

 

can turn into this:

@Override
public boolean useVoicedCommand(String command, L2PcInstance activeChar, String target)
{
    activeChar.setIsAutoLoot(!activeChar.isAutoLoot());
    activeChar.sendMessage("Auto loot is now " + (isAutoLoot(null) ? "enabled." : "disabled"));
    return true;
}

You don't need the if statement since there are only 2 cases and the useVoicedCommand will only executed when 1 of these handler that are in String[] are called. 

 

And this code: 

private boolean _autoLootEnabled = false;

public void setAutoLootEnabled(final boolean autoLootEnabled)
{
_autoLootEnabled = autoLootEnabled;

}

 

/**
* @param reference
* @return Returns the autoLootEnabled.
*/

public boolean isAutoLootEnabled(final L2Object reference)
{
return _autoLootEnabled && !(reference instanceof L2GrandBossInstance)
&& !(reference instanceof L2RaidBossInstance)
&& !(reference instanceof L2MinionInstance);

}

Should be better:

private boolean _isAutoLoot;

public void setIsAutoLoot(final boolean val)
{
   _isAutoLoot = val;
}

public boolean isAutoLoot(final L2Character target)
{
   return Objects.nonNull(target) && target.isRaid() ? false : _isAutoLoot;
}

 

Edited by EdithFinch
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Answer this question...

×   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.



×
×
  • Create New...