Блог

У меня есть класс abstract, который расширен на 15 классов. Все 15 классов имеют общий код метода с именем exec(map: HashMap) : Unit . Поэтому я подумал о реализации этого кода в абстрактном классе. Однако этот код exec() имеет побочный эффект. Он присваивает результирующую переменную выходным данным exec(). (Я не могу изменить подпись exec, потому что она реализована из некоторого внешнего интерфейса). Вывод основан на карте, которая была передана реализующим классом.

Итак, я в замешательстве, что, если я реализую exec() в абстрактном классе, будет ли повреждено значение result vairable ?

 abstract class AbstractMyClass extends externalIntefaceA{
  var result : DataFrame = _

  override def exec(map : HashMap): Unit{
  //some logic
  result = map.get(key1)
 }
  def eval

  def getResult : DataFrame = result
}
  
class A1 extends AbstractMyClass{
  override def eval(map : HashMap){
  map.add("key1","value1")
 }
}

class A2 extends AbstractMyClass{
  override def eval(map : HashMap){
  map.add("key1","value2")
}
}


class A3 extends AbstractMyClass{
  override def eval(map : HashMap){
  map.add("key1","value3")
 }
}
  

Хорошо, не уверен, что я смог это хорошо объяснить, но будет ли результирующая переменная повреждена различными реализующими классами? Я хотел бы использовать метод exec() в абстрактном классе, потому что остальная часть кода в нем одинакова во всех 15 классах. Поэтому нет смысла переопределять ее в каждом из 15 классов и иметь локальную результирующую переменную для каждого класса. Есть ли лучшая практика для такого рода вещей?

Возможно, вы путаете классы и экземпляры. A1 и A2 т.д. — это разные классы, которые разделяют некоторую реализацию AbstractMyClass . Но каждый раз, когда вы создаете новый объект одного из этих классов, он включает в себя новый объект типа AbstractMyClass . Каждый объект имеет разное result значение, поэтому вызов методов для одного объекта не повлияет на значение в другом объекте.

Однако общий дизайн здесь плохой и не функциональный. Использование var — плохой признак, и обнародование этого var делает невозможным узнать, кто мог изменять это значение.

Было бы лучше, если exec бы возвращался результат, а не сохранял его, устраняя необходимость в a var . Если новое значение требуется для последующих вычислений, то это новое значение может быть передано этим методам, а не считываться из состояния класса.

Если действительно необходимо иметь изменяемые значения, убедитесь, что они есть private , и предоставьте методы для их изменения. И избегайте записи этих значений в основной код обработки, чтобы сохранить эту часть кода функциональной. Легче проверять и тестировать код обработки, если он не записывает никаких изменяемых значений.