You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Sebb (JIRA)" <ji...@apache.org> on 2016/06/15 10:16:09 UTC
[jira] [Created] (CRYPTO-74) Full class names make code more
difficult to update
Sebb created CRYPTO-74:
--------------------------
Summary: Full class names make code more difficult to update
Key: CRYPTO-74
URL: https://issues.apache.org/jira/browse/CRYPTO-74
Project: Commons Crypto
Issue Type: Improvement
Reporter: Sebb
The method CryptoRandomFactory.getCryptoRandom uses the value of a property as the full name of the CryptoRandom class to be instantiated.
This is inherently non-portable if the code package names should ever be changed in future.
One way round this is to add a constant alias for the embedded implementations.
For example:
CryptoRandomFactory.java
public static final String OPENSSL_RANDOM = "OpensslCryptoRandom";
public static final String JAVA_RANDOM = "JavaCryptoRandom";
...
If the COMMONS_CRYPTO_SECURE_RANDOM_CLASSES_KEY string does not contain a full class name, then the code would preprend the appropriate package name (or there could be a lookup table).
This would also work for the case where the class is provided as a system property value.
Another advantage of this method is that it simplifies the user code.
Similar considerations apply to all other factories which use class name strings.
[Note: the constants must not contain the full package names as that would result in a binary incompatibility if the names changed.]
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)