Возврат дефолта или рейза Исключение?

Question

Я смущен о том, что считается хорошей практикой, зависит ли это от решения? Предположим, у меня есть следующий код Java:

public class Stack { 
    public Integer pop() { 
    if (isEmpty()) return null; // or some exception maybe? 
    // else get and return the top item in the stack. 
    }; 
} 

Клиент метода pop ожидает некоторое Integer значение так, что будет самым лучшим подходом, чтобы позволить клиенту знать, что стек пуст?

Answer 1

Возвращаемое значение null или значение по умолчанию обычно Неправильная практика и исключение должно быть предпочтительным. Причина в том, что вы всегда должны стремиться к сбою так быстро, как можете,, когда что-то пойдет не так. Если вы вернете null - вы получите ошибки, происходящие в каком-то другом месте в коде, и пользователи вашего API будут иметь проблемы с обнаружением проблемы. Это называется Fail Fast

Исключением из этого правила является то, что ваш API заставит пользователей полагаться на исключения для управления потоком, поэтому, если ваш стек не поддерживает isEmpty(), исключение здесь не очень хорошо.
Если у вас есть стек, который может позволить только peek(), pop() и add() - по какой-то причине isEmpty() не может быть частью API.

Что произойдет с кодом ваших пользователей для обоих подходов?

вариант 1 - с помощью нуля:

Integer x = stack.pop(); 
if (x != null) { 
    //do something with x 
} 

Вариант 2 - с использованием исключений:

Integer x = null; 
try { 
    x = stack.pop(); 
} catch (MyException e) { } 
//do something with x 

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

Answer 2

Вы можете выбросить IllegalStateException.

Сигналы о том, что метод был вызван в незаконное или ненадлежащее время.

Вызов pop() на пустой стек - это вызов в неподходящее время.

Более современный подход бы вернуть Optional<Integer>, который является рекомендуемым способом в Java 8, avoud null возвращается.

public class Stack { 
    public Optional<Integer> pop() { 
    if (isEmpty()) return Option.ofNullable(null); 
    return Optional.of(valueOnTheStack); 
    } 
} 

Answer 3

В соответствии с Java API, пытаясь выскочить элемент из пустого стека должен выход исключение.

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

Answer 4

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

public Integer pop() throws Exception 
{ 
    if (stack.isEmpty()) 
     throw new java.util.EmptyStackException(); 

    return stack.pop(); 
} 

---

Если я создаю это как веб-службы, то я хотел бы сделать это следующим образом. В основном, чтобы убедиться, что стандартизация различных типов внутренних исключений представлена ​​через возвращаемый объект использования стека.

StackService

@WebService 
class MyStack 
{ 
    StackProvider provider; 
    Stack stack; 

    public MyStack (StackProvider provider) 
    { 
     this.provider = provider; 
     this.stack = provider.readStack(); 
    } 

    public void sync() 
    { 
     provider.syncStack(stack); 
    } 

    public Integer pop() 
    { 
     if (stack.isEmpty()) 
      return new StackElement(StackElement.STACK_EMPTY); 

     return new StackElement(stack.pop()); 
    } 

    public void push (Integer val) 
    { 
     stack.push(val); 
    } 
} 

StackElement

class StackElement 
{ 
    public static final char STACK_EMPTY = 0; 
    public static final char ELEMENT_VALID = 1; 

    public Integer value; 
    public char flag; 

    public StackElement (Integer val) 
    { 
     this.value = val; 
     this.flag = StackElement.ELEMENT_VALID; 
    } 

    public StackElement (char flag) 
    { 
     this.value = null; 
     this.flag = flag; 
    } 

    public boolean isValid() 
    { 
     return (flag == 1); 
    } 
} 

Тест

class Test 
{ 
    public static void main (String [] args) 
    { 
     StackProvider provider = new StackProvider("..."); 

     // init 

     MyStackService service = new MyStackService(provide); 
     MyStackServiceSoap soap = service.getMyStackServiceSoap(); 

     // call pop operation 

     StackElement element = soap.pop(); 

     // checking the value 

     if (element.isValid()) 
      System.out.println("Stack.Pop ~ " + element.value); 
     else 
     { 
      if (element.flag == StackElement.STACK_EMPTY) 
      System.out.println("Stack is Empty"); 
      else 
      System.out.println("Unknown error"); 
     } 

    } 
}