Как преобразовать один enum в другой enum в java?

20

У меня есть:

public enum Detailed {
    PASSED, INPROCESS, ERROR1, ERROR2, ERROR3;
}

и нужно преобразовать его в следующее:

public enum Simple {
    DONE, RUNNING, ERROR;
}

Итак, сначала PASSED - & gt; DONE и INPROCESS - & gt; RUNNING , но все ошибки должны быть: ERROR . Очевидно, что можно писать случаи для всех значений, но может быть лучшее решение?

    
задан user710818 18.06.2012 в 07:43
источник

6 ответов

19

Один из способов - определить метод asSimple() в вашем Detailed перечислении:

public enum Detailed {
    PASSED {
        @Override
        Simple asSimple() {
            return PASSED;
        }
    },
    INPROCESS {
        @Override
        Simple asSimple() {
            return RUNNING;
        }
    },
    ERROR1,
    ERROR2,
    ERROR3;
    public Simple asSimple() {
        return Simple.ERROR; // default mapping
    }
}

Затем вы можете просто вызвать метод, когда хотите сделать сопоставление:

Detailed code = . . .
Simple simpleCode = code.asSimple();

Преимущество состоит в том, чтобы поместить знания в сопоставление с перечислением Detailed (где, возможно, оно принадлежит). Недостаток заключается в том, что вы знаете, что Simple смешивается с кодом для Detailed . Это может быть или не быть плохой, в зависимости от вашей архитектуры системы.

    
ответ дан Ted Hopp 18.06.2012 в 07:48
источник
  • Я думаю, что включение логики отображения в enum в другой enum прерывает инкапсуляцию, вводя связь и уменьшая сплоченность. Озабоченность перечисления должна состоять просто в том, чтобы представлять набор состояний (или концепций). Забота о переходе от одного перечня к другому должна быть инкапсулирована отдельно. –  Chomeh 10.02.2015 в 03:27
  • @Chomeh - Я заметил в своем ответе, что Детский в конечном итоге потребует знания Simple. (В принципе, я выражал ту же озабоченность, что и вы, в вашем комментарии, но с довольно расплывчатой ​​критикой назвать ее «недостатком». Вы были намного точнее.) Тем не менее, я думаю, что этот подход заслуживает рассмотрения. Названия перечислений OP указывают на то, что подробный концептуально является уточнением Simple. Если это так, то совсем не плохо, что Деталь связан с Простым (так как это не всегда плохо, что подкласс связан с его родительским классом). –  Ted Hopp 10.02.2015 в 06:10
18

Лично я бы просто создал Map<Detailed, Simple> и сделал это явно - или даже использовал оператор switch , возможно.

Другой альтернативой было бы передать отображение в конструктор - вы могли бы сделать это только в одну сторону, конечно:

public enum Detailed {
    PASSED(Simple.DONE),
    INPROCESS(Simple.RUNNING),
    ERROR1(Simple.ERROR),
    ERROR2(Simple.ERROR),
    ERROR3(Simple.ERROR);

    private final Simple simple;

    private Detailed(Simple simple) {
        this.simple = simple;
    }

    public Simple toSimple() {
        return simple;
    }
}

(Я нахожу это более простым, чем подход Теда к использованию полиморфизма, поскольку мы на самом деле не пытаемся обеспечить другое поведение - просто другое простое сопоставление.)

В то время как вы могли потенциально делать что-то хитрое с порядковым значением, было бы гораздо менее очевидным и принимать больше кода - я не думаю, что будет какая-то польза.

    
ответ дан Jon Skeet 18.06.2012 в 07:49
источник
  • Мое голосование за карту –  Nu2Overflow 10.01.2014 в 20:17
7

Используйте EnumMap

Я отключаю внешний внешний XML-интерфейс от своей внутренней модели домена, внедряя службу преобразования. Это включает в себя перечисление перечислений из jaxb сгенерированного кода для перечислений модели домена.

Использование статического EnumMap инкапсулирует проблему трансформации внутри класса, ответственного за преобразование. Его сплоченность.

@Service
public class XmlTransformer {

    private static final Map<demo.xml.Sense, Constraint.Sense> xmlSenseToSense;
    static {
        xmlSenseToSense = new EnumMap<demo.xml.Sense, Constraint.Sense> (
            demo.xml.Sense.class);
        xmlSenseToSense.put(demo.xml.planningInterval.Sense.EQUALS, 
            Constraint.Sense.EQUALS);
        xmlSenseToSense.put(demo.xml.planningInterval.Sense.GREATER_THAN_OR_EQUALS, 
            Constraint.Sense.GREATER_THAN_OR_EQUALS);
        xmlSenseToSense.put(demo.xml.planningInterval.Sense.LESS_THAN_OR_EQUALS, 
            Constraint.Sense.LESS_THAN_OR_EQUALS);
    }
    ...
}
    
ответ дан Chomeh 05.12.2014 в 09:22
источник
1

Ответ Теда очень Javaly, но выражение

passed == PASSED ? DONE : ERROR

тоже выполнит эту работу.

    
ответ дан tiwo 18.06.2012 в 07:52
источник
  • Что случилось с частью INPROCESS -> RUNNING? –  Ted Hopp 18.06.2012 в 07:54
  • @Ted: конечно, вы правы, спасибо за исправление. Я не буду предлагать ПАССАЖ? СОВЕРШЕННО: ИНПРОЦЕСС? RUNNING: ERROR, так как это, очевидно, менее читаемо. (Опять же, я видел это раньше и с хорошим форматированием, это не выглядит слишком странным ...) –  tiwo 27.06.2012 в 00:18
0

Для меня это больше похоже на концептуальную проблему, чем на проблему программирования. Почему бы вам просто не удалить «простой» тип перечисления и использовать его во всех местах программы?

Просто, чтобы сделать это более понятным с помощью другого примера: действительно ли вы попытаетесь определить тип перечисления для рабочих дней в неделю (с понедельника по пятницу) и другое перечисление для всех дней недели (с понедельника по воскресенье)?     

ответ дан Bananeweizen 18.06.2012 в 07:54
источник
  • Поскольку типы перечислений не могут наследовать от других типов перечислений в Java, вы в значительной степени зацикливаетесь на таком отображении такого рода время от времени. –  Ted Hopp 18.06.2012 в 08:03
0

Вот простой транслятор перечисления с тестом:

- ОСУЩЕСТВЛЕНИЕ

- ENUMS

public enum FirstEnum {

A(0), B(1);

private final int value;

private FirstEnum(int value) {
    this.value = value;
}

public int getValue() {
    return value;
}
}

public enum  SecondEnum {

C(0), D(1);

private final int valueId;

private SecondEnum(int valueId) {
    this.valueId = valueId;
}

public int getValueId() {
    return valueId;
}

}

- MAPPER

import java.lang.reflect.InvocationTargetException;
import java.util.HashMap;
import java.util.Map;

import org.apache.commons.beanutils.PropertyUtils;
import org.apache.commons.lang3.Validate;

import com.google.common.collect.Sets;

public class EnumPropertyMapping {

private final Map<?, ?> firstMap;
private final Map<?, ?> secondMap;

private final Class<?> firstType;
private final Class<?> secondType;

private EnumPropertyMapping(
        Map<?, ?> firstMap, Map<?, ?> secondMap, Class<?> firstType, Class<?> secondType) {

    this.firstMap = firstMap;
    this.secondMap = secondMap;
    this.firstType = firstType;
    this.secondType = secondType;
}

public static Builder builder() {
    return new Builder();
}

@SuppressWarnings("unchecked")
public <R> R getCorrespondingEnum(Object mappedEnum) {
    Validate.notNull(mappedEnum, "Enum must not be NULL");
    Validate.isInstanceOf(Enum.class, mappedEnum, "Parameter must be an Enum");

    if (firstType.equals(mappedEnum.getClass())) {
        return (R) firstMap.get(mappedEnum);
    }

    if (secondType.equals(mappedEnum.getClass())) {
        return (R) secondMap.get(mappedEnum);
    }

    throw new IllegalArgumentException("Didn't found mapping for enum value: " + mappedEnum);
}

public static class Builder {

    private final Map<Object, Object> firstEnumMap = new HashMap<>();
    private final Map<Object, Object> secondEnumMap = new HashMap<>();
    private Class<?> firstEnumType;
    private Class<?> secondEnumType;

    public <T extends Enum<T>> Builder addFirst(Class<T> enumType, String propertyName) {
        firstEnumType = enumType;
        initMap(firstEnumMap, enumType.getEnumConstants(), propertyName);
        return this;
    }

    public <T extends Enum<T>> Builder addSecond(Class<T> enumType, String propertyName) {
        secondEnumType = enumType;
        initMap(secondEnumMap, enumType.getEnumConstants(), propertyName);
        return this;
    }

    private void initMap(Map<Object, Object> enumMap, Object[] enumConstants, String propertyName) {
        try {
            for (Object constant : enumConstants) {
                enumMap.put(PropertyUtils.getProperty(constant, propertyName), constant);
            }
        } catch (InvocationTargetException | NoSuchMethodException | IllegalAccessException ex) {
            throw new IllegalStateException(ex);
        }
    }

    public EnumPropertyMapping mapEnums() {
        Validate.isTrue(firstEnumMap.size() == secondEnumMap.size());
        Validate.isTrue(Sets.difference(firstEnumMap.keySet(), secondEnumMap.keySet()).isEmpty());

        Map<Object, Object> mapA = new HashMap<>();
        Map<Object, Object> mapB = new HashMap<>();

        for (Map.Entry<Object, Object> obj : firstEnumMap.entrySet()) {
            Object secondMapVal = secondEnumMap.get(obj.getKey());
            mapA.put(obj.getValue(), secondMapVal);
            mapB.put(secondMapVal, obj.getValue());
        }
        return new EnumPropertyMapping(mapA, mapB, firstEnumType, secondEnumType);
    }
}

}

- ТЕСТ

import org.junit.Test;

import com.bondarenko.common.utils.lang.enums.FirstEnum;
import com.bondarenko.common.utils.lang.enums.SecondEnum;

import static junit.framework.TestCase.assertEquals;

public class EnumPropertyMappingTest {

@Test
public void testGetMappedEnum() {
    EnumPropertyMapping mapping = EnumPropertyMapping.builder()
                                                                                                     .addSecond(SecondEnum.class, "valueId")
                                                                                                     .addFirst(FirstEnum.class, "value")
                                                                                                     .mapEnums();

    assertEquals(SecondEnum.D, mapping.getCorrespondingEnum(FirstEnum.B));
    assertEquals(FirstEnum.A, mapping.getCorrespondingEnum(SecondEnum.C));
}

}
    
ответ дан nikopol86 15.01.2015 в 22:30
источник