Лучший способ к флагу статуса возврата и сообщению из метода в Java

Вместо того, чтобы создать специальный объект для типа возврата, я обычно просто возвращаю массив, где вся возвращенная информация хранится. Преимущество - то, что можно расширить этот массив с помощью новых элементов, не создавая новые типы и путаницу. Оборотная сторона необходимо знать точно, что должны представить элементы, когда массив возвращается из конкретного метода для парсинга ее правильно. Обычно я договариваюсь об определенной структуре, как первый элемент всегда булев успех признака, второй Строка с описанием, остальное является дополнительным. Пример:

public static void main(String[] args)
{
    Object[] result = methodReturningStatus();
    if(!(Boolean)result[0])
        System.out.println("Method return: "+ result[1]);
}

static Object[] methodReturningStatus()
{
    Object[] result = new Object[2];

    result[0] = false;
    result[1] = "Error happened";

    return result;
}

Я выбрал бы опцию Exception в первом месте.

, Но, во втором месте, я предпочел бы метод C-стиля:

public boolean authenticate(Client client, final StringBuilder sb) {
    if (sb == null)
        throw new IllegalArgumentException();
    if (isOK()) {
        sb.append("info message");
        return true;
    } else {
        sb.append("error message");
        return false;
    }
}

Это не настолько странно, и это сделано во многих местах в платформе.

Возвратите Объект. Это позволяет Вам помещать дополнительную функциональность в Класс при необходимости в нем. Недолгие объекты в Java быстры, чтобы создать и собраться.

Как насчет того, чтобы возвратить строку. Пустой или Пустой для успеха. Сообщение об ошибке в случае отказа. Самый простой, который работал бы. Однако не уверенный, если это читает хорошо.

Существует много хороших ответов здесь, таким образом, я сохраню его коротким.

я думаю, что отказ пользователя аутентифицировать можно считать допустимым случаем для контролируемой исключительной ситуации. Если бы Ваш стиль программирования привилегированных обрабатывающих исключений тогда не был бы никакой причиной не сделать это. Это также удаляет, "Как возвратить несколько значений из метода, мой метод делает одну вещь, Это аутентифицирует пользователя"

, Если Вы собираетесь возвратиться, несколько значений тогда проводят 10 минут, создавая универсальный PairTuple (могут также быть больше, чем парный TripleTuple, я не повторю упомянутый выше пример), и возвратите свои значения тот путь. Я очень не хочу иметь маленькие dto объекты стиля для возврата различный несколько значений, они просто создают помехи месту.

Успешная аутентификация должна быть "нормальным" случаем, таким образом, ошибка аутентификации является исключительным случаем.

, Каковы различные строки состояния для пользователя так или иначе. Я вижу только два, успешность или неуспешность. Дальнейшей информацией является потенциальная проблема безопасности. Другое преимущество решения за исключениями состоит в том, что это нельзя назвать неправильным способом, и случай возникновения отказов более очевиден. Без исключений Вы пишете:

if (authenticate()) {
  // normal behaviour...
}
else {
  // error case...
}

можно случайно назвать метод, игнорирующий возвращаемое значение. "Нормальное поведение" код тогда выполняется без успешной аутентификации:

authenticate();
// normal behaviour...

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

if (isAuthenticated()) {
//...
}

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

Почти тот же вопрос отправляется здесь и здесь

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

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

Вот кавычка о возврате двух значений от функции:

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

я нашел, что он в запросе новых функций для Java позволяет , несколько возвращаемых значений

смотрят на датированный раздел "оценки": 06.05.2005 9:40:08

Я по всей вероятности пошел бы для чего-то как:

class SomeClass {
public int authenticate (Client client) {
//returns 0 if success otherwise one value per possible failure
}
public String getAuthenticationResultMessage (int authenticateResult) {}
//returns message associated to authenticateResult
}

С этим "дизайном", можно попросить сообщение только, когда аутентификация перестала работать (который я надеюсь, сценарий, который происходит 99,99% времени;))

Это может также иметь хорошую практику для делегирования разрешения сообщения к другому Классу. Но это зависит Ваших потребностей приложения (главным образом, этому нужен i18n?)

Я использую "крошечный класс" сам, обычно с внутренним классом. Мне не нравится использовать аргументы для сбора сообщений.

кроме того, если метод, который мог бы перестать работать, является "низким уровнем" - как прибытие из сервера приложений или слоя базы данных, я предпочел бы возвращать Перечисление со статусом возврата, и затем переводить это в строку на уровне GUI. Не раздавайте пользовательские строки на низком уровне, если Вы когда-либо собираетесь интернационализировать свой код, потому что тогда Ваш сервер приложений может только ответить на одном языке за один раз, вместо того, чтобы иметь различные клиенты, работающие на различных языках.

Просто, потому что неудачная аутентификация распространена, не означает, что это не исключительно.

, По-моему, ошибки аутентификации ребенок плаката вариант использования для контролируемых исключительных ситуаций. (Хорошо..., возможно, несуществование файла является каноническим вариантом использования, но ошибка аутентификации является близким № 2.)

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

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

Еще некоторые опции:

• Возврат отдельное перечисление значений для каждого типа отказа. Перечислимый объект мог содержать сообщение
• Возврат интервал и иметь отдельный метод, который ищет, соответствующее сообщение от массива
• создают универсальный служебный класс кортежа, который может содержать два значения. Такой класс может быть полезным в значительно большем количестве мест.

простой пример кортежа, фактическая реализация, возможно, нуждается в больше:

class Tuple<L, R> {

    public final L left;
    public final R right;

    public Tuple( L left, R right) {
        this.left = left;
        this.right = right;
    }
}

Постарайтесь не возвращать "значение сигнальной метки", особенно пустой. Вы закончите с кодовой базой, где методы не могут быть поняты под вызывающей стороной, не читая реализацию. В случае пустого указателя вызывающие стороны могут закончить с NullPointerExceptions, если они забывают (или не знайте), что Ваш метод может возвратить пустой указатель.

предложение кортежа от Bas Leijdekkers является хорошим, которое я использую все время, если я хочу возвратить больше чем одно значение из метода. Тот, который мы используем, P2<A, B> от библиотека Functional Java . Этот вид типа является совместным объединением двух других типов (это содержит одно значение каждого типа).

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

1. Вы можете иметь Option<T> абстрактный класс с двумя подклассами Some<T> и None<T>. Это немного похоже на безопасную с точки зрения типов альтернативу пустому указателю и хороший способ реализовать частичные функции (функции, возвращаемое значение которых не определяется для некоторых аргументов). библиотека Functional Java имеет полнофункциональное Option класс, который реализует Iterable<T>, таким образом, можно сделать что-то вроде этого:

   public Option<String> authenticate(String arg) {
      if (success(arg))
         return Option.some("Just an example");
      else
         return Option.none();
   }
   
   ...
   
   for(String s : authenticate(secret)) {
      privilegedMethod();
   }
   

2. , С другой стороны, можно использовать дизъюнктное объединение двух типов, как Either<L, R> класс. Это содержит одно значение, которое имеет любой тип L или R. Эти реализации класса Iterable<T> и для [1 112] и для R, таким образом, можно сделать что-то вроде этого:

   public Either<Fail, String> authenticate(String arg) {
      if (success(arg))
         return Either.right("Just an example");
      else
         return Either.left(Fail.authenticationFailure());
   }
   
   ...
   
   Either<Fail, String> auth = authenticate(secret);
   for(String s : auth.rightProjection()) {
      privilegedMethod();
   }
   for(Fail f : auth.leftProjection()) {
      System.out.println("FAIL");
   }
   

Все эти классы, P2, Option, и Either полезны в большом разнообразии ситуаций.

Вы могли использовать исключения....

try {
    AuthenticateMethod();
} catch (AuthenticateError ae) {         
    // Display ae.getMessage() to user..
    System.out.println(ae.getMessage());
    //ae.printStackTrace();    
}

и затем если ошибка происходит в Вашем AuthenticateMethod, Вы отправляете, новый AuthenticateError (расширяет Исключение)

Действительно ли это - единственный метод, где у Вас есть такое требование? В противном случае просто генерируйте общий класс Ответа с флагом isSuccessful и строкой сообщения и использованием это везде.

Или у Вас мог просто быть пустой указатель возврата метода для проявления успеха (не симпатичный, и не позволяет возвращать успех И сообщение).