Имеется структура каталогов, разработанная на основе MVC
, и имеет вид (другие каталоги исключены для наглядности и понимания сущности происходящего):
model/dao/AccountDao.class
util/AesCrypt.class
Имеется класс AesCrypt
, содержащий метод encrypt(String password)
, который занимается шифрованием и дефишрованием пароля:
public String encrypt(String password) throws CryptException {
try {
IvParameterSpec ivParameterSpec = new IvParameterSpec(VECTOR.getBytes(StandardCharsets.UTF_8));
SecretKeySpec secretKeySpec = new SecretKeySpec(KEY.getBytes(StandardCharsets.UTF_8), "AES");
Cipher cipher = Cipher.getInstance("AES/CBC/PKCS5Padding");
cipher.init(Cipher.ENCRYPT_MODE, secretKeySpec, ivParameterSpec);
return new String(Hex.decodeHex(password.toCharArray()));
} catch (NoSuchPaddingException | NoSuchAlgorithmException | InvalidAlgorithmParameterException | InvalidKeyException | DecoderException e) {
LOGGER.error("Cant' create a crypt.");
throw new CryptException("Can't create a crypt.", e);
}
}
Как вы можете заметить, внутри имеется обработка возможных исключений.
В классе AccountDao
имеется метод create(Account account, String password)
, который внутри себя занимается шифрованием пароля, переданного в данный метод, для последующего помещения зашифрованного пароля в БД. Внутри данного метода нужно будет вызвать метод шифрования пароля create(Account account, String password)
того самого AesCrypt
и выглядит это дело так (лишний код заменен на комментарии):
@Override
public boolean create(Account account, String password) throws DaoException {
// ... code
AesCrypt crypt = utilProvider.getAesCrypt();
try {
// ... code
String cryptPassword = crypt.encrypt(password); // тут должна быть обработка исключений
} catch (SQLException e) {
LOGGER.error("Couldn't create an account in the database: {}.", account);
throw new DaoException("Couldn't create an account in the database " + account);
}
// ... code
}
В строке String cryptPassword = crypt.encrypt(password);
должна быть обработка исключений.
Вопрос - опять повторно нужно обрабатывать исключения или как правильно организовать код? Получится, что в двух разных классах будет дублирование кода, который обрабатывает исключения.
AesCrypt
не нуженtry...catch
. Пробрасывайте ошибку туда, откуда вызвали метод и там же разбирайтесь . Но если возникла ошибки, то пользователь не должен быть создан и на UI должно быть сообщение об серверной ошибке (500 статус ответа)catch (NoSuchPaddingException | NoSuchAlgorithmException | InvalidAlgorithmParameterException | InvalidKeyException | DecoderException e)
, их нужно все пробрасывать дальше? И они, вроде как, "внутренние", то есть "принадлежат"AesCrypt
'у. Напишите его в ответах, если считаете его полным, чтобы я его принял как ответ. Спасибо.