Не получается вычислить среднеарифметическое между двумя объектами

У Вас проблемы в алгоритме. Попробуйте решить её немного по другому.

Добавьте поле количество статистик по школе. по умолчанию = 1, увеличиваете на повторении школы. StatsCount

Сделайте среднее арифметическое вычислимым полем. (math+rus+info) / 3

где math = Сумма всех математик по школе / StatsCount

rus = Сумма всех языков по школе / StatsCount

info = Сумма всех информатик по школе / StatsCount

1. Ввод школ

2. Создать новый список

3. По каждой статистике по школе. Если нет в новым списке то добавить. Если есть то сложить статистики и увеличить StatsCount.

4. Вывести новый список.

У вас проблемы не с алгоритмом, а с базовыми принципами программирования (для начала почитайте про GRASP), в т.ч. с ООП. Вы написали абсолютно не читаемый код на очень тривиальной задаче, в котором не смогли разобраться даже сами. А ведь по сути задача сводилась к поиску среднего арифметического. Для начала запомните, что машина прочитает любой код. Главная ваша задача - сделать код читаемым для человека. Именно для этого существуют все парадигмы программирования, паттерны и т.д. Посему читаемость - главное! И первое, что на нее влияет - правила именования, именно поэтому их очень важно соблюдать. Классы пишутся с большой буквы, методы и переменные - с маленькой. Все имена должны быть осмысленные. Если вы написали исключительный код с точки зрения ООП, но при этом ваши переменные называются a,b,c,d, все ваши усилия напрасны и ваш код нужно выбросит в урну по той же причине - это не читаемо! Следующее правило : геттеры и сеттеры - устоявшиеся и общепринятые сигнатуры, посему сеттеры всегда void. Хотите сделать то, что вы сделали, используйте паттерн билдер, но не меняйте сигнатуры общепринятых методов. Причин тому много. Я назову лишь одну (остальные вы поймете со временем): поведение любой построенной вами инженерной системы должно быть интуитивно понятно и ожидаемо. Например, вы хотите перебрать все элементы коллекции. Вы можете пройти обычным циклом и это нормально. Однако вы можете реализовать свой итератор, который будет делать перебор всех элементов каким-то особым способом. Человек, который посмотрит ваш код с обычным циклом, разберет это за секунды. Кастомный итератор же вызовет справедливый вопрос - почему вы сделали именно так? может нужен особый порядок элементов и т.д. это усложнит читаемость. аналогично с геттерами и сеттерами. их сигнатура такая, какая есть. это ожидаемо и предсказуемо. заталкивать туда дополнительную логику либо менять сигнатуру нельзя. Когда усвоите это, перейдите к уже рекомендованному GRASP. а теперь к практике. ваш класс-модель, описывающий школу, в соответствии с паттероном GRASP information expert, обязан иметь все необходимые методы для работы со своими полями (информация обрабатывается там, где хранится). именно поэтому мы щзаставим наш класс считать среднее арифметическое для своих полей и не будем считать и присваивать его из вне. вместе с тем для обработки множества объектов методы одного объекта нам уже не подходят. решений может быть несколько. мы создадим класс-сервис для этих целей. здесь есть 2 метода : один добавляет информацию о новой школе, а второй предоставляет нам готовую информацию о накопившихся объектах (экземплярах класса школа). реализации довольно простые и выглядят так:

import java.util.Locale;
import java.util.stream.DoubleStream;

public class School {
    private int number;
    private double mathematics;
    private double russianLanguage;
    private double computerScience;
    private double average;

    public School(int number, double mathematics, double russianLanguage, double computerScience) {
        this.number = number;
        this.mathematics = mathematics;
        this.russianLanguage = russianLanguage;
        this.computerScience = computerScience;
        this.average();
    }

    public final School average() {
        this.average = DoubleStream.of(mathematics, russianLanguage, computerScience).average().getAsDouble();
        return this;
    }

    public int getNumber() {
        return number;
    }

    public void setNumber(int number) {
        this.number = number;
    }

    public double getMathematics() {
        return mathematics;
    }

    public void setMathematics(double mathematics) {
        this.mathematics = mathematics;
    }

    public double getRussianLanguage() {
        return russianLanguage;
    }

    public void setRussianLanguage(double russianLanguage) {
        this.russianLanguage = russianLanguage;
    }

    public double getComputerScience() {
        return computerScience;
    }

    public void setComputerScience(double computerScience) {
        this.computerScience = computerScience;
    }

    public double getAverage() {
        return average;
    }

    @Override
    public String toString() {
        return String.format(Locale.ENGLISH,"Школа № " + number + " математика - " + 
                "%.1f" + " русский язык - " + "%.1f" + " инфрматика - " + "%.1f" + " общий средний балл - " + "%.1f", 
                mathematics, russianLanguage,computerScience,average);
    }

}


import java.util.Arrays;
import java.util.Collection;
import java.util.Comparator;
import java.util.Map;
import java.util.HashMap;
import java.util.stream.Collectors;

public class SchoolService {

    private final Map<Integer, School> schools;

    public SchoolService() {
        this.schools = new HashMap<>();
    }

    public SchoolService add(int number, double mathematics, double russianLanguage, double computerScience) {
        if (schools.containsKey(number)) {
            final School school = schools.get(number);
            school.setMathematics(avarage(school.getMathematics(), mathematics));
            school.setRussianLanguage(avarage(school.getRussianLanguage(), russianLanguage));
            school.setComputerScience(avarage(school.getComputerScience(), computerScience));
            school.average();
        }
        else schools.put(number, new School(number, mathematics, russianLanguage, computerScience));
        return this;
    }

    private double avarage (double...args) {
        return Arrays.stream(args).average().getAsDouble();
    }

    public Collection<School> get() {
        return schools.values();
    }

    public Collection<School> get(Comparator<School> comparator) {
        return schools.values().stream().sorted(comparator).collect(Collectors.toList());
    }

}


import java.util.Comparator;

public class Main {

    public static void main(String[] args) {

        new SchoolService()
                .add(32, 80d, 80d, 80d)
                .add(71, 91d, 89d, 100d)
                .add(1, 15d, 60d, 30d)
                .add(32, 100d, 100d, 100d)
                .add(1, 75d, 99d, 67d)
                .add(17, 78d, 87d, 77d)
                .add(89, 66d, 54d, 78d)
                .get(Comparator.comparingDouble(School::getAverage).reversed())
                .forEach(System.out::println);

    }

}

теперь все хорошо? с точки зрения бизнесс-логики да, но с точки зрения программирования не совсем. я бы хотел обратить внимание на конструктор класса-модели. он принимает один int и 3 double. это не очень хорошо, потому что можно с легкостью запутаться в порядке параметров. посему, во-первых, попробуйте реализовать свой кастомный билдер для этого класса. во-вторых, наш сервис сейчас сам создает экземпляры класса-модели и принимает в аргументах метода множество параметров (аналогично конструктору). попробуйте решить эту проблему. в-третьих, билдер - не единственный порождающий паттерн. обратите внимание на остальные. главная проблема ооп- создание экземпляра класса. научитесь решать эту проблему - первый шаг в ооп. удачи