Danya Kozlovskiy

Скорее всего компилятор ругается на то, что у вас не во всех вариантах событий возвращается какое-либо значение, допустим в коде ниже если выполнится условие if(income1>income2), но не выполнится вложенное в него if(income1>income3), то функция просто ничего не вернет, ибо блок else для вложенного if отсутствует.

if (income1>income2) {
            if (income1>income3) { 
                return film1;  
            }