0

Подскажите, может есть какое-то более элегантное решение написания с заменой полотна из 20 ифов?

if (null != req.getParameter("removeExclude")) {
    String exclude_id = req.getParameter("exclude_id");
    db.removeExclude(exclude_id);
    return;
}
if (null != req.getParameter("getProdGroupDate")) {
    db.getProdGroupDate(out);
    return;
}
if (null != req.getParameter("getProdGroupByDate")) {
    db.getProdGroupByDate(out, req.getParameter("date"));
    return;
}
if (null != req.getParameter("createProdPhone")) {
    if(req.getParameter("pass").equals("androidapppass")){
        db.createProdPhone(req.getParameter("cat"), req.getParameter("code"));
    }
    return;
}

и так далее, и так далее....

2
  • Можно switch case
    – Denis
    Commented 7 дек. 2016 в 15:00
  • Шаблон проектирования нужно было внедрять раньше, чтобы не было такой портянки. Смотрите в сторону рефлексии.
    – user230141
    Commented 7 дек. 2016 в 15:47

1 ответ 1

1

Можно попробовать в функциональном стиле пописать, например так, но на java не особо то удобно получается.

 Predicate<String> predicate = paramName -> req.getParameter(paramName) != null;

 Map<String, Runnable> tasks = new LinkedHashMap<>();
 tasks.put("removeExclude", () -> {
        String exclude_id = req.getParameter("exclude_id");
        db.removeExclude(exclude_id);
 });
 tasks.put("getProdGroupDate", () -> db.getProdGroupDate(out));

 tasks.entrySet().stream().filter(entry -> predicate.test(entry.getKey())).findFirst();

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.