Чистый код. Создание, анализ и рефакторинг

private void parseSchemaElement(String element) throws ParseException {

  char elementId = element.charAt(0);

  String elementTail = element.substring(1);

  validateSchemaElementId(elementId);

  if (elementTail.length() == 0)

    marshalers.put(elementId, new BooleanArgumentMarshaler());

  else if (elementTail.equals("*"))

    marshalers.put(elementId, new StringArgumentMarshaler());

  else if (elementTail.equals("#"))

    marshalers.put(elementId, new IntegerArgumentMarshaler());

  else if (elementTail.equals("##"))

    marshalers.put(elementId, new DoubleArgumentMarshaler());

  else

    throw new ParseException(String.format(

      "Argument: %c has invalid format: %s.", elementId, elementTail), 0);

}

Затем пишется класс DoubleArgumentMarshaler.

private class DoubleArgumentMarshaler implements ArgumentMarshaler {

  private double doubleValue = 0;

  public void set(Iterator currentArgument) throws ArgsException {

    String parameter = null;

    try {

      parameter = currentArgument.next();

      doubleValue = Double.parseDouble(parameter);

    } catch (NoSuchElementException e) {

      errorCode = ErrorCode.MISSING_DOUBLE;

      throw new ArgsException();

    } catch (NumberFormatException e) {

      errorParameter = parameter;

      errorCode = ErrorCode.INVALID_DOUBLE;

      throw new ArgsException();

    }

  }

  public Object get() {

    return doubleValue;

  }

}

Для нового типа добавляются новые коды ошибок:

private enum ErrorCode {

  OK, MISSING_STRING, MISSING_INTEGER, INVALID_INTEGER, UNEXPECTED_ARGUMENT,

  MISSING_DOUBLE, INVALID_DOUBLE}

А еще понадобится функция getDouble:

public double getDouble(char arg) {

  Args.ArgumentMarshaler am = marshalers.get(arg);

  try {

    return am == null ? 0 : (Double) am.get();

  } catch (Exception e) {

    return 0.0;

  }

}

И все тесты успешно проходят! Добавление нового типа прошло в целом безболезненно. Теперь давайте убедимся в том, что обработка ошибок работает правильно. Следующий тестовый сценарий проверяет, что при передаче неразбираемой строки с аргументом ## выдается соответствующая ошибка:

  public void testInvalidDouble() throws Exception {

    Args args = new Args("x##", new String[] {"-x","Forty two"});

    assertFalse(args.isValid());

    assertEquals(0, args.cardinality());

    assertFalse(args.has('x'));

    assertEquals(0, args.getInt('x'));

    assertEquals("Argument -x expects a double but was 'Forty two'.",

               args.errorMessage());

  }

---

  public String errorMessage() throws Exception {

    switch (errorCode) {

      case OK:

        throw new Exception("TILT: Should not get here.");

      case UNEXPECTED_ARGUMENT:

        return unexpectedArgumentMessage();

      case MISSING_STRING:

        return String.format("Could not find string parameter for -%c.",

                             errorArgumentId);

      case INVALID_INTEGER:

        return String.format("Argument -%c expects an integer but was '%s'.",

                             errorArgumentId, errorParameter);

      case MISSING_INTEGER: