Как правильно обработать Unhandled exception?

Question

[Jake Taylor]

Есть у меня класс, в котором есть метод, который принимает строку и парсит её с помощью стримов в массив типа double, который выбрасывает исключение типа неверного формата данных:

Класс-парсер

public double[] parseString (String dataString) throws QuadrangleException {
        String withoutCommas = dataString.
                replace(REGEX_COMMA, REGEX_SPACE)
                .trim();

        double[] parseData;

        try {
            parseData = Arrays.stream(withoutCommas.split(REGEX_SEPARATOR))
                    .mapToDouble(Double::parseDouble)
                    .toArray();
        } catch (NumberFormatException ex) {
            LOGGER.log(Level.ERROR, "Incorrect input data", ex);
            throw new QuadrangleException("Incorrect input data " + ex);
        }

        LOGGER.log(Level.INFO, "From string \"{}\" turned out an array: {}", dataString, Arrays.toString(parseData));

        return parseData;
    }

Тут всё понятно. И вот наступает момент использования метода parseString() одного класса (код приведен выше) внутри другого метода с помощью стрима вот так:

Нажми, чтобы посмотреть код

@Override
    public List<Quadrangle> createQuadrangleFromGroupPoints (List<String> allGroupsPoint) {
        DataParser dataParser = new DataParser();

        List<Quadrangle> quadrangles = allGroupsPoint.stream()
                .map(dataParser::parseString)
                .map(this::convertToListPoint)
                .map(p -> new Quadrangle(p.get(0), p.get(1), p.get(2), p.get(3)))
                .collect(Collectors.toList());

        LOGGER.log(Level.INFO, "Was created {} quadrangles.", quadrangles.size());

        return quadrangles;
    }

// вот метод convertToListPoint
private List<Point> convertToListPoint (double[] doubleDigits) {
        int positionX;
        int positionY;
        List<Point> listPoints = new ArrayList<>();

        for (int i = 0; i < POINTS_IN_QUADRANGLE; i++) {
            positionX = i * 2;
            positionY = positionX + 1;
            listPoints.add(new Point(doubleDigits[positionX], doubleDigits[positionY]));
        }

        LOGGER.log(Level.INFO, "From {} digits was created {} points.", doubleDigits, listPoints.size());

        return listPoints;
    }

И вот на этой строке .map(dataParser::parseString) в стриме мне показывает, что у меня имеется необработанное исключение. Ок, как его правильно обработать? Добавить throws к методу - это не решение. Тогда как? Можно ли как-то использовать этот стрим или его нужно будет заменить?

Comments

[Dmitry Roo]

В случае возникновения исключения - какая должна быть бизес-логика?
Оно должно прокидываться дальше?
Как-то обрабатываться в методе createQuadrangleFromGroupPoints ?

[Сергей Горностаев]

мне показывает

Кто показывает? Как показывает? Небось parseString выбрасывает checked exception и IDE ругается?

[Jake Taylor]

Сергей Горностаев, Верно, Intellij IDEA ругается, не компилируется.

[Jake Taylor]

Dmitry Roo, Обрабатывать не надо, зачем? Это исключение уже обрабатывается в первом методе, второй раз мне не нужно.

[сергей кузьмин]

@n199a

заменить

mapToDouble(Double::parseDouble)
на

foreach (o->

	{
		try {
		  Double res= Double.parseDouble(o);
		  return res;
		} catch (NumberFormatException e) { 
			  //обработать исключение и вернуть то как требует бизнес логика например минус 1
                         return -1;
		}
	}
)

если помогло - отметьте решением

[Jake Taylor]

сергей кузьмин, конструкция выглядит громоздко, что затрудняет чтение кода. Ещё варианты.

[Сергей Горностаев]

Jake Taylor,
https://github.com/TheDeadOne/exception-unchecker

[сергей кузьмин]

Jake Taylor, не понял что имеете в виду. жалобы на синтаксис направляйте в оракл пожалуйста. почитайте кстати, https://mattgemmell.com/what-have-you-tried/

[Сергей Горностаев]

сергей кузьмин, Oracle больше не отвечает за синтаксис, Java - open source. Теперь, если что-то не устраивает в синтаксисе, нужно делать PR. Многоопытные инженеры из Oracle, IBM, RedHat, SAP, Amazon и ещё двух десятков компаний соответствующего масштаба этот PR обсудят и примут, если он того стоит.

[сергей кузьмин]

Сергей Горностаев это наверное надо предложить Jake Taylor
- на мой взгляд явный код вместо ссылки на готовый статический метод в лямбде *не* выглядит громоздко -

Answer 1

[Денис Загаевский]

Бизнес-логику на исключениях делать не надо. Заведи класс Result, сделай двух наследников, в одном массив даблов, другой - маркер ошибки. В правильном месте обрабатывай. Checked exceptions всё, можешь забыть про них. Никогда не используй в своём коде.

Comments

[Jake Taylor]

Можно пример, не совсем понимаю с точки зрения практический реализации.

[Денис Загаевский]

Jake Taylor,

public abstract class Result{
   Result(){}
   public final static class Ok extends Result {
        public final double [] data;
        ...
   }

    public final static class Error extends Result{
        private Error(){}
        public static final Error Instance = new Error ();
    }
}

try{ 
   parse 
   return new Result.Ok(parsed);
} catch(NFE){ 
   return Result.Error.Instance;
}


if(result instanceOf Result.Ok ok){
    use ok.data
} else {
   Handle error
}

Когда в джаву дотащат sealed classes/interfaces, можно будет использовать их, а пока так.

Answer 2

[Dmitry Roo]

Используйте @SneakyThrows из lombok https://projectlombok.org/features/SneakyThrows