Отрефакторим наш код, а уже затем закончим писать программу для Кекса.

Зачем снова рефакторинг?

Дело в том, что дальше нужно будет сравнить полученные результаты и вывести рекомендации для босса. Получается, нам придётся несколько раз использовать результаты работы функции calculateFlights. Поэтому логично и правильно записать эти результаты в переменные и использовать их дальше. Если мы вдруг захотим получить данные для других городов, мы просто изменим значения аргументов в одном единственном месте, а не по всему коду.

Как лучше всего назвать переменные, которые содержат в себе количество полётов до Лиссабона и Валенсии? Первое, что приходит на ум, дать переменным конкретные названия: flightsToValencia и flightsToLisbon — но такие название слишком частные. Если Кекс захочет полететь в какой-нибудь канадский Виннипег вместо Валенсии, переменную придётся переименовывать. И так каждый раз, когда направление полёта поменяется.

Можно отталкиваться не от направления, а от класса перелёта, и назвать переменные flightsEconom и flightsBusiness. Такой вариант тоже не самый лучший, потому что мы летаем в разные города, меняется не только класс, но и расстояния.

Назовём переменные максимально просто: flightsVariantFirst и flightsVariantSecond. Тогда внутри у них могут быть любые расстояния и классы перелёта.