Argument names out of order. Prefer preincrement for iterator classes. Potential null pointer dereferences.