Item Packs

Thought it was neat so I coded it. :)

Code:

---

public void openPack(int packId, int amount) {
                if(c.getItems().freeSlots() >= 0) {
                        switch(packId) {
                                case 15262: c.getItems().deleteItem(packId, 1 * amount); c.getItems().addItem(12183, 5000 * amount); break;
                                case 15362: c.getItems().deleteItem(packId, 1 * amount); c.getItems().addItem(230, 50 * amount); break;
                                case 15364: c.getItems().deleteItem(packId, 1 * amount); c.getItems().addItem(222, 50 * amount); break;
                                case 15365: c.getItems().deleteItem(packId, 1 * amount); c.getItems().addItem(9979, 50 * amount); break;
                                case 15366: c.getItems().deleteItem(packId, 1 * amount); c.getItems().addItem(5377, 50 * amount); break;
                                case 15367: c.getItems().deleteItem(packId, 1 * amount); c.getItems().addItem(5419, 50 * amount); break;
                        }
                }
        }

---

Usage:

Code:

---

case 15362: case 15364: case 15365: case 15366: case 15367: case 15262: c.getPA().openPack(itemId, 1); break;

---

And in ItemAssistant:

Code:

---

if(wearID == 15262 || wearID == 15362 || wearID == 15364 || wearID == 15365 || wearID == 15366 || wearID == 15367) {
                                        c.getPA().openPack(wearID, c.getItems().getItemAmount(wearID));
                                        return false;
                                }

---

Yes, I know it could probably be done better but it works just fine and is fairly clean.

Are those like those spirit shard packs? if so goodjob

Yes they are. I left 2 pack types out though. You need to add Vial of Water Pack and Juju Vial Pack

And where to add these?

Bro, you gotta get out of this habit:

Code:

---

case 15367: c.getItems().deleteItem(packId, 1 * amount); c.getItems().addItem(5419, 50 * amount); break;

---

Do this, for proper conventions:

Code:

---

case 15367:
    c.getItems().deleteItem(packId, 1 * amount);
    c.getItems().addItem(5419, 50 * amount);
    break;

---

Quote:

---

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

Bro, you gotta get out of this habit:

Code:

---

case 15367: c.getItems().deleteItem(packId, 1 * amount); c.getItems().addItem(5419, 50 * amount); break;

---

Do this, for proper conventions:

Code:

---

case 15367:
    c.getItems().deleteItem(packId, 1 * amount);
    c.getItems().addItem(5419, 50 * amount);
    break;

---

---

Probably wanted less lines of code :hurr:

Quote:

---

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

Probably wanted less lines of code :hurr:

---

less lines doesn't mean better

It does not affect the way it works in any way though. And I find it a hell of a lot easier to read. Lol. I'll try to get out of the habit though.

Quote:

---

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

And where to add these?

---

Not spoon feeding something as simple as this....