Как привести данный код в более красивый вид или в switch case?

Question

[Chvalov]

Ищу форму по id,name,class и action которая отвечает регулярному выражению.
Если такая нашлась передаю текущий WebElement методу direct для дальнейших тестов
Но меня смущает использование if - else if так как с временем их количество только увеличивается.

if (formElements.getAttribute("action").matches(regexUser)) {
 direct(formElements);
} else if (formElements.getAttribute("id").matches(regexMember)) {
 direct(formElements);
} else if (formElements.getAttribute("name").matches(regexPeople)) {
 direct(formElements);
} else if (formElements.getAttribute("class").matches(regexFollowers)) {
 direct(formElements);
} else if (formElements.getAttribute("class").matches(regexUser)) {
 direct(formElements);
} else {
 System.out.println("Сегодня нету нужных форм...");
}

Как переписать в более красивый / удобный вид ?

Answer 1

[GavriKos]

Запихнуть регекспы и соответствующие аттрибуты в массив и гонять простым foreach-ем.
Если реакция на положительное срабатывание может быть разная - то можно словарь использовать.

Comments

[Chvalov]

Реакция всегда одинакова, если true передаем методу direct...

Но Я не совсем догоняю как реализовать, то что вы написали (

Делать двумерный массив [attribute] [regex] ?

[GavriKos]

Можно двумерный. Можно хранить что то вроде KeyValuePair, елси такое есть в джаве. Можно завести свой класс-контейнер с двумя полями и уже его хранить.

[BorLaze]

Chvalov, Зачем массив?

Надо что-то типа (дальше псевдокод):

Set<Pair<String, String>> pairs = [
  {"action", regexUser},
  {"id", regexMember},
  ...
];

private void checkPairs() {
  for( Pair<String, String> p: pairs ) {
    if (formElements.getAttribute(p.getKey()).matches(p.getValue())) {
      direct(formElements);
      return;
    } 
  System.out.println("Сегодня нету нужных форм...");
}

Этого, по идее, будет достаточно.

[GavriKos]

BorLaze, ну массив в данном случае не конкретное название коллеции, а скоре именно "псевдокодное". Понятно что можно Set, можно Array, можно <что там еще есть в джаве>. Это уже вопрос конкретной реализации :-)