I feel that this class would be better if instances didn't contain the data to be encrypted or decrypted.  i.e. create an instance that knows how to decrypt and encrypt using a particular set of keys and parameters but is data agnostic.

Then the encrypt and decrypt methods would take data and return data with the appropriate encryption method applied.

What do you think?