От това имах нужада! Много Благодаря!!!

@MartinBG

Здравейте отново!

Направих каквото ме посъветвахте и мисля, че се получи, макар един от тестовете в judge да не минава :(

https://pastebin.com/WzZ2SDbB

Не използвах Stream API, защото все още не съм го учил, а за другите насоки, които ми дадехте, бяхте много прав.

Махнах списъка от Main класа, защото наистина не работеше, както отбелязахте.

Гетърите и сетърите в Team класа ги бях оставил, защото ползвах различни варианти за решение, но определено вашето мнение много ми помогна.

Благодаря още веднъж!

Виждам две грешки, поради които може да не минава решението.

1. Може да се създаде нов отбор от участник, който вече е създал друг отбор:

            if (isTeamExist(teams, team)) {
                teams.add(newTeam); // тук липсва проверката за създателя
                System.out.printf("Team %s has been created by %s!\n", team, creator);
            } else {
                for (Team currentTeam : teams) {
                    if (currentTeam.getName().equals(team)) {
                        System.out.printf("Team %s was already created!\n", team);
                    } else if (currentTeam.getCreator().equals(creator)) {
                        System.out.printf("%s cannot create another team!\n", creator);
                    }
                }
            }

Може да решите проблема примерно с нов метод (bool canCreateTeam(teams, creatorName), който извиквате в if/else if/else конструкция

2. Сортирането не е правилно конструирано - има две отделни и независими една от друга части:

        teams.sort(Comparator.comparing(Team::getName));
        teams.sort(Comparator.comparing(Team::membersCount).reversed());

Трябва да е примерно така:

        teams.sort(Comparator.comparing(Team::getName)
                .thenComparing(Comparator.comparingInt(Team::membersCount).reversed()));

Набързо няколко коментара по кода:

- isTeamExist в момента прави обратното на това, което твърди името ѝ (връща true за несъществуващ отбор и false за съществуващ)

- на много места в кода търсите отбор по име (for (Team team : teams) { if (team.getName().equals(teamToJoin))). Много по-удобно ще е ако пазите отборите в Map<String, Team>

- в Team класа getMembers() метода връща референция към members, което е опасно (някой може да ги модифицира отвън например). Трябва да върнете unmodifiableList или дори unmodifiableCollection

- части от кода в main могат да се изнесат като методи - например прочитането на входните данни и извеждането на резултата

@MartinBG

Да, грешката идваше от отва, че може да се създаде нов отбор от участник, който вече е създал друг отбор.

Добавих в проверката вече написаният метод за проверка дали даден член на отбор съществува и готово.

Направих и корекция при сортирането, но в judge ми даде грешка на първия тест, макар че вашият подход ми се струва правилен, тъй като прави две последователни сортировки.

Метода isTeamExist беше ок, но IDE-то ме посъветва да го обърна, защото и двата пъти, в които го извиквах беше с отрицание. Направих го, но забравих да му сменя името.

Защитих масива с метода unmodifiableList(), както ме посъветвахте.

И в крайна сметка взех 100/100 точки благодарение на вас.

Уви, направих няколко опита с map, но много обърках кода.

Ще се запозная по подробно с употребата на този вид масиви, защото не можах да извикам полетата на класа Team. Не намерих начин да обходя списъка с членовете и да правя проверки, дали в Map-а се съдържат конкретни стойности.

Бих желал да видя, как би могло да стане в тази задача, но ми помогнахте много и не искам да злоупотребявам.

Благодаря!