> +
> +/**
> + * Validates Vault names according to Amazon Vault conventions.
> + *
> + * @author Roman Coedo
> + */
> +@Singleton
> +public class VaultNameValidator extends Validator<String> {
> +
> +   private final static int MIN_LENGTH = 1;
> +   private final static int MAX_LENGTH = 255;
> +
> +   @Override
> +   public void validate(String vaultName) {
> +      if (vaultName == null || vaultName.length() < MIN_LENGTH || 
> vaultName.length() > MAX_LENGTH)
> +         throw exception(vaultName, "Can't be null or empty. Length must be 
> " + MIN_LENGTH + " to " + MAX_LENGTH

With MIN_LENGTH being 1, would `if (isNullOrEmpty(vaultName) || 
vaultName.length() > MAX_LENGTH)` be easier?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-aws/pull/4/files#r13034780

Reply via email to