Java - Removing an object from an object array

Question

I am writing a large program that involves an Object called Player. The Player definition is as follows:

public class Player
{
    public static String name;
    public static Item inventory[] = new Item[10];
    public static int items;

    /* ... */

    public void addItem(String itemName, int itemType)
    {
            if ((items + 1) <= 10) {
                    inventory[items] = new Item(itemName, itemType);
                    items++;
            }
    }

    public void removeItem(int x)
    {
            for (int i = x; i < items; i++)
                    inventory[i] = inventory[i+1];
    }
}

I am adding inventory handling now because it's much easier than adding it later, but inventory isn't going to be used until much later in development. I have no way to see if removeItem works. I modified a function I wrote called strstrip to get this... Would removeItem work? If not, why?

if you are going to remove stuff, things like Lists would be more suitable. Typically, ArrayList. Nonetheless, no removeItem isn't going to work, it crashes, but you would know that if you tested it. Also, you may want to learn what static means — njzk2
– njzk2, Commented Jun 13, 2016 at 22:02
I have no way to see if removeItem works: sure you do. Add items, then remove one, then see what inventory contains what it should contain. That said, you're reinventing ArrayList. — JB Nizet
– JB Nizet, Commented Jun 13, 2016 at 22:02
You may want to look into unit testing. It can help with situations like this. — user2201041
– user2201041, Commented Jun 13, 2016 at 22:10
Do you know what static means? Do you really intend to have state shared between all Player instances? — Andy Turner
– Andy Turner, Commented Jun 13, 2016 at 22:11

Mikhail Chibel · Accepted Answer · 2016-06-13 23:43:05Z

1

Create unit tests for your classes especially if you going to build 'big and complicated program'. This will guarantee you that the code written will work later and if you change your code the failure of unit tests should indicate a problem. The unit test also gives you ability to check that your method works as expected.

As per other comments, consider using List interface instead of array, unless you have some specific requirement (I cannot imagine any). And definitely, having public static fields in your class looks suspicious.

EDIT

Just to indicate how code can look like and how to call methods from the main method.

public class Player {

    private String name;
    private List<Item> inventory;
    private int items;

    public Player() {
        this.inventory = new ArrayList();
    }

    public void addItem(String itemName, int itemType) {
        this.inventory.add(new Item(itemName, itemType));
    }

    public void removeItem(int x) {
        Item itemToRemove = this.inventory.get(x);
        if (itemToRemove != null) {
            this.inventory.remove(itemToRemove);
        }
    }

    public static void main(String[] args) {
        // create a new instance
        Player player = new Player();
        // call a method on the instance
        player.addItem("bla", 0);
    }
}

edited Jun 13, 2016 at 23:43

answered Jun 13, 2016 at 22:47

Mikhail Chibel

1,9952 gold badges23 silver badges40 bronze badges

Sign up to request clarification or add additional context in comments.

6 Comments

112 Over a year ago

I use static for everything so that it can be accessed by public static void main. Otherwise I get an error - 'cannot access static ... from non-static context'.

Mikhail Chibel Over a year ago

Sure it will complain. The main method is static and in order to use any nonstatic methods from the class you need to create an instance of the class first. If you unsure about static... then check this post stackoverflow.com/questions/3903537/…

112 Over a year ago

what should i do to create an instance of the class - make a public interface?

Mikhail Chibel Over a year ago

I modified answer to indicate how to deal with the 'main' method.

112 Over a year ago

thank you very much, that was extremely helpful! out of curiosity, why should the variables all be private - what would be the benefits? and how would I put the elements in the linked list in a specific order?

|

Collectives™ on Stack Overflow

Java - Removing an object from an object array

1 Answer 1

6 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

6 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related