массив строк к нескольким объектам

Question

У меня есть массив строк

String[] weekDays 

будние дни представляет дни недели. для например

{1, 2, 4} means Monday, Tuesday, Thusday 
{2,5,6,7} means Tuesday, Friday, Saturday, Sunday 

У меня есть семь булевых, представляющие 7 дней недели и должен установить истинные или ложными на основе будней.

Мой код

private static void setWeekDays(final Object object, final String[] weekDays) { 
    for (String day : weekDays) { 
     if(day.equalsIgnoreCase("1")) { 
      object.setMonday(true); 
     } else if(day.equalsIgnoreCase("2")) { 
      object.setTuesday(true); 
     } else if(day.equalsIgnoreCase("3")) { 
      object.setWednesday(true); 
     } else if(day.equalsIgnoreCase("4")) { 
      object.setThrusday(true); 
     } else if(day.equalsIgnoreCase("5")) { 
      object.setFriday(true); 
     } else if(day.equalsIgnoreCase("6")) { 
      object.setSaturday(true); 
     } else if(day.equalsIgnoreCase("7")) { 
      object.setSunday(true); 
     } 
    } 
} 

Но это так много, если другого заявления и перед выполнением этого я явно установить все булевы ложными. Есть ли другой простой способ сделать то же самое? Любое предложение ??

Answer 1

Определить Enum быть ваши дни недели:

enum DayOfWeek { 
    MONDAY, TUESDAY, etc. 
} 

Создать карту, чтобы перейти от строки в день:

Map<String, DayOfWeek> dayByString = new HashMap<String, DayOfWeek>(); 

Заполните карту со всеми поисков, которые вы хотите, то есть

dayByString.put("1", DayOfWeek.MONDAY); 
dayByString.put("2", DayOfWeek.TUESDAY); 

Теперь посмотреть на следующий день просто сделать:

DayOfWeek day = dayByString.get(str); 

Это вернет null, если нет совпадения - или соответствующий день недели.

Теперь вместо того, чтобы ваши 7 булевых использовать EnumSet:

Set<DayOfWeek> days = new EnumSet<DayOfWeek>(DayOfWeek.class); 

Внутренне, который будет использовать битовый представлять дни так будет невероятно быстрым и эффективное пространством.

Чтобы установить флаг сделать days.add(day);

Чтобы сбросить это сделать days.remove(day);

Чтобы проверить, если он установлен сделать days.contains(day); или days.contains(DayOfWeek.MONDAY);.

private static void setWeekDays(EnumSet<DayOfWeek> set, final String[] weekDays) { 
    set.clear(); 
    for (String day : weekDays) { 
     set.add(dayByString.get(day)); 
    } 
} 

Все готово, это все, что вам нужно.

EnumSet - это правильный способ его хранения. Есть и другие варианты, но 7 булевых ошибок ошибочны.

Answer 2

Думаю, вы можете попробовать использовать ENUM для этого.

Например

public enum WeekDay { 
    MONDAY("1"); 

    private String value; 

    private WeekDay(String value) { 
     this.value = value; 
    } 

    public static WeekDay find(String value) { 
     for (WeekDay weekDay : values()) { 
      if (weekDay.value.equals(value)) { 
       return weekDay; 
      } 
     } 
    } 

Итак, то вы можете использовать этот ENUM в поле вашего DTO.

Answer 3

Если класс DTO не может быть изменен, вы можете попробовать сделать отображение методами ранее, и использовать отражение для вызова целевого метода, например:

private static Method[] methods; 

private static void init() throws Exception { 

    Class klass = BusTravelDetailDTO.class; 
    String[] methodNames = new String[]{null, "setMonday", "setTuesday", "setSunday"}; 
    methods = new Method[methodNames.length]; 
    for (int i = 0; i < methods.length; i++) { 
     if(methodNames[i] != null) { 
      methods[i] = klass.getMethod(methodNames[i], Boolean.class); 
     } 
    } 
} 


private static void setWeekDays(final Object object, final String[] weekDays) { 

    for (String day : weekDays) { 
     methods[Integer.parseInt(day)].invoke(object, Boolean.TRUE); 
    } 
} 

Но, поскольку у вас есть только семь вариантов , if-else может быть самым простым и эффективным способом.

Если класс DTO может быть изменен, используйте enum вместо семи булевых флагов.

Answer 4

Вы можете попробовать следующее:

1. Добавить следующий код в DTO объект

   Список weekDaysList;

   private void setWeekDays(String[] weekDays){    
       weekDaysList = Arrays.asList(weekDays); 
   } 
   
   private boolean isWeekdaySet(String weekday){ 
       if (weekDaysList == null || weekDaysList.size() == 0){ 
        return false; 
       } 
       return weekDaysList.contains(weekday); 
   } 
   

2. обновление каждого метода getMonday, getTuesday следующим образом:

 
public boolean getMonday() { 
    return isWeekdaySet("1"); 
} 

public boolean getTuesday(){ 
    return isWeekdaySet("2"); 
} 

Я надеюсь, что это помогает.