Question

There are some different opinions about simple inner classes, so I was wondering if there is a general consensus on what is good, and when to use private inner classes.

Here’s an example that I found, and for which I think it’s unnecessary to create an inner class. How good/bad practice is this?

private static class InternalCounter {
  int count;

  public InternalTabManager() {
    count = 0;
  }

  public int increment() {
    return count++;
  }
}

Mind you that in this particular case, one instance is kept in the surrounding class to keep track of a count.

Answer 1

Yeah, in this case it does seem very unnecessary but if you have a case where there is some significant functionality and you know that no other class will ever need your inner class and it makes no sense to create a class more globally available then do use an inner class.