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

    return s == null ? "" : s;

  }

  public String getString(char arg) {

    return blankIfNull(stringArgs.get(arg));

  }

  public int getInt(char arg) {

    return zeroIfNull(intArgs.get(arg));

  }

  public boolean getBoolean(char arg) {

    return falseIfNull(booleanArgs.get(arg));

  }

  public boolean has(char arg) {

    return argsFound.contains(arg);

  }

  public boolean isValid() {

    return valid;

  }

  private class ArgsException extends Exception {

  }

}

Надеюсь, при виде этой глыбы кода вам захотелось сказать: «Как хорошо, что она не осталась в таком виде!» Если вы почувствовали нечто подобное, вспомните, что другие люди чувствуют то же самое при виде вашего кода, оставшегося на стадии «черновика».

Вообще говоря, «черновик» — самое мягкое, что можно сказать об этом коде. Очевидно, что перед нами незавершенная работа. От одного количества переменных экземпляров можно прийти в ужас. Загадочные строки вроде "TILT”, контейнеры HashSet и TreeSet, конструкции try-catch-catch только увеличивают масштабы этого беспорядочного месива.

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

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

Листинг 14.9. Args.java (только Boolean)

package com.objectmentor.utilities.getopts;

import java.util.*;

public class Args {

  private String schema;

  private String[] args;

  private boolean valid;

  private Set unexpectedArguments = new TreeSet();

  private Map booleanArgs =

    new HashMap();

  private int numberOfArguments = 0;

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

    this.schema = schema;

    this.args = args;

    valid = parse();

  }

  public boolean isValid() {

    return valid;

  }

  private boolean parse() {

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

      return true;

    parseSchema();

    parseArguments();

    return unexpectedArguments.size() == 0;

  }

  private boolean parseSchema() {

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

      parseSchemaElement(element);

    }

    return true;

  }

  private void parseSchemaElement(String element) {

    if (element.length() == 1) {

      parseBooleanSchemaElement(element);

    }

  }

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

  private void parseBooleanSchemaElement(String element) {

    char c = element.charAt(0);

    if (Character.isLetter(c)) {

      booleanArgs.put(c, false);

    }

  }

  private boolean parseArguments() {

    for (String arg : args)

      parseArgument(arg);

    return true;

  }

  private void parseArgument(String arg) {

    if (arg.startsWith("-"))

      parseElements(arg);

  }

  private void parseElements(String arg) {

    for (int i = 1; i < arg.length(); i++)

      parseElement(arg.charAt(i));

  }

  private void parseElement(char argChar) {

    if (isBoolean(argChar)) {

      numberOfArguments++;

      setBooleanArg(argChar, true);

    } else

      unexpectedArguments.add(argChar);

  }

  private void setBooleanArg(char argChar, boolean value) {

    booleanArgs.put(argChar, value);

  }

  private boolean isBoolean(char argChar) {