Boas práticas

Olá Paulo!

O código da aula diz que a classe tem um atributo e que ele é usado para seus métodos (que neste caso é apenas um).

Usando seu código, ao usar o método, é feita a comparação e o valor desta comparação "morre" com o uso do método. Caso algum outro método (se existir na classe) precisar usar este atributo, ele já não existe mais.

Resumindo: Acredito que para este caso é até válido, pois a classe tem apenas um método e ninguém mais irá precisar deste atributo (desde que o funcionário seja autenticado e depois disso não precise mais desta informação). Porém, se for um caso onde mais de um método irá compartilhar a mesma informação do objeto, ou se precisar do valor deste booleano em outro trecho do código, então é aconselhável definir um atributo para a classe. Ou seja, é mais uma questão da necessidade do seu código.

Espero ter ajudado!