rather than throwing NPE better throw an ItemNotFoundException or probably an IllegalStateException since you should know at this point that there should be an entry for this item