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

    public void set(Iterator currentArgument) throws ArgsException {

      try {

        stringValue = currentArgument.next();

      } catch (NoSuchElementException e) {

        errorCode = ArgsException.ErrorCode.MISSING_STRING;

        throw new ArgsException();

      }

    }

    public Object get() {

      return stringValue;

    }

  }

  private class IntegerArgumentMarshaler implements ArgumentMarshaler {

    private int intValue = 0;

    public void set(Iterator currentArgument) throws ArgsException {

      String parameter = null;

      try {

        parameter = currentArgument.next();

        intValue = Integer.parseInt(parameter);

      } catch (NoSuchElementException e) {

        errorCode = ArgsException.ErrorCode.MISSING_INTEGER;

        throw new ArgsException();

      } catch (NumberFormatException e) {

        errorParameter = parameter;

        errorCode = ArgsException.ErrorCode.INVALID_INTEGER;

        throw new ArgsException();

      }

    }

    public Object get() {

      return intValue;

    }

  }

  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 = ArgsException.ErrorCode.MISSING_DOUBLE;

        throw new ArgsException();

      } catch (NumberFormatException e) {

        errorParameter = parameter;

        errorCode = ArgsException.ErrorCode.INVALID_DOUBLE;

        throw new ArgsException();

      }

    }

    public Object get() {

      return doubleValue;

    }

  }

}

Хорошо — теперь Args выдает единственное исключение ArgsException. Выделение ArgsException в отдельный модуль приведет к тому, что большой объем вспомогательного кода обработки ошибок переместится из модуля Args в этот модуль. Это наиболее естественное и очевидное место для размещения этого кода, вдобавок перемещение поможет очистить перерабатываемый модуль Args.

Итак, нам удалось полностью отделить код исключений и ошибок от модуля Args (листинги 14.13–14.16). Для решения этой задачи понадобилось примерно 30 промежуточных шагов, и после каждого шага проверялось прохождение всех тестов.

Листинг 14.13. ArgsTest.java

package com.objectmentor.utilities.args;

import junit.framework.TestCase;

public class ArgsTest extends TestCase {

  public void testCreateWithNoSchemaOrArguments() throws Exception {

    Args args = new Args("", new String[0]);

    assertEquals(0, args.cardinality());

  }

  public void testWithNoSchemaButWithOneArgument() throws Exception {

    try {

      new Args("", new String[]{"-x"});

      fail();

    } catch (ArgsException e) {

      assertEquals(ArgsException.ErrorCode.UNEXPECTED_ARGUMENT,

                   e.getErrorCode());

      assertEquals('x', e.getErrorArgumentId());

    }

  }

  public void testWithNoSchemaButWithMultipleArguments() throws Exception {

    try {

      new Args("", new String[]{"-x", "-y"});

      fail();

    } catch (ArgsException e) {

      assertEquals(ArgsException.ErrorCode.UNEXPECTED_ARGUMENT,

                   e.getErrorCode());

      assertEquals('x', e.getErrorArgumentId());

    }

  }