Блог

У меня есть некоторый код, который нуждается в сокращении, и это было рекомендовано мне на Reddit.com для этого я использую функции. Но проблема в том, что код в основном представляет собой набор public static final «ссылок», таких как PIXELS_PER_INCH = 72 .

Я открываю исходные коды своего приложения на GitHub, если кто-то хочет взглянуть на это таким образом.

Вот пример того, как я присваиваю значения своим константам:

 public static final int PICTURES_4 = 0;
public static final int PICTURES_6 = 1;
public static final int PICTURES_8 = 2;
public static final int PICTURES_10 = 3;
public static final int PICTURES_12 = 4;

public static final int PICTURES_OPTION_0 = 4;
public static final int PICTURES_OPTION_1 = 6;
public static final int PICTURES_OPTION_2 = 8;
public static final int PICTURES_OPTION_3 = 10;
  

И вот пример того, где я использую эти постоянные ссылки:

 switch (settings.getInt(SettingsActivity.PICTURES_OPTION, SettingsActivity.DEFAULT_PICTURES_OPTION)) {
        case SettingsActivity.PICTURES_4:
            totalImages = SettingsActivity.PICTURES_OPTION_0;
            break;
        case SettingsActivity.PICTURES_6:
            totalImages = SettingsActivity.PICTURES_OPTION_1;
            break;
        case SettingsActivity.PICTURES_8:
            totalImages = SettingsActivity.PICTURES_OPTION_2;
            break;
  

По сути, я хотел бы знать, как использовать функцию для сокращения кода, но при этом сохранить возможность использовать читаемые ссылки. Или моя идея использовать читаемую ссылку неверна? На онлайн-уроке программирования я узнал об использовании констант в качестве читаемых ссылок (таких как PIXELS_PER_INCH = 72) вместо использования абсолютных значений.

Спасибо, что уделили мне время.

Обновление: В итоге я воспользовался предложениями aioobe и Ллойда Озимандиаса Форса. Я использовал EnumMap в цикле for. Однако в цикле for я по какой-то причине не смог использовать myEnumMap.values() . Я погуглил это и нашел другой способ сделать это с помощью myEnumMap.class.getEnumConstants() , вместо этого.

• Вы можете поместить параметры в map и сделать get(option) вместо switch(option)

  Инициализируйте карту следующим образом:

   Map<Integer, Integer> optionsMap = new HashMap<Integer, Integer>() {{
      put(SettingsActivity.PICTURES_4, SettingsActivity.PICTURES_OPTION_0);
      put(SettingsActivity.PICTURES_6, SettingsActivity.PICTURES_OPTION_1);
      ...
  }};
    

  а затем замените переключатель следующим образом:

   totalImages = optionsMap.get(settings.getInt(SettingsActivity.PICTURES_OPTION,
                                     SettingsActivity.DEFAULT_PICTURES_OPTION));
    

• Или (поскольку у вас, похоже, простое преобразование целого числа в целое) вы могли бы поместить свои значения в таблицу поиска.

• Или вы можете поместить это в функцию и заменить все разрывы возвратами:

   public int getPictureOption() {
      switch (settings.getInt(SettingsActivity.PICTURES_OPTION,
                              SettingsActivity.DEFAULT_PICTURES_OPTION)) {
              case SettingsActivity.PICTURES_4:
                  return SettingsActivity.PICTURES_OPTION_0;
  
              case SettingsActivity.PICTURES_6:
                  return SettingsActivity.PICTURES_OPTION_1;
  
              case SettingsActivity.PICTURES_8:
                  return SettingsActivity.PICTURES_OPTION_2;
      }
  }
    

  а затем выполните

   totalImages = getPictureOption();
    

Если вы считаете, что ваши константы говорят сами за себя, вы могли бы избавиться от всех SettingsActivity. , добавив статический импорт в начало вашего файла:

 import static your.package.SettingsActivity.*;
  

Не включайте значения enum — это задание для полиморфизма или таблицы Map / look up, как упоминалось aiobe.

Чтобы проиллюстрировать один из методов работы с перечислениями в Java, рассмотрим следующий параметр SettingsActivity enum с полиморфным методом getPictureOption() . Он объявлен абстрактным в теле перечисления, заставляя каждую константу перечисления реализовывать этот метод. Этот метод более подробно описан в Effective Java 2nd Edition (Джош Блох).

 enum SettingsActivity {
    OPTION_1(){
        public int getPicureOption() {
            return  myValue * 6;
        }
    },
    OPTION_2(){
        public int getPictureOption() {
            return Math.pow(myValue,5);
        }
    },
    ...
    OPTION_N() {
        public int getPictureOption() {
            return myValue-6;
        }
    };

    private final int myValue;

    private SettingsActivity(int myValue) {
        this.myValue = myValue;
    }

    // enums can have abstract methods!
    public abstract int getPictureOption();
}

SettingsActivity foo = getSettingsActivityFromSomewhere();

// polymorphic getPictureOption method - we don't
// know which activity was clicked
int something = foo.getPictureOption();
  

Конечно, это всего лишь пример кода, иллюстрирующий, что вы можете сделать. Я не уверен, какова цель вашего оператора switch или откуда взято перечисление SettingsActivity, поэтому я не могу предложить больше рекомендаций.

В случае, если вы хотите использовать Map, класс EnumMap, предоставляемый JDK, предоставляет сверхэффективную реализацию Map для перечислений (он реализован в виде одного длинного значения для перечислений с менее чем 65 константами и массива длинных значений для > 64 констант). В любом случае, операции поиска равны O (1) (в отличие от O (lg n) для хэш-карты).