Недавно я подошел к интересному вопросу, какими должны быть быстрые методы? Должны ли они изменить состояние текущего объекта или создать новый с новым состоянием?
В случае, если это краткое описание не очень интуитивно понятно здесь (неудовлетворительно) длинный пример. Это калькулятор. Он выполняет очень тяжелые вычисления и поэтому возвращает результаты через асинхронный обратный вызов:
public interface ICalculator {
// because calcualations are too lengthy and run in separate thread
// these methods do not return values directly, but do a callback
// defined in IFluentParams
void Add();
void Mult();
// ... and so on
}
Итак, здесь свободный интерфейс, который устанавливает параметры и обратные вызовы:
public interface IFluentParams {
IFluentParams WithA(int a);
IFluentParams WithB(int b);
IFluentParams WithReturnMethod(Action<int> callback);
ICalculator GetCalculator();
}
У меня есть два интересных варианта для реализации этого интерфейса. Я покажу им обоим, и тогда я напишу, что я считаю хорошим и плохим каждый из них.
Итак, сначала обычный, который возвращает этот:
public class FluentThisCalc : IFluentParams {
private int? _a;
private int? _b;
private Action<int> _callback;
public IFluentParams WithA(int a) {
_a = a;
return this;
}
public IFluentParams WithB(int b) {
_b = b;
return this;
}
public IFluentParams WithReturnMethod(Action<int> callback) {
_callback = callback;
return this;
}
public ICalculator GetCalculator() {
Validate();
return new Calculator(_a, _b);
}
private void Validate() {
if (!_a.HasValue)
throw new ArgumentException("a");
if (!_b.HasValue)
throw new ArgumentException("bs");
}
}
Вторая версия сложнее, она возвращает новый объект при каждом изменении состояния:
public class FluentNewCalc : IFluentParams {
// internal structure with all data
private struct Data {
public int? A;
public int? B;
public Action<int> Callback;
// good - data logic stays with data
public void Validate() {
if (!A.HasValue)
throw new ArgumentException("a");
if (!B.HasValue)
throw new ArgumentException("b");
}
}
private Data _data;
public FluentNewCalc() {
}
// used only internally
private FluentNewCalc(Data data) {
_data = data;
}
public IFluentParams WithA(int a) {
_data.A = a;
return new FluentNewCalc(_data);
}
public IFluentParams WithB(int b) {
_data.B = b;
return new FluentNewCalc(_data);
}
public IFluentParams WithReturnMethod(Action<int> callback) {
_data.Callback = callback;
return new FluentNewCalc(_data);
}
public ICalculator GetCalculator() {
Validate();
return new Calculator(_data.A, _data.B);
}
private void Validate() {
_data.Validate();
}
}
Как они сравнивают:
Pro первая (эта) версия:
-
проще и короче
-
обычно используется
-
кажется более энергоемким
-
что еще?
Вторая (новая) версия:
-
хранит данные в отдельном контейнере, позволяет разделить логику данных и всю обработку
-
позволяет легко фиксировать часть данных, а затем заполнять другие данные и обрабатывать их отдельно. Посмотрите:
var data = new FluentNewCalc() .WithA(1); Parallel.ForEach(new[] {1, 2, 3, 4, 5, 6, 7, 8}, b => { var dt = data .WithB(b) .WithReturnMethod(res => {/* some tricky actions */}); // now, I have another data object for each value of b, // and they have different callbacks. // if I were to do it with first version, I would have to create each // and every data object from scratch var calc = dt.GetCalculator(); calc.Add(); });
Что может быть лучше во второй версии?
-
Я мог бы реализовать метод WiспасибоXX следующим образом:
public IFluentParams WithXXX(int xxx) { var data = _data; data.XXX = xxx; return new FluentNewCalc(data); }
и сделайте _data readonly (т.е. неизменяемый), который говорят некоторые умные люди.
Итак, вопрос в том, как вы считаете, что лучше и почему? Постскриптум Я использовал С#, но он вполне может применяться к java.