Substituir ModularState por State e injetar IQuizController via parâmetro em QuizContentState

[shonorio]

Descrição

Atualmente, a classe QuizContentState estende ModularState e utiliza o IQuizController diretamente através do Modular. Para melhorar a testabilidade e a clareza do código, propomos substituir o ModularState pelo State padrão do Flutter e injetar o IQuizController via passagem de parâmetro no construtor.

Alterações Propostas

1. Remover a herança de ModularState:

   • A classe QuizContentState deixará de estender ModularState e passará a estender State<QuizContent> diretamente.

2. Injetar IQuizController via construtor:

   • O IQuizController será passado como parâmetro no construtor de QuizContentState.
   • O QuizContent será responsável por fornecer o IQuizController ao QuizContentState.

3. Atualizar o método build do QuizContent:

   • O QuizContent passará o IQuizController para o QuizContentState ao criar o estado.

Código Atual

class QuizContentState extends ModularState<QuizContent, IQuizController>
    with TickerProviderStateMixin<QuizContent>
    implements ScrollOverflowListener {
  // ...
}

Código Proposto

class QuizContentState extends State<QuizContent>
    with TickerProviderStateMixin
    implements ScrollOverflowListener {
  final IQuizController controller;

  QuizContentState({required this.controller});

  // ...
}

Atualização do QuizContent

class QuizContent extends StatefulWidget {
  const QuizContent({
    Key? key,
    required this.constraints,
    required this.controller,
  }) : super(key: key);

  final BoxConstraints constraints;
  final IQuizController controller;

  static QuizContentState? of(BuildContext context) =>
      context.findAncestorStateOfType<QuizContentState>();

  @override
  State<QuizContent> createState() => QuizContentState(controller: controller);
}

Benefícios

• Testabilidade: A injeção de dependência via construtor facilita a criação de testes unitários, permitindo a injeção de mocks ou stubs do IQuizController.
• Clareza: A remoção do ModularState torna o código mais explícito sobre as dependências necessárias para o funcionamento do QuizContentState.
• Desacoplamento: A classe QuizContentState não está mais acoplada ao Modular, o que pode facilitar futuras migrações ou mudanças na arquitetura.

Impacto

• Compatibilidade: Essa mudança não deve impactar outras partes do código, desde que o IQuizController seja corretamente injetado.
• Testes: Os testes existentes podem precisar de ajustes para refletir a nova forma de injeção de dependência.

Passos para Implementação

1. Atualizar a classe QuizContentState para estender State<QuizContent> e receber o IQuizController via construtor.
2. Atualizar a classe QuizContent para passar o IQuizController ao criar o estado.
3. Atualizar os testes unitários para refletir a nova forma de injeção de dependência.

Considerações Finais

Essa mudança é um passo importante para melhorar a qualidade do código e facilitar a manutenção e testes futuros. A equipe deve revisar e validar as alterações propostas antes de implementá-las.