Prawy wyjątek rzucać za brak właściwości systemu

Question

Say mam właściwość systemu MY_PROP:

java -DMY_PROP="My value" 

Ta nieruchomość jest niezbędna dla mojego systemu do pracy.

Jaki jest właściwy wyjątek do rzucania, jeśli ta właściwość nie jest ustawiona?

@PostConstruct 
private void init() { 

    myProp = System.getProperty("MY_PROP"); 
    if (myProp == null) { 
     throw new ???? 
    } 
    // ... 
} 

Jakoś IllegalArgumentException nie czuje się dobrze. Może IllegalStateException, MissingResourceException, TypeNotPresentException? Jaka jest standardowa praktyka w tym scenariuszu?

Answer 1

Nie ma. Wrzuciłbym IllegalStateException, ponieważ brakuje tego parametru. Oznacza to, że weryfikator konfiguracji nie powiódł się, a aplikacja jest w nieprawidłowym stanie. Innymi słowy, nigdy nie powinieneś być w stanie zadzwonić na numer init().

Jeśli wartość parametru byłaby nieprawidłowa, to rzuciłbym IllegalArgumentException.

Jeśli piszesz weryfikator, powinieneś zdecydować pomiędzy użyciem RuntimeException lub zaznaczonym. Podczas używania na przykład javax.naming.ConfigurationException "lub utworzyć własny wyjątek konfiguracji. You API będzie w stanie obsłużyć taki wyjątek i właściwie zareagować w ramach spuścizny.

Definicje:

IllegalStateException - Sygnały, że metoda została przywołana w nielegalnym lub niewłaściwym czasie. Innymi słowy, środowisko Java lub aplikacja Java nie są w odpowiednim stanie dla żądanej operacji.

IllegalArgumentException - Generowany w celu wskazania, że ​​przyjęto metodę niezgodną z prawem lub niewłaściwą.

Answer 2

Ponieważ właściwość systemowa nie zawsze jest zdefiniowana, standardową praktyką jest użycie wartości domyślnej, gdy nie można znaleźć właściwości.

Właśnie sprawdziłem standardowy kod w java 7 (apache tomcat, java.lang, java.awt, ...), zawsze używają domyślnego "fallback", gdy właściwość to null.

Więc może twój problem jest gdzieś indziej?

Dlaczego nie bierzesz tych parametrów jako wymaganego argumentu swojego słoika? Następnie możesz użyć IllegalArgumentException.

Answer 3

Dodaję tylko odpowiedź Vasha na Wiosenną Ramę. Jeśli używasz Spring Framework i chcesz być spójny z tym, jak większość komponentów na wiosnę to robi, to powinieneś powiedzieć, że powinieneś użyć IllegalStateException (lub własnego wyprowadzenia).

Wiosną większość komponentów, które wykonują @PostConstruct lub @Override void afterPropertiesSet() rzut IllegalStateException przy użyciu narzędzia org.springframework.util.Assert.state(..).

Można to zobaczyć w jednym z przykładów: Spring AMQP.

Powiedziałbym, że zgłosiłem błędy przeciwko Spring MVC where they used IllegalArgumentException zamiast niestandardowej i bardziej jednoznacznej pochodnej klasy. Dzięki statycznym klasom wbudowanym bardzo łatwo jest utworzyć niestandardowy wyjątek bez tworzenia innego pliku Java.