Есть ли способ сделать следующий код более общим?

Question

Я использую gson для сериализации/десериализации и потока java 8. Ниже приведен фрагмент кода,

private long sumofTime() { 
    Line[] lines = gson.fromJson(jsonString, Line[].class); 
    return Arrays.stream(lines).filter(x -> x.hasTime()) 
       .mapToInt(x -> x.getTime).sum(); 
} 

класса Line выглядит,

public class Line { 
    String name; 
    String stamp; 
    Integer time; 
    Integer xx; 
    Integer yy; 
    Integer zz; 
    ... 
    ... 

    boolean hasTotalTime() { 
     return totalTime != null; 
    } 
    ... 
    getters...setters...} 

потока используется для проверки, если конкретные переменные (eg.time в приведенном выше примере) не является нулевой для каждого элемента массива (т.е. Line), а затем получить сумму всех раз.

Вопрос: В объектах линии есть сумма, равная 30 переменным, сумма которых необходима, поэтому как сделать решение более общим, вместо того, чтобы писать метод суммы для каждой переменной. Обратите внимание, что для обработки более 1000 объектов Line, поэтому я подумал, что Stream будет лучше.

Answer 1

Вы можете передать функцию в метод суммы для получения значения из каждой строки, которую вы хотите подвести:

public int sumLines(Function<Line, Integer> extractor){ 
    Line[] lines = ... 
    return Arrays.stream(lines).map(extractor) 
     .filter(Objects::nonNull).mapToInt(i -> i).sum(); 
} 
.... 
int time = sumLines(Line::getTime); 

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

Answer 2

Вместо того, чтобы иметь поле для каждого атрибута, можно определить перечисление для атрибутов, а затем отображение из перечисления в INT:

public enum LineAttribute { 
    XX, 
    YY, 
    ZZ, 
    ... 
} 

private final EnumMap<LineAttribute, Integer> attributes; 

public Line() { 
    attributes = new EnumMap<>(LineAttribute.class); 
    // init all attributes to 0 
    for (LineAttribute attr : LineAttribute.values()) { 
     attributes.put(attr, 0); 
    } 
} 

Затем вы можете цикл по каждому атрибуту, получение его значения линий и суммирования этих значений.

private long sumOf(LineAttribute attr) { 
    Line[] lines = gson.fromJson(jsonString, Line[].class); 
    return Arrays.stream(lines) 
      .filter(x -> x.has(attr)) 
      .mapToInt(x -> x.get(attr)) 
      .sum(); 
} 

Answer 3

То, что вы пытаетесь сделать, это частично динамическое извлечение собственности и частично частично применяется функция, ни один из которых особенно проста в Java. Вам было бы лучше определить ваш класс Line по-другому. Я хотел бы использовать Map хранить свойства следующим образом:

public class Line { 
    Map<String, Integer> props; 
    public Line() { 
     // Initialize props 
    } 

    public boolean has(String prop) { 
     return props.containsKey(prop); 
    } 

    public Integer get(String prop) { 
     return props.get(prop); 
    } 

    public void set(String prop, Object value) { 
     return props.put(prop, value); 
    } 
} 

Теперь, когда вы смотрите на сумму кучу вещей, вы можете позвонить

public int sumOf(Line[] lines, String prop) { 
    return Arrays.stream(lines) 
       .filter(l -> l.has(prop)) 
       .reduce(0, Integer::sum); 
}