I'm planning to create a file encryption application consisting of separate classes for the different activities like file/directory navigation, key management, actual encryption/decryption etc.
For key management, I'm creating a helper class called KeyManager that takes care of all key related activities like generating and storing symmetric keys, key pairs, generating and loading new key stores etc. All methods in this class are currently public and static so that I don't need to create an instance of KeyManager class to access its methods. I'm wondering if this approach has any drawbacks. Is there any advantage in converting all methods to non-static and creating an instance of KeyManager class to access its methods?