NPC Item Drop Interface

I've tried to convert this to 876 but interface is different. I don't think it'll work on it.

Delete.

Nice

For those who wants a fix for the famous getSortedList nullpointer,

Here is the code with that fixed:

Spoiler for code:

Code:

---

private static Drop[] getSortedList(Player player, Drop[] drops) {
                /*
                * Populate DropList
                */
                int dropsLength = drops == null ? 0: drops.length;
                List<Drop> dropList = new ArrayList<Drop>();
                if (drops == null) {
                        return null;
                }
                        for (int i = 0; i < drops.length; i++) {
                                dropList.add(drops[i]);
                }
                switch (player.getDropSortingType()) {
                        case 0: //Sort Drop List by highest drop rate
                                Collections.sort(dropList, (s1, s2) -> Double.compare(s2.getRate(), s1.getRate()));
                                break;
                        case 1: //Sort Drop List by lowest drop rate
                                Collections.sort(dropList, (s1, s2) -> Double.compare(s1.getRate(), s2.getRate()));
                                break;
                        case 2: //Sort by highest market value
                                //Collections.sort(dropList, (s1, s2) -> Integer.compare(ItemDefinitions.getItemDefinitions(s2.getItemId()).getValue(), ItemDefinitions.getItemDefinitions(s1.getItemId()).getValue()));
                                Collections.sort(dropList, (s1, s2) -> Integer.compare(ItemDefinitions.getItemDefinitions(ItemManager.getPrice(s1.getItemId())).getValue(), ItemDefinitions.getItemDefinitions(ItemManager.getPrice(s2.getItemId())).getValue()));               
                                break;
                        case 3: //Sort by lowest market value
                                //Collections.sort(dropList, (s1, s2) -> Integer.compare(ItemDefinitions.getItemDefinitions(s1.getItemId()).getValue(), ItemDefinitions.getItemDefinitions(s2.getItemId()).getValue()));
                                Collections.sort(dropList, (s1, s2) -> Integer.compare(ItemDefinitions.getItemDefinitions(ItemManager.getPrice(s2.getItemId())).getValue(), ItemDefinitions.getItemDefinitions(ItemManager.getPrice(s1.getItemId())).getValue()));               
                                break;
                }
       
                return drops = false ? null : dropList.toArray(drops);
        }

---

the fixes were just adding a if to check if the drops are null, and a return asking if the response of the void method is false (which means it couldnt execute) then send null if the drop doesnt exist :)

[Only registered and activated users can see links. Click Here To Register...]

Sorry for gravedigging :P but does anyone know how to fix this?? I imported a calendar... but still having an issue..

done everything you said of the forst part added proper alot of the codes did the fix and the drops do ot come up at all

Quote:

---

Originally Posted by benvenga888 [Only registered and activated users can see links. Click Here To Register...]

done everything you said of the forst part added proper alot of the codes did the fix and the drops do ot come up at all

---

If you have done everything correctly then it should work.
Did you check if you have toggled the method? since it checks if you enabled or disabled the drops at the examine packet.

If you always want the drops to come up on examine, just remove the - if (boolean) so the showdrops methods is always called on examine.