É muito complicado falar em boas práticas, tem muita opinião sobre isto. Também não dá para falar muito sem saber os requisitos, o caso concreto. Não existe boa prática em caso abstrato, só é uma prática boa se aplicar dentro do que precisa. Exemplos artificiais não servem para ensinar boas práticas, serve apenas para ensinar receita de bolo que as pessoas aplicarão de forma equivocada em casos reais.
Um exemplo que muita gente vai dizer que é boa prática é criar os getters e setters que criou. Isto é má prática. Sabe por que? Porque fez sem entender porque. Poderia até estar certo, como eu disse antes, depende do caso concreto, mas como fez porque alguém fez antes e disse que é bom, já é uma má prática. Pode começar estudar mais o assunto em Métodos Getters e Setters.
Não tem um construtor, quase sempre, mas não sempre, é uma boa prática ter um. E em boa parte das vezes o setter é inadequado.
Tem certeza que é adequado manter os campos que são listas como públicos ou internos ao pacote como fez? Tanto declarado assim explicitamente como implicitamente. Isso é vazamento de abstração. E se for mais adequado usar de um outro jeito mais pra frente? Pode ler também Programar voltado para interface e não para a implementação, por quê?.
Não sei se Aluno
deveria ter um método chamado matricula()
. Isso pode ser um acoplamento indevido, mas novamente, sem requisito não tem como saber. Veja mais em O que são os conceitos de coesão e acoplamento?. Também pode ser sobre ter responsabilidade indevida, veja O que é e como se usa o SRP?.
Nem dá para ir muito além porque o código está pela metade, não usa o array criado, usa alguns campos (e não atributos como as pessoas chamam) que não foram declarados na classe.
Programar orientado a objeto é difícil e sem requisitos claros é impossível até para os mais experientes. Mesmo com tudo isto nem sempre existem respostas definitivas e é ingênuo achar isto e pode virar cagação de regra :)