Pytanie o poprawność konstruktora i metodę

Witam
Mam do was dwa pytanka:

1. Posiadam taki konstruktor

 public Match(FootballClub teamA, FootballClub teamB, int teamAScore, int teamBScore, Date date) {
        this.teamA = teamA;
        this.teamB = teamB;
        this.teamAScore = teamAScore;
        this.TeamBScore = teamBScore;
        this.date = date;
        if(teamAScore>teamBScore){
            teamA.setWinsCount(teamA.getWinsCount()+1);
            teamA.setScoredGoalsCount(teamA.getScoredGoalsCount()+teamAScore);
            teamA.setRecivedGoalsCount(teamA.getRecivedGoalsCount()+teamBScore);
            teamB.setDefeatsCount(teamB.getDefeatsCount()+1);
            teamB.setScoredGoalsCount(teamB.getScoredGoalsCount()+teamBScore);
            teamA.setRecivedGoalsCount(teamA.getRecivedGoalsCount()+teamBScore);
            teamA.setPoints(teamA.getPoints()+3);
        }
        else if(teamAScore==teamBScore){
            teamA.setDrawsCount(teamA.getDrawsCount()+1);
            teamB.setDrawsCount(teamB.getDrawsCount()+1);
            teamA.setRecivedGoalsCount(teamA.getRecivedGoalsCount()+teamBScore);
            teamA.setRecivedGoalsCount(teamA.getRecivedGoalsCount()+teamBScore);
            teamA.setPoints(teamA.getPoints()+1);
            teamB.setPoints(teamB.getPoints()+1);
        }
        else if(teamBScore>teamAScore){
            teamB.setWinsCount(teamB.getWinsCount()+1);
            teamB.setScoredGoalsCount(teamB.getScoredGoalsCount()+teamBScore);
            teamB.setRecivedGoalsCount(teamB.getRecivedGoalsCount()+teamAScore);
            teamA.setDefeatsCount(teamA.getDefeatsCount()+1);
            teamA.setScoredGoalsCount(teamA.getScoredGoalsCount()+teamAScore);
            teamB.setRecivedGoalsCount(teamB.getRecivedGoalsCount()+teamAScore);
            teamB.setPoints(teamB.getPoints()+3);
        }
    }

Czy umieszczenie takiej logiki w nim jest zgodne z normami, konwencjami, czy powinna być gdzie indziej?

2 Mam taką metodę i wywołanie

 private Date setDate(){
        System.out.println("Podaj datę: (dd-mm-rrrr)");
        String line = scanner.nextLine();
        Date date = null;
        try {
            date = new SimpleDateFormat("dd-MM-yyyy").parse(line);
        } catch (ParseException ex) {
            System.out.println("niepoprawny format daty");
        }
        return date;
    }

  private void playMatch() {
        Date date;
        do {
            date = setDate();
        }
        while(date==null);
}

Czy jest to dobre rozwiązanie, jeśli chce pytać aż do uzyskania poprawnej odpowiedzi? Czy jest lepsza opcja?
Z góry dzięki

Nie. W konstruktorach nie piszemy logiki. Pętlę do-while ostatni raz widziałem 10 lat temu, więc tutaj się nie wypowiem :)

EDIT: czytanka https://www.yegor256.com/2015/05/07/ctors-must-be-code-free.html

Chłopaki nie przepisuje książek. Na YT jest wszystko.

Ad. 1. Zgadzam się z @Charles_Ray że konstruktor powinien być bez logiki. Powinieneś mieć jakąś metodę statyczną/wytwórczą calculateMatch/createMatch i tam robić te wyliczenia, a w konstruktorze ustawiać tylko wyliczone wartości
Ad. 2. Metoda o sygnaturze private Date setDate() która nie ustawia daty tylko ją pobiera od użytkownika wygląda jak ponury żart. BTW używanie nulli, zwłaszcza jako wartości zwracane, jest złe

Tu raczej trzeba się zastanowić jak to jest za-modelowane. Ja bym miał takie klasy:

• Match - reprezentuje mecz, może być jeszcze nie rozegrany lub już rozegrany
• MatchResult - reprezentuje wynik meczu (można też mieć MatchStats dla dokładnych statystyk)
• Team reprezentuje drużyne
• TeamStats - reprezentuje statystyki drużyny
• Player i PlayerStats podobnie

A potem bym miał:

match.finish();

// W match finish można mieć już
teams().foreach(t => t.updateStats(this.getMatchStats()));

Można też miec bardziej globalny obiekt typu FootballSeasonSimulator który by pilnował uaktualnienia statystyk po zakończeniu meczu.

I jeszcze jedno - java.util.Date (i java.util.Calendar) to bryndza. Omijaj.
Uzywaj klas z java.time:
https://docs.oracle.com/javase/8/docs/api/java/time/LocalDate.html
java.time.format.DateTimeFormatter

Konkstruktor powinien być jak najprostszy. Jeżeli Twój przyjmuje zbyt dużo argumentów na wejściu, staje się również mniej czytelny. Zainteresuj się wtedy wzorcem Builder (nie wariantem z Gang of Four).

Zrób sobie klasę FootballCup niemutowalną i wtedy kulturalnie tworzenie tych klas będziesz musiał zrobić przed Match.
Poza tym potrzebujesz więcej klas:
np. klasę ze statystykami drużyny

Ja bym takim modelem pojechał np.:

public class TeamStatstics {

    public final Integer matchesPlayed;
    public final Integer winMatches;
    public final Integer lostMatches;
    public final Integer scoredGoals;
    public final Integer receivedGoals;
    public final Integer points;

    // buildera se zrób

    public TeamStatstics(Integer matchesPlayed, Integer winMatches, Integer lostMatches, Integer scoredGoals, Integer receivedGoals, Integer points) {
        this.matchesPlayed = matchesPlayed;
        this.winMatches = winMatches;
        this.lostMatches = lostMatches;
        this.scoredGoals = scoredGoals;
        this.receivedGoals = receivedGoals;
        this.points = points;
    }


}

public class LeaugeStatistics {

    // jakies atrybuty

    private Map<Long, TeamStatstics> statistics = new HashMap<>();

    // jakis konstruktor

    public synchronized void addTeamStatistics(Long teamId, TeamStatstics teamStat) {
        statistics.put(teamId, teamStat);
    }

    public TeamStatstics getTeamStatistics(Long teamId) {
        return statistics.get(teamId);
    }

}