I could see that if I was adding new functionality Iwould want to extend it, but all I'm doing is adding to the Data class, the functionality it is suppose to have.
I agree. However, looking back at it. If they were in a CriteriaFinder class then it would be cleaner. Meaning the only thing you would have to touch in the Data class is the two deprectaed methods, implement lock and unlock and maybe just adding the criteriaFind method which instantiates the CriteriaFinder class. Or that could be in the interfaces. Then it would follow the specs to a tee. It's just a thought. And since they never tell you where you lost points. I can only guess that this might have been where I lost my 4 points. It's either here or because I didn't use a LockManager class.
Speaking of which If you had a CriteriaFinder class and a LockManager class, then the only thing you would need to do to the Data class would be fix the 2 deprecated methods.
Mark
Mark