Help Help With Code

L2shaken · August 30, 2016

someone help me with this code? this error in the SQL database that collects information. "SELECT boss_id FROM raidboss_spawnlist ORDER BY boss_id ASC LIMIT "+ stpoint + Config.RAID_LIST_RESULTS);"

original code: "SELECT id, name, level FROM npc WHERE type='L2RaidBoss' AND EXISTS (SELECT * FROM raidboss_spawnlist WHERE raidboss_spawnlist.boss_id = npc.id) ORDER BY `level` " + sort + " Limit " + stpoint + ", " + Config.RAID_LIST_RESULTS);

code for ACIS

entire code.

package net.sf.l2j.gameserver.communitybbs;

import net.sf.l2j.Config;
import net.sf.l2j.L2DatabaseFactory;
import net.sf.l2j.gameserver.datatables.NpcTable;

import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.util.logging.Level;
import java.util.logging.Logger;

public class RaidList
{
  protected static final Logger _log = Logger.getLogger(RaidList.class.getName());
  private final StringBuilder _raidList = new StringBuilder();
  
  public RaidList(String rfid)
  {
    loadFromDB(rfid);
  }
  
  private void loadFromDB(String rfid)
  {
    int type = Integer.parseInt(rfid);
    int stpoint = 0;
    int pos = 0;
    
    for (int count = 1; count != type; count++) 
    {
      stpoint += Config.RAID_LIST_RESULTS;
    }
    try
    {
      Connection con = L2DatabaseFactory.getInstance().getConnection();Throwable localThrowable3 = null;
      try
      {
        PreparedStatement statement = con.prepareStatement("SELECT boss_id FROM raidboss_spawnlist ORDER BY boss_id ASC LIMIT "+ stpoint + Config.RAID_LIST_RESULTS);
        ResultSet result = statement.executeQuery();
        pos = stpoint;
        while (result.next())
        {
           int npcid = result.getInt("boss_id");
           String npcname = NpcTable.getInstance().getTemplate(npcid).getName();
           byte rlevel = NpcTable.getInstance().getTemplate(npcid).getLevel();
          PreparedStatement statement2 = con.prepareStatement("SELECT respawn_time, spawn_time, random_time FROM raidboss_spawnlist WHERE boss_id=" + npcid);
          ResultSet result2 = statement2.executeQuery();
          while (result2.next())
          {
            pos++;
            boolean rstatus = false;
            long respawn = result2.getLong("respawn_time");
            if (respawn == 0L) {
              rstatus = true;
            }
            int mindelay = result2.getInt("spawn_time");
            int maxdelay = result2.getInt("random_time");
            mindelay = mindelay / 60 / 60;
            maxdelay = maxdelay / 60 / 60;
            addRaidToList(pos, npcname, rlevel, mindelay, maxdelay, rstatus);
          }
          result2.close();
          statement2.close();
        }
        result.close();
        statement.close();
      }
      catch (Throwable localThrowable1)
      {
        localThrowable3 = localThrowable1;throw localThrowable1;
      }
      finally
      {
        if (con != null) {
          if (localThrowable3 != null) {
            try
            {
              con.close();
            }
            catch (Throwable localThrowable2)
            {
              localThrowable3.addSuppressed(localThrowable2);
            }
          } 
          else 
          {
            con.close();
          }
        }
      }
    }
    catch (Exception e)
	 {
		 _log.log(Level.WARNING, "Failed to load Raid List");
		 e.printStackTrace();
	 }
   }
  
  private void addRaidToList(int pos, String npcname, byte rlevel, int mindelay, int maxdelay, boolean rstatus)
  {
    _raidList.append("<table border=0 cellspacing=0 cellpadding=2 width=610 height=" + Config.RAID_LIST_ROW_HEIGHT + ">");
    _raidList.append("<tr>");
    _raidList.append("<td FIXWIDTH=5></td>");
    _raidList.append("<td FIXWIDTH=25>" + pos + "</td>");
    _raidList.append("<td FIXWIDTH=270>" + npcname + "</td>");
    _raidList.append("<td FIXWIDTH=50>" + rlevel + "</td>");
    _raidList.append("<td FIXWIDTH=120 align=center>" + mindelay + " - " + maxdelay + "</td>");
    _raidList.append("<td FIXWIDTH=50 align=center>" + (rstatus ? "<font color=99FF00>Alive</font>" : "<font color=CC0000>Dead</font>") + "</td>");
    _raidList.append("<td FIXWIDTH=5></td>");
    _raidList.append("</tr>");
    _raidList.append("</table>");
    _raidList.append("<img src=\"L2UI.Squaregray\" width=\"610\" height=\"1\">");
  }
  
  public String loadRaidList()
  {
    return _raidList.toString();
  }
}

code for ACIS

Gam3Master · August 30, 2016

it's a very ugly.

You can get Raid bosses without sql connection.

L2shaken · August 30, 2016

it's a very ugly.

You can get Raid bosses without sql connection.

Can you help me ?

Edited August 30, 2016 by L2shaken

Gam3Master · August 30, 2016

private static void showRaidBosses(L2PcInstance activeChar, int page)

{

String content = HtmCache.getInstance().getHtm("data/html/CommunityBoard/Statistics/RaidBosses.htm");

StringBuilder _raidList = new StringBuilder();

Collection<NpcTemplate> npcs = NpcTable.getInstance().getAllNpcs();

List<NpcTemplate> list = new ArrayList<>(npcs);

Collections.sort(list, new Comparator<NpcTemplate>()

{

@Override

public int compare(NpcTemplate left, NpcTemplate right)

{

return right.getLevel() - left.getLevel();

}

});

pos = 0;

int myPage = 1;

int i = 0;

boolean hasMore = false;

for(final NpcTemplate boss : list)

{

int bossId = boss.getNpcId();

if(!boss.isType("L2RaidBoss"))

continue;

if(bossId >= 25333 && boss.getNpcId() <=25338)

continue;

if(bossId == 25286 || boss.getNpcId() == 25283 || boss.getNpcId() == 29054 || boss.getNpcId() == 25273 || boss.getNpcId() == 29062 || boss.getNpcId() == 29040 || boss.getNpcId() == 25328)

continue;

if(bossId == 29060 || boss.getNpcId() == 29056 || boss.getNpcId() == 29030 || boss.getNpcId() == 29033 || boss.getNpcId() == 29036 || boss.getNpcId() == 29037)

continue;

String color = "";

boolean rstatus = false;

String npcname = boss.getName();

int level = boss.getLevel();

long delay = RaidBossSpawnManager.getInstance().getRespawntime(boss.getIdTemplate());

int hours = (int)((delay - System.currentTimeMillis()) / 1000L / 60L / 60L);

int minutes = (int)((delay - hours * 60 * 60 * 1000 - System.currentTimeMillis()) / 1000L / 60L);

if(delay == 0)

rstatus = true;

if (pos == PAGE_LIMIT)

{

hasMore = true;

break;

}

if (myPage != page)

{

i++;

if (i == PAGE_LIMIT)

{

myPage++;

i = 0;

}

continue;

}

pos++;

if(pos % 2 == 0)

color="bgcolor=000000";

_raidList.append("<table "+color+" border=0 cellspacing=0 cellpadding=0 width=500>");

_raidList.append("<tr>");

_raidList.append("<td FIXWIDTH=30 align=center>" + pos + "</td>");

_raidList.append("<td FIXWIDTH=220>" + npcname + "</td>");

_raidList.append("<td FIXWIDTH=50 align=center>" + level + "</td>");

_raidList.append("<td FIXWIDTH=200 align=center>" + ((rstatus) ? "<font color=008000>Alive</font>" : "<font color=13eedd>"+hours+" hours "+minutes+" mins</font>") + "</td>");

_raidList.append("</tr>");

_raidList.append("</table>");

_raidList.append("<img src=\"L2UI.Squaregray\" width=\"500\" height=\"1\">");

}

int nextpage = page + 1;

int prevpage = page - 1;

_raidList.append("<center><table><tr>");

if (page > 1)

{

_raidList.append("<td><button action=\"bypass _bbsbosses;"+prevpage+ "\" value=\"Previous\" width=75 height=21 back=\"L2UI_ch3.Btn1_normalOn\" fore=\"L2UI_ch3.Btn1_normal\"></td>");

if (!hasMore)

_raidList.append("<td FIXWIDTH=80><center>Page "+ page+ "</center></td></tr>");

}

if (hasMore)

{

if (page <= 1)

_raidList.append("<td></td>");

_raidList.append("<td FIXWIDTH=80><center>Page "+ page+ "</center></td><td><button action=\"bypass _bbsbosses;"+nextpage+ "\" value=\"Next ->\" width=75 height=21 back=\"L2UI_ch3.Btn1_normalOn\" fore=\"L2UI_ch3.Btn1_normal\"></td></tr>");

}

_raidList.append("</table></center>");

content = content.replaceAll("%raidlist%", _raidList.toString());

separateAndSend(content, activeChar);

}

L2shaken · August 30, 2016

	

Thank you , thank you , you have the rest of the code ? I think I'll have to add a few more things in the community .

Edited August 30, 2016 by L2shaken

Elfo · August 30, 2016

Wow Gam3Master you nailed it. You managed to triple penetrate SOLID principles, coding standards and basic coding concepts all in 50 lines of code.

I can't even be mad, i will use this in tomorrows scrum.

L2shaken · August 30, 2016

Wow Gam3Master you nailed it. You managed to triple penetrate SOLID principles, coding standards and basic coding concepts all in 50 lines of code.

I can't even be mad, i will use this in tomorrows scrum.

You can help me by to see the list of raid boss in the community ?

Tessa · August 30, 2016

Wow Gam3Master [/size] you nailed it. You managed to triple penetrate SOLID principles, coding standards and basic coding concepts all in 50 lines of code.

I can't even be mad, i will use this in tomorrows scrum.

Most devs around have a lot to learn, before they can even understand what "SOLID" stands for... :lol:

Elfo · August 30, 2016

Love how splitting this if statement to three different ones is his idea of clean code.

if(bossId >= 25333 && boss.getNpcId() <=25338)
	continue;
			
if(bossId == 25286 || boss.getNpcId() == 25283 || boss.getNpcId() == 29054 || boss.getNpcId() == 25273 || boss.getNpcId() == 29062 || boss.getNpcId() == 29040 || boss.getNpcId() == 25328)
	continue;
			
if(bossId == 29060 || boss.getNpcId() == 29056 || boss.getNpcId() == 29030 || boss.getNpcId() == 29033 || boss.getNpcId() == 29036 || boss.getNpcId() == 29037)
	continue;

Tessa · August 30, 2016

I already stopped judging others' code quality. If someone want to learn, there are a lot of books out there... if not, he obviously don't care. :D

Gam3Master · August 31, 2016

I don't understand what you mean but I am beginner, I write codes by searching in google and watching other examples. Sorry if I disappointed you.

At least I'm trying to help noobs like me.

Reborn12 · September 1, 2016

I don't understand what you mean but I am beginner, I write codes by searching in google and watching other examples. Sorry if I disappointed you.

At least I'm trying to help noobs like me.

Why you didnt use this with a split config ? And you can write all this ids with a config line?

Elfo · September 1, 2016

Why you didnt use this with a split config ? And you can write all this ids with a config line?

From one stupid way to the other.

Reborn12 · September 1, 2016

From one stupid way to the other.

Buts its an easy way to add remove without compile etc..everyone have his way

SweeTs · September 1, 2016

Why you getAllNpcs() lol, it's fat and highly unnecessary. Create an int[] with ids you want and then for loop it, profit.

Gam3Master · September 1, 2016

Why you getAllNpcs() lol, it's fat and highly unnecessary. Create an int[] with ids you want and then for loop it, profit.

You mean I must put Ids of all raids in int[]?

Help Help With Code

Question

L2shaken

16 answers to this question

Recommended Posts

Gam3Master

L2shaken

Gam3Master

L2shaken

Elfo

L2shaken

Tessa

Elfo

Tessa

Gam3Master

Reborn12

Elfo

Reborn12

SweeTs

Gam3Master

Join the conversation

Posts

Topics

Browse

Activity

Store