Stich25

Полностью согласен с тем, что написали мои коллеги.
От себя добавлю, что ваш код можно несколько упростить. Если вам придется сравнивать не 2 или 3 фильма, а допустим штук 100, то вы просто с ума сойдете для каждого из них подбирать комбинацию и строить ветки if else. А значит, должно быть другое "общее" решение.
Можете ознакомиться с одним из вариантов решения с помощью, которого вы можете сравнить сколько угодно фильмов и без всякого if .. else:

import java.util.Arrays;

public class Main {

    public static void main(String[] args) {
        // Создаем фильмы
        Cinema cinema1 = new Cinema("Титаник", 2194);
        Cinema cinema2 = new Cinema("Аватар", 2810);
        Cinema cinema3 = new Cinema("Тёмный рыцарь", 1084);
        Cinema[] cinemas = new Cinema[]{cinema1, cinema2, cinema3};
        // Находим фильм с большим доходом
        String cinemaTitle = findHighestGrossingFilm(cinemas);
        System.out.println(cinemaTitle);
    }

    /**
     * Метод находит фильм с большим доходом
     * @param cinemas массив фильмов для сравнения
     * @return название фильма с большим доходом
     */
    public static String findHighestGrossingFilm(Cinema ... cinemas) {
        Cinema cinemaWithMaxIncome = Arrays.stream(cinemas).max(Cinema::compareTo).get();
        return cinemaWithMaxIncome.name;
    }
}

class Cinema implements Comparable<Cinema> {

    String name;
    int income;

    public Cinema(String name, int income) {
        this.name = name;
        this.income = income;
    }

    // Имплементируем интерфейс Comparable и сравниваем income объектов
    @Override
    public int compareTo(Cinema c) {
        return Integer.compare(this.income, c.income);
    }
}