Рефакторинг участков кода с одинаковой сигнатурой, но разными именами объектов

@Serbis · Регистрация: 13.07.2012

Студворк — интернет-сервис помощи студентам

В процессе роста программы, в довольно большом количестве стали возникать участки кода с одинаковой сигнатурой, но с разными именами объектов. Т. е. в сути код делает одно и то же, но оперирует разными именами. Естественно пришла мысль, упаковать весь код в одну функцию, так как одинаковый по структуре код, в множестве разных мест, приводит к сумятице и лишней раздутости классов. Идеальным решение в данном случае было бы динамическое назначение имен объектов, но увы, язык этого, как я понимаю делать не умеет. Можно ли как-то упаковать весь этот повторяющийся код, в одну функцию с входящими параметрами, не прибегая к приемам черной магии.

Сам код :

Java

fragment1.addLine(etName.getText().toString(), etCount.getText().toString());       //Добавляем строку в таблицц Белки
m.put(ATTRIBUTE_NAME_NAME, etName.getText().toString());        //Загружаем в HashMap имя
m.put(ATTRIBUTE_NAME_VALUE, etCount.getText().toString());      //Загружаем в HashMap количество
String id = String.valueOf(rand.nextInt(1000000000));      //Генерируем случайный id
m.put(ATTRIBUTE_NAME_ID, id);       //Загружаем в HashMap id
Toast.makeText(context, String.valueOf(id), Toast.LENGTH_LONG).show();
arrayProt.add(m);    //Добавление значения в массив текущих значений белков
tvProtSum.setText(calcSumElements("prot"));        //Пересчет столбца суммы белков и ккал
sql.writeToDB("PROT", etName.getText().toString(), etCount.getText().toString(), id);

Тот же самый код в другой формации:

Java

fragment2.addLine(etName.getText().toString(), etCount.getText().toString());       //Добавляем строку в таблицц Жиры
m.put(ATTRIBUTE_NAME_NAME, etName.getText().toString());        //Загружаем в HashMap имя
m.put(ATTRIBUTE_NAME_VALUE, etCount.getText().toString());      //Загружаем в HashMap количество
String id = String.valueOf(rand.nextInt(1000000000));      //Генерируем случайный id
m.put(ATTRIBUTE_NAME_ID, id);       //Загружаем в HashMap id
Toast.makeText(context, String.valueOf(id), Toast.LENGTH_LONG).show();
arrayFat.add(m);    //Добавление значения в массив текущих значений белков
tvFatSum.setText(calcSumElements("fat"));        //Пересчет столбца суммы жиров и ккал
sql.writeToDB("FAT", etName.getText().toString(), etCount.getText().toString(), id);

И так далее и тому подобное еще пять раз в разными именами.

@turbanoff · 13.02.2015, 14:18

Все что различается - выносите в параметры. Как-то так:

Java

public void doSomething(Fragment fragment, MyArray array, MyTv tvSum, String typeForCalc, String typeForDb) {
   fragment.addLine(etName.getText().toString(), etCount.getText().toString());       //Добавляем строку в таблицц Белки
   m.put(ATTRIBUTE_NAME_NAME, etName.getText().toString());        //Загружаем в HashMap имя
   m.put(ATTRIBUTE_NAME_VALUE, etCount.getText().toString());      //Загружаем в HashMap количество
   String id = String.valueOf(rand.nextInt(1000000000));      //Генерируем случайный id
   m.put(ATTRIBUTE_NAME_ID, id);       //Загружаем в HashMap id
   Toast.makeText(context, String.valueOf(id), Toast.LENGTH_LONG).show();
   array.add(m);    //Добавление значения в массив текущих значений белков
   tvSum.setText(calcSumElements(type));        //Пересчет столбца суммы белков и ккал
   sql.writeToDB(typeForDb, etName.getText().toString(), etCount.getText().toString(), id);
}

Java

1	doSomething(fragment1, "prot", "PROT", arrayProt, tvProtSum);

	13.02.2015, 14:18

Опции темы