Реализация булевой инструкции в коммутаторе

Question

Итак, я сейчас занимаюсь университетским заданием первого года, и я немного застрял в одном из вопросов. Это как таковое. Измените методы setType() и setPlan(), чтобы вернуть логическое значение true/false, если выбранный тип или план недействительны. I.e, если тип был «Bogus», игнорировать тип и вернуть «ложь» в вызов. Измените класс Test, чтобы добавить оператор if, который сообщит об ошибке.

Моя текущая строка кода для метода выглядит следующим образом:

public void setType(String newType) { 
    switch (newType) { 
     case "Basic": 
      mType = newType; 
      break 

     case "Enhanced": 
      mType = newType; 
      break 

     default: 
      break 
} 

Мой вопрос, как я могу идти о добавлении, если заявление и не нужно ничего менять, чтобы сделать метод недействительным возвращать значение true/false?

Answer 1

Попробуйте это:

public boolean setType(String newType) { 
    switch (newType) { 
    case "Basic": 
    case "Enhanced": 
     mType = newType; 
     return true; 
    default: 
     return false; 
    } 
} 

, Вам не нужно дополнительное if заявление для этого, и вы можете проверить два или более случаев в switch с помощью проваливаемся (два или более последовательных case) ,

Answer 2

Вы можете определить массив со всеми допустимыми типами в некоторых глобально доступном месте в вашем приложении:

public static final String[] VALID_TYPES = {"Basic","Enhanced"}; 

, а затем есть метод, который просто итерацию все эти типы и проверяет, является ли данное значение соответствует одному из них :

public static boolean isValidType(String candidate){ 
    for(String validType : VALID_TYPES){ 
    if(validType.equals(candidate)){ 
     return true; 
    } 
    } 
    return false; 
} 

хорошая вещь об этом - в отличие от использования заявлений переключателя - это то, что вы можете легко modifiy список допустимых типов (добавление/типы изменений/удалить) без необходимости трогать метод, который проверяет данный тип для действительности. Таким образом, его легко поддерживать и меньше подвергать ошибкам (в коммутаторе вы можете забыть добавить оператор case для каждого возможного значения, или вы можете забыть добавить оператор break где-то и т. Д.)

Answer 3

Лучшая практика говорит об этом является «проверка параметров», и это уборщик тоже неудачу рано:

public boolean setType(String newType) { 
    if (!newType.matches("Basic|Enhanced")) { 
     return false; 
    } 
    type = newType; // or whatever you need to do 
    return true; 
} 

Варианты тестирования также включают в себя:

if (!Arrays.asList("Basic", "Enhanced").contains(newType)) 

который также аккуратно обрабатывает newType быть пустой без дополнительной с ода.

Но лучший способ справиться с этим - использовать enum, что в первую очередь не допустило бы отрицательных значений.

Answer 4

Другие ответы являются хорошими. Вот мое занятие. Тернарный оператор, ? и : выражение, логически является «if then else».

public boolean setType(String newType) { 
    boolean result = (newType.equals("Basic") || newType.equals("Enhanced")); 
    mType = result ? newType : mType; 
    return result; 
} 

Answer 5

public class YourClass{ 

    private String mType; 

    private static final List<String> VALID_TYPES = Arrays.asList("Basic","Enhanced"); 

    public boolean setType(String newType){ 
     if(!VALID_TYPES.contains(newType)){ 
     return false; 
     } 
     mType = newType; 
     return true; 
    } 
} 

И аналогично для setPlan метода, просто определить другой список VALID_PLANS.

А о тестовом классе:

if(!yourClassObject.setType("Some Invalid Type")){ 
    System.err.println("Invalid type!"); 
} 

Answer 6

добавив булево свойство класса, например:

public class Test 
{ 
    private boolean flag; 
}