Решил,но терзает сомнения, что гавнокод

Могу предложить ознакомиться с моим вариантом. Он написан проще, как мне кажется. :) Я бы лишний раз не тормошил календарь. Всё-таки он не особо тут и нужен, хотя заполнить тот же мой monthList можно было и через него, но зачем? Да, это бы сократило метод по инициализации листа на пару строк, но создало лишние объекты и усложнило чтение кода. :)

public static void main(String[] args) throws IOException {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        String monthName = reader.readLine();
        System.out.printf("%s is the %d month", monthName, getMonth(monthName));
    }

    private static int getMonth(String monthName) {
        List<String> monthList = getMonthList();

        if (monthList.contains(monthName)) {
            return monthList.indexOf(monthName) + 1;
        }

        return -1;
    }

    private static List<String> getMonthList() {
        List<String> monthList = new ArrayList<>();
        monthList.add("January");
        monthList.add("February");
        monthList.add("March");
        monthList.add("April");
        monthList.add("May");
        monthList.add("June");
        monthList.add("July");
        monthList.add("August");
        monthList.add("September");
        monthList.add("October");
        monthList.add("November");
        monthList.add("December");
        return monthList;
    }
}