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

    return booleanArgs.containsKey(argChar);

  }

  public int cardinality() {

    return numberOfArguments;

  }

  public String usage() {

    if (schema.length() > 0)

       return "-["+schema+"]";

    else

      return "";

  }

  public String errorMessage() {

    if (unexpectedArguments.size() > 0) {

      return unexpectedArgumentMessage();

    } else

      return "";

  }

  private String unexpectedArgumentMessage() {

    StringBuffer message = new StringBuffer("Argument(s) -");

    for (char c : unexpectedArguments) {

      message.append(c);

    }

    message.append(" unexpected.");

    return message.toString();

  }

  public boolean getBoolean(char arg) {

    return booleanArgs.get(arg);

  }

}

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

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

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

Листинг 14.10. Args.java (Boolean и String)

package com.objectmentor.utilities.getopts;

import java.text.ParseException;

import java.util.*;

public class Args {

  private String schema;

  private String[] args;

  private boolean valid = true;

  private Set unexpectedArguments = new TreeSet();

  private Map booleanArgs =

    new HashMap();

  private Map stringArgs =

Листинг 14.10 (продолжение)

    new HashMap();

  private Set argsFound = new HashSet();

  private int currentArgument;

  private char errorArgument = '\0';

  enum ErrorCode {

    OK, MISSING_STRING}

  private ErrorCode errorCode = ErrorCode.OK;

  public Args(String schema, String[] args) throws ParseException {

    this.schema = schema;

    this.args = args;

    valid = parse();

  }

  private boolean parse() throws ParseException {

    if (schema.length() == 0 && args.length == 0)

      return true;

    parseSchema();

    parseArguments();

    return valid;

  }

  private boolean parseSchema() throws ParseException {

    for (String element : schema.split(",")) {

      if (element.length() > 0) {

        String trimmedElement = element.trim();

        parseSchemaElement(trimmedElement);

      }

    }

    return true;

  }

  private void parseSchemaElement(String element) throws ParseException {

    char elementId = element.charAt(0);

    String elementTail = element.substring(1);

    validateSchemaElementId(elementId);

    if (isBooleanSchemaElement(elementTail))

      parseBooleanSchemaElement(elementId);

    else if (isStringSchemaElement(elementTail))

      parseStringSchemaElement(elementId);

  }

  private void validateSchemaElementId(char elementId) throws ParseException {

    if (!Character.isLetter(elementId)) {

      throw new ParseException(

        "Bad character:" + elementId + "in Args format: " + schema, 0);

    }

  }