Is this a descriptive enough implementation of toString? Logging and testing frameworks often use the toString of an object, so if it's not going to identify it enough you may have to add more.