Не сочтите за назойливость. А насколько в индустрии разрешается тянуть вот так зависимости? Я читал где-то, что чем меньше создано переменных, чем меньше тянется и импортируется классов, пускай даже стандартных, тем правильнее и лучше. Или это из разряда "сам себя объегорил" и "изобретение велосипеда"?
1) Стандартная библиотека у тебя один фиг именно в зависимостях есть, ты не тянешь зависимость
2) Лишний импорт - вообще не беда, на количество импортов на моём опыте не смотрит вообще никто
2.5) Ну в данном случае можно конечно завести у себя локально int max(int a, int b) {if (a > b) return a else return b}
, разницы никакой
3) Тянуть зависимость ради одной функции было бы скорее всего плохо, если функция тривиальная
4) Само по себе количество переменных/классов/зависимостей не коррелирует с качеством кода - надо избегать избыточность, но если добавление переменной/класса например помогает заметно улучшить читаемость или поддерживаемость кода - это почти однозначно стоит сделать