The change you made from Regexp to Integer.parseInt sounds like the right way to go anyway. Why pay the overhead of the regexp processing if all that's needed is a simple parsing already supported elsewhere.
I dont know where and how often this piece of code gets called and how often a wrong value will be passed, but to throw an exception might be more expensive.
Think of the stacktrace which needs to be filled.
Maybe the type of the term should then be cached (per field).
Mario
smime.p7s
Description: S/MIME Cryptographic Signature
